[PATCH RFC v4 3/4] virtio-mmio: read shm region page size
Sergio Lopez
slp at redhat.com
Thu Feb 20 22:40:32 UTC 2025
Use the newly introduced SHM_PAGE_SHIFT register to read the page shift
for the shared memory region, derive the page size from it and store the
resulting value into virtio_shm_region.
Reviewed-by: Dmitry Osipenko <dmitry.osipenko at collabora.com>
Signed-off-by: Sergio Lopez <slp at redhat.com>
---
drivers/virtio/virtio_mmio.c | 20 +++++++++++++++++++-
include/uapi/linux/virtio_mmio.h | 3 +++
2 files changed, 22 insertions(+), 1 deletion(-)
diff --git a/drivers/virtio/virtio_mmio.c b/drivers/virtio/virtio_mmio.c
index 1f594b626d7a7734e8ec58766737a118c26bad94..5532450c9e9935399efbe9c5c6257293b6dcfc9e 100644
--- a/drivers/virtio/virtio_mmio.c
+++ b/drivers/virtio/virtio_mmio.c
@@ -537,6 +537,7 @@ static bool vm_get_shm_region(struct virtio_device *vdev,
struct virtio_shm_region *region, u8 id)
{
struct virtio_mmio_device *vm_dev = to_virtio_mmio_device(vdev);
+ u8 page_shift = 0;
u64 len, addr;
/* Select the region we're interested in */
@@ -560,7 +561,24 @@ static bool vm_get_shm_region(struct virtio_device *vdev,
region->addr = addr;
- region->page_size = 4096;
+ /* If supported by the device transport, read the region page size.
+ * The page_shift variable is initialized to zero above, so if this
+ * feature isn't supported it will result in a page_size of 4096, a
+ * default safe value.
+ */
+ if (__virtio_test_bit(vdev, VIRTIO_F_SHM_PAGE_SIZE)) {
+ page_shift = (u8) readl(vm_dev->base + VIRTIO_MMIO_SHM_PAGE_SHIFT);
+ /* The value derived from page_shift must fit into a 32-bit
+ * unsigned integer.
+ */
+ if ((page_shift + 12) >= 32) {
+ dev_err(&vdev->dev, "%s: device returned an invalid page shift: %d\n",
+ __func__, page_shift);
+ return false;
+ }
+ }
+
+ region->page_size = 1U << (page_shift + 12);
return true;
}
diff --git a/include/uapi/linux/virtio_mmio.h b/include/uapi/linux/virtio_mmio.h
index 0650f91bea6c70f935764070d825d181a2379afb..43348be30eff90ee228b6490b9d3c35ba4c50aa5 100644
--- a/include/uapi/linux/virtio_mmio.h
+++ b/include/uapi/linux/virtio_mmio.h
@@ -133,6 +133,9 @@
#define VIRTIO_MMIO_SHM_BASE_LOW 0x0b8
#define VIRTIO_MMIO_SHM_BASE_HIGH 0x0bc
+/* Shared memory region page shift */
+#define VIRTIO_MMIO_SHM_PAGE_SHIFT 0x0c4
+
/* Configuration atomicity value */
#define VIRTIO_MMIO_CONFIG_GENERATION 0x0fc
--
2.48.1
More information about the dri-devel
mailing list