[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

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


  • To: Jan Beulich <JBeulich@xxxxxxxx>
  • From: "Liu, Jinsong" <jinsong.liu@xxxxxxxxx>
  • Date: Thu, 28 Feb 2013 08:40:36 +0000
  • Accept-language: en-US
  • Cc: "Ren, Yongjie" <yongjie.ren@xxxxxxxxx>, xen-devel <xen-devel@xxxxxxxxxxxxx>
  • Delivery-date: Thu, 28 Feb 2013 08:47:48 +0000
  • List-id: Xen developer discussion <xen-devel.lists.xen.org>
  • Thread-index: AQHOFY1DaDPIeN3VQXC+k/Nn+vsjXZiO8zqg
  • Thread-topic: [PATCH 2/2] Minor fix for rdmsrl

Jan Beulich wrote:
>>>> 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

No need backporting, just minor cleanup to drop unnecessary ';'

Thanks,
Jinsong

> 
>> --- 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


 


Rackspace

Lists.xenproject.org is hosted with RackSpace, monitoring our
servers 24x7x365 and backed by RackSpace's Fanatical Support®.