tests: undef NDEBUG in tests

So we can run tests even when compiled with -DNDEBUG

Change-Id: Iebb8399409e2b0a5172529e08bcac188956fd925
Signed-off-by: Rashmica Gupta <rashmica@linux.ibm.com>

format: reformat with clang-format

Reformat everything with clang-format and remove .clang-ignore and
custom code formatter.

Signed-off-by: Patrick Williams <patrick@stwcx.xyz>
Change-Id: I93dfca67

format: reformat with clang-format

Reformat everything with clang-format and remove .clang-ignore and
custom code formatter.

Signed-off-by: Patrick Williams <patrick@stwcx.xyz>
Change-Id: I93dfca67948d3fbcc0f05ece4ae3e4dd9495ab57

show more ...

tests/test_seq.c: Avoid duplicate definition of ARRAY_SIZE

Avoid duplicate definition of "ARRAY_SIZE" macro in environments
that already define the macro

Signed-off-by: Younghyun Park <younghyunpar

tests/test_seq.c: Avoid duplicate definition of ARRAY_SIZE

Avoid duplicate definition of "ARRAY_SIZE" macro in environments
that already define the macro

Signed-off-by: Younghyun Park <younghyunpark@google.com>
Change-Id: I8fa5ddd0d0279cbcda160fcb232c9c481594b0f0

show more ...

core: Add TX/RX API that exposes message tag and tag owner

MCTP received packets can carry a message tag and tag owner bit
which is set by a remote MCTP endpoint. This can be used by the
remote MCTP

core: Add TX/RX API that exposes message tag and tag owner

MCTP received packets can carry a message tag and tag owner bit
which is set by a remote MCTP endpoint. This can be used by the
remote MCTP endpoint to track the responses. Thus, libmctp should
provide a mechanism for the upper layer MCTP applications to
respond with the same message tag.

This patchset extends TX and RX API with message tag and
tag owner bits.

Signed-off-by: Sumanth Bhat <sumanth.bhat@linux.intel.com>
Change-Id: I6d07eafa86c653abdd4313ab7cc77e5a93124477

show more ...

libmctp: Introduce compiler.h

Define __unused here and include compiler.h in all the places we can
make use of it. Clean up some header ordering and include styles while
we're at it.

Signed-off-by:

libmctp: Introduce compiler.h

Define __unused here and include compiler.h in all the places we can
make use of it. Clean up some header ordering and include styles while
we're at it.

Signed-off-by: Andrew Jeffery <andrew@aj.id.au>
Change-Id: I33ddbdacbddbbf557dc02e42d8dde71a16e1cabe

show more ...

test_seq: Clean up after each test case

Free all allocated memory to avoid false-positive leak reports.
Resolves:

==11106==ERROR: LeakSanitizer: detected memory leaks

Indirect leak of 1228

test_seq: Clean up after each test case

Free all allocated memory to avoid false-positive leak reports.
Resolves:

==11106==ERROR: LeakSanitizer: detected memory leaks

Indirect leak of 12288 byte(s) in 3 object(s) allocated from:
#0 0x7f92a1347f1e in __interceptor_realloc (/usr/lib/x86_64-linux-gnu/libasan.so.5+0x10df1e)
#1 0x7f92a1207065 in __mctp_realloc /home/andrew/src/openbmc/libmctp/alloc.c:48
#2 0x7f92a1201979 in mctp_msg_ctx_add_pkt /home/andrew/src/openbmc/libmctp/core.c:213
#3 0x7f92a1204a09 in mctp_bus_rx /home/andrew/src/openbmc/libmctp/core.c:383
#4 0x55ff747e1655 in mctp_binding_test_rx_raw tests/test-utils.c:50
#5 0x55ff747df302 in run_one_test tests/test_seq.c:122
#6 0x55ff747df6d6 in main tests/test_seq.c:141
#7 0x7f92a06b21e2 in __libc_start_main (/lib/x86_64-linux-gnu/libc.so.6+0x271e2)

