>> @@ -837,10 +837,8 @@ static int add_module_usage(struct module *a, struct
>> module *b)
>>
>> pr_debug("Allocating new usage for %s.\n", a->name);
>> use = kmalloc(sizeof(*use), GFP_ATOMIC);
>> - if (!use) {
>> - pr_warn("%s: out of memory loading\n", a->name);
>> + if (!use)
>> return -ENOMEM;
>> - }
>
> IMO this is removing useful information.
How do you think about to clarify the circumstances any further?
> Although stack traces are generated on alloc failures,
Do you ever want to switch them off for special use cases?
> the extra print also tells us which module we were trying to load
> at the time the memory allocation failed.
Can a default allocation failure report provide the same information already?
Regards,
Markus
