Re: [edk2] [Patch 1/2] UefiCpuPkg/RegisterCpuFeaturesLib: Separate semaphore container.

2018-11-09 Thread Ni, Ruiyu
Eric, Patch is good. Reviewed-by: Ruiyu Ni Some modification to commit message to help understanding the changes. Please rewording if you think some of them is not proper. You can push the changes if no concerns about the new commit message. - In current implementation, core and package l

[edk2] [Patch 1/2] UefiCpuPkg/RegisterCpuFeaturesLib: Separate semaphore container.

2018-11-07 Thread Eric Dong
In current implementation, core level semaphore use same container with package level semaphore. This design will let the core level semaphore not works as expected in below case: 1. Feature A has CPU_FEATURE_CORE_BEFORE dependence with Feature B. 2. Feature C has CPU_FEATURE_PACKAGE_AFTER dependen