Skip to content

Add a pcap dump to the the mctpd process wrapper#156

Open
jk-ozlabs wants to merge 2 commits into
CodeConstruct:mainfrom
jk-ozlabs:dev/pcap
Open

Add a pcap dump to the the mctpd process wrapper#156
jk-ozlabs wants to merge 2 commits into
CodeConstruct:mainfrom
jk-ozlabs:dev/pcap

Conversation

@jk-ozlabs

Copy link
Copy Markdown
Member

We use the test suite's mctpd process wrapper to provide a mock implementation of (remote) MCTP endpoints. Debugging test issues is much easier if we can analyse the MCTP communicatons between the mctpd under test, and the mock endpoint(s).

This series adds a pcap writer to the mock MCTP socket implementation. This can either be enabled on the mctpenv main() command-line, or via a temporary change to an invocation of the McptdWrapper constructor.

We don't currently capture the netlink messaging, but that could be a nice future addition.

We're about to introduce a new option, so switch from manual sys.argv
parsing.

Signed-off-by: Jeremy Kerr <jk@codeconstruct.com.au>
When debugging tests, it's helpful to view the sequence of MCTP commands
between the emulated environment and mctpd.

Add a small pcap writer, storing the packet data to a file.

This is typically enabled by passing a file path to the MctpdWrapper
constructor, but pcap output from the mctpenv main() can be enabled with
a new --pcap argument too.

Signed-off-by: Jeremy Kerr <jk@codeconstruct.com.au>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant