Skip to content

mctp-client: add cci message type code - #168

Open
LeoYangQuanta wants to merge 1 commit into
CodeConstruct:mainfrom
LeoYangQuanta:main
Open

mctp-client: add cci message type code#168
LeoYangQuanta wants to merge 1 commit into
CodeConstruct:mainfrom
LeoYangQuanta:main

Conversation

@LeoYangQuanta

Copy link
Copy Markdown

According to Specification DSP0239 1.12.0
Add type code 8 for use with the CXL Type 3 Device Component Command Interface over MCTP.

Testing

cci commands

mctp-client net 1 eid 10 type cci data 00 00 00 00 03 00 00 00 00 00 cd ab
01 00 00 00 03 08 00 00 00 00 00 00 2B 01 AD 9C A4 8A D2 16

According to Specification DSP0239 1.12.0
Add type code 8 for use with the CXL Type 3 Device Component Command Interface over MCTP.

Testing
---
cci commands
```
mctp-client net 1 eid 10 type cci data 00 00 00 00 03 00 00 00 00 00 cd ab
01 00 00 00 03 08 00 00 00 00 00 00 2B 01 AD 9C A4 8A D2 16
```

Signed-off-by: Leo Yang <Leo-Yang@quantatw.com>

@jk-ozlabs jk-ozlabs left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Change itself looks okay, just a comment on the chosen name here.

However, the commit message just needs a little work. The wording is a little off, and no need for the section break there, nor that Testing line (we assume that you have tested this before submission).

Maybe just:

Add type code 8 for CXL Type 3 Device Component Command Interface,
as defined by DSP0239 1.9.0 and later.

For example:

$ mctp-client eid 10 type cxl-cci data 00 00 00 00 03 00 00 00 00 00 cd ab
01 00 00 00 03 08 00 00 00 00 00 00 2B 01 AD 9C A4 8A D2 16

Comment thread src/mctp-client.c
"secured messages using spdm over mctp per DSP0276",
},
{
.name = "cci",

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

should we clarify this is CXL related? perhaps cxl-cci?

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