This is a note to let you know that I've just added the patch titled
debugobject: Ensure pool refill (again)
to the 5.10-stable tree which can be found at:
http://www.kernel.org/git/?p=linux/kernel/git/stable/stable-queue.git;a=summary
The filename of the patch is:
debugobject-ensure-pool-refill-again.patch
and it can be found in the queue-5.10 subdirectory.
If you, or anyone else, feels it should not be added to the stable tree,
please let <stable@xxxxxxxxxxxxxxx> know about it.
>From 0af462f19e635ad522f28981238334620881badc Mon Sep 17 00:00:00 2001
From: Thomas Gleixner <tglx@xxxxxxxxxxxxx>
Date: Mon, 1 May 2023 17:42:06 +0200
Subject: debugobject: Ensure pool refill (again)
From: Thomas Gleixner <tglx@xxxxxxxxxxxxx>
commit 0af462f19e635ad522f28981238334620881badc upstream.
The recent fix to ensure atomicity of lookup and allocation inadvertently
broke the pool refill mechanism.
Prior to that change debug_objects_activate() and debug_objecs_assert_init()
invoked debug_objecs_init() to set up the tracking object for statically
initialized objects. That's not longer the case and debug_objecs_init() is
now the only place which does pool refills.
Depending on the number of statically initialized objects this can be
enough to actually deplete the pool, which was observed by Ido via a
debugobjects OOM warning.
Restore the old behaviour by adding explicit refill opportunities to
debug_objects_activate() and debug_objecs_assert_init().
Fixes: 63a759694eed ("debugobject: Prevent init race with static objects")
Reported-by: Ido Schimmel <idosch@xxxxxxxxxx>
Signed-off-by: Thomas Gleixner <tglx@xxxxxxxxxxxxx>
Tested-by: Ido Schimmel <idosch@xxxxxxxxxx>
Link: https://lore.kernel.org/r/871qk05a9d.ffs@tglx
Signed-off-by: Greg Kroah-Hartman <gregkh@xxxxxxxxxxxxxxxxxxx>
---
lib/debugobjects.c | 16 +++++++++++++++-
1 file changed, 15 insertions(+), 1 deletion(-)
--- a/lib/debugobjects.c
+++ b/lib/debugobjects.c
@@ -590,6 +590,16 @@ static struct debug_obj *lookup_object_o
return NULL;
}
+static void debug_objects_fill_pool(void)
+{
+ /*
+ * On RT enabled kernels the pool refill must happen in preemptible
+ * context:
+ */
+ if (!IS_ENABLED(CONFIG_PREEMPT_RT) || preemptible())
+ fill_pool();
+}
+
static void
__debug_object_init(void *addr, const struct debug_obj_descr *descr, int onstack)
{
@@ -598,7 +608,7 @@ __debug_object_init(void *addr, const st
struct debug_obj *obj;
unsigned long flags;
- fill_pool();
+ debug_objects_fill_pool();
db = get_bucket((unsigned long) addr);
@@ -683,6 +693,8 @@ int debug_object_activate(void *addr, co
if (!debug_objects_enabled)
return 0;
+ debug_objects_fill_pool();
+
db = get_bucket((unsigned long) addr);
raw_spin_lock_irqsave(&db->lock, flags);
@@ -892,6 +904,8 @@ void debug_object_assert_init(void *addr
if (!debug_objects_enabled)
return;
+ debug_objects_fill_pool();
+
db = get_bucket((unsigned long) addr);
raw_spin_lock_irqsave(&db->lock, flags);
Patches currently in stable-queue which might be from tglx@xxxxxxxxxxxxx are
queue-5.10/x86-fpu-prevent-fpu-state-corruption.patch
queue-5.10/tick-sched-reduce-seqcount-held-scope-in-tick_do_upd.patch
queue-5.10/rcu-fix-missing-tick_dep_mask_rcu_exp-dependency-che.patch
queue-5.10/x86-ioapic-don-t-return-0-from-arch_dynirq_lower_bou.patch
queue-5.10/tick-sched-optimize-tick_do_update_jiffies64-further.patch
queue-5.10/debugobject-prevent-init-race-with-static-objects.patch
queue-5.10/tick-sched-use-tick_next_period-for-lockless-quick-c.patch
queue-5.10/tick-common-align-tick-period-with-the-hz-tick.patch
queue-5.10/posix-cpu-timers-implement-the-missing-timer_wait_running-callback.patch
queue-5.10/tick-get-rid-of-tick_period.patch
queue-5.10/debugobject-ensure-pool-refill-again.patch
