Re: [PATCH 3/3] kvm-s390: implement KVM_CAP_MAX_VCPUS

[Avi Kivity <avi@redhat.com>]

On 04/24/2012 10:24 AM, Christian Borntraeger wrote:
> Let userspace know the number of supported cpus fro kvm.
>
> Signed-off-by: Christian Borntraeger <borntraeger@de.ibm.com>
> ---
>  arch/s390/kvm/kvm-s390.c |    3 +++
>  1 file changed, 3 insertions(+)
>
> diff --git a/arch/s390/kvm/kvm-s390.c b/arch/s390/kvm/kvm-s390.c
> index fd98914..0054cb0 100644
> --- a/arch/s390/kvm/kvm-s390.c
> +++ b/arch/s390/kvm/kvm-s390.c
> @@ -136,6 +136,9 @@ int kvm_dev_ioctl_check_extension(long ext)
>  	case KVM_CAP_SYNC_REGS:
>  		r = 1;
>  		break;
> +	case KVM_CAP_NR_VCPUS:
> +		r = KVM_MAX_VCPUS;
> +		break;
>  	default:
>  		r = 0;
>  	}

This looks like a bugfix, but having survived so long without it, I
imagine it's not urgent for 3.4.

btw, a better place might be in kvm_devel_ioctl_check_extension_generic().

-- 
error compiling committee.c: too many arguments to function