Messages in this thread | | | Date | Fri, 7 Feb 2020 07:46:30 +0100 | From | Gerd Hoffmann <> | Subject | Re: [PATCH] drm/virtio: fix ring free check |
| |
Hi,
> > + indirect = virtio_has_feature(vgdev->vdev, VIRTIO_RING_F_INDIRECT_DESC); > > + vqcnt = indirect ? 1 : elemcnt; > Is the feature dynamic and require the lock held? If not, the result > can be cached and the fixup can happen before grabbing the lock
Not dynamic, so yes, caching makes sense.
cheers, Gerd
| |