On Fri, Feb 27, 2026 at 02:46:49PM -0500, Steven Rostedt wrote: > On Fri, 27 Feb 2026 16:00:27 +0000 > Dmitry Ilvokhin <[email protected]> wrote: > > > static inline void zone_lock_init(struct zone *zone) > > { > > @@ -12,26 +59,41 @@ static inline void zone_lock_init(struct zone *zone) > > > > #define zone_lock_irqsave(zone, flags) \ > > do { \ > > + bool success = true; \ > > + \ > > + __zone_lock_trace_start_locking(zone); \ > > spin_lock_irqsave(&(zone)->_lock, flags); \ > > + __zone_lock_trace_acquire_returned(zone, success); \ > > Why the "success" variable and not just: > > __zone_lock_trace_acquire_returned(zone, true); > > ?
Good point, passing true directly is cleaner. Happy to respin if needed.
