[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <45DD7271.6000503@qumranet.com>
Date: Thu, 22 Feb 2007 12:37:37 +0200
From: Avi Kivity <avi@...ranet.com>
To: Joerg Roedel <joerg.roedel@....com>
CC: Dor Laor <dor.laor@...ranet.com>, Pavel Machek <pavel@....cz>,
kvm-devel@...ts.sourceforge.net, akpm@...l.org,
linux-kernel@...r.kernel.org
Subject: Re: [kvm-devel] [PATCH 10/13] KVM: Wire up hypercall handlers to
a central arch-independent location
Joerg Roedel wrote:
> On Thu, Feb 22, 2007 at 02:17:54AM -0800, Dor Laor wrote:
>
>> I have Ingo's network PV hypercalls to commit in my piplien.
>> Till then we can just add the test hypercall:
>> case __NR_hypercall_test:
>> printk(KERN_DEBUG "%s __NR_hypercall_test\n",
>> __FUNCTION__);
>> ret = 0x5a5a;
>> break;
>> default:
>> BUG();
>>
>
> Is it a good idea to call BUG() on an undefined hypercall? I assume its
> better to inform the guest that is has issued an illegal call...
>
>
The code returns -KVM_EINVAL if the hypercall is not handled (and BUG()
on the host as a response to something controlled by a guest is not a
good idea).
--
error compiling committee.c: too many arguments to function
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
Powered by blists - more mailing lists