[<prev] [next>] [<thread-prev] [day] [month] [year] [list]
Message-ID: <1516783522.2230.6.camel@tiscali.nl>
Date: Wed, 24 Jan 2018 09:45:22 +0100
From: Paul Bolle <pebolle@...cali.nl>
To: Matthew Wilcox <willy@...radead.org>
Cc: Matthew Wilcox <mawilcox@...rosoft.com>, linux-mm@...ck.org,
linux-fsdevel@...r.kernel.org,
linux-f2fs-devel@...ts.sourceforge.net,
linux-nilfs@...r.kernel.org, linux-btrfs@...r.kernel.org,
linux-xfs@...r.kernel.org, linux-usb@...r.kernel.org,
Bjorn Andersson <bjorn.andersson@...aro.org>,
Stefano Stabellini <sstabellini@...nel.org>,
iommu@...ts.linux-foundation.org, linux-remoteproc@...r.kernel.org,
linux-s390@...r.kernel.org, intel-gfx@...ts.freedesktop.org,
cgroups@...r.kernel.org, linux-sh@...r.kernel.org,
David Howells <dhowells@...hat.com>,
linux-kernel@...r.kernel.org
Subject: Re: [PATCH v6 05/99] xarray: Add definition of struct xarray
Mathhew,
Just a minor question.
On Wed, 2018-01-17 at 12:20 -0800, Matthew Wilcox wrote:
> This is a direct replacement for struct radix_tree_root. Some of the
> struct members have changed name; convert those, and use a #define so
> that radix_tree users continue to work without change.
>
> Signed-off-by: Matthew Wilcox <mawilcox@...rosoft.com>
> --- a/include/linux/xarray.h
> +++ b/include/linux/xarray.h
> @@ -10,6 +10,8 @@
> */
>
> #include <linux/bug.h>
> +#include <linux/compiler.h>
> +#include <linux/kconfig.h>
The top Makefile includes linux/kconfig.h globally. (See the odd USERINCLUDE
variable, which is actually part of the LINUXINCLUDE variable, but split off
to make things confusing.)
Why do you need to include linux/kconfig.h here?
> #include <linux/spinlock.h>
> #include <linux/types.h>
Thanks,
Paul Bolle
Powered by blists - more mailing lists