Re: [edk2-devel] [PATCH v1 05/19] DynamicTablesPkg: Fix Proc node length assignment

2019-08-23 Thread Alexei Fedorov
Reviewed-by: Alexei Fedorov -=-=-=-=-=-=-=-=-=-=-=- Groups.io Links: You receive all messages sent to this group. View/Reply Online (#46281): https://edk2.groups.io/g/devel/message/46281 Mute This Topic: https://groups.io/mt/32999779/21656 Group Owner: devel+ow...@edk2.groups.io Unsubscribe: htt

[edk2-devel] [PATCH v1 05/19] DynamicTablesPkg: Fix Proc node length assignment

2019-08-23 Thread Sami Mujawar
The length field for the Processor Hierarchy node structure is 8-bit wide while the number of private resource field is 32-bit wide. Therefore, the GetProcHierarchyNodeSize() returns the size as a 32-bit value. The VS2017 compiler reports 'warning C4244: '=': conversion from 'UINT32' to 'UINT8', p