public inbox for devel@edk2.groups.io
 help / color / mirror / Atom feed
* Re: [edk2-devel] IpmiCommandLib's IpmiGetSystemUuid clarification
       [not found] <176C4F04309F3E96.4004@groups.io>
@ 2023-06-27 20:19 ` Aaron Pop
  2023-06-28  8:43   ` Chang, Abner
  0 siblings, 1 reply; 2+ messages in thread
From: Aaron Pop @ 2023-06-27 20:19 UTC (permalink / raw)
  To: devel

Re sending and adding maintainers directly


On Monday, 6/26/2023 12:43 PM, Aaron Pop wrote:
> I was hoping to get a clarification on the IpmiGetSystemUuid from 
> MdeModulePkg...
> 
> IPMI specification's Get Device Guid Command says that data is returned 
> in "least signification byte first" and "this is the reverse of 
> convention described in RFC4122".
> 
> 
> IpmiGetSystemUuid specifies that it is returning an EFI_GUID. Does this 
> mean that the IpmigetSystemUuid is going to reverse the formatting and 
> return an EFI_GUID as opposed to what is directly returned from the 
> Management Device?
> 
> To help clarify:
> 
> 
> typedef struct {
>    UINT32    Data1;
>    UINT16    Data2;
>    UINT16    Data3;
>    UINT8     Data4[8];
> } EFI_GUID;
> 
> vs
> 
> 
> typedef struct {
>    UINT8     Data4[8];
>    UINT16    Data3;
>    UINT16    Data2;
>    UINT32    Data1;
> } IPMI_GUID;
> 
> Thank you,
> Aaron
> 
> 
> 
> 

^ permalink raw reply	[flat|nested] 2+ messages in thread

* Re: [edk2-devel] IpmiCommandLib's IpmiGetSystemUuid clarification
  2023-06-27 20:19 ` [edk2-devel] IpmiCommandLib's IpmiGetSystemUuid clarification Aaron Pop
@ 2023-06-28  8:43   ` Chang, Abner
  0 siblings, 0 replies; 2+ messages in thread
From: Chang, Abner @ 2023-06-28  8:43 UTC (permalink / raw)
  To: devel@edk2.groups.io, aaronpop@linux.microsoft.com

[AMD Official Use Only - General]

Hi Aaron,
Yes, this is a bug. We should copy the return GUID follow IPMI GUID format. Could you please create a Bugzilla ticket?  You can also send the patch to mailing list if you already had a fix.

Thank you.
Abner

> -----Original Message-----
> From: devel@edk2.groups.io <devel@edk2.groups.io> On Behalf Of Aaron
> Pop via groups.io
> Sent: Wednesday, June 28, 2023 4:19 AM
> To: devel@edk2.groups.io
> Subject: Re: [edk2-devel] IpmiCommandLib's IpmiGetSystemUuid clarification
>
> Caution: This message originated from an External Source. Use proper caution
> when opening attachments, clicking links, or responding.
>
>
> Re sending and adding maintainers directly
>
>
> On Monday, 6/26/2023 12:43 PM, Aaron Pop wrote:
> > I was hoping to get a clarification on the IpmiGetSystemUuid from
> > MdeModulePkg...
> >
> > IPMI specification's Get Device Guid Command says that data is returned
> > in "least signification byte first" and "this is the reverse of
> > convention described in RFC4122".
> >
> >
> > IpmiGetSystemUuid specifies that it is returning an EFI_GUID. Does this
> > mean that the IpmigetSystemUuid is going to reverse the formatting and
> > return an EFI_GUID as opposed to what is directly returned from the
> > Management Device?
> >
> > To help clarify:
> >
> >
> > typedef struct {
> >    UINT32    Data1;
> >    UINT16    Data2;
> >    UINT16    Data3;
> >    UINT8     Data4[8];
> > } EFI_GUID;
> >
> > vs
> >
> >
> > typedef struct {
> >    UINT8     Data4[8];
> >    UINT16    Data3;
> >    UINT16    Data2;
> >    UINT32    Data1;
> > } IPMI_GUID;
> >
> > Thank you,
> > Aaron
> >
> >
> >
> >
>
>
> 
>


^ permalink raw reply	[flat|nested] 2+ messages in thread

end of thread, other threads:[~2023-06-28  8:43 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
     [not found] <176C4F04309F3E96.4004@groups.io>
2023-06-27 20:19 ` [edk2-devel] IpmiCommandLib's IpmiGetSystemUuid clarification Aaron Pop
2023-06-28  8:43   ` Chang, Abner

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox