Skip site navigation (1)Skip section navigation (2)
Date:      Mon, 11 Nov 2013 17:00:39 +0100
From:      =?ISO-8859-1?Q?Roger_Pau_Monn=E9?= <roger.pau@citrix.com>
To:        Sergey Kandaurov <pluknet@gmail.com>, Pavel Timofeev <timp87@gmail.com>
Cc:        "freebsd-virtualization@freebsd.org" <freebsd-virtualization@freebsd.org>
Subject:   Re: Running `sysctl -a` causes crash (Hyper-V)
Message-ID:  <5280FF27.1090405@citrix.com>
In-Reply-To: <CAE-mSOLszJLfHBgmSTaUH5%2B5dTe23XLUFbp-nhfPpz2UVWMCJQ@mail.gmail.com>
References:  <CAAoTqfvrdKH6BC=KrpWLgBKDjx402oTt_bp1TnV_Z3FC1if88A@mail.gmail.com> <CAE-mSOLKgk_h10mUPLuMXHgPfYJMEog_FFdyZvEoAOXMh=rbfg@mail.gmail.com> <CAAoTqfu9NFuVFYnLFo%2BLG7_kXd%2BAaxEPTm4nCKETzWvje7=fyA@mail.gmail.com> <CAE-mSOLeMHkh7gKLZPYcyu3GdJKqN4hAMBgEf5TwMUxB4nQ7zA@mail.gmail.com> <CAAoTqftAbsDNUiCv4QRnNfamf61dPhJ%2BWAzgfVVL4QMe9qHxcA@mail.gmail.com> <CAE-mSOLszJLfHBgmSTaUH5%2B5dTe23XLUFbp-nhfPpz2UVWMCJQ@mail.gmail.com>

next in thread | previous in thread | raw e-mail | index | archive | help
On 11/11/13 15:31, Sergey Kandaurov wrote:
> On 11 November 2013 18:13, Pavel Timofeev <timp87@gmail.com> wrote:
>> [...]
>> (kgdb) p vm_guest
>> $1 = 3
>> Current language:  auto; currently minimal
>> (kgdb)
>>
> 
> What if you try this change?
> 
> Index: sys/kern/subr_param.c
> ===================================================================
> --- sys/kern/subr_param.c    (revision 257695)
> +++ sys/kern/subr_param.c    (working copy)
> @@ -153,6 +153,7 @@
>      "none",
>      "generic",
>      "xen",
> +    "hv",
>      NULL
>  };

This is my fault, I've added VM_GUEST_HV to the guest type enum without
knowing there was another array in a completely different file that
contained a description for those values.

If you (or someone else) are going to commit this change, I would
recommend adding a BIG comment in sys/systm.h above the declaration of
VM_GUEST explaining that if you add a new guest type there you also need
to add a description in subr_param.c, this will probably prevent
something like this happening again.




Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?5280FF27.1090405>