Hi Ray,


Thanks,
Chao
On 2024/4/15 10:10, Ni, Ray wrote:
>
> +
> +typedef struct {
> +  UINT32    NumberOfProcessor;
> +  UINTN     *ApicId;
>
> [Ray] How about "UINTN  ApicId[]"? The difference between the two is 
> your version contains an 8-byte pointer pointing to somewhere else. My 
> version implies the APIC ID array is just after "NumberOfProcessor".
Of curse, using the flexible arrays is better than my version and I will 
adjust this in V3.
> 


-=-=-=-=-=-=-=-=-=-=-=-
Groups.io Links: You receive all messages sent to this group.
View/Reply Online (#117816): https://edk2.groups.io/g/devel/message/117816
Mute This Topic: https://groups.io/mt/105478501/7686176
Group Owner: devel+owner@edk2.groups.io
Unsubscribe: https://edk2.groups.io/g/devel/unsub [rebecca@openfw.io]
-=-=-=-=-=-=-=-=-=-=-=-