| |
|
 |
|
 |
|
|
|
| |
|
|
xen-devel
[Xen-devel] Re: [PATCH] Linux: 6 arguments hypercall v2
On 24/06 03:46, Ian Campbell wrote:
> On Fri, 2011-06-24 at 15:00 +0100, Jean Guyader wrote:
> > Hi,
> >
> > This patch implements the 6 arguments hypercalls.
> > The sixth argument is passed using ebp or r9.
>
> Looks pretty good.
>
> I was able to get rid of more of the ifdefery etc with the following. It
> appears to do the right thing (by inspection of compiled code, not
> actual testing).
>
> Defining both __res and __arg6 as register("eax") seems slightly
> dubious, as is including them both in __HYPERCALL_6PARAM, but gcc
> doesn't seem to mind and it makes __HYPERCALL_6PARAM the same in both
> cases and gets rid of __HYPERCALL_DECLS6.
>
> Your version is probably less likely to tickle ggc bugs/behaviour
> changes in the future, so it's likely safer I think.
>
Thanks that makes the code much nicer, I didn't really want to declare
to variable in eax but if that works fine.
Also my patch modifies HYPERVISOR_set_trap_table by mistake but
shouldn't. I only used that for testing _hypercall6. I'll send
another version with your changes in but without the mistake
at the end.
Jean
_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxxxxxxxx
http://lists.xensource.com/xen-devel
|
|
|
| |
|
Home