[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH v3 2/2] lockable: replaced locks with lock guard macros where
Dr. David Alan Gilbert
Re: [PATCH v3 2/2] lockable: replaced locks with lock guard macros where appropriate
Fri, 20 Mar 2020 13:55:31 +0000
* Paolo Bonzini (address@hidden) wrote:
> On 20/03/20 13:56, Dr. David Alan Gilbert wrote:
> >> According to https://wiki.qemu.org/ToDo/LockGuards cases that are trivial
> >> (no
> >> conditional logic) shouldn't be replaced.
> > OK
> I don't think that has to be either-or. Trivial lock/unlock sequences
> are not the first ones that should be converted, but there's an
> advantage in having a single patch that converts all possible uses of a
> lock. Trivial sequences certainly do not belong in a bigger patch like
> this, as they would make the patch even bigger.
> > So for what you've already got there,
> > For migration:
> > Acked-by: Dr. David Alan Gilbert <address@hidden>
> Can you just extract that and queue it yourself (for 5.1 probably)?
I can, although it would be easier if Daniel did that; there's no rush
given it's for 5.1
Dr. David Alan Gilbert / address@hidden / Manchester, UK
Re: [PATCH v3 0/2] Replaced locks with lock guard macros, no-reply, 2020/03/20