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

2015-06-18 Thread Michael S. Tsirkin
On Thu, Jun 18, 2015 at 12:33:20PM +0200, Gerd Hoffmann wrote: Hi, + if (offset + length offset || + offset + length pci_resource_len(dev, bar)) { + dev_err(dev-dev, + virtio_pci: virtio capability %u@%u + out of range on bar %i

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

2015-06-18 Thread Michael S. Tsirkin
On Thu, Jun 18, 2015 at 11:27:34AM +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 v2] virtio-pci: alloc only resources actually used.

2015-06-18 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 kra...@redhat.com --- drivers/virtio/virtio_pci_common.c | 7

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

2015-06-18 Thread Gerd Hoffmann
Hi, + if (offset + length offset || + offset + length pci_resource_len(dev, bar)) { + dev_err(dev-dev, + virtio_pci: virtio capability %u@%u + out of range on bar %i length %lu\n, + length, offset, bar, +