Thread (17 messages) flat view 17 messages, 5 authors, 2021-10-13

Re: [PATCH v2 3/4] lib/rhashtable: Replace kernel.h with the necessary inclusions

From: Andy Shevchenko <andriy.shevchenko@linux.intel.com>
Date: 2021-10-07 11:44:58
Also in: linux-kselftest, linux-media, lkml

On Thu, Oct 07, 2021 at 07:23:28PM +0800, Herbert Xu wrote:
On Thu, Oct 07, 2021 at 12:51:28PM +0300, Andy Shevchenko wrote:
quoted
When kernel.h is used in the headers it adds a lot into dependency hell,
especially when there are circular dependencies are involved.
quoted
Replace kernel.h inclusion with the list of what is really being used.
Nack.  I can see the benefits of splitting things out of kernel.h
but there is no reason to add crap to end users like rhashtable.c.
Crap is in the kernel.h. Could you elaborate how making a proper list
of the inclusions is a crap?

-- 
With Best Regards,
Andy Shevchenko

Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help