On Tue, 10 Mar 2020 14:21:23 +0100 David Hildenbrand david@redhat.com wrote:
On 10.03.20 14:12, Christian Borntraeger wrote:
When we do the initial CPU reset we must not only clear the registers in the internal data structures but also in kvm_run sync_regs. For modern userspace sync_regs is the only place that it looks at.
Cc: stable@vger.kernel.org
# v?
Signed-off-by: Christian Borntraeger borntraeger@de.ibm.com
arch/s390/kvm/kvm-s390.c | 18 +++++++++++++++++- 1 file changed, 17 insertions(+), 1 deletion(-)
However, I do wonder if that ioctl *originally* was designed for that - IOW if this is rally a stable patch or just some change that makes sense. IIRC, userspace/QEMU always did the right thing, no? There was no documentation about the guarantees AFAIK.
The documentation only refers to the PoP for what is actually reset... should it also mention the sync regs?