From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mail02.groups.io (mail02.groups.io [66.175.222.108]) by spool.mail.gandi.net (Postfix) with ESMTPS id 9937778042F for ; Fri, 20 Oct 2023 13:01:06 +0000 (UTC) DKIM-Signature: a=rsa-sha256; bh=sk/fDrMStOomgR4A/V024lVPWveDDFQBCWTYykzTgqI=; c=relaxed/simple; d=groups.io; h=MIME-Version:References:In-Reply-To:From:Date:Message-ID:Subject:To:Cc:Precedence:List-Subscribe:List-Help:Sender:List-Id:Mailing-List:Delivered-To:Reply-To:List-Unsubscribe-Post:List-Unsubscribe:Content-Type:Content-Transfer-Encoding; s=20140610; t=1697806865; v=1; b=wHoNqu5B/FjQ8Z9PQf/L/cF3KYcCeXWcu863a7feEohA5lNpj96w31eS5SijoPHudrGs/Kw2 SzC92KjUTJ5QLf49Cy+QpET2kjAAmeYS57qzq/OXpa3HxGby+G2FXvBm5D6P1TRmcY/R/HGXI1z DppXBnO/q4IslBBswKG+vrrY= X-Received: by 127.0.0.2 with SMTP id Ou2kYY7687511xsMstrgFLUr; Fri, 20 Oct 2023 06:01:05 -0700 X-Received: from mail-vk1-f172.google.com (mail-vk1-f172.google.com [209.85.221.172]) by mx.groups.io with SMTP id smtpd.web11.53760.1697806864055893496 for ; Fri, 20 Oct 2023 06:01:04 -0700 X-Received: by mail-vk1-f172.google.com with SMTP id 71dfb90a1353d-49dc95be894so267376e0c.2; Fri, 20 Oct 2023 06:01:03 -0700 (PDT) X-Gm-Message-State: sO5nLFzK2jmIJK3riNWxYy9Ax7686176AA= X-Google-Smtp-Source: AGHT+IGgFWzZkgrndxvR79V2qq4jzj270sssK3K1cI8bgw3lXJ3Gh3E1zePrsSeYuD/j3DabcHdCUR8iy3hBfsVd1Io= X-Received: by 2002:a05:6122:4686:b0:49e:3f5:7678 with SMTP id di6-20020a056122468600b0049e03f57678mr1625013vkb.10.1697806862337; Fri, 20 Oct 2023 06:01:02 -0700 (PDT) MIME-Version: 1.0 References: <178DEE4D03C504AF.14388@groups.io> In-Reply-To: From: "Konstantin Aladyshev" Date: Fri, 20 Oct 2023 16:00:49 +0300 Message-ID: Subject: Re: [edk2-devel] [edk2-discuss] PLDM messages via MCTP over KCS To: "Chang, Abner" Cc: "discuss@edk2.groups.io" , "devel@edk2.groups.io" Precedence: Bulk List-Subscribe: List-Help: Sender: devel@edk2.groups.io List-Id: Mailing-List: list devel@edk2.groups.io; contact devel+owner@edk2.groups.io Reply-To: devel@edk2.groups.io,aladyshev22@gmail.com List-Unsubscribe-Post: List-Unsubscribe=One-Click List-Unsubscribe: Content-Type: text/plain; charset="UTF-8" Content-Transfer-Encoding: quoted-printable X-GND-Status: LEGIT Authentication-Results: spool.mail.gandi.net; dkim=pass header.d=groups.io header.s=20140610 header.b=wHoNqu5B; dmarc=fail reason="SPF not aligned (relaxed), DKIM not aligned (relaxed)" header.from=gmail.com (policy=none); spf=pass (spool.mail.gandi.net: domain of bounce@groups.io designates 66.175.222.108 as permitted sender) smtp.mailfrom=bounce@groups.io Hi Abner! With several minor changes I was able to perform PLDM communication via the EDKII_PLDM_PROTOCOL. I've sent these changes to you via the pull request to your branch https://github.com/changab/edk2-platforms/pull/7 I think this way it would be easier for you to review them. I've also sent the PATCH v2 series to the edk2-platforms mailing list. Here I did the following things: - squashed correction like https://github.com/changab/edk2-platforms/commit/8c792c2ba0d6d814ee91cd5e02= 9fdeb21668313e with earlier patches, - added your new patches with proper commit messages and squashed some redundant commits, - added my new PLDM patches. The correctly formatted patches I maintain here: https://github.com/Kostr/edk2-platforms/tree/MCTP_KCS (Basically the content of this branch is the same as our MCTP_OVER_KCS_UPDATE branch, only the patches are formatted more nicely) Do I need to perform a pull request directly to the tianocore/edk2-platforms repo? Also don't forget that we also need a couple of patches for the edk2 code. Here is a branch with these patches: https://github.com/Kostr/edk2/tree/MCTP_PLDM Do I need to perform PR to tianocore/edk2 as well? Best regards, Konstantin Aladyshev On Fri, Oct 20, 2023 at 11:33=E2=80=AFAM Chang, Abner = wrote: > > [AMD Official Use Only - General] > > > -----Original Message----- > > From: Konstantin Aladyshev > > Sent: Thursday, October 19, 2023 1:14 AM > > To: Chang, Abner ; discuss@edk2.groups.io; > > devel@edk2.groups.io > > Subject: Re: [edk2-devel] [edk2-discuss] PLDM messages via MCTP over KC= S > > > > Caution: This message originated from an External Source. Use proper ca= ution > > when opening attachments, clicking links, or responding. > > > > > > Thanks! > > > > I've found some issues with the current PLDM code, I hope I will get > > back with complete feedback on how to make everything work by Friday. > > > > I was also thinking about the response buffer interface in the > > MctpProtocol/PldmProtocols. > > Currently the caller should provide the response buffer big enough to > > accommodate the actual response. Otherwise the code will return with > > error. > > I'm thinking, wouldn't it be better to just dynamically allocate the > > response buffer inside the protocols and leave the responsibility of > > calling FreePool(...) to the caller? > > What do you think? > I don=E2=80=99t have strong opinions on how to manage the response buffer= . How about we get back to this after you verifying the functionality of th= e code changes we made and pushing the code to edk2-platform? This seems to= me simpler than having all the code changes on hand. > > BTW, I would like to merge the patch set you sent. Did you give me the PR= or repo that has this code changes? Or I missed it? > > Thanks > Abner > > > > > Best regards, > > Konstantin Aladyshev > > > > On Wed, Oct 18, 2023 at 6:45=E2=80=AFAM Chang, Abner > > wrote: > > > > > > [AMD Official Use Only - General] > > > > > > Code update is on https://github.com/changab/edk2- > > platforms/tree/MCTP_OVER_KCS_UPDATE, the last commit. > > > > > > Abner > > > > > > > -----Original Message----- > > > > From: Chang, Abner > > > > Sent: Wednesday, October 18, 2023 10:49 AM > > > > To: Konstantin Aladyshev > > > > Subject: RE: [edk2-devel] [edk2-discuss] PLDM messages via MCTP ove= r > > KCS > > > > > > > > Sure, this works for me. I will update the code. > > > > > > > > Thanks > > > > Abner > > > > > > > > > -----Original Message----- > > > > > From: Konstantin Aladyshev > > > > > Sent: Tuesday, October 17, 2023 11:58 PM > > > > > To: Chang, Abner > > > > > Subject: Re: [edk2-devel] [edk2-discuss] PLDM messages via MCTP o= ver > > KCS > > > > > > > > > > Caution: This message originated from an External Source. Use pro= per > > > > caution > > > > > when opening attachments, clicking links, or responding. > > > > > > > > > > > > > > > Oops, sorry, never mind the last letter. I think I've got confuse= d > > > > > between the different PldmSubmit functions. > > > > > We don't need to change the protocol interface. > > > > > > > > > > But still I think we can get rid of special values like > > > > > PLDM_TERMINUS_ID_UNASSIGNED and PLDM_TERMINUS_ID_SPECIAL > > > > with > > > > > something like this: > > > > > > > > > > https://github.com/changab/edk2- > > > > > > > > > > > platforms/blob/56087215747be4a6a73e17e8478d5a8c6e35c801/Features/ > > > > > > > ManageabilityPkg/Library/PldmProtocolLibrary/Dxe/PldmProtocolLib.c#L16 > > > > > - UINT8 mSourcePldmTerminusId =3D > > > > > PLDM_TERMINUS_ID_UNASSIGNED; > > > > > - UINT8 mDestinationPldmTerminusId =3D > > > > > PLDM_TERMINUS_ID_UNASSIGNED; > > > > > + UINT8 mSourcePldmTerminusId =3D > > > > > PcdGet8(PcdPldmSourceTerminusId); > > > > > + UINT8 mDestinationPldmTerminusId =3D > > > > > PcdGet8(PcdPldmDestinationTerminusId); > > > > > > > > > > https://github.com/changab/edk2- > > > > > > > > > > > platforms/blob/56087215747be4a6a73e17e8478d5a8c6e35c801/Features/ > > > > > > > > > > > ManageabilityPkg/Library/PldmProtocolLibrary/Dxe/PldmProtocolLib.c#L105 > > > > > ``` > > > > > Status =3D mEdkiiPldmProtocol->Functions.Version1_0- > > > > >PldmSubmitCommand > > > > > ( > > > > > mEdkiiPldm= Protocol, > > > > > PldmType, > > > > > Command, > > > > > - > > > > > (mSourcePldmTerminusId =3D=3D PLDM_TERMINUS_ID_SPECIAL)? (UINT8 > > > > > *)NULL: > > > > > &mSourcePldmTerminusId, > > > > > - > > > > > (mDestinationPldmTerminusId =3D=3D PLDM_TERMINUS_ID_SPECIAL)? > > (UINT8 > > > > > *)NULL: &mDestinationPldmTerminusId, > > > > > + &mSourcePld= mTerminusId, > > > > > + > > > > > &mDestinationPldmTerminusId, > > > > > RequestDat= a, > > > > > RequestDat= aSize, > > > > > ResponseDa= ta, > > > > > ResponseDa= taSize > > > > > ); > > > > > ``` > > > > > > > > > > I'll try to test everything on the hardware tomorrow. > > > > > > > > > > Best regards, > > > > > Konstantin Aladyshev > > > > > > > > > > On Tue, Oct 17, 2023 at 4:12=E2=80=AFPM Konstantin Aladyshev > > > > > wrote: > > > > > > > > > > > > One more thought. > > > > > > > > > > > > Shouldn't we also put the "PldmSetTerminus" to the > > > > > EDKII_PLDM_PROTOCOL? > > > > > > https://github.com/changab/edk2- > > > > > > > > > > > platforms/blob/56087215747be4a6a73e17e8478d5a8c6e35c801/Features/ > > > > > ManageabilityPkg/Include/Protocol/PldmProtocol.h#L71 > > > > > > > > > > > > As you remember in my test application I've manually constructe= d the > > > > > > MCTP buffer for the "GetPLDMTypes (0x04)" PLDM command and sent > > it > > > > via > > > > > > the EDKII_MCTP_PROTOCOL: > > > > > > https://github.com/changab/edk2- > > > > > > > > > > > platforms/blob/56087215747be4a6a73e17e8478d5a8c6e35c801/Features/ > > > > > ManageabilityPkg/Include/Protocol/PldmProtocol.h#L71 > > > > > > > > > > > > The next test should be to send the same command, but via the > > > > > > EDKII_PLDM_PROTOCOL. > > > > > > And to route the MCTP packet correctly, we somehow need to call= the > > > > > > "PldmSetTerminus" function, before calling "PldmSubmit". > > > > > > So I think we need to add PldmSetTerminus to the > > > > EDKII_PLDM_PROTOCOL > > > > > interface: > > > > > > https://github.com/changab/edk2- > > > > > > > > > > > platforms/blob/56087215747be4a6a73e17e8478d5a8c6e35c801/Features/ > > > > > ManageabilityPkg/Include/Protocol/PldmProtocol.h#L71 > > > > > > > > > > > > typedef struct { > > > > > > PLDM_SET_TERMINOUS PldmSetTerminous; > > > > > > PLDM_SUBMIT_COMMAND PldmSubmitCommand; > > > > > > } EDKII_PLDM_PROTOCOL_V1_0; > > > > > > > > > > > > What do you think? > > > > > > > > > > > > Best regards, > > > > > > Konstantin Aladyshev > > > > > > > > > > > > On Tue, Oct 17, 2023 at 2:37=E2=80=AFPM Konstantin Aladyshev > > > > > > wrote: > > > > > > > > > > > > > > Sorry, maybe I've described this wrong. > > > > > > > Initially mSourcePldmTerminusId/mDestinationPldmTerminusId > > would > > > > get > > > > > > > their values from the build-time PCDs: > > > > > > > """ > > > > > > > UINT8 mSourcePldmTerminusId =3D > > > > > > > PcdGet8(PcdPldmSourceTerminusId); > > > > > > > UINT8 mDestinationPldmTerminusId =3D > > > > > > > PcdGet8(PcdPldmDestinationTerminusId); > > > > > > > """ > > > > > > > > > > > > > > So by default the PCD values are used in the PLDM protocols. = If the > > > > > > > user want to use something different, he would execute someth= ing > > like > > > > > > > 'PldmSmbiosTransferProtocol.PldmSetTerminus'. This function a= s > > written > > > > > > > would change values of the > > > > > > > "mSourcePldmTerminusId/mDestinationPldmTerminusId". > > > > > > > """ > > > > > > > PldmSmbiosTransferProtocol.PldmSetTerminus -> > > > > > > > PldmProtocolLib.PldmSetTerminus -> modify > > > > > > > mSourcePldmTerminusId/mDestinationPldmTerminusId > > > > > > > """ > > > > > > > https://github.com/changab/edk2- > > > > > > > > > > > platforms/blob/56087215747be4a6a73e17e8478d5a8c6e35c801/Features/ > > > > > > > ManageabilityPkg/Library/PldmProtocolLibrary/Dxe/PldmProtocolLib.c#L44 > > > > > > > > > > > > > > In this case PldmSubmit can always just use > > > > > > > mSourcePldmTerminusId/mDestinationPldmTerminusId values for > > TIDs. > > > > > > > > > > > > > > Best regards, > > > > > > > Konstantin Aladyshev > > > > > > > > > > > > > > On Tue, Oct 17, 2023 at 1:39=E2=80=AFPM Chang, Abner > > > > > > > > > wrote: > > > > > > > > > > > > > > > > [AMD Official Use Only - General] > > > > > > > > > > > > > > > > > -----Original Message----- > > > > > > > > > From: Konstantin Aladyshev > > > > > > > > > Sent: Tuesday, October 17, 2023 4:54 PM > > > > > > > > > To: Chang, Abner > > > > > > > > > Cc: discuss@edk2.groups.io; devel@edk2.groups.io > > > > > > > > > Subject: Re: [edk2-devel] [edk2-discuss] PLDM messages vi= a > > MCTP > > > > > over KCS > > > > > > > > > > > > > > > > > > Caution: This message originated from an External Source.= Use > > proper > > > > > caution > > > > > > > > > when opening attachments, clicking links, or responding. > > > > > > > > > > > > > > > > > > > > > > > > > > > Hi Abner, > > > > > > > > > > > > > > > > > > In the code you are using: > > > > > > > > > PLDM_TERMINUS_ID_UNASSIGNED > > > > > > > > > PLDM_TERMINUS_ID_RESERVED > > > > > > > > > PLDM_TERMINUS_ID_SPECIAL > > > > > > > > > but I don't see any definitions for these values. > > > > > > > > > I'm particularly confused about the meaning and usage of = the > > > > > > > > > PLDM_TERMINUS_ID_SPECIAL value. > > > > > > > > Oops. That is in edk2. Here it is: > > > > > https://github.com/changab/edk2/tree/MCTP_OVER_KCS_UPDATE > > > > > > > > > > > > > > > > > > > > > > > > > > My understanding was that we give initial values for the > > TerminusId > > > > > > > > > values as PCDs: > > > > > > > > > https://github.com/changab/edk2- > > > > > > > > > > > > > > > > > > > > platforms/blob/56087215747be4a6a73e17e8478d5a8c6e35c801/Features/ > > > > > > > > > > > > > > > > > > > > ManageabilityPkg/Library/PldmProtocolLibrary/Dxe/PldmProtocolLib.c#L16C > > > > > > > > > 1-L17C79 > > > > > > > > > ``` > > > > > > > > > UINT8 mSourcePldmTerminusId =3D > > > > > > > > > PcdGet8(PcdPldmSourceTerminusId); > > > > > > > > > UINT8 mDestinationPldmTerminusId =3D > > > > > > > > > PcdGet8(PcdPldmDestinationTerminusId); > > > > > > > > > ``` > > > > > > > > If we do this then we have to change the platform PCD setti= ng in > > build > > > > > time to give the different value of PLDM terminus ID. So the EFI = shell > > > > > application binary (e.g., 3rd party EFI app if any) has no way to= use > > different > > > > > TID value, unless we declare PCD as dynamic and EFI application h= as to > > use > > > > > PCD protocol to change the value. This would be the burden of EFI > > > > application > > > > > and unnecessary in my option. > > > > > > > > So the implementation is if SetPldmSmbiosTransferTerminusId= is > > > > invoked, > > > > > PldmSetTerminus in PldmProtocolLib is invoked to set the new valu= e of > > TID > > > > > and only for PldmSmbiosTransferProtocol. Otherwise the default PC= D > > > > defined > > > > > by platform developer are used. This is the most flexible to supp= ort > > different > > > > > use cases. > > > > > > > > > > > > > > > > > > > > > > > > > > And in the PldmSubmit function just use them: > > > > > > > > > https://github.com/changab/edk2- > > > > > > > > > > > > > > > > > > > > platforms/blob/56087215747be4a6a73e17e8478d5a8c6e35c801/Features/ > > > > > > > > > > > > > > > > > > > > ManageabilityPkg/Library/PldmProtocolLibrary/Dxe/PldmProtocolLib.c#L100 > > > > > > > > > ``` > > > > > > > > > Status =3D mEdkiiPldmProtocol->Functions.Version1_0- > > > > > >PldmSubmitCommand > > > > > > > > > ( > > > > > > > > > mE= dkiiPldmProtocol, > > > > > > > > > Pl= dmType, > > > > > > > > > Co= mmand, > > > > > > > > > mS= ourcePldmTerminusId, > > > > > > > > > > > > > > > > > > mDestinationPldmTerminusId, > > > > > > > > > Re= questData, > > > > > > > > > Re= questDataSize, > > > > > > > > > Re= sponseData, > > > > > > > > > Re= sponseDataSize > > > > > > > > > ); > > > > > > > > > ``` > > > > > > > > > Writing this it looks like we don't even need to pass TID= s as > > pointers > > > > > > > > > in this case. > > > > > > > > > Wouldn't it work like we want it to? > > > > > > > > > > > > > > > > > > One more minor observation, there are actually 2 reserved= values > > for > > > > > > > > > the TID according to the specification: 0x00 and 0xff. > > > > > > > > Yes, we can use 0x00 as the unsigned value as it is defined= in GetTID > > > > > command. > > > > > > > > > > > > > > > > Thanks > > > > > > > > Abner > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > Best regards, > > > > > > > > > Konstantin Aladyshev > > > > > > > > > > > > > > > > > > On Tue, Oct 17, 2023 at 6:41=E2=80=AFAM Chang, Abner > > > > > > > > > > > > > > wrote: > > > > > > > > > > > > > > > > > > > > [AMD Official Use Only - General] > > > > > > > > > > > > > > > > > > > > Hi Aladyshev, > > > > > > > > > > I had updated code according to our discussion on PLDM = over > > > > MCTP, > > > > > please > > > > > > > > > take a look at https://github.com/changab/edk2- > > > > > > > > > platforms/tree/MCTP_OVER_KCS_UPDATE and verify it at you = end > > if > > > > > possible. > > > > > > > > > The PLDM command table is also removed at this version. > > > > > > > > > > > > > > > > > > > > Thanks > > > > > > > > > > Abner > > > > > > > > > > > > > > > > > > > > > -----Original Message----- > > > > > > > > > > > From: discuss@edk2.groups.io = On > > > > > Behalf Of > > > > > > > > > > > Chang, Abner via groups.io > > > > > > > > > > > Sent: Saturday, October 14, 2023 5:11 PM > > > > > > > > > > > To: Konstantin Aladyshev > > > > > > > > > > > Cc: discuss@edk2.groups.io; devel@edk2.groups.io > > > > > > > > > > > Subject: Re: [edk2-devel] [edk2-discuss] PLDM message= s via > > > > MCTP > > > > > over > > > > > > > > > KCS > > > > > > > > > > > > > > > > > > > > > > [AMD Official Use Only - General] > > > > > > > > > > > > > > > > > > > > > > Caution: This message originated from an External Sou= rce. Use > > > > > proper > > > > > > > > > caution > > > > > > > > > > > when opening attachments, clicking links, or respondi= ng. > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > [AMD Official Use Only - General] > > > > > > > > > > > > > > > > > > > > > > > -----Original Message----- > > > > > > > > > > > > From: Konstantin Aladyshev > > > > > > > > > > > > Sent: Saturday, October 14, 2023 4:26 PM > > > > > > > > > > > > To: Chang, Abner > > > > > > > > > > > > Cc: discuss@edk2.groups.io; devel@edk2.groups.io > > > > > > > > > > > > Subject: Re: [edk2-devel] [edk2-discuss] PLDM messa= ges via > > > > > MCTP over > > > > > > > > > KCS > > > > > > > > > > > > > > > > > > > > > > > > Caution: This message originated from an External S= ource. > > Use > > > > > proper > > > > > > > > > > > caution > > > > > > > > > > > > when opening attachments, clicking links, or respon= ding. > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > Sorry I got confused a little bit by your answer. > > > > > > > > > > > > You were talking about PLDM TIDs (Terminus IDs) and= I was > > > > asking > > > > > about > > > > > > > > > > > > MCTP EIDs. Don't they like different concepts? Or d= id you > > mean > > > > > that we > > > > > > > > > > > > need to have a mapping of TIDs<-->EIDs in PldmProto= colLib? > > > > > > > > > > > I just distinguish the ID naming for PLDM and MCTP us= ing the > > > > > > > > > terminologies > > > > > > > > > > > defined in spec. For PLDM to MCTP, the values are the= same as > > the > > > > > SID and > > > > > > > > > EID > > > > > > > > > > > are passed to MCTP from PLDM. > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > I reviewed these patches just now, that looks goo= d to me. > > > > > However this > > > > > > > > > > > way > > > > > > > > > > > > we don=E2=80=99t have the history in the mailing li= st. > > > > > > > > > > > > If the code looks good to you, I'll try to properly= format all our > > > > > > > > > > > > patches and send them to the mailing list next week= . > > > > > > > > > > > Ok, I see. > > > > > > > > > > > > > > > > > > > > > > > > > BTW, are you an individual contributor or you are= the > > > > > representative of > > > > > > > > > a > > > > > > > > > > > > firm? > > > > > > > > > > > > As a part of my job I develop UEFI and BMC firmware= . But all > > the > > > > > > > > > > > > open-source work that I do I perform on my own free= time. > > So > > > > you > > > > > can > > > > > > > > > > > > count me as an individual contributor) > > > > > > > > > > > I am asking this because I am looking for an addition= al reviewer > > for > > > > > > > > > > > ManageabilityPkg as the replacement of Isaac from Int= el as he > > is > > > > no > > > > > longer > > > > > > > > > > > works on edk2 stuff. > > > > > > > > > > > > > > > > > > > > > > Thanks > > > > > > > > > > > Abner > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > Best regards, > > > > > > > > > > > > Konstantin Aladyshev > > > > > > > > > > > > > > > > > > > > > > > > On Sat, Oct 14, 2023 at 11:06=E2=80=AFAM Chang, Abn= er > > > > > > > > > > > > > > > > > > > > > wrote: > > > > > > > > > > > > > > > > > > > > > > > > > > [AMD Official Use Only - General] > > > > > > > > > > > > > > > > > > > > > > > > > > > -----Original Message----- > > > > > > > > > > > > > > From: Konstantin Aladyshev > > > > > > > > > > > > > > Sent: Friday, October 13, 2023 8:16 PM > > > > > > > > > > > > > > To: Chang, Abner > > > > > > > > > > > > > > Cc: discuss@edk2.groups.io; devel@edk2.groups.i= o > > > > > > > > > > > > > > Subject: Re: [edk2-devel] [edk2-discuss] PLDM m= essages > > via > > > > > MCTP > > > > > > > > > over > > > > > > > > > > > > KCS > > > > > > > > > > > > > > > > > > > > > > > > > > > > Caution: This message originated from an Extern= al Source. > > > > Use > > > > > proper > > > > > > > > > > > > caution > > > > > > > > > > > > > > when opening attachments, clicking links, or re= sponding. > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > Hi Chang! > > > > > > > > > > > > > > > > > > > > > > > > > > > > Thanks for the response! > > > > > > > > > > > > > > Is it ok that because of 1) we would need to in= clude MCTP > > > > > > > > > > > > > > SrcEID/DstEID in each function of the > > > > > > > > > > > > PLDM_SMBIOS_TRANSFER_PROTOCOL? > > > > > > > > > > > > > > > > > > > > > > > > > > > > https://github.com/tianocore/edk2- > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > platforms/blob/06f6274d56422084281886fad447ab117fd0e487/Features/ > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > ManageabilityPkg/Universal/PldmSmbiosTransferDxe/PldmSmbiosTransferDx > > > > > > > > > > > > > > e.c#L459 > > > > > > > > > > > > > > ``` > > > > > > > > > > > > > > EDKII_PLDM_SMBIOS_TRANSFER_PROTOCOL_V1_0 > > > > > > > > > > > > > > mPldmSmbiosTransferProtocolV10 =3D { > > > > > > > > > > > > > > GetSmbiosStructureTableMetaData, > > > > > > > > > > > > > > SetSmbiosStructureTableMetaData, > > > > > > > > > > > > > > GetSmbiosStructureTable, > > > > > > > > > > > > > > SetSmbiosStructureTable, > > > > > > > > > > > > > > GetSmbiosStructureByType, > > > > > > > > > > > > > > GetSmbiosStructureByHandle > > > > > > > > > > > > > > }; > > > > > > > > > > > > > > ``` > > > > > > > > > > > > > For the PLDM_SMBIOS_TRANSFER_PROTOCOL, we can add > > a > > > > > API in > > > > > > > > > > > > EDKII_PLDM_SMBIOS_TRANSFER_PROTOCOL_V1_0 for > > setting > > > > > the PLDM > > > > > > > > > > > > terminus. As adding Terminus STID and DTID seems a = burden > > to > > > > > this > > > > > > > > > > > protocol. > > > > > > > > > > > > > So, we can do this, > > > > > > > > > > > > > > > > > > > > > > > > > > Define SetPldmTerminus (UINT8 SourceTid, UINT8 > > > > > DestinationTid) API > > > > > > > > > and > > > > > > > > > > > > put it in the first member of > > > > > > > > > > > > EDKII_PLDM_SMBIOS_TRANSFER_PROTOCOL_V1_0. > > > > > > > > > > > > > Define PldmSetTerminus in PldmProtocolLib and inv= oked by > > > > > > > > > > > > SetPldmTerminus API, this way each PLDM application= driver > > > > (e.g., > > > > > > > > > SMBIOS) > > > > > > > > > > > > has its own PLDM terminus ID in the PldmProtocolLib > > instance. > > > > We > > > > > still > > > > > > > > > > > define > > > > > > > > > > > > PCDs for these two PLDM ID which is similar to MCTP > > protocol. > > > > > Use PCD > > > > > > > > > > > > instead in case SourceTid and DestinationTid had ne= ver been > > set > > > > in > > > > > > > > > > > > PldmProtocolLib. > > > > > > > > > > > > > > > > > > > > > > > > > > EDKII_PLDM_SMBIOS_TRANSFER_PROTOCOL_V1_0 =3D > > > > > > > > > > > > > mPldmSmbiosTransferProtocolV10 =3D { > > > > > > > > > > > > > SetPldmTerminus, > > > > > > > > > > > > > GetSmbiosStructureTableMetaData, > > > > > > > > > > > > > SetSmbiosStructureTableMetaData, > > > > > > > > > > > > > GetSmbiosStructureTable, > > > > > > > > > > > > > SetSmbiosStructureTable, > > > > > > > > > > > > > GetSmbiosStructureByType, > > > > > > > > > > > > > GetSmbiosStructureByHandle > > > > > > > > > > > > > }; > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > I've also send you pull requests with my change= s to the > > > > > > > > > > > > > > edk2/edk2-platforms repos: > > > > > > > > > > > > > > https://github.com/changab/edk2/pull/1 > > > > > > > > > > > > > > https://github.com/changab/edk2-platforms/pull/= 6 > > > > > > > > > > > > > > > > > > > > > > > > > > > > If you don't mind I would send patches in this = fashion > > before > > > > > posting > > > > > > > > > > > > > > them to the mailing list. So you can review the= m and > > provide > > > > > > > > > feedback. > > > > > > > > > > > > > I reviewed these patches just now, that looks goo= d to me. > > > > > However this > > > > > > > > > > > way > > > > > > > > > > > > we don=E2=80=99t have the history in mailing list. > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > Also I wanted to ask if you plan to work on the= required > > > > > changes > > > > > > > > > > > > > > yourself, or am I on my own from here?) > > > > > > > > > > > > > That would be good if you can help, that is also = fine for me > > to > > > > > update > > > > > > > > > the > > > > > > > > > > > > code, however it would be later next week as I have= to work > > on > > > > > some > > > > > > > > > tasks. > > > > > > > > > > > > > > > > > > > > > > > > > > BTW, are you an individual contributor or you are= the > > > > > representative of > > > > > > > > > a > > > > > > > > > > > > firm? > > > > > > > > > > > > > > > > > > > > > > > > > > Thanks > > > > > > > > > > > > > Abner > > > > > > > > > > > > > > > > > > > > > > > > > > > > Best regards, > > > > > > > > > > > > > > Konstantin Aladyshev > > > > > > > > > > > > > > > > > > > > > > > > > > > > On Wed, Oct 11, 2023 at 8:58=E2=80=AFAM Chang, = Abner > > > > > > > > > > > > > > > > > > > > > > > > > > wrote: > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > [AMD Official Use Only - General] > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > Hi Aladyshev, > > > > > > > > > > > > > > > Here is my response, > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > 1. Shouldn't we update the PLDM protocol's > > 'PldmSubmit' > > > > > function > > > > > > > > > to > > > > > > > > > > > > > > receive 'MctpSrcEID'/'MctpDestEID'. > > > > > > > > > > > > > > > Yes, I see the use case of EFI shell applicat= ion. I would like > > to > > > > > have the > > > > > > > > > > > > input > > > > > > > > > > > > > > parameters similar with MctpSubmitCommand. Use > > pointers > > > > > for the > > > > > > > > > > > given > > > > > > > > > > > > > > SEID and DEID, adopt the fix PCD value of the p= ointer is > > NULL. > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > 2. The 'PldmProtocolCommon' code contains arr= ay > > > > > ofexpected > > > > > > > > > > > response > > > > > > > > > > > > > > sizes. > > > > > > > > > > > > > > > The idea is as an high level PLDM driver, suc= h as SMBIOS, > > it > > > > > doesn't > > > > > > > > > have > > > > > > > > > > > > to > > > > > > > > > > > > > > handle the full size of PLDM message. It should= only care > > > > about > > > > > the > > > > > > > > > > > > response > > > > > > > > > > > > > > data belong to it. That's why I added the array= to define > > the > > > > > PLDM > > > > > > > > > > > > response > > > > > > > > > > > > > > size for any PLDM spec. > > > > > > > > > > > > > > > As the PLDM message and PLDM complete code is= in a > > fixe > > > > > size. I > > > > > > > > > think > > > > > > > > > > > we > > > > > > > > > > > > > > can remove the array and just adjust the respon= se size in > > > > PLDM > > > > > > > > > Protocol > > > > > > > > > > > > > > driver. > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > 3. The 'AdditionalTransferError' is designed = to > > accommodate > > > > > error > > > > > > > > > > > codes > > > > > > > > > > > > for > > > > > > > > > > > > > > the specific transport or protocol. We don=E2= =80=99t have much > > > > > > > > > implementation > > > > > > > > > > > of > > > > > > > > > > > > > > 'AdditionalTransferError' as the entire Managea= bility needs > > > > > more > > > > > > > > > > > practices > > > > > > > > > > > > to > > > > > > > > > > > > > > make it more mature. > > > > > > > > > > > > > > > I suggest we can categorize 'AdditionalTrans= ferError' > > using > > > > > the most > > > > > > > > > > > > > > significant byte, for example > > > > > > > > > > > > > > > 0x00 - Common additional transfer error. > > > > > > > > > > > > > > > 0x80 - KCS transport error > > > > > > > > > > > > > > > 0x81 - MCTP > > > > > > > > > > > > > > > ... > > > > > > > > > > > > > > > 0xC0 - IPMI protocol error > > > > > > > > > > > > > > > 0xC1 - PLDM protocol error > > > > > > > > > > > > > > > ... > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > How do you think? > > > > > > > > > > > > > > > Thanks > > > > > > > > > > > > > > > Abner > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > -----Original Message----- > > > > > > > > > > > > > > > > From: Konstantin Aladyshev > > > > > > > > > > > > > > > > > > Sent: Thursday, October 5, 2023 11:18 PM > > > > > > > > > > > > > > > > To: Chang, Abner > > > > > > > > > > > > > > > > Cc: discuss@edk2.groups.io; devel@edk2.grou= ps.io > > > > > > > > > > > > > > > > Subject: Re: [edk2-devel] [edk2-discuss] PL= DM > > messages > > > > > via > > > > > > > > > MCTP > > > > > > > > > > > > over > > > > > > > > > > > > > > KCS > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > Caution: This message originated from an Ex= ternal > > Source. > > > > > Use > > > > > > > > > proper > > > > > > > > > > > > > > caution > > > > > > > > > > > > > > > > when opening attachments, clicking links, o= r > > responding. > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > Sorry for many messages today, but I have o= ne more > > > > > observation. > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > I think the MCTP code should set specific > > > > > 'AdditionalTransferError' > > > > > > > > > > > > > > > > for the caller when it fails on each of the= response > > header > > > > > checks: > > > > > > > > > > > > > > > > https://github.com/changab/edk2- > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > platforms/blob/5af7ca1555863288bd2887ca465ad07d74b9868e/Features/ > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > ManageabilityPkg/Universal/MctpProtocol/Common/MctpProtocolCommon. > > > > > > > > > > > > > > > > c#L464 > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > But I'm not sure how 'AdditionalTransferErr= or' can be > > > > > utilized for > > > > > > > > > > > > > > > > this. Right now the errors are generic for = the > > IPMI/MCTP > > > > > protocols > > > > > > > > > > > > > > > > https://github.com/changab/edk2- > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > platforms/blob/5af7ca1555863288bd2887ca465ad07d74b9868e/Features/ > > > > > > > > > > > > > > > > > > > > > > > > > > > ManageabilityPkg/Include/Library/ManageabilityTransportLib.h#L63 > > > > > > > > > > > > > > > > Maybe it would be good to reserve some bits= for the > > > > > protocol > > > > > > > > > specific > > > > > > > > > > > > > > > > errors? What do you think? > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > Also I don't see anything like 'AdditionalT= ransferError' > > in > > > > the > > > > > PLDM > > > > > > > > > > > > > > > > protocol code. Shouldn't it also support re= porting > > > > additional > > > > > error > > > > > > > > > > > > > > > > information to the user? > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > I've also created PRs for your edk2/edk2-pl= atforms > > forks > > > > > with > > > > > > > > > some > > > > > > > > > > > > > > > > things that I've found today. > > > > > > > > > > > > > > > > https://github.com/changab/edk2/pull/1 > > > > > > > > > > > > > > > > https://github.com/changab/edk2-platforms/p= ull/6 > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > Best regards, > > > > > > > > > > > > > > > > Konstantin Aladyshev > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > On Thu, Oct 5, 2023 at 3:31=E2=80=AFPM Kons= tantin Aladyshev > > > > > > > > > > > > > > > > wrote: > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > I was thinking more about it and I think = the right > > > > solution > > > > > would > > > > > > > > > be > > > > > > > > > > > > > > > > > to move these checks to the > > > > > > > > > > > > > > EDKII_PLDM_SMBIOS_TRANSFER_PROTOCOL > > > > > > > > > > > > > > > > code > > > > > > > > > > > > > > > > > (i.e. https://github.com/changab/edk2- > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > platforms/blob/MCTP_OVER_KCS_UPDATE/Features/ManageabilityPkg/Univ > > > > > > > > > > > > > > > > > > ersal/PldmSmbiosTransferDxe/PldmSmbiosTransferDxe.c) > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > Because only the PldmSmbios protocol code= should > > > > know > > > > > such > > > > > > > > > > > > > > information > > > > > > > > > > > > > > > > > as expected response sizes for its comman= ds. > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > Best regards, > > > > > > > > > > > > > > > > > Konstantin Aladyshev > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > On Thu, Oct 5, 2023 at 3:19=E2=80=AFPM Ko= nstantin > > Aladyshev > > > > > > > > > > > > > > > > > wrote: > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > Also I see that the 'PldmProtocolCommon= ' code > > > > contains > > > > > array > > > > > > > > > of > > > > > > > > > > > > > > > > > > expected response sizes for every suppo= rted PLDM > > > > > command: > > > > > > > > > > > > > > > > > > https://github.com/changab/edk2- > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > platforms/blob/5af7ca1555863288bd2887ca465ad07d74b9868e/Features/ > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > ManageabilityPkg/Universal/PldmProtocol/Common/PldmProtocolCommon. > > > > > > > > > > > > > > > > c#L24 > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > I'm not sure that we should have this, = since we > > don't > > > > > need to > > > > > > > > > > > know > > > > > > > > > > > > the > > > > > > > > > > > > > > > > > > MCTP response size apriori. > > > > > > > > > > > > > > > > > > This only limits allowed PLDM commands = to the > > > > > supported > > > > > > > > > array: > > > > > > > > > > > > > > > > > > https://github.com/changab/edk2- > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > platforms/blob/5af7ca1555863288bd2887ca465ad07d74b9868e/Features/ > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > ManageabilityPkg/Universal/PldmProtocol/Common/PldmProtocolCommon. > > > > > > > > > > > > > > > > c#L261 > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > This means that right now I can't execu= te my simple > > > > > > > > > > > 'GetPLDMTypes' > > > > > > > > > > > > > > > > > > command through the 'PldmSubmit' protoc= ol > > > > function. > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > Best regards, > > > > > > > > > > > > > > > > > > Konstantin Aladyshev > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > On Thu, Oct 5, 2023 at 12:55=E2=80=AFPM= Konstantin > > > > Aladyshev > > > > > > > > > > > > > > > > > > wrote: > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > Shouldn't we update the PLDM protocol= 's > > > > > 'PldmSubmit' > > > > > > > > > function > > > > > > > > > > > > to > > > > > > > > > > > > > > > > > > > receive 'MctpSrcEID'/'MctpDestEID' as= incoming > > > > > parameters? > > > > > > > > > Or > > > > > > > > > > > > > > maybe > > > > > > > > > > > > > > > > > > > add some 'PldmInit' function to set t= hose > > parameters > > > > > for > > > > > > > > > further > > > > > > > > > > > > > > PLDM > > > > > > > > > > > > > > > > > > > communication? > > > > > > > > > > > > > > > > > > > Because right now the MCTP EIDs are f= ixed to > > PCDs > > > > > with no > > > > > > > > > > > > flexibility > > > > > > > > > > > > > > > > > > > https://github.com/tianocore/edk2- > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > platforms/blob/d6e36a151ff8365cdc55a6914cc5e6138d5788dc/Features/ > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > ManageabilityPkg/Universal/PldmProtocol/Common/PldmProtocolCommon. > > > > > > > > > > > > > > > > c#L121 > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > Best regards, > > > > > > > > > > > > > > > > > > > Konstantin Aladyshev > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > On Thu, Oct 5, 2023 at 7:03=E2=80=AFA= M Chang, Abner > > > > > > > > > > > > > > > > wrote: > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > [AMD Official Use Only - General] > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > -----Original Message----- > > > > > > > > > > > > > > > > > > > > > From: discuss@edk2.groups.io > > > > > > > > > > > > > > > > On > > > > > > > > > > > > > > Behalf > > > > > > > > > > > > > > > > Of > > > > > > > > > > > > > > > > > > > > > Konstantin Aladyshev via groups.i= o > > > > > > > > > > > > > > > > > > > > > Sent: Thursday, October 5, 2023 1= :57 AM > > > > > > > > > > > > > > > > > > > > > To: Chang, Abner > > > > > > > > > > > > > > > > > > > > > Cc: devel@edk2.groups.io; > > > > > discuss@edk2.groups.io > > > > > > > > > > > > > > > > > > > > > Subject: Re: [edk2-devel] [edk2-d= iscuss] > > PLDM > > > > > messages > > > > > > > > > via > > > > > > > > > > > > > > MCTP > > > > > > > > > > > > > > > > over KCS > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > Caution: This message originated = from an > > External > > > > > Source. > > > > > > > > > > > Use > > > > > > > > > > > > > > proper > > > > > > > > > > > > > > > > caution > > > > > > > > > > > > > > > > > > > > > when opening attachments, clickin= g links, or > > > > > responding. > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > That is great, and I'm surprise= d there are > > some > > > > > build > > > > > > > > > errors > > > > > > > > > > > at > > > > > > > > > > > > > > your > > > > > > > > > > > > > > > > end. > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > I'm surprised your compiler didn'= t catch that > > since > > > > it > > > > > is all > > > > > > > > > basic > > > > > > > > > > > > > > > > > > > > > syntax errors. > > > > > > > > > > > > > > > > > > > > > I've used your > > > > https://github.com/changab/edk2- > > > > > > > > > > > > > > > > > > > > > platforms/tree/MCTP_OVER_KCS_UPDA= TE > > > > > > > > > > > > > > > > > > > > > directly. > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > Ah I know why, I forget to rebuild = the changes > > of > > > > > both PEC > > > > > > > > > and > > > > > > > > > > > > > > MCTP > > > > > > > > > > > > > > > > EID after I verifying the functionality of = IPMI on the > > new > > > > > > > > > > > > KbcCommonLib.c. > > > > > > > > > > > > > > > > Yes, I do see the build error now and was f= ixed at my > > end. > > > > > My fault. > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > How do you think we just send i= t to the > > mailing > > > > > list for > > > > > > > > > > > review > > > > > > > > > > > > > > and > > > > > > > > > > > > > > > > keep > > > > > > > > > > > > > > > > > > > > > working on other problems based o= n it.? > > > > > > > > > > > > > > > > > > > > > > Could you please send the patch= es out, with > > > > you > > > > > as the > > > > > > > > > > > > author > > > > > > > > > > > > > > and > > > > > > > > > > > > > > > > I'm the > > > > > > > > > > > > > > > > > > > > > coauthor? I will review it again= on dev mailing > > list. > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > No problem, I can send a patch to= the 'edk2- > > > > devel' > > > > > mailing > > > > > > > > > > > list. > > > > > > > > > > > > > > > > > > > > > But before that I think I'll writ= e a test app to > > check > > > > if > > > > > PLDM > > > > > > > > > > > > > > > > > > > > > protocols work correctly. > > > > > > > > > > > > > > > > > > > > Ok. > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > Also earlier I've pointed to a fa= ct that > > > > > > > > > 'MctpSourceEndpointId' > > > > > > > > > > > > and > > > > > > > > > > > > > > > > > > > > > 'MctpDestinationEndpointId' aren'= t actually > > used > > > > in > > > > > the > > > > > > > > > > > > > > > > > > > > > 'MctpSubmitMessage' function. > > > > > > > > > > > > > > > > > > > > > EIDs are always taken from the PC= Ds: > > > > > > > > > > > > > > > > > > > > > https://github.com/changab/edk2- > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > platforms/blob/1c8d0d3fa403b47a34667f7f690add7822163111/Features/ > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > ManageabilityPkg/Universal/MctpProtocol/Common/MctpProtocolCommon. > > > > > > > > > > > > > > > > > > > > > c#L178 > > > > > > > > > > > > > > > > > > > > > What can we do about that? > > > > > > > > > > > > > > > > > > > > Ah yes, we should update the algori= thm, it is > > done > > > > > here: > > > > > > > > > > > > > > > > https://github.com/changab/edk2- > > > > > > > > > > > > > > > > platforms/tree/MCTP_OVER_KCS_UPDATE. You ha= ve > > to > > > > > update > > > > > > > > > your > > > > > > > > > > > > code > > > > > > > > > > > > > > > > here: > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > https://github.com/Kostr/PLDM/blob/master/PldmMessage/PldmMessage.c > > > > > > > > > > > > > > > > > > > > And we also need the fix the typo o= n edk2, > > > > > > > > > > > > > > > > > > > > > > > > > > > https://github.com/changab/edk2/tree/MCTP_OVER_KCS_UPDATE > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > Btw, how long do you think I wo= uld take to > > > > merge > > > > > your > > > > > > > > > > > > changes > > > > > > > > > > > > > > on > > > > > > > > > > > > > > > > > > > > > openBMC? > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > So as I've described in the > > > > > > > > > > > > > > > > https://github.com/Kostr/PLDM/tree/master > > > > > > > > > > > > > > > > > > > > > there are basically 2 approaches = for the MCTP > > > > stack > > > > > in > > > > > > > > > > > > OpenBMC: > > > > > > > > > > > > > > > > > > > > > (1) userspace approach (legacy, s= houldn't be > > > > really > > > > > used > > > > > > > > > now) > > > > > > > > > > > > > > > > > > > > > (2) kernel approach > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > It is hard to tell if OpenBMC pat= ches for the (1) > > > > > approach > > > > > > > > > will > > > > > > > > > > > be > > > > > > > > > > > > > > > > > > > > > merged. Since I've developed the = Linux kernel > > > > driver > > > > > (2) > > > > > > > > > > > nobody > > > > > > > > > > > > > > really > > > > > > > > > > > > > > > > > > > > > cares about (1). > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > For the (2) there are a couple of= OpenBMC > > > > patches > > > > > which > > > > > > > > > I've > > > > > > > > > > > > > > helped > > > > > > > > > > > > > > > > to > > > > > > > > > > > > > > > > > > > > > develop, but I'm just a coathor i= n them. So it is > > > > hard > > > > > for > > > > > > > > > me to > > > > > > > > > > > > tell > > > > > > > > > > > > > > > > > > > > > when they would be merged. For me= it looks > > like > > > > > they are > > > > > > > > > > > > mostly > > > > > > > > > > > > > > > > ready: > > > > > > > > > > > > > > > > > > > > > 66591: transport: af-mctp: Add > > > > > > > > > > > > pldm_transport_af_mctp_bind() | > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > https://gerrit.openbmc.org/c/openbmc/libpldm/+/66591 > > > > > > > > > > > > > > > > > > > > > 63652: pldm: Convert to using lib= pldm > > transport > > > > > APIs | > > > > > > > > > > > > > > > > > > > > > > > > > > https://gerrit.openbmc.org/c/openbmc/pldm/+/63652 > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > For the (2) I also need to push t= he mctp Linux > > > > kernel > > > > > driver > > > > > > > > > > > > > > upstream. > > > > > > > > > > > > > > > > > > > > > Linux kernel development is not w= hat I do > > every > > > > > day, so > > > > > > > > > I'm > > > > > > > > > > > not > > > > > > > > > > > > > > sure > > > > > > > > > > > > > > > > > > > > > how long it would take. But I'm p= retty > > determined > > > > > to > > > > > > > > > finish > > > > > > > > > > > the > > > > > > > > > > > > > > work > > > > > > > > > > > > > > > > > > > > > and push my driver upstream. Curr= ently there > > are > > > > > some > > > > > > > > > > > > questions > > > > > > > > > > > > > > > > > > > > > regarding Linux KCS subsystem, so= along with > > the > > > > > KCS > > > > > > > > > > > > subsystem > > > > > > > > > > > > > > > > creator > > > > > > > > > > > > > > > > > > > > > we have to figure out how to rewr= ite the > > > > > subsystem > > > > > > > > > correctly. > > > > > > > > > > > > So > > > > > > > > > > > > > > this > > > > > > > > > > > > > > > > > > > > > can take some time. > > > > > > > > > > > > > > > > > > > > > After the code is pushed to the t= orvalds/linux, > > it > > > > > would be > > > > > > > > > > > > picked > > > > > > > > > > > > > > up > > > > > > > > > > > > > > > > > > > > > by the openbmc/linux automaticall= y. > > > > > > > > > > > > > > > > > > > > Ok, I got it. Thanks for the detail= ed information. > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > Regards, > > > > > > > > > > > > > > > > > > > > Abner > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > Best regards, > > > > > > > > > > > > > > > > > > > > > Konstantin Aladyshev > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > On Wed, Oct 4, 2023 at 7:12=E2=80= =AFPM Chang, > > Abner > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > wrote: > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > [AMD Official Use Only - Genera= l] > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > That is great, and I'm surprise= d there are > > some > > > > > build > > > > > > > > > errors > > > > > > > > > > > at > > > > > > > > > > > > > > your > > > > > > > > > > > > > > > > end. > > > > > > > > > > > > > > > > > > > > > > How do you think we just send i= t to the > > mailing > > > > > list for > > > > > > > > > > > review > > > > > > > > > > > > > > and > > > > > > > > > > > > > > > > keep > > > > > > > > > > > > > > > > > > > > > working on other problems based o= n it.? > > > > > > > > > > > > > > > > > > > > > > Could you please send the patch= es out, with > > > > you > > > > > as the > > > > > > > > > > > > author > > > > > > > > > > > > > > and > > > > > > > > > > > > > > > > I'm the > > > > > > > > > > > > > > > > > > > > > coauthor? I will review it again= on dev mailing > > list. > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > I will take a look on kernal ch= ange. Btw, how > > long > > > > > do you > > > > > > > > > > > think > > > > > > > > > > > > I > > > > > > > > > > > > > > > > would take > > > > > > > > > > > > > > > > > > > > > to merge your changes on openBMC? > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > Thanks > > > > > > > > > > > > > > > > > > > > > > Abner > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > Get Outlook for Android > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > _______________________________= _ > > > > > > > > > > > > > > > > > > > > > > From: Konstantin Aladyshev > > > > > > > > > > > > > > > > > > > > > > > > > > > Sent: Wednesday, October 4, 202= 3 11:59:16 > > > > PM > > > > > > > > > > > > > > > > > > > > > > To: Chang, Abner > > > > > > > > > > > > > > > > > > > > > > Cc: devel@edk2.groups.io > > > > > ; > > > > > > > > > > > > > > > > > > > > > discuss@edk2.groups.io > > > > > > > > > > > > > > > > > > > > > > > > > > > Subject: Re: [edk2-devel] [edk2= -discuss] > > PLDM > > > > > > > > > messages > > > > > > > > > > > via > > > > > > > > > > > > > > MCTP > > > > > > > > > > > > > > > > over KCS > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > Caution: This message originate= d from an > > > > External > > > > > > > > > Source. > > > > > > > > > > > > Use > > > > > > > > > > > > > > > > proper > > > > > > > > > > > > > > > > > > > > > caution when opening attachments,= clicking > > links, > > > > > or > > > > > > > > > > > > responding. > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > Hi Chang! > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > Thanks! > > > > > > > > > > > > > > > > > > > > > > There were a couple of trivial = compilation > > errors, > > > > > but > > > > > > > > > after > > > > > > > > > > > I've > > > > > > > > > > > > > > > > > > > > > > fixed them everything seems to = work fine! > > > > > > > > > > > > > > > > > > > > > > Just in case I've tested the Op= enBMC side > > with > > > > the > > > > > mctp > > > > > > > > > > > Linux > > > > > > > > > > > > > > kernel > > > > > > > > > > > > > > > > > > > > > > driver approach > > > > > > > > > > > > > > > > > > > > > > > > > > > (https://github.com/Kostr/PLDM/tree/master/mctp- > > > > > > > > > > > kernel) > > > > > > > > > > > > > > > > > > > > > > The latest kernel patches can b= e found here: > > > > > > > > > > > > > > > > > > > > > > > > > > > https://lore.kernel.org/lkml/20231003131505.337-1- > > > > > > > > > > > > > > > > > > > > > aladyshev22@gmail.com/ > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > Here is a fix for the build err= ors that I've > > found: > > > > > > > > > > > > > > > > > > > > > > ``` > > > > > > > > > > > > > > > > > > > > > > diff --git > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > a/Features/ManageabilityPkg/Universal/MctpProtocol/Common/MctpProtoc > > > > > > > > > > > > > > > > > > > > > olCommon.c > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > b/Features/ManageabilityPkg/Universal/MctpProtocol/Common/MctpProto > > > > > > > > > > > > > > > > > > > > > colCommon.c > > > > > > > > > > > > > > > > > > > > > > index 79501d27aa..345c6da81a 10= 0644 > > > > > > > > > > > > > > > > > > > > > > --- > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > a/Features/ManageabilityPkg/Universal/MctpProtocol/Common/MctpProtoc > > > > > > > > > > > > > > > > > > > > > olCommon.c > > > > > > > > > > > > > > > > > > > > > > +++ > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > b/Features/ManageabilityPkg/Universal/MctpProtocol/Common/MctpProto > > > > > > > > > > > > > > > > > > > > > colCommon.c > > > > > > > > > > > > > > > > > > > > > > @@ -193,7 +193,7 @@ > > > > > > > > > SetupMctpRequestTransportPacket > > > > > > > > > > > ( > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > // > > > > > > > > > > > > > > > > > > > > > > // Generate PEC follow SMB= US 2.0 > > > > > specification. > > > > > > > > > > > > > > > > > > > > > > - *MctpKcsTrailer->Pec =3D > > > > > > > > > > > HelperManageabilityGenerateCrc8 > > > > > > > > > > > > > > > > > > > > > > (MCTP_KCS_PACKET_ERROR_CODE_POL= Y, 0, > > > > > > > > > ThisPackage, > > > > > > > > > > > > > > > > > > > > > > MctpKcsHeader->ByteCount); > > > > > > > > > > > > > > > > > > > > > > + MctpKcsTrailer->Pec =3D > > > > > > > > > HelperManageabilityGenerateCrc8 > > > > > > > > > > > > > > > > > > > > > > (MCTP_KCS_PACKET_ERROR_CODE_POL= Y, 0, > > > > > > > > > ThisPackage, > > > > > > > > > > > > > > > > > > > > > > MctpKcsHeader->ByteCount); > > > > > > > > > > > > > > > > > > > > > > *PacketBody =3D (UI= NT8 *)ThisPackage; > > > > > > > > > > > > > > > > > > > > > > *PacketBodySize =3D Mct= pKcsHeader- > > > > > >ByteCount; > > > > > > > > > > > > > > > > > > > > > > *PacketTrailer =3D > > > > > > > > > > > > > > > > > > > > > > > > > > (MANAGEABILITY_TRANSPORT_TRAILER)MctpKcsTrailer; > > > > > > > > > > > > > > > > > > > > > > diff --git > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > a/Features/ManageabilityPkg/Universal/MctpProtocol/Dxe/MctpProtocol.c > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > b/Features/ManageabilityPkg/Universal/MctpProtocol/Dxe/MctpProtocol.c > > > > > > > > > > > > > > > > > > > > > > index 863b8d471c..247d032b9b 10= 0644 > > > > > > > > > > > > > > > > > > > > > > --- > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > a/Features/ManageabilityPkg/Universal/MctpProtocol/Dxe/MctpProtocol.c > > > > > > > > > > > > > > > > > > > > > > +++ > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > b/Features/ManageabilityPkg/Universal/MctpProtocol/Dxe/MctpProtocol.c > > > > > > > > > > > > > > > > > > > > > > @@ -79,17 +79,17 @@ > > MctpSubmitMessage ( > > > > > > > > > > > > > > > > > > > > > > } > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > // > > > > > > > > > > > > > > > > > > > > > > - // Chec source EID and desti= nation EDI. > > > > > > > > > > > > > > > > > > > > > > + // Check source EID and dest= ination EID > > > > > > > > > > > > > > > > > > > > > > // > > > > > > > > > > > > > > > > > > > > > > if ((MctpSourceEndpointId >= =3D > > > > > > > > > > > > > > > > MCTP_RESERVED_ENDPOINT_START_ID) && > > > > > > > > > > > > > > > > > > > > > > - MctpSourceEndpointId <= =3D > > > > > > > > > > > > > > > > MCTP_RESERVED_ENDPOINT_END_ID) > > > > > > > > > > > > > > > > > > > > > > + (MctpSourceEndpointId <= =3D > > > > > > > > > > > > > > > > MCTP_RESERVED_ENDPOINT_END_ID) > > > > > > > > > > > > > > > > > > > > > > ) { > > > > > > > > > > > > > > > > > > > > > > DEBUG ((DEBUG_ERROR, "%a: = The value > > of > > > > > MCTP > > > > > > > > > source > > > > > > > > > > > > EID > > > > > > > > > > > > > > (%x) > > > > > > > > > > > > > > > > is > > > > > > > > > > > > > > > > > > > > > > reserved.\n", func, > > MctpSourceEndpointId)); > > > > > > > > > > > > > > > > > > > > > > return EFI_INVALID_PARAMET= ER; > > > > > > > > > > > > > > > > > > > > > > } > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > if ((MctpDestinationEndpoint= Id >=3D > > > > > > > > > > > > > > > > > > > > > MCTP_RESERVED_ENDPOINT_START_ID) = && > > > > > > > > > > > > > > > > > > > > > > - MctpDestinationEndpoint= Id <=3D > > > > > > > > > > > > > > > > MCTP_RESERVED_ENDPOINT_END_ID) > > > > > > > > > > > > > > > > > > > > > > + (MctpDestinationEndpoint= Id <=3D > > > > > > > > > > > > > > > > MCTP_RESERVED_ENDPOINT_END_ID) > > > > > > > > > > > > > > > > > > > > > > ) { > > > > > > > > > > > > > > > > > > > > > > DEBUG ((DEBUG_ERROR, "%a: = The value > > of > > > > > MCTP > > > > > > > > > > > > destination > > > > > > > > > > > > > > EID > > > > > > > > > > > > > > > > (%x) > > > > > > > > > > > > > > > > > > > > > > is reserved.\n", func, > > > > > MctpDestinationEndpointId)); > > > > > > > > > > > > > > > > > > > > > > return EFI_INVALID_PARAMET= ER; > > > > > > > > > > > > > > > > > > > > > > ``` > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > Best regards, > > > > > > > > > > > > > > > > > > > > > > Konstantin Aladyshev > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > On Wed, Oct 4, 2023 at 2:52=E2= =80=AFPM Chang, > > Abner > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > wrote: > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > [AMD Official Use Only - Gene= ral] > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > Hi Aladyshev, > > > > > > > > > > > > > > > > > > > > > > > I have updated the change you= made and > > put > > > > > those > > > > > > > > > code > > > > > > > > > > > on > > > > > > > > > > > > > > below > > > > > > > > > > > > > > > > link, > > > > > > > > > > > > > > > > > > > > > > > https://github.com/changab/ed= k2- > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > platforms/commit/1c8d0d3fa403b47a34667f7f690add7822163111 > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > I combined MCTP over KCS chan= ges and > > IPMI > > > > > over KCS > > > > > > > > > > > > > > > > functionality in > > > > > > > > > > > > > > > > > > > > > KcsCommonLib.c. I also created > > > > > > > > > > > > > > > > MANAGEABILITY_MCTP_KCS_TRAILER as you > > > > > > > > > > > > > > > > > > > > > suggested. The source EID and des= tination EID > > are > > > > > checked > > > > > > > > > in > > > > > > > > > > > > > > > > > > > > > MctpSubmitCommand as well. IPMI/K= CS > > > > > functionality is > > > > > > > > > > > > verified > > > > > > > > > > > > > > and > > > > > > > > > > > > > > > > works > > > > > > > > > > > > > > > > > > > > > fine after this change. > > > > > > > > > > > > > > > > > > > > > > > As I am no able to use the co= rresponding > > > > change > > > > > you > > > > > > > > > > > made > > > > > > > > > > > > on > > > > > > > > > > > > > > > > OpenBMC > > > > > > > > > > > > > > > > > > > > > site at my end, could you please = help to verify > > my > > > > > updates > > > > > > > > > on > > > > > > > > > > > > your > > > > > > > > > > > > > > > > machine? > > > > > > > > > > > > > > > > > > > > > Let's see how it works. > > > > > > > > > > > > > > > > > > > > > > > I also consider to migrate th= e code that > > > > > generates > > > > > > > > > MCTP > > > > > > > > > > > > over > > > > > > > > > > > > > > KCS > > > > > > > > > > > > > > > > > > > > > header/trailer from MctpProtocolC= ommon.c > > to > > > > > > > > > > > > KcsCommonLib.c, > > > > > > > > > > > > > > > > maybe after > > > > > > > > > > > > > > > > > > > > > we verifying PLDM->MCTP->KCS rout= e works > > well > > > > > on > > > > > > > > > > > > > > > > ManageabilityPkg. > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > Thank you > > > > > > > > > > > > > > > > > > > > > > > Abner > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > -----Original Message----- > > > > > > > > > > > > > > > > > > > > > > > > From: Konstantin Aladyshev > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > Sent: Friday, September 29,= 2023 2:18 > > AM > > > > > > > > > > > > > > > > > > > > > > > > To: Chang, Abner > > > > > > > > > > > > > > > > > > > > > > > > > > Cc: devel@edk2.groups.io; > > > > > discuss@edk2.groups.io > > > > > > > > > > > > > > > > > > > > > > > > Subject: Re: [edk2-devel] [= edk2-discuss] > > > > > PLDM > > > > > > > > > > > messages > > > > > > > > > > > > via > > > > > > > > > > > > > > > > MCTP over > > > > > > > > > > > > > > > > > > > > > KCS > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > Caution: This message origi= nated from an > > > > > External > > > > > > > > > > > Source. > > > > > > > > > > > > > > Use > > > > > > > > > > > > > > > > proper > > > > > > > > > > > > > > > > > > > > > caution > > > > > > > > > > > > > > > > > > > > > > > > when opening attachments, c= licking links, > > or > > > > > > > > > responding. > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > Hi, Chang! > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > Did you have time to test l= ibmctp MCTP > > KCS > > > > > binding > > > > > > > > > > > > solution? > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > Here are some updates from = my end. As I > > > > was > > > > > saying, > > > > > > > > > I > > > > > > > > > > > > was > > > > > > > > > > > > > > > > working on > > > > > > > > > > > > > > > > > > > > > > > > the Linux kernel binding so= lution. > > > > > > > > > > > > > > > > > > > > > > > > And now I've finished the i= nitial > > > > > implementation of > > > > > > > > > the > > > > > > > > > > > > Linux > > > > > > > > > > > > > > > > kernel > > > > > > > > > > > > > > > > > > > > > > > > binding driver for the MCTP= -over-KCS > > > > binding > > > > > and > > > > > > > > > > > > proposed > > > > > > > > > > > > > > all > > > > > > > > > > > > > > > > the > > > > > > > > > > > > > > > > > > > > > > > > patches upstream > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > (https://www.spinics.net/lists/kernel/msg4949173.ht= ml). > > > > > > > > > > > > > > > > > > > > > > > > I've also updated instructi= ons in my repo > > > > > > > > > > > > > > > > > > > > > > > > https://github.com/Kostr/PL= DM (the > > guide > > > > > for the > > > > > > > > > > > kernel > > > > > > > > > > > > > > > > binding > > > > > > > > > > > > > > > > > > > > > > > > solution and all the necess= ary Linux kernel > > > > > patches > > > > > > > > > can > > > > > > > > > > > be > > > > > > > > > > > > > > found > > > > > > > > > > > > > > > > here > > > > > > > > > > > > > > > > > > > > > > > > > > > > > https://github.com/Kostr/PLDM/tree/master/mctp- > > > > > > > > > > > > kernel). > > > > > > > > > > > > > > > > > > > > > > > > So now you can use Linux dr= iver instead > > of > > > > the > > > > > > > > > libmctp > > > > > > > > > > > > utility > > > > > > > > > > > > > > on > > > > > > > > > > > > > > > > the BMC > > > > > > > > > > > > > > > > > > > > > side. > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > Couple of things that I've = noticed in the > > > > > development > > > > > > > > > > > > > > process: > > > > > > > > > > > > > > > > > > > > > > > > - `MctpSubmitCommand` recei= ves > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > 'MctpSourceEndpointId'/'MctpDestinationEndpointId' > > > > > > > > > > > as > > > > > > > > > > > > > > > > arguments. But > > > > > > > > > > > > > > > > > > > > > > > > these values aren't actuall= y used. The > > current > > > > > code > > > > > > > > > just > > > > > > > > > > > > uses > > > > > > > > > > > > > > EIDs > > > > > > > > > > > > > > > > > > > > > > > > that were set via PCDs > > > > > > > > > > > > > > > > > > > > > > > > (https://github.com/tianoco= re/edk2- > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > platforms/blob/d03a60523a6086d200d3eb1e2f25530bf1cb790e/Features/ > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > ManageabilityPkg/Universal/MctpProtocol/Common/MctpProtocolCommon. > > > > > > > > > > > > > > > > > > > > > > > > c#L178) > > > > > > > > > > > > > > > > > > > > > > > > - According to the specific= ation DSP0236 > > > > > (section > > > > > > > > > 8.2) > > > > > > > > > > > > MCTP > > > > > > > > > > > > > > EID > > > > > > > > > > > > > > > > 0 to 7 > > > > > > > > > > > > > > > > > > > > > > > > are reserved. It is critica= l that we do not > > use > > > > > them > > > > > > > > > since > > > > > > > > > > > > MCTP > > > > > > > > > > > > > > > > Linux > > > > > > > > > > > > > > > > > > > > > > > > kernel subsystem checks tha= t part. So we > > > > > probably > > > > > > > > > need > > > > > > > > > > > > to > > > > > > > > > > > > > > add > > > > > > > > > > > > > > > > some > > > > > > > > > > > > > > > > > > > > > > > > check to the `MctpSubmitCom= mand` that > > > > > would > > > > > > > > > verify > > > > > > > > > > > > that > > > > > > > > > > > > > > we > > > > > > > > > > > > > > > > don't use > > > > > > > > > > > > > > > > > > > > > > > > reserved EIDs. > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > Best regards, > > > > > > > > > > > > > > > > > > > > > > > > Konstantin Aladyshev > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > On Thu, Sep 21, 2023 at 5:3= 2=E2=80=AFAM Chang, > > > > > Abner > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > wrote: > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > [AMD Official Use Only - = General] > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > Hi Aladyshev, > > > > > > > > > > > > > > > > > > > > > > > > > Thanks for providing the = details, I will > > take a > > > > > look at > > > > > > > > > > > your > > > > > > > > > > > > > > code > > > > > > > > > > > > > > > > first, > > > > > > > > > > > > > > > > > > > > > > > > implement it at my end and = then > > response to > > > > > your > > > > > > > > > > > > question. > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > Abner > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > -----Original Message--= --- > > > > > > > > > > > > > > > > > > > > > > > > > > From: Konstantin Aladys= hev > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > Sent: Friday, September= 8, 2023 8:57 > > PM > > > > > > > > > > > > > > > > > > > > > > > > > > To: Chang, Abner > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > Cc: devel@edk2.groups.i= o; > > > > > > > > > discuss@edk2.groups.io > > > > > > > > > > > > > > > > > > > > > > > > > > Subject: Re: [edk2-deve= l] [edk2- > > discuss] > > > > > PLDM > > > > > > > > > > > > messages > > > > > > > > > > > > > > via > > > > > > > > > > > > > > > > MCTP > > > > > > > > > > > > > > > > > > > > > over > > > > > > > > > > > > > > > > > > > > > > > > KCS > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > Caution: This message o= riginated from > > an > > > > > External > > > > > > > > > > > > Source. > > > > > > > > > > > > > > > > Use proper > > > > > > > > > > > > > > > > > > > > > > > > caution > > > > > > > > > > > > > > > > > > > > > > > > > > when opening attachment= s, clicking > > links, > > > > > or > > > > > > > > > > > > responding. > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > Hi, Chang! > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > I've finished my initia= l implementation > > of > > > > > the > > > > > > > > > MCTP > > > > > > > > > > > > over > > > > > > > > > > > > > > KCS > > > > > > > > > > > > > > > > binding. > > > > > > > > > > > > > > > > > > > > > > > > > > You can find 'edk2-plat= form' patches > > and > > > > > > > > > 'openbmc' > > > > > > > > > > > > > > patches > > > > > > > > > > > > > > > > along > > > > > > > > > > > > > > > > > > > > > with > > > > > > > > > > > > > > > > > > > > > > > > > > all of the instructions= in my repository > > > > > > > > > > > > > > > > > > > > > > > > > > https://github.com/Kost= r/PLDM. I > > hope > > > > > you'll be > > > > > > > > > > > able > > > > > > > > > > > > to > > > > > > > > > > > > > > > > reproduce > > > > > > > > > > > > > > > > > > > > > > > > > > everything on your hard= ware > > > > > configuration. Feel > > > > > > > > > free > > > > > > > > > > > > to > > > > > > > > > > > > > > ask > > > > > > > > > > > > > > > > any > > > > > > > > > > > > > > > > > > > > > > > > > > questions. > > > > > > > > > > > > > > > > > > > > > > > > > > Also I've sent all the = openbmc patches > > > > > upstream, > > > > > > > > > > > hope > > > > > > > > > > > > > > they > > > > > > > > > > > > > > > > will get > > > > > > > > > > > > > > > > > > > > > > > > > > accepted soon. > > > > > > > > > > > > > > > > > > > > > > > > > > As for the 'edk2-platfo= rm' patches, > > right > > > > > now I > > > > > > > > > don't > > > > > > > > > > > > fully > > > > > > > > > > > > > > > > understand > > > > > > > > > > > > > > > > > > > > > > > > > > how to write them corre= ctly to keep > > IPMI > > > > > over > > > > > > > > > KCS > > > > > > > > > > > > stack > > > > > > > > > > > > > > > > working. I > > > > > > > > > > > > > > > > > > > > > > > > > > think here I would need= your help. > > Right > > > > > now I've > > > > > > > > > > > > > > commited > > > > > > > > > > > > > > > > them to > > > > > > > > > > > > > > > > > > > > > my > > > > > > > > > > > > > > > > > > > > > > > > > > `edk2-platforms` fork > > > > > > > > > > > > > > > > > > > > > > > > > > https://github.com/tian= ocore/edk2- > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > platforms/commit/99a6c98a63b37f955c0d0480149b84fcc3a03f74 > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > Couple of questions/not= ices: > > > > > > > > > > > > > > > > > > > > > > > > > > 1) You've said that we = can > > differentiate > > > > > MCTP by > > > > > > > > > the > > > > > > > > > > > > > > transfer > > > > > > > > > > > > > > > > token, > > > > > > > > > > > > > > > > > > > > > > > > > > but it is not passed to= the > > > > > > > > > > > > 'KcsTransportSendCommand' > > > > > > > > > > > > > > > > function > > > > > > > > > > > > > > > > > > > > > > > > > > https://github.com/tian= ocore/edk2- > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > platforms/blob/bb6841e3fd1c60b3f8510b4fc0a380784e05d326/Features/ > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > ManageabilityPkg/Library/ManageabilityTransportKcsLib/Common/KcsComm > > > > > > > > > > > > > > > > > > > > > > > > > > on.c#L414 > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > 2) What function should= know about > > the > > > > > > > > > > > > > > > > > > > > > > > > > > > > MANAGEABILITY_MCTP_KCS_HEADER? > > > > > > > > > > > > > > > > > > > > > > > > > > Keep in mind that this = header includes > > > > > > > > > 'ByteCount' > > > > > > > > > > > for > > > > > > > > > > > > the > > > > > > > > > > > > > > > > incoming > > > > > > > > > > > > > > > > > > > > > > > > > > data size that we need = to read. > > > > > > > > > > > > > > > > > > > > > > > > > > - KcsTransportSendComma= nd or > > > > > > > > > > > > > > > > CommonMctpSubmitMessage ? > > > > > > > > > > > > > > > > > > > > > > > > > > (https://github.com/tia= nocore/edk2- > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > platforms/blob/master/Features/ManageabilityPkg/Library/ManageabilityTr= a > > > > > > > > > > > > > > > > > > > > > > > > > > > > nsportKcsLib/Common/KcsCommon.c) > > > > > > > > > > > > > > > > > > > > > > > > > > (https://github.com/tia= nocore/edk2- > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > platforms/blob/master/Features/ManageabilityPkg/Universal/MctpProtocol/ > > > > > > > > > > > > > > > > > > > > > > > > > > Common/MctpProtocolComm= on.c)? > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > 3) As I've said earlier= I think it would > > be > > > > > good to > > > > > > > > > add > > > > > > > > > > > > > > > > > > > > > > > > > > MANAGEABILITY_MCTP_KCS_= TRAILER > > to > > > > > the > > > > > > > > > Mctp.h > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > 4) Not sure if it is a = good idea to pass > > > > these > > > > > > > > > > > parameters > > > > > > > > > > > > to > > > > > > > > > > > > > > the > > > > > > > > > > > > > > > > > > > > > > > > > > MctpSubmitCommand proto= col > > function: > > > > > > > > > > > > > > > > > > > > > > > > > > ``` > > > > > > > > > > > > > > > > > > > > > > > > > > UINT8 Mc= tpType, > > > > > > > > > > > > > > > > > > > > > > > > > > BOOLEAN > > > > RequestDataIntegrityCheck, > > > > > > > > > > > > > > > > > > > > > > > > > > ``` > > > > > > > > > > > > > > > > > > > > > > > > > > (https://github.com/tia= nocore/edk2- > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > platforms/blob/master/Features/ManageabilityPkg/Include/Protocol/MctpPr > > > > > > > > > > > > > > > > > > > > > > > > > > otocol.h) > > > > > > > > > > > > > > > > > > > > > > > > > > Shouldn't it be in the = `RequestData` > > > > directly > > > > > since > > > > > > > > > it is > > > > > > > > > > > > more > > > > > > > > > > > > > > of > > > > > > > > > > > > > > > > a > > > > > > > > > > > > > > > > > > > > > > > > > > payload than a header f= or the MCTP? I > > > > > don't > > > > > > > > > know > > > > > > > > > > > the > > > > > > > > > > > > > > > > specification > > > > > > > > > > > > > > > > > > > > > > > > > > very well, but what if = the > > > > > > > > > RequestDataIntegrityCheck > > > > > > > > > > > > > > would > > > > > > > > > > > > > > > > be set in > > > > > > > > > > > > > > > > > > > > > > > > > > the response? Who would= need to > > check > > > > > the > > > > > > > > > integrity > > > > > > > > > > > > of > > > > > > > > > > > > > > the > > > > > > > > > > > > > > > > payload > > > > > > > > > > > > > > > > > > > > > > > > > > buffer in that case? MC= TP library or > > the > > > > > code > > > > > > > > > calling > > > > > > > > > > > the > > > > > > > > > > > > > > MCTP > > > > > > > > > > > > > > > > > > > > > > > > > > library? > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > 5) Haven't tested the P= ldmProtocol, > > > > maybe > > > > > it also > > > > > > > > > > > > needs > > > > > > > > > > > > > > > > some > > > > > > > > > > > > > > > > > > > > > corrections. > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > Feel free to ask any qu= estions about > > my > > > > > solution. > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > Right now I'll probably= focus on the > > Linux > > > > > kernel > > > > > > > > > > > driver > > > > > > > > > > > > for > > > > > > > > > > > > > > the > > > > > > > > > > > > > > > > MCTP > > > > > > > > > > > > > > > > > > > > > > > > > > over KCS binding. So if= you want to > > finish > > > > > edk2- > > > > > > > > > > > > platforms > > > > > > > > > > > > > > > > code based > > > > > > > > > > > > > > > > > > > > > > > > > > on my patches, feel fre= e to do it. If > > not, I'll > > > > > try to > > > > > > > > > get > > > > > > > > > > > > back > > > > > > > > > > > > > > to it > > > > > > > > > > > > > > > > > > > > > > > > > > after I finish the Linu= x kernel driver. > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > Best regards, > > > > > > > > > > > > > > > > > > > > > > > > > > Konstantin Aladyshev > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > On Fri, Sep 1, 2023 at = 8:58=E2=80=AFAM > > Chang, > > > > > Abner > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > wrote: > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > [AMD Official Use Onl= y - General] > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > See my answer below, > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > -----Original Messa= ge----- > > > > > > > > > > > > > > > > > > > > > > > > > > > > From: devel@edk2.gr= oups.io > > > > > > > > > > > > > > > > > > > > > > > > > > On > > > > > > > > > > > > > > > > Behalf > > > > > > > > > > > > > > > > > > > > > Of > > > > > > > > > > > > > > > > > > > > > > > > > > > > Konstantin Aladyshe= v via > > groups.io > > > > > > > > > > > > > > > > > > > > > > > > > > > > Sent: Friday, Septe= mber 1, 2023 > > > > 12:02 > > > > > AM > > > > > > > > > > > > > > > > > > > > > > > > > > > > To: Chang, Abner > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > Cc: devel@edk2.grou= ps.io; > > > > > > > > > > > > discuss@edk2.groups.io > > > > > > > > > > > > > > > > > > > > > > > > > > > > Subject: Re: [edk2-= devel] [edk2- > > > > > discuss] > > > > > > > > > PLDM > > > > > > > > > > > > > > messages > > > > > > > > > > > > > > > > via > > > > > > > > > > > > > > > > > > > > > MCTP > > > > > > > > > > > > > > > > > > > > > > > > over > > > > > > > > > > > > > > > > > > > > > > > > > > KCS > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > Caution: This messa= ge originated > > from > > > > > an > > > > > > > > > External > > > > > > > > > > > > > > Source. > > > > > > > > > > > > > > > > Use > > > > > > > > > > > > > > > > > > > > > proper > > > > > > > > > > > > > > > > > > > > > > > > > > caution > > > > > > > > > > > > > > > > > > > > > > > > > > > > when opening attach= ments, > > clicking > > > > > links, or > > > > > > > > > > > > > > responding. > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > As I've said there = is nothing like > > "KCS > > > > > > > > > completion > > > > > > > > > > > > code" > > > > > > > > > > > > > > in > > > > > > > > > > > > > > > > the > > > > > > > > > > > > > > > > > > > > > MCTP > > > > > > > > > > > > > > > > > > > > > > > > > > > > over KCS binding sp= ecification > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > (https://www.dmtf.org/sites/default/files/standards/documents/DSP0254_1 > > > > > > > > > > > > > > > > > > > > > > > > > > > > .0.0.pdf). > > > > > > > > > > > > > > > > > > > > > > > > > > > > The response packet= should have > > the > > > > > same > > > > > > > > > > > > structure as > > > > > > > > > > > > > > a > > > > > > > > > > > > > > > > request. > > > > > > > > > > > > > > > > > > > > > I.e. > > > > > > > > > > > > > > > > > > > > > > > > > > > > a packet with > > > > > > > > > > > > MANAGEABILITY_MCTP_KCS_HEADER > > > > > > > > > > > > > > and > > > > > > > > > > > > > > > > PEC. > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > Currently I'm writi= ng "MCTP over > > KCS" > > > > > binding > > > > > > > > > for > > > > > > > > > > > > the > > > > > > > > > > > > > > > > OpenBMC > > > > > > > > > > > > > > > > > > > > > > > > libmctp > > > > > > > > > > > > > > > > > > > > > > > > > > > > project. So I can s= end whatever I > > > > want, I > > > > > don't > > > > > > > > > > > think > > > > > > > > > > > > > > my > > > > > > > > > > > > > > > > output > > > > > > > > > > > > > > > > > > > > > would > > > > > > > > > > > > > > > > > > > > > > > > > > > > be any useful to yo= u. But I've > > asked > > > > this > > > > > > > > > question > > > > > > > > > > > in > > > > > > > > > > > > the > > > > > > > > > > > > > > > > > > > > > community > > > > > > > > > > > > > > > > > > > > > > > > > > > > and they also confi= rmed that the > > > > > response > > > > > > > > > packet > > > > > > > > > > > > has > > > > > > > > > > > > > > the > > > > > > > > > > > > > > > > same > > > > > > > > > > > > > > > > > > > > > > > > > > > > structure. > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > (https://discord.com/channels/775381525260664832/7787906385638850 > > > > > > > > > > > > > > > > > > > > > > > > > > > > 86/1146782595334549= 554) > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > Now I'm a little bi= t confused about > > the > > > > > > > > > > > > > > > > > > > > > `KcsTransportSendCommand` > > > > > > > > > > > > > > > > > > > > > > > > > > > > function. It has th= e following > > > > arguments > > > > > for > > > > > > > > > the > > > > > > > > > > > > > > function > > > > > > > > > > > > > > > > output: > > > > > > > > > > > > > > > > > > > > > > > > > > > > ``` > > > > > > > > > > > > > > > > > > > > > > > > > > > > OUT UINT8 > > > > > *ResponseData > > > > > > > > > > > > > > OPTIONAL, > > > > > > > > > > > > > > > > > > > > > > > > > > > > IN OUT UINT32 > > > > > > > > > > > *ResponseDataSize > > > > > > > > > > > > > > > > OPTIONAL > > > > > > > > > > > > > > > > > > > > > > > > > > > > ``` > > > > > > > > > > > > > > > > > > > > > > > > > > > > Should we include > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > MCTP_TRANSPORT_HEADER/MCTP_MESSAGE_HEADER > > > > > > > > > > > > > > > > > > > > > > > > > > to > > > > > > > > > > > > > > > > > > > > > > > > > > > > this > > > > > > > > > > > > > > > > > > > > > > > > > > > > output or not? > > > > > > > > > > > > > > > > > > > > > > > > > > > If the MCTP KCS packe= t for host- > > >BMC > > > > > and > > > > > > > > > BMC- > > > > > > > > > > > > >host > > > > > > > > > > > > > > are > > > > > > > > > > > > > > > > in the > > > > > > > > > > > > > > > > > > > > > same > > > > > > > > > > > > > > > > > > > > > > > > > > structure, then yes, th= e response data > > > > > from BMC > > > > > > > > > > > > should > > > > > > > > > > > > > > > > includes > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > MCTP_TRANSPORT_HEADER/MCTP_MESSAGE_HEADER > > > > > > > > > > > > > > in > > > > > > > > > > > > > > > > my > > > > > > > > > > > > > > > > > > > > > opinion, as > > > > > > > > > > > > > > > > > > > > > > > > this > > > > > > > > > > > > > > > > > > > > > > > > > > is defined in MCTP base= protocol. > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > So let me explain the > > implementation > > > > for > > > > > MCTP > > > > > > > > > > > over > > > > > > > > > > > > KCS > > > > > > > > > > > > > > > > and what > > > > > > > > > > > > > > > > > > > > > do we > > > > > > > > > > > > > > > > > > > > > > > > > > miss now. > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > A. MCTP protocol dri= ver linked with > > KCS > > > > > > > > > Transport > > > > > > > > > > > > > > interface > > > > > > > > > > > > > > > > library > > > > > > > > > > > > > > > > > > > > > > > > > > > - In MCTP protocol= driver, if the > > > > > transport > > > > > > > > > > > interface > > > > > > > > > > > > is > > > > > > > > > > > > > > KCS > > > > > > > > > > > > > > > > then > > > > > > > > > > > > > > > > > > > > > > > > > > > 1. MCTP protoc= ol driver builds > > up > > > > the > > > > > MCTP > > > > > > > > > KCS > > > > > > > > > > > > > > > > transport > > > > > > > > > > > > > > > > > > > > > header, > > > > > > > > > > > > > > > > > > > > > > > > which > > > > > > > > > > > > > > > > > > > > > > > > > > is DefBody, NetFunc and= ByeCount > > > > > > > > > > > > > > > > > > > > > > > > > > > 2. MCTP protoc= ol driver builds > > up > > > > the > > > > > MCTP > > > > > > > > > > > > payload > > > > > > > > > > > > > > > > > > > > > > > > > > > 3. MCTP protoc= ol driver builds > > up > > > > the > > > > > MCTP > > > > > > > > > KCS > > > > > > > > > > > > > > > > transport trailer, > > > > > > > > > > > > > > > > > > > > > > > > which > > > > > > > > > > > > > > > > > > > > > > > > > > is PEC. > > > > > > > > > > > > > > > > > > > > > > > > > > > Above three st= eps are already > > > > > implemented. > > > > > > > > > > > > > > > > > > > > > > > > > > > PEC is calcula= ted by MCTP > > protocol > > > > > driver > > > > > > > > > and > > > > > > > > > > > > should > > > > > > > > > > > > > > be > > > > > > > > > > > > > > > > verified > > > > > > > > > > > > > > > > > > > > > by > > > > > > > > > > > > > > > > > > > > > > > > KCS > > > > > > > > > > > > > > > > > > > > > > > > > > using the same algorith= m in my > > > > > understanding of > > > > > > > > > > > spec. > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > B. In KCS Transport = interface library > > > > > > > > > > > > > > > > > > > > > > > > > > > 1. KCS Transpor= t interface library > > > > sends > > > > > the > > > > > > > > > > > > transport > > > > > > > > > > > > > > > > header got > > > > > > > > > > > > > > > > > > > > > from > > > > > > > > > > > > > > > > > > > > > > > > > > TransportToken. Same be= havior for > > IPMI > > > > > protocol, > > > > > > > > > > > but > > > > > > > > > > > > > > > > different > > > > > > > > > > > > > > > > > > > > > content. > > > > > > > > > > > > > > > > > > > > > > > > KCS > > > > > > > > > > > > > > > > > > > > > > > > > > Transport interface lib= rary doesn't > > have to > > > > > > > > > > > understand > > > > > > > > > > > > > > this. > > > > > > > > > > > > > > > > > > > > > > > > > > > 2. KCS Transpor= t interface library > > > > sends > > > > > the > > > > > > > > > > > > payload > > > > > > > > > > > > > > > > > > > > > > > > > > > 3. KCS Transpor= t interface library > > > > sends > > > > > the > > > > > > > > > > > > transport > > > > > > > > > > > > > > > > trailer got > > > > > > > > > > > > > > > > > > > > > from > > > > > > > > > > > > > > > > > > > > > > > > > > TransportToken. Same be= havior for > > IPMI > > > > > protocol, > > > > > > > > > > > but > > > > > > > > > > > > > > > > different > > > > > > > > > > > > > > > > > > > > > content. > > > > > > > > > > > > > > > > > > > > > > > > KCS > > > > > > > > > > > > > > > > > > > > > > > > > > Transport interface lib= rary doesn't > > have to > > > > > > > > > > > understand > > > > > > > > > > > > > > this. > > > > > > > > > > > > > > > > > > > > > > > > > > > Above three ste= ps are already > > > > > implemented. > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > Then, if Manag= eability protocol is > > > > > MCTP, we > > > > > > > > > skip > > > > > > > > > > > > > > reading > > > > > > > > > > > > > > > > > > > > > responses > > > > > > > > > > > > > > > > > > > > > > > > > > header (Not implemented= ) > > > > > > > > > > > > > > > > > > > > > > > > > > > For reading res= ponse data > > > > > > > > > > > > > > > > > > > > > > > > > > > 1. If the Re= sponseData and > > > > > ResponseSize is > > > > > > > > > > > valid > > > > > > > > > > > > in > > > > > > > > > > > > > > the > > > > > > > > > > > > > > > > given > > > > > > > > > > > > > > > > > > > > > > > > > > TransportToken, then we= read > > > > > ResponseSize data > > > > > > > > > > > from > > > > > > > > > > > > > > KCS. > > > > > > > > > > > > > > > > (Already > > > > > > > > > > > > > > > > > > > > > > > > > > implemented) > > > > > > > > > > > > > > > > > > > > > > > > > > > 2. if Manage= ability protocol is > > > > MCTP, > > > > > then > > > > > > > > > we > > > > > > > > > > > > skip > > > > > > > > > > > > > > > > reading > > > > > > > > > > > > > > > > > > > > > responses > > > > > > > > > > > > > > > > > > > > > > > > > > header again (Not imple= mented) > > > > > > > > > > > > > > > > > > > > > > > > > > > Now the response= is returned to > > > > MCTP > > > > > > > > > protocol > > > > > > > > > > > > driver > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > C. In MCTP protocol = driver, we > > expect > > > > to > > > > > get the > > > > > > > > > > > > whole > > > > > > > > > > > > > > > > MCTP over > > > > > > > > > > > > > > > > > > > > > KCS > > > > > > > > > > > > > > > > > > > > > > > > > > packet response, that i= ncludes > > DefBody, > > > > > NetFunc > > > > > > > > > and > > > > > > > > > > > > > > > > ByeCount, > > > > > > > > > > > > > > > > > > > > > MCTP > > > > > > > > > > > > > > > > > > > > > > > > > > message and PEC. > > > > > > > > > > > > > > > > > > > > > > > > > > > 1. MCTP prot= ocol driver verifies > > the > > > > > > > > > returned > > > > > > > > > > > > PEC > > > > > > > > > > > > > > with > > > > > > > > > > > > > > > > the > > > > > > > > > > > > > > > > > > > > > payload. > > > > > > > > > > > > > > > > > > > > > > > > > > > 2. Strip out= DefBody, NetFunc, > > > > > ByeCount > > > > > > > > > and > > > > > > > > > > > > PEC > > > > > > > > > > > > > > and > > > > > > > > > > > > > > > > then > > > > > > > > > > > > > > > > > > > > > returns it > > > > > > > > > > > > > > > > > > > > > > > > to > > > > > > > > > > > > > > > > > > > > > > > > > > upper layer (e.g., MCTP= transport > > > > interface > > > > > > > > > library). > > > > > > > > > > > > > > Returns > > > > > > > > > > > > > > > > only > > > > > > > > > > > > > > > > > > > > > MCTP > > > > > > > > > > > > > > > > > > > > > > > > > > Transport header and MC= TP packet > > > > > payload as it > > > > > > > > > > > shows > > > > > > > > > > > > in > > > > > > > > > > > > > > > > MCTP base > > > > > > > > > > > > > > > > > > > > > > > > protocol > > > > > > > > > > > > > > > > > > > > > > > > > > spec. > > > > > > > > > > > > > > > > > > > > > > > > > > > Above is not imp= lemented > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > D. In MCTP transport = interface > > library, > > > > we > > > > > can > > > > > > > > > strip > > > > > > > > > > > > out > > > > > > > > > > > > > > > > MCTP > > > > > > > > > > > > > > > > > > > > > transport > > > > > > > > > > > > > > > > > > > > > > > > > > header and then return = it to upper > > layer > > > > > (e.g., > > > > > > > > > PLDM > > > > > > > > > > > > > > protocol > > > > > > > > > > > > > > > > driver). > > > > > > > > > > > > > > > > > > > > > > > > > > > Above is not impl= emented. > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > E. In PLDM protocol d= river, > > > > > > > > > > > > > > > > > > > > > > > > > > > 1. we verify = the Message > > Integrity > > > > > Check if > > > > > > > > > the > > > > > > > > > > > > > > Message > > > > > > > > > > > > > > > > Type > > > > > > > > > > > > > > > > > > > > > > > > requests it. > > > > > > > > > > > > > > > > > > > > > > > > > > > 2. we can rem= ove MCTP > > message > > > > > type then > > > > > > > > > > > > return it > > > > > > > > > > > > > > to > > > > > > > > > > > > > > > > upper > > > > > > > > > > > > > > > > > > > > > layer > > > > > > > > > > > > > > > > > > > > > > > > (e.g., > > > > > > > > > > > > > > > > > > > > > > > > > > PLDM SMBIOS transfer) > > > > > > > > > > > > > > > > > > > > > > > > > > > Above is not = implemented. > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > We didn=E2=80=99t imp= lement BMC->Host in > > > > step > > > > > C, D > > > > > > > > > and E > > > > > > > > > > > > as > > > > > > > > > > > > > > our > > > > > > > > > > > > > > > > current > > > > > > > > > > > > > > > > > > > > > > > > demand is > > > > > > > > > > > > > > > > > > > > > > > > > > to send the SMBIOS tabl= e to BMC, > > which > > > > > doesn't > > > > > > > > > > > > require > > > > > > > > > > > > > > the > > > > > > > > > > > > > > > > response > > > > > > > > > > > > > > > > > > > > > data > > > > > > > > > > > > > > > > > > > > > > > > if I > > > > > > > > > > > > > > > > > > > > > > > > > > am not wrong. > > > > > > > > > > > > > > > > > > > > > > > > > > > Let me know if it is = problematic in > > the > > > > > above > > > > > > > > > > > process. > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > Thanks and regards, > > > > > > > > > > > > > > > > > > > > > > > > > > > Abner > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > Best regards, > > > > > > > > > > > > > > > > > > > > > > > > > > > > Konstantin Aladyshe= v > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > On Thu, Aug 31, 202= 3 at 6:52=E2=80=AFPM > > > > > Chang, > > > > > > > > > Abner > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > wrote: > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > [AMD Official Use= Only - General] > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > But wait, wee my = another > > comment > > > > > below, > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > -----Original M= essage----- > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > From: Chang, Ab= ner > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > Sent: Thursday,= August 31, > > 2023 > > > > > 11:42 PM > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > To: devel@edk2.= groups.io; > > > > > > > > > > > > > > aladyshev22@gmail.com > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > Cc: discuss@edk= 2.groups.io > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > Subject: RE: [e= dk2-devel] > > [edk2- > > > > > discuss] > > > > > > > > > > > PLDM > > > > > > > > > > > > > > > > messages via > > > > > > > > > > > > > > > > > > > > > MCTP > > > > > > > > > > > > > > > > > > > > > > > > > > over > > > > > > > > > > > > > > > > > > > > > > > > > > > > KCS > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > -----Original= Message----- > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > From: devel@e= dk2.groups.io > > > > > > > > > > > > > > > > On > > > > > > > > > > > > > > > > > > > > > Behalf > > > > > > > > > > > > > > > > > > > > > > > > Of > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > Konstantin Al= adyshev via > > > > > groups.io > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > Sent: Thursda= y, August 31, > > 2023 > > > > > 10:57 > > > > > > > > > PM > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > To: Chang, Ab= ner > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > Cc: discuss@e= dk2.groups.io; > > > > > > > > > > > > > > devel@edk2.groups.io > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > Subject: Re: = [edk2-devel] > > [edk2- > > > > > discuss] > > > > > > > > > > > > PLDM > > > > > > > > > > > > > > > > messages via > > > > > > > > > > > > > > > > > > > > > > > > MCTP > > > > > > > > > > > > > > > > > > > > > > > > > > over > > > > > > > > > > > > > > > > > > > > > > > > > > > > KCS > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > Caution: This= message > > originated > > > > > from an > > > > > > > > > > > > External > > > > > > > > > > > > > > > > Source. > > > > > > > > > > > > > > > > > > > > > Use > > > > > > > > > > > > > > > > > > > > > > > > proper > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > caution > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > when opening = attachments, > > > > > clicking links, > > > > > > > > > or > > > > > > > > > > > > > > > > responding. > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > (I don see = what is the > > response > > > > > header > > > > > > > > > for > > > > > > > > > > > > MCTP > > > > > > > > > > > > > > > > KCS in spec > > > > > > > > > > > > > > > > > > > > > > > > though, > > > > > > > > > > > > > > > > > > > > > > > > > > > > does > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > it > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > mention the K= CS response?). > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > The spec does= n't explicitly > > > > mention > > > > > that > > > > > > > > > the > > > > > > > > > > > > > > format of > > > > > > > > > > > > > > > > a send > > > > > > > > > > > > > > > > > > > > > and > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > response pack= ets differ. So I > > > > > assume it is > > > > > > > > > the > > > > > > > > > > > > same > > > > > > > > > > > > > > > > and it is > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > described at = the "Figure 1 =E2=80=93 > > > > MCTP > > > > > over > > > > > > > > > KCS > > > > > > > > > > > > Packet > > > > > > > > > > > > > > > > Format" > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > (https://www.dmtf.org/sites/default/files/standards/documents/DSP0254_1 > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > .0.0.pdf) > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > Therefore the= format of a > > > > response > > > > > would > > > > > > > > > > > look > > > > > > > > > > > > > > like > > > > > > > > > > > > > > > > this: > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > ``` > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > MANAGEABILITY_MCTP_KCS_HEADER > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > (https://github.com/tianocore/edk2- > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > platforms/blob/master/Features/ManageabilityPkg/Include/Library/Managea > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > bilityTranspo= rtMctpLib.h) > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > MCTP_TRANSPOR= T_HEADER > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > (https://github.com/tianocore/edk2/blob/master/MdePkg/Include/Industry > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > Standard/Mctp= .h) > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > MCTP_MESSAGE_= HEADER > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > (https://github.com/tianocore/edk2/blob/master/MdePkg/Include/Industry > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > Standard/Mctp= .h) > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > < response da= ta> > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > What do you see= the KCS > > response > > > > > from > > > > > > > > > BMC? > > > > > > > > > > > > You > > > > > > > > > > > > > > > > probably > > > > > > > > > > > > > > > > > > > > > right as > > > > > > > > > > > > > > > > > > > > > > > > > > the > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > header and trai= ler are labeled in > > > > > different > > > > > > > > > > > colors > > > > > > > > > > > > =F0=9F=98=8A. > > > > > > > > > > > > > > > > Could you > > > > > > > > > > > > > > > > > > > > > > > > please > > > > > > > > > > > > > > > > > > > > > > > > > > > > enable > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > the debug messa= ge to capture > > it? > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > PEC > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > (P= robably we need to > > > > define > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > MANAGEABILITY_MCTP_KCS_TRAILER) > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > ``` > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > We have > > > > > > > > > > > MANAGEABILITY_MCTP_KCS_HEADER > > > > > > > > > > > > > > > > defined but no > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > MANAGEABILITY_MCTP_KCS_TRAILER as it > > > > > > > > > is > > > > > > > > > > > > > > > > hardcoded to one > > > > > > > > > > > > > > > > > > > > > > > > byte. > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > If the KCS resp= onse is PEC, then > > > > yes, > > > > > we can > > > > > > > > > > > > create > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > MANAGEABILITY_MCTP_KCS_TRAILER for > > > > > > > > > KCS > > > > > > > > > > > > > > > > transport > > > > > > > > > > > > > > > > > > > > > interface. > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > In the implementa= tion, PEC is > > > > > calculated in > > > > > > > > > > > MCTP > > > > > > > > > > > > > > > > protocol and > > > > > > > > > > > > > > > > > > > > > send > > > > > > > > > > > > > > > > > > > > > > > > > > through > > > > > > > > > > > > > > > > > > > > > > > > > > > > KCS as the KCS pack= et trailer. So, > > when > > > > > we > > > > > > > > > send > > > > > > > > > > > the > > > > > > > > > > > > > > MCTP > > > > > > > > > > > > > > > > request > > > > > > > > > > > > > > > > > > > > > > > > through > > > > > > > > > > > > > > > > > > > > > > > > > > > > KCS, KCS shouldn't = respond the > > PEC to > > > > > upper > > > > > > > > > > > stack, > > > > > > > > > > > > > > right? > > > > > > > > > > > > > > > > I still > > > > > > > > > > > > > > > > > > > > > think > > > > > > > > > > > > > > > > > > > > > > > > the > > > > > > > > > > > > > > > > > > > > > > > > > > > > response should be = the KCS > > > > completion > > > > > code. > > > > > > > > > The > > > > > > > > > > > > > > debug > > > > > > > > > > > > > > > > message > > > > > > > > > > > > > > > > > > > > > from > > > > > > > > > > > > > > > > > > > > > > > > > > your > > > > > > > > > > > > > > > > > > > > > > > > > > > > end may help to cla= rify this as your > > > > BMC > > > > > has > > > > > > > > > the > > > > > > > > > > > > MCTP > > > > > > > > > > > > > > > > KCS > > > > > > > > > > > > > > > > > > > > > > > > > > implementation. > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > Thanks > > > > > > > > > > > > > > > > > > > > > > > > > > > > > Abner > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > So in the > > > > > "KcsTransportSendCommand" > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > (https://github.com/tianocore/edk2- > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > platforms/blob/14553d31c72afa7289f6a2555b6e91f4f715a05a/Features/ > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > ManageabilityPkg/Library/ManageabilityTransportKcsLib/Common/KcsComm > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > on.c#L414) > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > we can check = if we transfer is > > > > > MCTP > > > > > > > > > (based > > > > > > > > > > > on > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > "TransportTok= en- > > > > > > > > > > > > > > >ManagebilityProtocolSpecification > > > > > > > > > > > > > > > > =3D=3D > > > > > > > > > > > > > > > > > > > > > MCTP" like > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > you've sugges= ted) and handle > > > > > response > > > > > > > > > > > > > > accordingly. > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > But which hea= ders should we > > > > > check in > > > > > > > > > this > > > > > > > > > > > > > > function? > > > > > > > > > > > > > > > > Only > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > MANAGEABILITY_MCTP_KCS_HEADER/MANAGEABILITY_MCTP_KCS_TRAILER > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > ? > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > Yes, only check= header and > > trailer > > > > for > > > > > > > > > transport > > > > > > > > > > > > > > > > interface. > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > What about > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > MCTP_TRANSPORT_HEADER/MCTP_MESSAGE_HEADER? > > > > > > > > > > > > > > > > > > > > > > > > > > > > Do > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > we > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > need to > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > check them he= re as well? Or > > do > > > > we > > > > > need > > > > > > > > > to > > > > > > > > > > > > check > > > > > > > > > > > > > > > > them > > > > > > > > > > > > > > > > > > > > > > > > somewhere > > > > > > > > > > > > > > > > > > > > > > > > > > > > upper > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > the call stac= k? > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > We should leave= this to MCTP > > > > > protocol > > > > > > > > > driver > > > > > > > > > > > as > > > > > > > > > > > > > > this is > > > > > > > > > > > > > > > > belong > > > > > > > > > > > > > > > > > > > > > to > > > > > > > > > > > > > > > > > > > > > > > > > > protocol > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > layer, the uppe= r layer stack. > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > Thanks > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > Abner > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > Best regards, > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > Konstantin Al= adyshev > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > On Thu, Aug 3= 1, 2023 at > > > > 7:59=E2=80=AFAM > > > > > Chang, > > > > > > > > > > > > Abner > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > wrote: > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > [AMD Offici= al Use Only - > > > > General] > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > Hi Aladyshe= v, > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > -----Orig= inal Message----- > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > From: Kon= stantin > > Aladyshev > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > Sent: Wed= nesday, August > > 30, > > > > > 2023 > > > > > > > > > > > 11:09 > > > > > > > > > > > > PM > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > To: Chang= , Abner > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > Cc: > > discuss@edk2.groups.io; > > > > > > > > > > > > > > > > devel@edk2.groups.io > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > Subject: = Re: [edk2- > > discuss] > > > > > PLDM > > > > > > > > > > > > messages > > > > > > > > > > > > > > via > > > > > > > > > > > > > > > > MCTP > > > > > > > > > > > > > > > > > > > > > over KCS > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > Caution: = This message > > > > > originated from > > > > > > > > > an > > > > > > > > > > > > > > External > > > > > > > > > > > > > > > > Source. > > > > > > > > > > > > > > > > > > > > > Use > > > > > > > > > > > > > > > > > > > > > > > > > > proper > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > caution > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > when open= ing > > attachments, > > > > > clicking > > > > > > > > > links, > > > > > > > > > > > > or > > > > > > > > > > > > > > > > responding. > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > Hi! > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > I've star= ted to implement > > > > MCTP > > > > > over > > > > > > > > > KCS > > > > > > > > > > > > > > binding > > > > > > > > > > > > > > > > for the > > > > > > > > > > > > > > > > > > > > > > > > libmctp > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > (https://github.com/openbmc/libmctp) > > > > > > > > > > > > and > > > > > > > > > > > > > > test > > > > > > > > > > > > > > > > it with > > > > > > > > > > > > > > > > > > > > > the > > > > > > > > > > > > > > > > > > > > > > > > > > current > > > > > > > > > > > > > > > > > > > > > > > > > > > > code > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > in the Ma= nageabilityPkg. > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > I was abl= e successfully > > send > > > > the > > > > > MCTP > > > > > > > > > > > > packet > > > > > > > > > > > > > > to > > > > > > > > > > > > > > > > the BMC, > > > > > > > > > > > > > > > > > > > > > but > > > > > > > > > > > > > > > > > > > > > > > > right > > > > > > > > > > > > > > > > > > > > > > > > > > > > now > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > I'm havin= g some troubles > > > > with > > > > > > > > > receiving > > > > > > > > > > > the > > > > > > > > > > > > > > > > answer back. > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > I think I= 've found some > > bug in > > > > > the > > > > > > > > > > > > > > > > > > > > > > > > `KcsTransportSendCommand` > > > > > > > > > > > > > > > > > > > > > > > > > > code. > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > After it = sends data over > > KCS in > > > > > expects > > > > > > > > > a > > > > > > > > > > > > > > responce > > > > > > > > > > > > > > > > starting > > > > > > > > > > > > > > > > > > > > > with > > > > > > > > > > > > > > > > > > > > > > > > a > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > 'IPMI_KCS_RESPONSE_HEADER' > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > https://github.com/tianocore/edk2- > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > platforms/blob/14553d31c72afa7289f6a2555b6e91f4f715a05a/Features/ > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > ManageabilityPkg/Library/ManageabilityTransportKcsLib/Common/KcsComm > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > on.c#L476 > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > Isn't it = wrong, assuming > > that > > > > > the right > > > > > > > > > > > > header > > > > > > > > > > > > > > in > > > > > > > > > > > > > > > > case of > > > > > > > > > > > > > > > > > > > > > MCTP > > > > > > > > > > > > > > > > > > > > > > > > > > should > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > be > > > > > > > > > > > > 'MANAGEABILITY_MCTP_KCS_HEADER' ? > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > I guess t= he > > > > > > > > > > > > 'IpmiHelperCheckCompletionCode' > > > > > > > > > > > > > > > > check after > > > > > > > > > > > > > > > > > > > > > the > > > > > > > > > > > > > > > > > > > > > > > > data > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > receive i= s also not relevant > > for > > > > > the > > > > > > > > > MCTP. > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > This is som= ething I don=E2=80=99t > > really > > > > > sure as I > > > > > > > > > > > can't > > > > > > > > > > > > > > verify > > > > > > > > > > > > > > > > the > > > > > > > > > > > > > > > > > > > > > response > > > > > > > > > > > > > > > > > > > > > > > > > > > > payload > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > because our B= MC doesn't > > have > > > > the > > > > > code > > > > > > > > > to > > > > > > > > > > > > handle > > > > > > > > > > > > > > > > MCTP over > > > > > > > > > > > > > > > > > > > > > KCS > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > command. > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > However it is= appreciated if > > > > > community > > > > > > > > > can > > > > > > > > > > > > help > > > > > > > > > > > > > > to > > > > > > > > > > > > > > > > verify this. > > > > > > > > > > > > > > > > > > > > > As I > > > > > > > > > > > > > > > > > > > > > > > > can > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > remember, I c= an see the > > return > > > > KCS > > > > > status > > > > > > > > > is > > > > > > > > > > > > 0xC1, > > > > > > > > > > > > > > > > the invalid > > > > > > > > > > > > > > > > > > > > > > > > > > command. > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > Thus I > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > think if we d= o a MCTP over > > KCS, > > > > > the first > > > > > > > > > > > > response > > > > > > > > > > > > > > is > > > > > > > > > > > > > > > > still KCS > > > > > > > > > > > > > > > > > > > > > > > > response > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > header. > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > This is not= what do you see > > on > > > > > the BCM > > > > > > > > > it > > > > > > > > > > > > does > > > > > > > > > > > > > > > > support > > > > > > > > > > > > > > > > > > > > > MCTP > > > > > > > > > > > > > > > > > > > > > > > > over > > > > > > > > > > > > > > > > > > > > > > > > > > > > KCS? If > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > so, then I wo= uld like to have > > your > > > > > help to > > > > > > > > > > > > correct > > > > > > > > > > > > > > this > > > > > > > > > > > > > > > > code. > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > Since > > > > > 'ManageabilityTransportKcsLib' > > > > > > > > > can > > > > > > > > > > > > be > > > > > > > > > > > > > > used > > > > > > > > > > > > > > > > both for > > > > > > > > > > > > > > > > > > > > > IPMI > > > > > > > > > > > > > > > > > > > > > > > > > > and > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > MCTP, how= should we > > deal > > > > > with this? > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > If KcsCommo= n.c, we can > > have > > > > > different > > > > > > > > > > > code > > > > > > > > > > > > > > path > > > > > > > > > > > > > > > > for the > > > > > > > > > > > > > > > > > > > > > given > > > > > > > > > > > > > > > > > > > > > > > > > > protocol > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > GUID. e.g., i= f > > (TransportToken- > > > > > > > > > > > > > > > > > > > > > >ManagebilityProtocolSpecificatio= n > > > > > > > > > > > > > > > > > > > > > > > > =3D=3D > > > > > > > > > > > > > > > > > > > > > > > > > > > > MCTP). > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > Then skip r= eading the > > > > > > > > > > > > KCS_REPOSNSE_HEADER > > > > > > > > > > > > > > or > > > > > > > > > > > > > > > > to read > > > > > > > > > > > > > > > > > > > > > the > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > MCTP_RESPONSE= _HEADER (I > > > > don > > > > > see > > > > > > > > > what > > > > > > > > > > > is > > > > > > > > > > > > the > > > > > > > > > > > > > > > > response > > > > > > > > > > > > > > > > > > > > > header > > > > > > > > > > > > > > > > > > > > > > > > for > > > > > > > > > > > > > > > > > > > > > > > > > > > > MCTP > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > KCS in spec t= hough, does it > > > > > mention the > > > > > > > > > KCS > > > > > > > > > > > > > > > > response?). > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > Thanks > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > Abner > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > Best rega= rds, > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > Konstanti= n Aladyshev > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > On Wed, A= ug 23, 2023 at > > > > > 5:18=E2=80=AFAM > > > > > > > > > > > Chang, > > > > > > > > > > > > > > Abner > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > wrote: > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > [AMD Of= ficial Use Only - > > > > > General] > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > Please = see my answers > > > > inline. > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > -----= Original Message-- > > --- > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > From: > > > > > discuss@edk2.groups.io > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > On > > > > > > > > > > > > > > > > > > > > > > > > > > > > Behalf > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > Of > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > Konst= antin Aladyshev > > via > > > > > > > > > groups.io > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > Sent:= Wednesday, > > August > > > > > 23, > > > > > > > > > 2023 > > > > > > > > > > > > 1:54 > > > > > > > > > > > > > > AM > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > To: C= hang, Abner > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > Cc: > > > > > discuss@edk2.groups.io; > > > > > > > > > > > > > > > > devel@edk2.groups.io > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > Subje= ct: Re: [edk2- > > > > discuss] > > > > > PLDM > > > > > > > > > > > > > > messages > > > > > > > > > > > > > > > > via MCTP > > > > > > > > > > > > > > > > > > > > > over > > > > > > > > > > > > > > > > > > > > > > > > KCS > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > Cauti= on: This message > > > > > originated > > > > > > > > > from > > > > > > > > > > > > an > > > > > > > > > > > > > > > > External > > > > > > > > > > > > > > > > > > > > > Source. > > > > > > > > > > > > > > > > > > > > > > > > Use > > > > > > > > > > > > > > > > > > > > > > > > > > > > proper > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > caution > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > when = opening > > > > attachments, > > > > > > > > > clicking > > > > > > > > > > > > links, > > > > > > > > > > > > > > or > > > > > > > > > > > > > > > > > > > > > responding. > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > Thank= s for the answer! > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > I was= a little bit > > confused > > > > > about the > > > > > > > > > > > > part, > > > > > > > > > > > > > > that > > > > > > > > > > > > > > > > in the > > > > > > > > > > > > > > > > > > > > > same > > > > > > > > > > > > > > > > > > > > > > > > > > package > > > > > > > > > > > > > > > > > > > > > > > > > > > > I > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > actua= lly need to > > provide > > > > > different > > > > > > > > > > > library > > > > > > > > > > > > > > > > > > > > > implementations > > > > > > > > > > > > > > > > > > > > > > > > for > > > > > > > > > > > > > > > > > > > > > > > > > > the > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > same > > > > > 'ManageabilityTransportLib', > > > > > > > > > > > > thanks > > > > > > > > > > > > > > for > > > > > > > > > > > > > > > > the > > > > > > > > > > > > > > > > > > > > > > > > clarification! > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > I thi= nk your DSC > > example > > > > > should > > > > > > > > > go > > > > > > > > > > > into > > > > > > > > > > > > > > the > > > > > > > > > > > > > > > > package > > > > > > > > > > > > > > > > > > > > > > > > > > > > documentation. > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > Yes, th= is is a good idea. I > > will > > > > > update > > > > > > > > > it. > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > As fo= r me, I'm working > > > > with > > > > > the > > > > > > > > > > > > OpenBMC > > > > > > > > > > > > > > > > distribution > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > (https://github.com/openbmc/openbmc) > > > > > > > > > > > > > > and > > > > > > > > > > > > > > > > my goal > > > > > > > > > > > > > > > > > > > > > is to > > > > > > > > > > > > > > > > > > > > > > > > > > > > transfer > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > data > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > from = the BIOS to the > > BMC > > > > > via > > > > > > > > > > > > MCTP/PLDM. > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > Curre= ntly there is no > > > > > solution for > > > > > > > > > the > > > > > > > > > > > > MCTP > > > > > > > > > > > > > > > > over KCS > > > > > > > > > > > > > > > > > > > > > binding > > > > > > > > > > > > > > > > > > > > > > > > in > > > > > > > > > > > > > > > > > > > > > > > > > > > > Linux, > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > so I = need to add this > > > > > support: > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > - eit= her to the MCTP > > > > > userspace > > > > > > > > > library > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > (https://github.com/openbmc/libmctp) > > > > > > > > > > > > > > [old > > > > > > > > > > > > > > > > > > > > > OpenBMC > > > > > > > > > > > > > > > > > > > > > > > > way, > > > > > > > > > > > > > > > > > > > > > > > > > > but > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > probably > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > easie= r] > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > - or = to the MCTP kernel > > > > > binding > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > (https://github.com/torvalds/linux/tree/master/drivers/net/mctp) > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > [mode= rn mctp Linux > > driver > > > > > > > > > approach] > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > Both = don't sound like > > an > > > > > easy task, > > > > > > > > > so > > > > > > > > > > > > can I > > > > > > > > > > > > > > > > ask, what > > > > > > > > > > > > > > > > > > > > > MC > > > > > > > > > > > > > > > > > > > > > > > > (i.e. > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > manag= ement > > controller) > > > > > device > > > > > > > > > and > > > > > > > > > > > > > > firmware > > > > > > > > > > > > > > > > do you > > > > > > > > > > > > > > > > > > > > > use on > > > > > > > > > > > > > > > > > > > > > > > > > > the > > > > > > > > > > > > > > > > > > > > > > > > > > > > other > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > side = of the MCTP KCS > > > > > > > > > transmissions? > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > We use = OpenBMC as > > well, > > > > but > > > > > as > > > > > > > > > you > > > > > > > > > > > > > > mention > > > > > > > > > > > > > > > > there are > > > > > > > > > > > > > > > > > > > > > some > > > > > > > > > > > > > > > > > > > > > > > > > > > > missing > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > pieces > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > to fully = support > > manageability > > > > > > > > > between > > > > > > > > > > > > host > > > > > > > > > > > > > > and > > > > > > > > > > > > > > > > BMC. > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > We don= =E2=80=99t have code to > > > > handle > > > > > MCTP > > > > > > > > > > > > IPMI > > > > > > > > > > > > > > > > either, the > > > > > > > > > > > > > > > > > > > > > edk2 > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > Manageability= Pkg > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > provides = the framework > > while > > > > > > > > > > > > > > MCTP/PLDM/KCS > > > > > > > > > > > > > > > > > > > > > > > > implementation > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > provides > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > a > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > sample ot= her than > > IPMI/KCS > > > > to > > > > > prove > > > > > > > > > the > > > > > > > > > > > > > > > > flexibility of > > > > > > > > > > > > > > > > > > > > > > > > > > > > ManageabilityPkg. > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > Actuall= y, MCTP over KCS > > is > > > > > not > > > > > > > > > > > supported > > > > > > > > > > > > in > > > > > > > > > > > > > > our > > > > > > > > > > > > > > > > BMC > > > > > > > > > > > > > > > > > > > > > > > > firmware > > > > > > > > > > > > > > > > > > > > > > > > > > yet, > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > thus > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > BMC just = returns the > > invalid > > > > > > > > > command. > > > > > > > > > > > > > > However, > > > > > > > > > > > > > > > > the > > > > > > > > > > > > > > > > > > > > > transport > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > framework > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > has been = verified to make > > > > sure > > > > > the > > > > > > > > > > > > > > > > implementation works > > > > > > > > > > > > > > > > > > > > > fine > > > > > > > > > > > > > > > > > > > > > > > > as > > > > > > > > > > > > > > > > > > > > > > > > > > > > expect. > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > We need= help from > > > > > community to > > > > > > > > > > > > provide > > > > > > > > > > > > > > more > > > > > > > > > > > > > > > > > > > > > > > > manageability > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > protocols > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > and > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > transport= interface > > libraries to > > > > > this > > > > > > > > > > > package. > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > You'v= e also mentioned > > > > > PLDM > > > > > > > > > SMBIOS, > > > > > > > > > > > > isn't > > > > > > > > > > > > > > it > > > > > > > > > > > > > > > > covered > > > > > > > > > > > > > > > > > > > > > by > > > > > > > > > > > > > > > > > > > > > > > > the > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > https://github.com/tianocore/edk2- > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > platforms/blob/master/Features/ManageabilityPkg/Universal/PldmSmbiosTr > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > ansferDxe/PldmSmbiosTransferDxe.c > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > ? > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > Ah hah,= yes I forget I > > > > > upstream it. > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > Please = just feel free to > > send > > > > > patch to > > > > > > > > > > > > make > > > > > > > > > > > > > > more > > > > > > > > > > > > > > > > > > > > > > > > functionalities to > > > > > > > > > > > > > > > > > > > > > > > > > > > > this > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > package. > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > Thanks > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > Abner > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > Best = regards, > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > Konst= antin Aladyshev > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > On Tu= e, Aug 22, 2023 > > at > > > > > 7:26=E2=80=AFPM > > > > > > > > > > > > Chang, > > > > > > > > > > > > > > > > Abner > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > wrote= : > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > [AM= D Official Use > > Only - > > > > > General] > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > Hi = Aladyshev, > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > We = use library class > > to > > > > > specify the > > > > > > > > > > > > desire > > > > > > > > > > > > > > > > transport > > > > > > > > > > > > > > > > > > > > > > > > interface > > > > > > > > > > > > > > > > > > > > > > > > > > for > > > > > > > > > > > > > > > > > > > > > > > > > > > > the > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > manag= ement protocol, > > > > such > > > > > as > > > > > > > > > MCTP, > > > > > > > > > > > > > > PLDM > > > > > > > > > > > > > > > > and IPMI. > > > > > > > > > > > > > > > > > > > > > This > > > > > > > > > > > > > > > > > > > > > > > > > > way > > > > > > > > > > > > > > > > > > > > > > > > > > > > we > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > can > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > flexi= bly support any > > > > > transport > > > > > > > > > > > interface > > > > > > > > > > > > for > > > > > > > > > > > > > > the > > > > > > > > > > > > > > > > > > > > > management > > > > > > > > > > > > > > > > > > > > > > > > > > > > protocol. > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > Her= e is the example > > of > > > > > using > > > > > > > > > > > > > > > > ManageabilityPkg, which > > > > > > > > > > > > > > > > > > > > > is > > > > > > > > > > > > > > > > > > > > > > > > > > PLDM > > > > > > > > > > > > > > > > > > > > > > > > > > > > over > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > MCTP > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > over = KCS. > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > ManageabilityPkg/Universal/IpmiProtocol/Dxe/IpmiProtocolDxe.inf > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > { > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > = > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > ManageabilityTransportLib|ManageabilityPkg/Library/ManageabilityTranspo= r > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > tKcsLib/Dxe/DxeManageabilityTransportKcs.in= f > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > } > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > ManageabilityPkg/Universal/MctpProtocol/Dxe/MctpProtocolDxe.inf > > > > > > > > > > > > > > > > > > > > > > > > > > { > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > = > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > ManageabilityTransportLib|ManageabilityPkg/Library/ManageabilityTranspo= r > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > tKcsLib/Dxe/DxeManageabilityTransportKcs.in= f > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > } > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > ManageabilityPkg/Universal/PldmProtocol/Dxe/PldmProtocolDxe.inf > > > > > > > > > > > > > > > > > > > > > > > > > > { > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > = > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > ManageabilityTransportLib|ManageabilityPkg/Library/ManageabilityTranspo= r > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > tMctpLib/Dxe/DxeManageabilityTransportMctp.= inf > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > } > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > So = you can > > implement > > > > > > > > > > > > > > > > ManageabilityTransport library > > > > > > > > > > > > > > > > > > > > > for > > > > > > > > > > > > > > > > > > > > > > > > > > either > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > industry > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > stand= ard or > > proprietary > > > > > > > > > > > implementation > > > > > > > > > > > > for > > > > > > > > > > > > > > > > the specific > > > > > > > > > > > > > > > > > > > > > > > > > > > > management > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > proto= col. > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > BTW= , We do have > > PLDM > > > > > SMBIOS > > > > > > > > > > > over > > > > > > > > > > > > > > MCTP > > > > > > > > > > > > > > > > > > > > > > > > implementation > > > > > > > > > > > > > > > > > > > > > > > > > > but > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > not > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > upstr= eam yet. > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > Hop= e this > > information > > > > > helps. > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > Tha= nks > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > Abn= er > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > -= ----Original > > Message-- > > > > -- > > > > > - > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > F= rom: > > > > > discuss@edk2.groups.io > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > On > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > Behalf Of > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > K= onstantin > > Aladyshev > > > > > via > > > > > > > > > > > groups.io > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > S= ent: Tuesday, > > August > > > > > 22, 2023 > > > > > > > > > > > > 7:00 > > > > > > > > > > > > > > PM > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > T= o: discuss > > > > > > > > > > > > ; > > > > > > > > > > > > > > > > > > > > > > > > > > devel@edk2.groups.io > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > S= ubject: [edk2- > > discuss] > > > > > PLDM > > > > > > > > > > > > messages > > > > > > > > > > > > > > via > > > > > > > > > > > > > > > > MCTP > > > > > > > > > > > > > > > > > > > > > over > > > > > > > > > > > > > > > > > > > > > > > > KCS > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > C= aution: This > > message > > > > > > > > > originated > > > > > > > > > > > > from > > > > > > > > > > > > > > an > > > > > > > > > > > > > > > > External > > > > > > > > > > > > > > > > > > > > > > > > Source. > > > > > > > > > > > > > > > > > > > > > > > > > > Use > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > proper > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > cauti= on > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > w= hen opening > > > > > attachments, > > > > > > > > > > > clicking > > > > > > > > > > > > > > links, > > > > > > > > > > > > > > > > or > > > > > > > > > > > > > > > > > > > > > responding. > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > H= i! > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > I= 'm trying to build > > > > > > > > > > > > `ManageabilityPkg` > > > > > > > > > > > > > > from > > > > > > > > > > > > > > > > the > > > > > > > > > > > > > > > > > > > > > edk2- > > > > > > > > > > > > > > > > > > > > > > > > > > platforms > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > = repo to issue PLDM > > > > > messages > > > > > > > > > via > > > > > > > > > > > > MCTP > > > > > > > > > > > > > > > > over KCS. > > > > > > > > > > > > > > > > > > > > > Is it > > > > > > > > > > > > > > > > > > > > > > > > > > possible > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > with > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > t= he current code? I > > see > > > > > all the > > > > > > > > > > > > building > > > > > > > > > > > > > > > > blocks, but > > > > > > > > > > > > > > > > > > > > > have > > > > > > > > > > > > > > > > > > > > > > > > > > trouble > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > p= utting it all > > together. > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > T= he main question > > that > > > > > bothers > > > > > > > > > > > me > > > > > > > > > > > > is > > > > > > > > > > > > > > what > > > > > > > > > > > > > > > > > > > > > > > > implementation > > > > > > > > > > > > > > > > > > > > > > > > > > > > should > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > I > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > set > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > f= or the > > > > > > > > > > > `ManageabilityTransportLib`? > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > B= y default it is set to > > > > > dummy > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > `BaseManageabilityTransportNull.inf` > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > (https://github.com/tianocore/edk2- > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > platforms/blob/master/Features/ManageabilityPkg/ManageabilityPkg.dsc). > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > O= n one case to get > > > > PLDM > > > > > via > > > > > > > > > MCTP > > > > > > > > > > > it > > > > > > > > > > > > > > looks > > > > > > > > > > > > > > > > that I > > > > > > > > > > > > > > > > > > > > > need to > > > > > > > > > > > > > > > > > > > > > > > > set > > > > > > > > > > > > > > > > > > > > > > > > > > it > > > > > > > > > > > > > > > > > > > > > > > > > > > > to > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > `DxeManageabilityTransportMctp.inf` > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > ManageabilityTransportLib| > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > <...>/DxeManageabilityTransportMctp.inf > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > (https://github.com/tianocore/edk2- > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > platforms/blob/master/Features/ManageabilityPkg/Library/ManageabilityTr= a > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > nsportMctpLib/Dxe/DxeManageabilityTransportMctp.inf) > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > B= ut on the other > > case if > > > > I > > > > > want > > > > > > > > > > > MCTP > > > > > > > > > > > > > > over > > > > > > > > > > > > > > > > KCS I > > > > > > > > > > > > > > > > > > > > > need to > > > > > > > > > > > > > > > > > > > > > > > > set > > > > > > > > > > > > > > > > > > > > > > > > > > it to > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > `DxeManageabilityTransportKcs.inf` > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > ManageabilityTransportLib| > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > <...>/DxeManageabilityTransportKcs.inf > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > (https://github.com/tianocore/edk2- > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > platforms/blob/master/Features/ManageabilityPkg/Library/ManageabilityTr= a > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > nsportKcsLib/Dxe/DxeManageabilityTransportKcs.inf) > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > W= hat is the right > > way > > > > to > > > > > resolve > > > > > > > > > > > > this? > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > T= here are no > > platforms > > > > in > > > > > the > > > > > > > > > repo > > > > > > > > > > > > that > > > > > > > > > > > > > > > > actually > > > > > > > > > > > > > > > > > > > > > > > > implement > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > PLDM/MCTP > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > f= unctionality, so > > there > > > > is > > > > > no > > > > > > > > > > > example > > > > > > > > > > > > > > that I > > > > > > > > > > > > > > > > can use > > > > > > > > > > > > > > > > > > > > > as a > > > > > > > > > > > > > > > > > > > > > > > > > > > > reference. > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > B= est regards, > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > K= onstantin > > Aladyshev > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > >=20 > > > > > > > > > > > > > > > > > > > > > -=3D-=3D-=3D-=3D-=3D-=3D-=3D-=3D-=3D-=3D-=3D- Groups.io Links: You receive all messages sent to this group. View/Reply Online (#109866): https://edk2.groups.io/g/devel/message/109866 Mute This Topic: https://groups.io/mt/100897530/7686176 Group Owner: devel+owner@edk2.groups.io Unsubscribe: https://edk2.groups.io/g/devel/unsub [rebecca@openfw.io] -=3D-=3D-=3D-=3D-=3D-=3D-=3D-=3D-=3D-=3D-=3D-