tests: Add test_serial

Uses pipes to connect two instances (A and B) of the serial binding and
passes a message exceeding the BTU from A to B.

Increases test converage for libmctp as a whole from ~

tests: Add test_serial

Uses pipes to connect two instances (A and B) of the serial binding and
passes a message exceeding the BTU from A to B.

Increases test converage for libmctp as a whole from ~60% (lines and
functions) to ~75% (lines and functions).

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

show more ...

tests: Add test_astlpc

Exercise message passing and packetisation over a dummy LPC interface
injected via the ops struct. The main() plays the role of the host.

Test suite code coverage for libmctp

tests: Add test_astlpc

Exercise message passing and packetisation over a dummy LPC interface
injected via the ops struct. The main() plays the role of the host.

Test suite code coverage for libmctp as a whole is lifted from ~34%
(lines and functions) to ~53% (lines and functions).

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

show more ...

libmctp: Clarify BTU-related symbols

DSP0236 defines a Baseline Transmission Unit (BTU), which dictates the
minimum supported packet payload size. The packet size is the BTU + the
packet header. The

libmctp: Clarify BTU-related symbols

DSP0236 defines a Baseline Transmission Unit (BTU), which dictates the
minimum supported packet payload size. The packet size is the BTU + the
packet header. The maximum (payload) transport unit size for a given
binding may be larger than the BTU.

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

show more ...

core,API: Add bridge support

This change introduces a facility to bridge messages between two
bindings.

This is implemented through a new mctp_bridge_busses() API, which
applies a new routing polic

core,API: Add bridge support

This change introduces a facility to bridge messages between two
bindings.

This is implemented through a new mctp_bridge_busses() API, which
applies a new routing policy, sending packets from one binding to the
other. This is in contrast to the current policy of dropping all
non-local packets.

To do this, the message context code needs to know both source and
destination EIDs, so add them to the mctp_msg_ctx_lookup() criteria.

Also, add a small test for bridge mode.

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

show more ...

core,bindings: Allow bindings to specify packet size

Currently, we fix all packet sizes to the baseline MTU size. However,
bindings may support larger packet sizes.

This change makes the packet all

core,bindings: Allow bindings to specify packet size

Currently, we fix all packet sizes to the baseline MTU size. However,
bindings may support larger packet sizes.

This change makes the packet allocator use binding-specific parameters
to suit the binding itself, and uses the max packet size in the
packetisation path.

Since packet sizes may now exceed 255, we change the size and offset
types from uint8_t to size_t.

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

show more ...

core: Free packet in mctp_bus_rx

Change mctp_bus_rx to take ownership of a received packet, freeing it
after use.

This prevents potential leaks in binding implementations.

Signed-off-by: Jeremy Ke

core: Free packet in mctp_bus_rx

Change mctp_bus_rx to take ownership of a received packet, freeing it
after use.

This prevents potential leaks in binding implementations.

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

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>

tests: Add EID addressing test

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

tests: Add simple test infrastructure

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

utils: move small utilities from tests/ dir to utils/

We want to start building up tests under tests, and the current contents
are more utilities.

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

tests/mctp-pipe: Use SOCK_STREAM for internal serial link

Currently, we have no way to determine when the MCTP session should be
closed, when using SOCK_DGRAM. While this will be the case for a "rea

tests/mctp-pipe: Use SOCK_STREAM for internal serial link

Currently, we have no way to determine when the MCTP session should be
closed, when using SOCK_DGRAM. While this will be the case for a "real"
serial link, we want the test program to be able to terminate.

This change uses SOCK_STREAM instead, which means that we can terminate
the MCTP session on EOF on stdin. This allows the pipe to exit.

Additionally, this means that the read() boundaries may not correspond
to the write() boundaries, which gives more testing to the serial state
mechanism.

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

show more ...

Initial MCTP core code

Just a skeleton of the MCTP library at present.

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