On 12/20/2010 07:57 PM, Avi Kivity wrote:
On 12/20/2010 07:53 PM, Randy Dunlap wrote:
On 12/20/10 09:47, Avi Kivity wrote:
> On 12/20/2010 06:56 PM, Randy Dunlap wrote:
>> On 12/20/10 01:06, Avi Kivity wrote:
>> > 'error' is byte sized, so use a byte register constraint.
>>
>> Hm, OK, but I still get the same build error.
>
> Not okay then. Can you verify that by removing the content of the
> function the problem goes away?
Nope. I put #if 0 / #endif around the asm() and if (error) lines
but I still get the error message:
linux-next-20101217/arch/x86/kvm/vmx.c: Assembler messages:
linux-next-20101217/arch/x86/kvm/vmx.c:488: Error: bad register name
`%sil'
And line 488 was one of those under #if 0?
which is on the "u8 error;" line below:
static void vmcs_load(struct vmcs *vmcs)
{
u64 phys_addr = __pa(vmcs);
u8 error;
Er, you just answered me. Are you sure gcc saw your update? what
happens if you comment that line as well? assign to 'error' instead?
Wierd.
--
I have a truly marvellous patch that fixes the bug which this
signature is too narrow to contain.
--
To unsubscribe from this list: send the line "unsubscribe kvm" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
