Re: [PATCH 10/12] net/mlx4: replace <linux/radix-tree.h> with <linux/radix-tree-root.h>
From: Masahiro Yamada
Date: Mon Oct 09 2017 - 08:03:46 EST
2017-10-09 2:32 GMT+09:00 Joe Perches <joe@xxxxxxxxxxx>:
> On Mon, 2017-10-09 at 02:29 +0900, Masahiro Yamada wrote:
>> The idea is simple; include necessary headers explicitly.
>
> Try that for kernel.h
>
> There's a reason aggregation of #includes is useful.
>
BTW, talking about <linux/kernel.h>, it is too much aggregation, isn't it?
It exceed 850 lines, and contains lots of unrelated stuff in one header.
Perhaps, it could be a good time to think about splitting?
For example, I see many trace_... things in it.
I wonder if linux/kernel.h is a good home for them, or a separate file.
--
Best Regards
Masahiro Yamada