Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) client-ip=2620:137:e000::1:20;
Feedback-ID: i494840e7:Fastmail
Date:   Tue, 2 May 2023 08:53:29 +0300
From:   Ido Schimmel <idosch@idosch.org>
To:     Thomas Gleixner <tglx@linutronix.de>
Cc:     Schspa Shi <schspa@gmail.com>, longman@redhat.com,
        swboyd@chromium.org, linux@roeck-us.net, wuchi.zero@gmail.com,
        linux-kernel@vger.kernel.org,
        syzbot+5093ba19745994288b53@syzkaller.appspotmail.com,
        danieller@nvidia.com, petrm@nvidia.com
Subject: Re: [PATCH] debugobject: Prevent init race with static objects
Message-ID: <ZFClWdkHOA0Eoybu@shredder>
References: <20230303161906.831686-1-schspa@gmail.com>
 <87bkl9jt3a.ffs@tglx>
 <m2ttycd8kx.fsf@gmail.com>
 <87sfdw8yru.ffs@tglx>
 <87pm908xvu.ffs@tglx>
 <87zg7dzgao.ffs@tglx>
 <ZE/BQ5yUQZUwhlTu@shredder>
 <871qk05a9d.ffs@tglx>
MIME-Version: 1.0
Content-Type: text/plain; charset=us-ascii
Content-Disposition: inline
In-Reply-To: <871qk05a9d.ffs@tglx>
Precedence: bulk

On Mon, May 01, 2023 at 05:42:06PM +0200, Thomas Gleixner wrote:
> The change in that patch is neither debug_objects_activate() nor
> debug_objecs_assert_init() no longer invoke debug_object_init() which is
> now the only place doing pool refills. So depending on the number of
> statically allocated objects this might deplete the pool quick enough.
> 
> Does the patch below restore the old behaviour?

Yes. Couldn't reproduce the issue with the proposed fix. Feel free to
add:

Tested-by: Ido Schimmel <idosch@nvidia.com>

Thanks for the quick fix!