Re: [PATCH] mm: slub: Fix slub_debug disablement for list of slabs

From: Vlastimil Babka
Date: Tue Jul 27 2021 - 08:22:40 EST


On 7/27/21 6:43 AM, Vijayanand Jitta wrote:
>
>
> On 7/27/2021 4:02 AM, Vlastimil Babka wrote:
>> On 7/13/21 1:45 PM, vjitta@xxxxxxxxxxxxxx wrote:
>>> From: Vijayanand Jitta <vjitta@xxxxxxxxxxxxxx>
>>>
>>> Consider the scenario where CONFIG_SLUB_DEBUG_ON is set
>>> and we would want to disable slub_debug for few slabs.
>>> Using boot parameter with slub_debug=-,slab_name syntax
>>> doesn't work as expected i.e; only disabling debugging for
>>> the specified list of slabs, instead it disables debugging
>>> for all slabs. Fix this.
>>>
>>> Signed-off-by: Vijayanand Jitta <vjitta@xxxxxxxxxxxxxx>
>>
>> Would the following work too, and perhaps be easier to follow?
>
> Right, the below change would also work and its easier to follow as you
> said. We can go with this.
>
> Reviewed-by: Vijayanand Jitta <vjitta@xxxxxxxxxxxxxx>

Thanks! Here's the full patch
----8<----