Re: [PATCH 4/6] KVM: x86: Introduce KVM_GET_SREGS2 / KVM_SET_SREGS2

From: Paolo Bonzini
Date: Thu Apr 01 2021 - 13:51:42 EST


On 01/04/21 19:10, Maxim Levitsky wrote:
I haven't yet studied in depth the locking that is used in the kvm,
so I put this to be on the safe side.

I looked at it a bit and it looks like the pdptr reading code takes
this lock because it accesses the memslots, which is not done here,
and therefore the lock is indeed not needed here.

I need to study in depth how locking is done in kvm to be 100% sure
about this.

Yes, SRCU protects reading the memslots (and therefore accessing guest memory).

195, not 196.

I am also planning to add KVM_CAP_SET_GUEST_DEBUG2 for which I
used 195.

Sounds good then.

Paolo