On Mon, Sep 28, 2020 at 07:20:42AM +0000, Krish Sadhukhan wrote: > According to section "CR3" in APM vol. 2, the non-MBZ reserved bits in CR3 > need to be set by software as follows: > > "Reserved Bits. Reserved fields should be cleared to 0 by software > when writing CR3." Nothing in the shortlog or changelog actually states what this patch does. "Test non-MBZ reserved bits in CR3 in long mode" is rather ambiguous, and IIUC, the changelog is straight up misleading. Based on the discussion from v1, I _think_ this test verifies that KVM does _not_ fail nested VMRUN if non-MBZ bits are set, correct? If so, then something like: KVM: nSVM: Verify non-MBZ CR3 reserved bits can be set in long mode with further explanation in the changelog would be very helpful.
