On Fri, Dec 01, 2017 at 02:45:52PM +0900, Masatake YAMATO wrote:
> * kvm.c (kvm_ioctl): Handle KVM_RUN, KVM_GET_VCPU_MMAP_SIZE
>   and KVM_GET_API_VERSION.
> 
> Signed-off-by: Masatake YAMATO <yam...@redhat.com>
> ---
>  kvm.c | 4 ++++
>  1 file changed, 4 insertions(+)
> 
> diff --git a/kvm.c b/kvm.c
> index 8251ce7d..9fa7e248 100644
> --- a/kvm.c
> +++ b/kvm.c
> @@ -37,6 +37,10 @@ int
>  kvm_ioctl(struct tcb *const tcp, const unsigned int code, const 
> kernel_ulong_t arg)
>  {
>       switch (code) {
> +     /* Commands not taking any arguments. */

These commands don't use the argument, but the kernel checks whether the
argument is equal to zero or not, so the comment is not quite correct.

> +     case KVM_RUN:
> +     case KVM_GET_VCPU_MMAP_SIZE:
> +     case KVM_GET_API_VERSION:
>       default:
>               return RVAL_DECODED;

The change is correct, though.


-- 
ldv

Attachment: signature.asc
Description: PGP signature

------------------------------------------------------------------------------
Check out the vibrant tech community on one of the world's most
engaging tech sites, Slashdot.org! http://sdm.link/slashdot
_______________________________________________
Strace-devel mailing list
Strace-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/strace-devel

Reply via email to