[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index] [Xen-devel] RE: main_pcilist_assignable
Stefano, I found libxl missing many of the checking we had in xm python code when we passthrough pci devices to the guest. Attached patch makes sure the passthrough device belongs to pciback before allow them passthrough to the guest. There are still many other checks missing. Xm terminates the guest startup process when this type of condition is found. This patch just allows the guest to continue to boot but with no device passthrough. I tried to call exit() by the guest windows still hangs around. Is there an example code for calling exit that also closes the guest (QEMU VGA) window? Allen -----Original Message----- From: Stefano Stabellini [mailto:stefano.stabellini@xxxxxxxxxxxxx] Sent: Monday, February 28, 2011 3:14 AM To: Kay, Allen M Cc: Stefano Stabellini; xen-devel@xxxxxxxxxxxxxxxxxxx Subject: Re: main_pcilist_assignable On Sat, 26 Feb 2011, Kay, Allen M wrote: > Hi Stefano, > > I found xl allows me to passthrough PCI devices to guests even when I did not > hide them to either pciback or pci-stub (xm has this check). > > Libxl has main_pcilist_assignable() that seems suppose to do something > similar but I'm having trouble finding the callers. > > Can you tell me which code calls this function? main_pcilist_assignable is called when the user executes xl pci-list-assignable-devices the function main_pcilist_assignable is in the cmd_table (tools/libxl/xl_cmdtable.c). In any case main_pcilist_assignable is just a wrapper around libxl_device_pci_list_assignable, that contains the interesting implementation details. libxl_device_pci_list_assignable is only called by main_pcilist_assignable, but it could be called before actually doing any device assignement, for example in xl_cmdimpl.c:pciattach. Attachment:
libxl_pci.patch _______________________________________________ Xen-devel mailing list Xen-devel@xxxxxxxxxxxxxxxxxxx http://lists.xensource.com/xen-devel
|
Lists.xenproject.org is hosted with RackSpace, monitoring our |