Re: [ANNOUNCE] kvm-36 release

Avi Kivity
2007-09-14T17:19:30Z

Wei Huang wrote:
> The compilation errors also showed up on my boxes. The attached patch
> fixes the errors by renaming variables.
>
> Signed-off-by: Wei Huang <wei.huang2-5C7GfCeVMHo@public.gmane.org>
> diff --git a/drivers/kvm/kvm_main.c b/drivers/kvm/kvm_main.c
> index 99e4917..f04ca6b 100644
> --- a/drivers/kvm/kvm_main.c
> +++ b/drivers/kvm/kvm_main.c
> @@ -2967,25 +2967,25 @@ static long kvm_vcpu_ioctl(struct file *
> break;
> }
> case KVM_GET_LAPIC: {
> - struct kvm_lapic_state lapic;
> + struct kvm_lapic_state lapic_state;
>
> - memset(&lapic, 0, sizeof lapic);
> - r = kvm_vcpu_ioctl_get_lapic(vcpu, &lapic);
> + memset(&lapic_state, 0, sizeof lapic_state);
> + r = kvm_vcpu_ioctl_get_lapic(vcpu, &lapic_state);
> if (r)

I committed an equivalent fix, but one that only affects the external
module, since the issue is not present with current kernels.

--
Do not meddle in the internals of kernels, for they are subtle and quick to panic.

-------------------------------------------------------------------------
This SF.net email is sponsored by: Microsoft
Defy all challenges. Microsoft(R) Visual Studio 2005.