search for: dev_to_virtio

to_vp_device is worth changing to inlined definition. Signed-off-by: Wanlong Gao <gaowanlong at cn.fujitsu.com> --- drivers/virtio/virtio_pci.c | 6 +----- 1 file changed, 1 insertion(+), 5 deletions(-) diff --git a/drivers/virtio/virtio_pci.c b/drivers/virtio/virtio_pci.c index e3ecc94..7681fe3 100644 --- a/drivers/virtio/virtio_pci.c +++ b/drivers/virtio/virtio_pci.c @@ -98,11 +98,7 @@

to_vp_device is worth changing to inlined definition. Signed-off-by: Wanlong Gao <gaowanlong at cn.fujitsu.com> --- drivers/virtio/virtio_pci.c | 6 +----- 1 file changed, 1 insertion(+), 5 deletions(-) diff --git a/drivers/virtio/virtio_pci.c b/drivers/virtio/virtio_pci.c index e3ecc94..7681fe3 100644 --- a/drivers/virtio/virtio_pci.c +++ b/drivers/virtio/virtio_pci.c @@ -98,11 +98,7 @@

Use dev_to_virtio() instead of open-coding it. Signed-off-by: Geliang Tang <geliangtang at 163.com> --- drivers/s390/virtio/virtio_ccw.c | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git a/drivers/s390/virtio/virtio_ccw.c b/drivers/s390/virtio/virtio_ccw.c index 1b83159..97231e1 100644 ---...

Use dev_to_virtio() instead of open-coding it. Signed-off-by: Geliang Tang <geliangtang at 163.com> --- drivers/s390/virtio/virtio_ccw.c | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git a/drivers/s390/virtio/virtio_ccw.c b/drivers/s390/virtio/virtio_ccw.c index 1b83159..97231e1 100644 ---...

From: Geliang Tang <geliangtang at 163.com> Use dev_to_virtio() instead of open-coding it. Signed-off-by: Geliang Tang <geliangtang at 163.com> Message-Id: <912bf59bd3a48f2d4d4994681e898dc084fe29d3.1451484163.git.geliangtang at 163.com> Signed-off-by: Cornelia Huck <cornelia.huck at de.ibm.com> --- drivers/s390/virtio/virtio_ccw.c | 3 +--...

This is a leftover from the drm_bus days, where we've had a bus-specific device type for every bus type in drm_device. Except for pci (which we can't remove because dri1 drivers) this is all gone. And the virt driver also doesn't really need it, dev_to_virtio works perfectly fine. Cc: David Airlie <airlied at linux.ie> Cc: Gerd Hoffmann <kraxel at redhat.com> Cc: virtualization at lists.linux-foundation.org Signed-off-by: Daniel Vetter <daniel.vetter at intel.com> --- drivers/gpu/drm/virtio/virtgpu_drm_bus.c | 1 - drivers/gpu/drm/vi...

This is a leftover from the drm_bus days, where we've had a bus-specific device type for every bus type in drm_device. Except for pci (which we can't remove because dri1 drivers) this is all gone. And the virt driver also doesn't really need it, dev_to_virtio works perfectly fine. Cc: David Airlie <airlied at linux.ie> Cc: Gerd Hoffmann <kraxel at redhat.com> Cc: virtualization at lists.linux-foundation.org Signed-off-by: Daniel Vetter <daniel.vetter at intel.com> --- drivers/gpu/drm/virtio/virtgpu_drm_bus.c | 1 - drivers/gpu/drm/vi...

