Re: [PATCH v5 9/9] Documentation: virtual: kvm: Support vcpu preempted check

[xinhui]

On 2016å10æ22æ 02:39, rkrcmar@xxxxxxxxxx wrote:
> 2016-10-21 11:27+0000, David Laight:
> > From: Pan Xinhui
> > > Sent: 20 October 2016 22:28
> > > Commit ("x86, kvm: support vcpu preempted check") add one field "__u8
> > > preempted" into struct kvm_steal_time. This field tells if one vcpu is
> > > running or not.
> > >
> > > It is zero if 1) some old KVM deos not support this filed. 2) the vcpu is
> > > preempted. Other values means the vcpu has been preempted.
> > >
> > > Signed-off-by: Pan Xinhui <xinhui.pan@xxxxxxxxxxxxxxxxxx>
> > > ---
> > >   Documentation/virtual/kvm/msr.txt | 8 +++++++-
> > >   1 file changed, 7 insertions(+), 1 deletion(-)
> > >
> > > diff --git a/Documentation/virtual/kvm/msr.txt b/Documentation/virtual/kvm/msr.txt
> > > index 2a71c8f..3376f13 100644
> > > --- a/Documentation/virtual/kvm/msr.txt
> > > +++ b/Documentation/virtual/kvm/msr.txt
> > > @@ -208,7 +208,8 @@ MSR_KVM_STEAL_TIME: 0x4b564d03
> > >   		__u64 steal;
> > >   		__u32 version;
> > >   		__u32 flags;
> > > -		__u32 pad[12];
> > > +		__u8  preempted;
> > > +		__u32 pad[11];
> > >   	}
> >
> > I think I'd be explicit about the 3 pad bytes you've left.
>
> Seconded.
>
> With that change are all KVM bits
like below?
__u8 preempted;
__u8 kvm_pad[3];


> Acked-by: Radim KrÄmÃÅ <rkrcmar@xxxxxxxxxx>
thanks!