Re: [PATCH] virtio-pci: alloc only resources actually used.

2015-06-23 Thread Michael S. Tsirkin
On Wed, Jun 24, 2015 at 07:54:15AM +0200, Gerd Hoffmann wrote: > Move resource allocation from common code to legacy and modern code. > Only request resources actually used, i.e. bar0 in legacy mode and > the bar(s) specified by capabilities in modern mode. > > Signed-off-by: Gerd Hoffmann > ---

[PATCH] virtio-pci: alloc only resources actually used.

2015-06-23 Thread Gerd Hoffmann
Move resource allocation from common code to legacy and modern code. Only request resources actually used, i.e. bar0 in legacy mode and the bar(s) specified by capabilities in modern mode. Signed-off-by: Gerd Hoffmann --- drivers/virtio/virtio_pci_common.c | 7 --- drivers/virtio/virtio_pci

Re: [PATCH] virtio-pci: alloc only resources actually used.

2015-06-23 Thread Michael S. Tsirkin
On Tue, Jun 23, 2015 at 03:54:48PM +0200, Gerd Hoffmann wrote: > Move resource allocation from common code to legacy and modern code. > Only request resources actually used, i.e. bar0 in legacy mode and > the bar(s) specified by capabilities in modern mode. > > Signed-off-by: Gerd Hoffmann Looks

[PATCH] virtio-pci: alloc only resources actually used.

2015-06-23 Thread Gerd Hoffmann
Move resource allocation from common code to legacy and modern code. Only request resources actually used, i.e. bar0 in legacy mode and the bar(s) specified by capabilities in modern mode. Signed-off-by: Gerd Hoffmann --- drivers/virtio/virtio_pci_common.c | 7 --- drivers/virtio/virtio_pci

Re: [PATCH] virtio-pci: alloc only resources actually used.

2015-06-18 Thread Michael S. Tsirkin
On Thu, Jun 18, 2015 at 10:14:40AM +0200, Gerd Hoffmann wrote: > Hi, > > > > +static struct resource *request_capability(struct pci_dev *dev, int off, > > > +const char *name) > > > +{ > > > + u8 bar; > > > + u32 offset, length; > > > + > > > + pci_read_config

Re: [PATCH] virtio-pci: alloc only resources actually used.

2015-06-18 Thread Gerd Hoffmann
Hi, > > +static struct resource *request_capability(struct pci_dev *dev, int off, > > + const char *name) > > +{ > > + u8 bar; > > + u32 offset, length; > > + > > + pci_read_config_byte(dev, off + offsetof(struct virtio_pci_cap, > > +

Re: [PATCH] virtio-pci: alloc only resources actually used.

2015-06-16 Thread Michael S. Tsirkin
On Tue, Jun 16, 2015 at 03:57:52PM +0200, Gerd Hoffmann wrote: > Move resource allocation from common code to legacy and modern code. > Only request resources actually used, i.e. bar0 in legacy mode and > the regions specified by capabilities in modern mode. > > Signed-off-by: Gerd Hoffmann > ---

[PATCH] virtio-pci: alloc only resources actually used.

2015-06-16 Thread Gerd Hoffmann
Move resource allocation from common code to legacy and modern code. Only request resources actually used, i.e. bar0 in legacy mode and the regions specified by capabilities in modern mode. Signed-off-by: Gerd Hoffmann --- drivers/virtio/virtio_pci_common.c | 7 - drivers/virtio/virtio_pci_