Re: [PATCH] mm: zswap: make the lock critical section obvious in shrink_worker()

[Nhat Pham <nphamcs@xxxxxxxxx>]

On Fri, Aug 2, 2024 at 10:33 PM Yosry Ahmed <yosryahmed@xxxxxxxxxx> wrote:
>
> Move the comments and spin_{lock/unlock}() calls around in
> shrink_worker() to make it obvious the lock is protecting the loop
> updating zswap_next_shrink.
>
> Signed-off-by: Yosry Ahmed <yosryahmed@xxxxxxxxxx>

Thanks, it looks cleaner to me too.

Reviewed-by: Nhat Pham <nphamcs@xxxxxxxxx>