Hi Ray,
Of curse, using the flexible arrays is better than my version and I will adjust this in V3.
+
+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".