linux-toradex.git/drivers/virtio/virtio_pci_modern.c, branch v4.2-rc2

virtio-pci: alloc only resources actually used.

2015-06-24T06:15:09+00:00

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 

Signed-off-by: Michael S. Tsirkin

virtio_pci: support non-legacy balloon devices

2015-04-15T03:11:13+00:00

virtio_device_is_legacy_only is always false now,
drop the test from virtio pci.

Signed-off-by: Michael S. Tsirkin 
Signed-off-by: Rusty Russell

virtio_pci_modern: switch to type-safe io accessors

2015-04-01T04:13:34+00:00

As Rusty noted, we were accessing queue_enable with an incorrect width.
Switch to type-safe accessors so we don't make this mistake again in the
future.

Signed-off-by: Michael S. Tsirkin 
Signed-off-by: Rusty Russell

virtio_pci_modern: type-safe io accessors

2015-04-01T04:07:15+00:00

The spec is very clear on this:

4.1.3.1 Driver Requirements: PCI Device Layout

The driver MUST access each field using the “natural” access method,
i.e. 32-bit accesses for 32-bit fields, 16-bit accesses for 16-bit
fields and 8-bit accesses for 8-bit fields.

Add type-safe wrappers to prevent access with incorrect width.

Signed-off-by: Michael S. Tsirkin 
Signed-off-by: Rusty Russell

virtio_pci: use 16-bit accessor for queue_enable.

2015-02-11T04:33:16+00:00

Since PCI is little endian, 8-bit access might work, but the spec section
is very clear on this:

  4.1.3.1 Driver Requirements: PCI Device Layout

  The driver MUST access each field using the “natural” access method,
  i.e. 32-bit accesses for 32-bit fields, 16-bit accesses for 16-bit
  fields and 8-bit accesses for 8-bit fields.

Signed-off-by: Rusty Russell 
Acked-by: Michael S. Tsirkin

virtio_pci_modern: drop an unused function

2015-01-21T05:59:01+00:00

release function in modern driver is unused:
it's a left-over from when each driver had
to have its own release.

Signed-off-by: Michael S. Tsirkin

virtio_pci_modern: support devices with no config

2015-01-21T05:58:55+00:00

Virtio 1.0 spec lists device config as optional.
Set get/set callbacks to NULL. Drivers can check that
and fail gracefully.

Signed-off-by: Michael S. Tsirkin 
Signed-off-by: Rusty Russell

virtio_pci_modern: reduce number of mappings

2015-01-21T05:58:55+00:00

We don't know the # of VQs that drivers are going to use so it's hard to
predict how much memory we'll need to map. However, the relevant
capability does give us an upper limit.
If that's below a page, we can reduce the number of required
mappings by mapping it all once ahead of the time.

Signed-off-by: Michael S. Tsirkin 
Signed-off-by: Rusty Russell

virtio_pci: macros for PCI layout offsets

2015-01-21T05:58:54+00:00

QEMU wants it, so why not?  Trust, but verify.

Signed-off-by: Rusty Russell 
Signed-off-by: Michael S. Tsirkin

virtio_pci: modern driver

2015-01-21T05:58:53+00:00

Lightly tested against qemu.

One thing *not* implemented here is separate mappings
for descriptor/avail/used rings. That's nice to have,
will be done later after we have core support.

This also exposes the PCI layout to userspace, and
adds macros for PCI layout offsets:

QEMU wants it, so why not?  Trust, but verify.

Signed-off-by: Rusty Russell 
Signed-off-by: Michael S. Tsirkin