[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [Xen-devel] [PATCH 3/3] xen/include: Include xen/kconfig.h automatically



>>> On 16.02.17 at 11:40, <andrew.cooper3@xxxxxxxxxx> wrote:
> On 16/02/17 10:27, Jan Beulich wrote:
>>>>> On 15.02.17 at 19:10, <andrew.cooper3@xxxxxxxxxx> wrote:
>>> generated/autoconf.h is already included automatically so CONFIG_* defines 
>>> are
>>> avaialble.  However, the companion macros such as IS_ENABLED() are not
>>> included.
>>>
>>> Include them uniformally everywhere.
>> Well, if you really think this is a good idea, I'm not going to stand in
>> the way, but why do we need this included everywhere? Many files
>> don't even care about any CONFIG_*, and hence even less so about
>> kconfig.h.
> 
> I am sorry, but you are complaining if I include it unilaterally, and
> also complaining if I include kconfig.h in the specific location where I
> need it.  These are mutually exclusive.

I don't understand - when did I complain about its inclusion where
it's needed? Iirc my complaint was about you adding the inclusion
to */config.h without that header actually using the macros. My
point really is that ideally each C file would get as little cruft as
possible, while at present quite a number of header are being
included by virtually every source file.

Jan


_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxx
https://lists.xen.org/xen-devel

 


Rackspace

Lists.xenproject.org is hosted with RackSpace, monitoring our
servers 24x7x365 and backed by RackSpace's Fanatical Support®.