Re: [PATCH 12/36] drm/amd: remove _unlocked suffix in drm_object_put_unlocked

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Hi Christian,

On Fri, 8 May 2020 at 10:23, Christian König <christian.koenig@xxxxxxx> wrote:
>
> Am 07.05.20 um 17:07 schrieb Emil Velikov:
> > From: Emil Velikov <emil.velikov@xxxxxxxxxxxxx>
> >
> > Spelling out _unlocked for each and every driver is a annoying.
> > Especially if we consider how many drivers, do not know (or need to)
> > about the horror stories involving struct_mutex.
> >
> > Just drop the suffix. It makes the API cleaner.
>
> In general sounds like a good idea to me, but I don't see the full
> patchset in my mailbox.
>
The series is quite long, you can see it here
https://patchwork.freedesktop.org/series/77043/

> With the current code this change would probably trigger the
> "WARN_ON(!mutex_is_locked(&obj->dev->struct_mutex));" in
> drm_gem_object_put().
>
Earlier patch renames drm_gem_object_put, adding explicit "_locked"
suffix to it.

If the series is applied atomically, then that issue will not happen.
Doing so might cause trivial merge conflicts, for individual drivers.
Nothing a simple build check won't highlight/catch.

Although if the AMD team has pending work which will seriously clash
with this, could delay the "remove _unlocked suffix" part of the
series.

-Emil
_______________________________________________
dri-devel mailing list
dri-devel@xxxxxxxxxxxxxxxxxxxxx
https://lists.freedesktop.org/mailman/listinfo/dri-devel




[Index of Archives]     [Linux DRI Users]     [Linux Intel Graphics]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]     [XFree86]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Linux Kernel]     [Linux SCSI]     [XFree86]
  Powered by Linux