Re: [Linaro-mm-sig] Re: [PATCH v3] drm/virtio: use uninterruptible resv lock for plane updates

[Deepanshu Kartikey]

On Sun, May 17, 2026 at 9:44 PM Dmitry Osipenko
<dmitry.osipenko@xxxxxxxxxxxxx> wrote:
>
> On 5/15/26 11:40, Deepanshu Kartikey wrote:
> > +int virtio_gpu_array_lock_resv_uninterruptible(struct virtio_gpu_object_array *objs)
> > +{
> > +     unsigned int i;
> > +     int ret = 0;
> > +
> > +     if (objs->nents == 1) {
> > +             dma_resv_lock(objs->objs[0]->resv, NULL);
> > +     } else {
> > +             ret = drm_gem_lock_reservations(objs->objs, objs->nents,
> > +                                             &objs->ticket);
>
> drm_gem_lock_reservations() is interruptible. Given that only one BO
> needs to be locked for the fix, make it
> virtio_gpu_lock_one_resv_uninterruptible() and fail with -EINVAL if
> objs->nents > 1
>
> --
> Best regards,
> Dmitry
>

I have sent patch v4.

Thanks

Deepanshu