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]> -- Cheers, David
