Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

This commit supports spdm-requester-emu communicate with other endpoints #341

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

NguyenTanNhutQuang
Copy link

via MCTP in kernel. It also supports sending single command GET_VERSION to test the interface.

Tested:
Send command:
spdm_requester_emu --trans MCTP_KERNEL --eid 20 --cmd GET_VERSION
Sample output:
spdm_requester_emu version 0.1
trans - 0x5
spdm_cmd - 0x01
connect success!
context_size - 0x21d8
libspdm_send_spdm_request[0] msg SPDM_GET_VERSION(0x84), size (0x4):
0000: 10 84 00 00
Platform port Transmit transport_type: 00 00 00 05
Platform port Transmit size: 00 00 00 05
Platform port Transmit buffer:
05 10 84 00 00
Platform port Receive buffer:
05 10 04 00 00 00 03 00 10 00 11 00 12
libspdm_receive_spdm_response[0] msg SPDM_VERSION(0x4), size (0xc):
0000: 10 04 00 00 00 03 00 10 00 11 00 12
Client stopped

This commit supports spdm-requester-emu communicate with other endpoints
via MCTP in kernel. It also supports sending single command GET_VERSION
to test the interface.

Tested:
Send command:
    spdm_requester_emu --trans MCTP_KERNEL --eid 20 --cmd GET_VERSION
Sample output:
    spdm_requester_emu version 0.1
    trans - 0x5
    spdm_cmd - 0x01
    connect success!
    context_size - 0x21d8
    libspdm_send_spdm_request[0] msg SPDM_GET_VERSION(0x84), size (0x4):
    0000: 10 84 00 00
    Platform port Transmit transport_type: 00 00 00 05
    Platform port Transmit size: 00 00 00 05
    Platform port Transmit buffer:
        05 10 84 00 00
    Platform port Receive buffer:
        05 10 04 00 00 00 03 00 10 00 11 00 12
    libspdm_receive_spdm_response[0] msg SPDM_VERSION(0x4), size (0xc):
    0000: 10 04 00 00 00 03 00 10 00 11 00 12
    Client stopped

Signed-off-by: Quang Nguyen <[email protected]>
@jyao1
Copy link
Member

jyao1 commented Apr 4, 2024

@NguyenTanNhutQuang , please fix CI failure.

Please note that spdm-emu need to be built in both Linux and Windows OS environment.
I feel you need to have a way (maybe MACO) to exclude Linux kernel code if it is Windows/Visual Studio.

@NguyenTanNhutQuang
Copy link
Author

@NguyenTanNhutQuang , please fix CI failure.

Please note that spdm-emu need to be built in both Linux and Windows OS environment. I feel you need to have a way (maybe MACO) to exclude Linux kernel code if it is Windows/Visual Studio.

Hi jyao1,

Is there anyway that I can check CI on my local server before push to github? Thank you

@jyao1
Copy link
Member

jyao1 commented Apr 6, 2024

@steven-bellock , do you have any idea?

@steven-bellock
Copy link
Contributor

In terms of bandwidth / utilization it's fine to test your changes by pushing to this repository. The CI/CD utilization of this repository is low. If you really want to run locally you'd need something like https://github.com/nektos/act.

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.

3 participants