On 05/03/20 02:34, Sean Christopherson wrote: > > In theory, everything up to the refactoring is non-controversial, i.e. we > can bikeshed the refactoring without delaying the bug fixes. Even the refactoring itself is much less controversial. I queued everything, there's always time to unqueue. Paolo > v2: > - Use Jan's patch to fix the trace bug. [Everyone] > - Rework Hypervisor/Centaur handling so that only the Hypervisor > sub-ranges get the restrictive 0xffffff00 mask, and so that Centaur's > range only gets recognized when the guest vendor is Centaur. [Jim] > - Add the aforementioned bug fixes. > - Add a patch to do build time assertions on the vendor string, which > are hand coded u32s in the emulator (for direct comparison against > CPUID register output). > - Drop the patch to add CPUID.maxphyaddr emulator helper. [Paolo] > - Redo refactoring patches to land them after all the bug fixes > and to do the refactoring without any semantic changes in the > emulator.