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

spdm: large-transfer: set chunk capability #6

Open
wants to merge 1 commit into
base: spdm-future
Choose a base branch
from

Conversation

twilfredo
Copy link

@twilfredo twilfredo commented Aug 30, 2024

In the flags field of a GET_CAPABILITIES request, ensure we indicate that large transfers are supported by setting the CHUNK_CAP bit [1].

[1] SPDM Spec 1.3: Table 13

libspdm picked this up during testing here: https://github.com/DMTF/libspdm/blob/704d5652b2eeb451fffbd1bb1129046e377c9cf3/library/spdm_responder_lib/libspdm_rsp_capabilities.c#L221

@l1k l1k force-pushed the spdm-future branch 10 times, most recently from b048e74 to fbc92d2 Compare September 10, 2024 14:47
@twilfredo
Copy link
Author

@l1k This needs a rebase, but does it look right?

@l1k
Copy link
Owner

l1k commented Sep 12, 2024

Sorry for the delay Wilfred! I'm underwater with Plumbers preparations and the P1363 series. I'll see to it that I come up with a solution this weekend.

@twilfredo
Copy link
Author

Sorry for the delay Wilfred! I'm underwater with Plumbers preparations and the P1363 series. I'll see to it that I come up with a solution this weekend.

No worries, understandable. Hope to see you there!

In the flags field of a GET_CAPABILITIES request, ensure we indicate
that large transfers are supported by setting the CHUNK_CAP bit [1].

[1] SPDM Spec 1.3: Table 13

Signed-off-by: Wilfred Mallawa <wilfred.mallawa@wdc.com>
@twilfredo twilfredo force-pushed the wilfred/fixup-large-transfer-cap branch from 5fc03a9 to bc9f23e Compare October 8, 2024 01:49
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.

2 participants