Re: [PATCH v2] misc: Reserve minor for VFIO

2013-12-19 Thread Alex Williamson
On Thu, 2013-12-19 at 21:38 -0500, Konrad Rzeszutek Wilk wrote: > On Wed, Dec 18, 2013 at 02:02:38PM -0800, H. Peter Anvin wrote: > > On 12/18/2013 01:01 PM, Greg KH wrote: > > > On Wed, Dec 18, 2013 at 01:56:32PM -0700, Alex Williamson wrote: > > >> VFIO currently allocates it's own dynamic charde

Re: [PATCH v2] misc: Reserve minor for VFIO

2013-12-19 Thread Konrad Rzeszutek Wilk
On Wed, Dec 18, 2013 at 02:02:38PM -0800, H. Peter Anvin wrote: > On 12/18/2013 01:01 PM, Greg KH wrote: > > On Wed, Dec 18, 2013 at 01:56:32PM -0700, Alex Williamson wrote: > >> VFIO currently allocates it's own dynamic chardev range, reserving the > >> first minor for the control part of the inte

Re: [PATCH v2] misc: Reserve minor for VFIO

2013-12-18 Thread H. Peter Anvin
On 12/18/2013 03:55 PM, One Thousand Gnomes wrote: >> I think Alan Cox was the last person to man ... Alan, >> are you still doing that? (Otherwise patching the file in the Linux >> kernel tree seems eminently sensible... there really isn't any need to >> reserve numbers for out-of-tree drivers an

Re: [PATCH v2] misc: Reserve minor for VFIO

2013-12-18 Thread One Thousand Gnomes
> I think Alan Cox was the last person to man ... Alan, > are you still doing that? (Otherwise patching the file in the Linux > kernel tree seems eminently sensible... there really isn't any need to > reserve numbers for out-of-tree drivers anymore. Just another perk of > being in-tree.) I am so

Re: [PATCH v2] misc: Reserve minor for VFIO

2013-12-18 Thread H. Peter Anvin
On 12/18/2013 01:01 PM, Greg KH wrote: > On Wed, Dec 18, 2013 at 01:56:32PM -0700, Alex Williamson wrote: >> VFIO currently allocates it's own dynamic chardev range, reserving the >> first minor for the control part of the interface (/dev/vfio/vfio) and >> the remainder for VFIO groups (/dev/vfio/$

Re: [PATCH v2] misc: Reserve minor for VFIO

2013-12-18 Thread Alex Williamson
On Wed, 2013-12-18 at 13:01 -0800, Greg KH wrote: > On Wed, Dec 18, 2013 at 01:56:32PM -0700, Alex Williamson wrote: > > VFIO currently allocates it's own dynamic chardev range, reserving the > > first minor for the control part of the interface (/dev/vfio/vfio) and > > the remainder for VFIO group

Re: [PATCH v2] misc: Reserve minor for VFIO

2013-12-18 Thread Greg KH
On Wed, Dec 18, 2013 at 01:56:32PM -0700, Alex Williamson wrote: > VFIO currently allocates it's own dynamic chardev range, reserving the > first minor for the control part of the interface (/dev/vfio/vfio) and > the remainder for VFIO groups (/dev/vfio/$GROUP). This works, but it > doesn't suppor

[PATCH v2] misc: Reserve minor for VFIO

2013-12-18 Thread Alex Williamson
VFIO currently allocates it's own dynamic chardev range, reserving the first minor for the control part of the interface (/dev/vfio/vfio) and the remainder for VFIO groups (/dev/vfio/$GROUP). This works, but it doesn't support auto loading. For instance when libvirt checks for VFIO support it loo