On Mon, Jun 21, 2021 at 11:46 AM Paolo Bonzini <pbonzini@xxxxxxxxxx> wrote: > > On 19/06/21 00:27, Jing Zhang wrote: > > + struct kvm_vcpu_stat stat; > > struct kvm_dirty_ring dirty_ring; > > + char stats_id[KVM_STATS_NAME_SIZE]; > > I think stats_id needs to be part of the usercopy region too. > > You can also use > > offsetofend(struct kvm_vcpu, stats_id) - offsetof(struct kvm_vcpu, arch) > > to compute the size. Sure! Will do. > > Paolo > Thanks, Jing