> -----Original Message----- > From: Xen-devel [mailto:xen-devel-boun...@lists.xenproject.org] On Behalf > Of George Dunlap > Sent: 25 January 2018 18:15 > To: Anthony Perard <anthony.per...@citrix.com> > Cc: Kevin Stange <ke...@steadfast.net>; xen-devel@lists.xenproject.org; > George Dunlap <george.dun...@citrix.com> > Subject: Re: [Xen-devel] PCI Device Subtree Change from Traditional to > Upstream > > On 01/25/2018 06:04 PM, Anthony PERARD wrote: > > On Thu, Jan 25, 2018 at 05:54:36PM +0000, George Dunlap wrote: > >> On 01/04/2018 12:52 PM, Anthony PERARD wrote: > >>> From: Anthony PERARD <anthony.per...@citrix.com> > >>> Subject: [PATCH] libxl_dm: Explicitly put xen-platform device on PCI slot > 3 > >>> > >>> In order to do that, we don't use xenfv machine anymore and explicity > >>> add the platform device on the command line. > >>> > >>> Signed-off-by: Anthony PERARD <anthony.per...@citrix.com> > >> > >> Anthony, > >> > >> It seems like we might want to add the ability to specify which slot we > >> want the xen-platform device to occupy. Is it worth thinking of the best > >> way to add a patch like this upstream? > > > > I think that would be nice for people who switch from qemu-trad to > > QEMU. The only question that remain is, how to name the xl config > > option? The rest is to simply take my libxl patch and make it use > > the new config option. > > Well the other half would be to make sure something like this doesn't > happen by accident in the future -- i.e., that no future changes in QEMU > will accidentally move it away from whatever the current slot is now. >
IMO, it would be best if libxl specified the bus topology exactly (i.e. specified devfn for everything that appears on the bus). If we, as I hope, make a move to having Xen rather than QEMU own the topology then this should hopefully ensure at least some degree of forward compatibility. Paul > -George > > _______________________________________________ > Xen-devel mailing list > Xen-devel@lists.xenproject.org > https://lists.xenproject.org/mailman/listinfo/xen-devel _______________________________________________ Xen-devel mailing list Xen-devel@lists.xenproject.org https://lists.xenproject.org/mailman/listinfo/xen-devel