Re: [PATCH 2/7] genalloc: selftest

From: Igor Stoppa
Date: Mon Feb 26 2018 - 07:12:36 EST




On 24/02/18 00:42, J Freyensee wrote:
>
>> + locations[action->location] = gen_pool_alloc(pool, action->size);
>> + BUG_ON(!locations[action->location]);
>
> Again, I'd think it through if you really want to use BUG_ON() or not:
>
> https://lwn.net/Articles/13183/
> https://lkml.org/lkml/2016/10/4/1

Is it acceptable to display only a WARNing, in case of risking damaging
a mounted filesystem?

--
igor