Re: [PATCH v4 06/19] irqdomain: Drop revmap mutex

On Wed, Jan 18 2023 at 14:10, Johan Hovold wrote:

On Wed, Jan 18, 2023 at 02:05:29PM +0100, Thomas Gleixner wrote:

quoted

You can do this in a two step approach.

    1) Add the new locking and ensure that the lock is held when
       the functions are called

But the new locking has nothing to do with these functions. They are
added because they fix various races elsewhere. Adding lockdep
assertions in unrelated function as part of those fixes doesn't really
make much sense.

Seriously? The point is that revmap_mutex is not protecting against any
of the races which you are trying to protect against. Ergo, any callsite
which does not hold the irqdomain mutex is part of the problem you are
trying to solve, no?

The removal of the revmap_mutex becomes possible due to the overall
protection scheme rework _after_ you established that all callers hold
the irqdomain mutex.

Thanks,

        tglx

_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel