Re: [Xen-devel] [PATCH 2/2] Minor fix for rdmsrl

["Jan Beulich" <JBeulich@xxxxxxxx>]

>>> On 27.02.13 at 17:46, "Liu, Jinsong" <jinsong.liu@xxxxxxxxx> wrote:
> Minor fix for rdmsrl
> 
> Signed-off-by: Liu Jinsong <jinsong.liu@xxxxxxxxx>

I applied this one, but could you clarify whether the stray
semicolon caused any actual problem? I'm mainly asking to
determine whether this needs backporting.

Jan

> --- a/xen/include/asm-x86/msr.h       Thu Feb 28 08:05:15 2013 +0800
> +++ b/xen/include/asm-x86/msr.h       Thu Feb 28 08:15:17 2013 +0800
> @@ -20,7 +20,7 @@
>                           : "=a" (a__), "=d" (b__) \
>                           : "c" (msr)); \
>         val = a__ | ((u64)b__<<32); \
> -} while(0);
> +} while(0)
>  
>  #define wrmsr(msr,val1,val2) \
>       __asm__ __volatile__("wrmsr" \




_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxx
http://lists.xen.org/xen-devel