Re: [patch] v2 mm/slub: restore/expand unfreeze_partials() local exclusion scope
From: Mike Galbraith
Date: Sun Jul 18 2021 - 04:12:12 EST
On Sun, 2021-07-18 at 09:58 +0200, Vlastimil Babka wrote:
> On 7/17/21 4:58 PM, Mike Galbraith wrote:
> > On Thu, 2021-07-15 at 18:34 +0200, Mike Galbraith wrote:
> > > Greetings crickets,
> > >
> > > Methinks he problem is the hole these patches opened only for RT.
> > >
> > > static void put_cpu_partial(struct kmem_cache *s, struct page
> > > *page,
> > > int drain)
> > > {
> > > #ifdef CONFIG_SLUB_CPU_PARTIAL
> > > struct page *oldpage;
> > > int pages;
> > > int pobjects;
> > >
> > > slub_get_cpu_ptr(s->cpu_slab);
> > > ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
> >
> > Bah, I'm tired of waiting to see what if anything mm folks do about
> > this little bugger, so I'm gonna step on it my damn self and be
> > done
> > with it. Fly or die little patchlet.
>
> Sorry, Mike. Got some badly timed sickness. Will check ASAP.
Hey, no apology necessary. I have no way of knowing what's going on
behind the scenes, but I also didn't like the notion of just walking
away from a busted thing, so given the silence, figured it's better to
post the working something I've got (wart and all) than to let the bug
go unchallenged. (take that damn bug.. pokes it with sharp stick;)
-Mike