Re: [Xen-devel] [PATCH 3 of 7] xen: allows more hypercalls from stubdoms

[Stefano Stabellini <stefano.stabellini@xxxxxxxxxxxxx>]

On Mon, 12 Oct 2009, Keir Fraser wrote:
> On 12/10/2009 18:20, "Stefano Stabellini" <stefano.stabellini@xxxxxxxxxxxxx>
> wrote:
> 
> > +        case XEN_DOMCTL_ioport_mapping:
> > +        case XEN_DOMCTL_memory_mapping:
> > +        case XEN_DOMCTL_bind_pt_irq:
> > +        case XEN_DOMCTL_unbind_pt_irq:
> > +        case XEN_DOMCTL_assign_device:
> > +        case XEN_DOMCTL_deassign_device:
> 
> This kind of thing, for example, while we're talking about least
> privilege... I think it's wrong-headed in the first place for this kind of
> control-plane activity to be going on in qemu. Surely it belongs in the
> toolstack? Yes, I know it's a pain in the bum that this means modifying
> multiple toolstacks! :-)
> 

I agree with you that we need to redesign these hypercalls, but I am a
fan of "doing one thing at a time" so I think we should decouple this
goal from the other one of making passthrough work with stubdom for the
moment.
This way we could first let people test it as it is, fix some bugs that
probably still affect the code, fix pci coldplug and add MSI-X support,
then redesign the hypercalls.

In any case it is just a matter of ordering the items in the todo list,
all these things have to happen anyway.


_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxxxxxxxx
http://lists.xensource.com/xen-devel