Re: [PATCH 03/16] mm/slab: remove the checks for slab implementation bug

From: Joonsoo Kim
Date: Thu Jan 14 2016 - 11:20:47 EST


2016-01-15 0:23 GMT+09:00 Christoph Lameter <cl@xxxxxxxxx>:
> On Thu, 14 Jan 2016, Joonsoo Kim wrote:
>
>> Some of "#if DEBUG" are for reporting slab implementation bug
>> rather than user usecase bug. It's not really needed because slab
>> is stable for a quite long time and it makes code too dirty. This
>> patch remove it.
>
> Maybe better convert them to VM_BUG_ON() or so?

It's one possible solution but I'd like to make slab.c clean
as much as possible. Nowadays, SLAB code isn't changed
so much, therefore I don't think we need to keep them.

Thanks.