Re: [PATCH v2 1/8] Revert "binder: switch alloc->mutex to spinlock_t"

From: Carlos Llamas
Date: Thu Nov 07 2024 - 09:30:30 EST


On Thu, Nov 07, 2024 at 09:56:16AM +0100, Greg Kroah-Hartman wrote:
> On Thu, Nov 07, 2024 at 04:02:23AM +0000, Carlos Llamas wrote:
> > This reverts commit 7710e2cca32e7f3958480e8bd44f50e29d0c2509.
> >
> > In preparation for concurrent page installations, restore the original
> > alloc->mutex which will serialize zap_page_range_single() against page
> > installations in subsequent patches (instead of the mmap_sem).
> >
> > Cc: Mukesh Ojha <quic_mojha@xxxxxxxxxxx>
> > [cmllamas: fix trivial conflict due to 2c10a20f5e84a]
>
> Nit, commits have the full name in them as well, not just the sha1 one,
> i.e.:
> 2c10a20f5e84 ("binder_alloc: Fix sleeping function called from invalid context")
>

Sounds good, done for v3.

Cheers,
Carlos Llamas