On 14.03.2016 15:53, Cédric Le Goater wrote: > Hello Thomas, > > On 03/11/2016 03:46 PM, Thomas Huth wrote: >> I'd maybe also squash the "Add KVM numbers to some P8 SPRs" patch into >> the "Add a bunch of hypervisor SPRs to Book3s" to reduce the code churn >> a little bit (note that the patch description for the "Add a bunch of >> hypervisor SPRs to Book3s" patch is misleading - DAWR and DAWRX need to >> be synchronized for migration, since the guest can set this register via >> the H_SET_MODE hypercall). > > So shall I just remove the comment : > > "We don't give them a KVM reg number yet as no current KVM version > supports HV mode." > > from patch "Add a bunch of hypervisor SPRs to Book3s" after squashing in > "Add KVM numbers to some P8 SPRs" ?
Either that, or change it into something like: "We don't give them a KVM reg number to most of the registers yet as no current KVM version supports HV mode. For DAWR and DAWRX, the KVM reg number is needed since this register can be set by the guest via the H_SET_MODE hypercall." ? Thomas
