On Fri, Feb 27, 2026 at 09:39:11PM +0100, David Hildenbrand (Arm) wrote: > On 2/27/26 17:00, Dmitry Ilvokhin wrote: > > Replace direct zone lock acquire/release operations with the > > newly introduced wrappers. > > > > The changes are purely mechanical substitutions. No functional change > > intended. Locking semantics and ordering remain unchanged. > > > > The compaction path is left unchanged for now and will be > > handled separately in the following patch due to additional > > non-trivial modifications. > > > > Signed-off-by: Dmitry Ilvokhin <[email protected]> > > Acked-by: Shakeel Butt <[email protected]> > > Reviewed-by: SeongJae Park <[email protected]> > > --- > > [...] > > > #ifdef CONFIG_COMPACTION > > @@ -530,11 +531,14 @@ static bool compact_lock_irqsave(spinlock_t *lock, > > unsigned long *flags, > > * Returns true if compaction should abort due to fatal signal pending. > > * Returns false when compaction can continue. > > */ > > -static bool compact_unlock_should_abort(spinlock_t *lock, > > - unsigned long flags, bool *locked, struct compact_control *cc) > > + > > +static bool compact_unlock_should_abort(struct zone *zone, > > + unsigned long flags, > > + bool *locked, > > + struct compact_control *cc) > > We tend to use two-tabs on second parameter line; like the existing code > did. > > > Besides that > > Acked-by: David Hildenbrand (Arm) <[email protected]> >
Thanks, David. Noted. Appreciate the review and ack. > -- > Cheers, > > David