...s > > + * > > + * This API is only for pre-mapped buffers, for non premapped buffers virtio > > + * core handles DMA API internally. > > + */ > > +bool virtio_dma_need_sync(struct device *dev, dma_addr_t addr) > > +{ > > + struct virtio_device *vdev = dev_to_virtio(dev); > > + > > + if (!vring_use_dma_api(vdev)) > > + return 0; > > + > > + return dma_need_sync(vdev->dev.parent, addr); > > +} > > +EXPORT_SYMBOL_GPL(virtio_dma_need_sync); > > + > > +/** > > + * virtio_dma_s...

...; > > > This (and the reset) needs to be done per virtqueue instead per device > > after b0e504e5505d184b0be248b7dcdbe50b79f03758 ("virtio_ring: per > > virtqueue dma device"). > > > YES. > > > > > > > + struct virtio_device *vdev = dev_to_virtio(dev); > > > + > > > + if (!vring_use_dma_api(vdev)) > > > + return page_to_phys(page) + offset; > > > + > > > + return dma_map_page(vdev->dev.parent, page, offset, length, dir); > > > +} > > > > Need eith...

...; > > > This (and the reset) needs to be done per virtqueue instead per device > > after b0e504e5505d184b0be248b7dcdbe50b79f03758 ("virtio_ring: per > > virtqueue dma device"). > > > YES. > > > > > > > + struct virtio_device *vdev = dev_to_virtio(dev); > > > + > > > + if (!vring_use_dma_api(vdev)) > > > + return page_to_phys(page) + offset; > > > + > > > + return dma_map_page(vdev->dev.parent, page, offset, length, dir); > > > +} > > > > Need eith...

Hi Michael, here are two virtio/s390 patches (one cleanup, one bugfix), prepared against your vhost branch of mst/vhost.git. Please apply. Cornelia Huck (1): virtio/s390: size of SET_IND payload Geliang Tang (1): virtio/s390: use dev_to_virtio drivers/s390/virtio/virtio_ccw.c | 15 +++++++++------ 1 file changed, 9 insertions(+), 6 deletions(-) -- 2.3.9

Hi Michael, here are two virtio/s390 patches (one cleanup, one bugfix), prepared against your vhost branch of mst/vhost.git. Please apply. Cornelia Huck (1): virtio/s390: size of SET_IND payload Geliang Tang (1): virtio/s390: use dev_to_virtio drivers/s390/virtio/virtio_ccw.c | 15 +++++++++------ 1 file changed, 9 insertions(+), 6 deletions(-) -- 2.3.9

...ngth, enum dma_data_direction dir) > > +{ > > This (and the reset) needs to be done per virtqueue instead per device > after b0e504e5505d184b0be248b7dcdbe50b79f03758 ("virtio_ring: per > virtqueue dma device"). YES. > > > + struct virtio_device *vdev = dev_to_virtio(dev); > > + > > + if (!vring_use_dma_api(vdev)) > > + return page_to_phys(page) + offset; > > + > > + return dma_map_page(vdev->dev.parent, page, offset, length, dir); > > +} > > Need either inline or EXPORT_SYMBOL_GPL() here....

...1 file changed, 19 insertions(+), 8 deletions(-) diff --git a/drivers/virtio/virtio.c b/drivers/virtio/virtio.c index ee59b74..fed0ce1 100644 --- a/drivers/virtio/virtio.c +++ b/drivers/virtio/virtio.c @@ -13,18 +13,24 @@ static ssize_t device_show(struct device *_d, struct virtio_device *dev = dev_to_virtio(_d); return sprintf(buf, "0x%04x\n", dev->id.device); } +static DEVICE_ATTR_RO(device); + static ssize_t vendor_show(struct device *_d, struct device_attribute *attr, char *buf) { struct virtio_device *dev = dev_to_virtio(_d); return sprintf(buf, "0x%04x\n", d...

...1 file changed, 19 insertions(+), 8 deletions(-) diff --git a/drivers/virtio/virtio.c b/drivers/virtio/virtio.c index ee59b74..fed0ce1 100644 --- a/drivers/virtio/virtio.c +++ b/drivers/virtio/virtio.c @@ -13,18 +13,24 @@ static ssize_t device_show(struct device *_d, struct virtio_device *dev = dev_to_virtio(_d); return sprintf(buf, "0x%04x\n", dev->id.device); } +static DEVICE_ATTR_RO(device); + static ssize_t vendor_show(struct device *_d, struct device_attribute *attr, char *buf) { struct virtio_device *dev = dev_to_virtio(_d); return sprintf(buf, "0x%04x\n", d...

...file changed, 19 insertions(+), 8 deletions(-) diff --git a/drivers/virtio/virtio.c b/drivers/virtio/virtio.c index ee59b747..fed0ce19 100644 --- a/drivers/virtio/virtio.c +++ b/drivers/virtio/virtio.c @@ -13,18 +13,24 @@ static ssize_t device_show(struct device *_d, struct virtio_device *dev = dev_to_virtio(_d); return sprintf(buf, "0x%04x\n", dev->id.device); } +static DEVICE_ATTR_RO(device); + static ssize_t vendor_show(struct device *_d, struct device_attribute *attr, char *buf) { struct virtio_device *dev = dev_to_virtio(_d); return sprintf(buf, "0x%04x\n", d...

...file changed, 19 insertions(+), 8 deletions(-) diff --git a/drivers/virtio/virtio.c b/drivers/virtio/virtio.c index ee59b747..fed0ce19 100644 --- a/drivers/virtio/virtio.c +++ b/drivers/virtio/virtio.c @@ -13,18 +13,24 @@ static ssize_t device_show(struct device *_d, struct virtio_device *dev = dev_to_virtio(_d); return sprintf(buf, "0x%04x\n", dev->id.device); } +static DEVICE_ATTR_RO(device); + static ssize_t vendor_show(struct device *_d, struct device_attribute *attr, char *buf) { struct virtio_device *dev = dev_to_virtio(_d); return sprintf(buf, "0x%04x\n", d...

...t; > > > after b0e504e5505d184b0be248b7dcdbe50b79f03758 ("virtio_ring: per > > > > virtqueue dma device"). > > > > > > > > > YES. > > > > > > > > > > > > > > > + struct virtio_device *vdev = dev_to_virtio(dev); > > > > > + > > > > > + if (!vring_use_dma_api(vdev)) > > > > > + return page_to_phys(page) + offset; > > > > > + > > > > > + return dma_map_page(vdev->dev.parent, page, offset, length, dir...

...0be248b7dcdbe50b79f03758 ("virtio_ring: per > > > > > virtqueue dma device"). > > > > > > > > > > > > YES. > > > > > > > > > > > > > > > > > > > + struct virtio_device *vdev = dev_to_virtio(dev); > > > > > > + > > > > > > + if (!vring_use_dma_api(vdev)) > > > > > > + return page_to_phys(page) + offset; > > > > > > + > > > > > > + return dma_map_page(vdev->dev.parent,...

...t; > > > > > > > > > > > > > > > > > > > > YES. > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > + struct virtio_device *vdev = dev_to_virtio(dev); > > > > > > > > > + > > > > > > > > > + if (!vring_use_dma_api(vdev)) > > > > > > > > > + return page_to_phys(page) + offset; > > > > > > > > > + > > > >...