Indirect leak of 2208 byte(s) in 4 object(s) allocated from:
#0 0x7f92a1347ae8 in malloc (/usr/lib/x86_64-linux-gnu/libasan.so.5+0x10dae8)
#1 0x7f92a1206cf0 in __mctp_alloc /home/andrew/src/openbmc/libmctp/alloc.c:28
#2 0x7f92a1201edc in mctp_init /home/andrew/src/openbmc/libmctp/core.c:234
#3 0x55ff747e1738 in mctp_test_stack_init tests/test-utils.c:63
#4 0x55ff747def62 in run_one_test tests/test_seq.c:111
#5 0x55ff747df6d6 in main tests/test_seq.c:141
#6 0x7f92a06b21e2 in __libc_start_main (/lib/x86_64-linux-gnu/libc.so.6+0x271e2)

Indirect leak of 224 byte(s) in 4 object(s) allocated from:
#0 0x7f92a1347ae8 in malloc (/usr/lib/x86_64-linux-gnu/libasan.so.5+0x10dae8)
#1 0x7f92a1206cf0 in __mctp_alloc /home/andrew/src/openbmc/libmctp/alloc.c:28
#2 0x55ff747e1169 in mctp_binding_test_init tests/test-utils.c:27
#3 0x55ff747e17b1 in mctp_test_stack_init tests/test-utils.c:66
#4 0x55ff747def62 in run_one_test tests/test_seq.c:111
#5 0x55ff747df6d6 in main tests/test_seq.c:141
#6 0x7f92a06b21e2 in __libc_start_main (/lib/x86_64-linux-gnu/libc.so.6+0x271e2)

Indirect leak of 160 byte(s) in 4 object(s) allocated from:
#0 0x7f92a1347ae8 in malloc (/usr/lib/x86_64-linux-gnu/libasan.so.5+0x10dae8)
#1 0x7f92a1206cf0 in __mctp_alloc /home/andrew/src/openbmc/libmctp/alloc.c:28
#2 0x7f92a120255f in mctp_register_bus /home/andrew/src/openbmc/libmctp/core.c:277
#3 0x55ff747e16eb in mctp_binding_test_register_bus tests/test-utils.c:56
#4 0x55ff747e18d5 in mctp_test_stack_init tests/test-utils.c:69
#5 0x55ff747def62 in run_one_test tests/test_seq.c:111
#6 0x55ff747df6d6 in main tests/test_seq.c:141
#7 0x7f92a06b21e2 in __libc_start_main (/lib/x86_64-linux-gnu/libc.so.6+0x271e2)

Signed-off-by: Andrew Jeffery <andrew@aj.id.au>
Change-Id: I7fbc729740357eb89b679dd51bbf03d6735a75dc

show more ...

LICENSE: add GPLv2 license option.

As per the MCTP design document
(https://github.com/openbmc/docs/blob/master/designs/mctp.md), a
dual-licence (GPLv2+ and Apache-2.0) gives us flexibility with inc

LICENSE: add GPLv2 license option.

As per the MCTP design document
(https://github.com/openbmc/docs/blob/master/designs/mctp.md), a
dual-licence (GPLv2+ and Apache-2.0) gives us flexibility with including
libcmtp code into various firmware environments.

This change adds GPLv2+ as a license option. All current contributors
have agreed to this change.

Signed-off-by: Jeremy Kerr <jk@ozlabs.org>

show more ...

tests/seq: More sequencing tests

Signed-off-by: Jeremy Kerr <jk@ozlabs.org>

tests/seq: Expand sequencing test

Signed-off-by: Jeremy Kerr <jk@ozlabs.org>

tests: Add sequencing test

Add a small test (to be expanded later) to check behaviour of MCTP
packet sequencing.

Signed-off-by: Jeremy Kerr <jk@ozlabs.org>