[<prev] [next>] [thread-next>] [day] [month] [year] [list]
Message-ID: <alpine.LFD.1.00.0802091345150.14859@localhost.localdomain>
Date: Sat, 9 Feb 2008 14:07:39 -0500 (EST)
From: "Robert P. J. Day" <rpjday@...shcourse.ca>
To: Linux Kernel Mailing List <linux-kernel@...r.kernel.org>
Subject: some questions about generated headers
just a few observations about the generated headers and whether
there's some cleanup to be done there.
1)
the generated header linux/tty.h consists solely of:
=====
#ifndef _LINUX_TTY_H
#define _LINUX_TTY_H
/*
* 'tty.h' defines some structures used by tty_io.c and some defines.
*/
#endif
=====
i once submitted a patch that, if this header were included from
userspace, would print, "Don't include me from user space, I'm empty."
or something to that effect. is that worth doing? or should user
space code be including that header at all? (should an empty header
even be *exported* to user space?)
2)
include/linux/soundcard.h still contains some weird, non-standard
checks:
...
#if (!defined(__KERNEL__) && !defined(KERNEL) && !defined(INKERNEL) &&
!defined(_KERNEL)) || defined(USE_SEQ_MACROS)
...
so what's with KERNEL or INKERNEL or _KERNEL? do those tests still
have any value? or can they be tossed?
3)
related to 2), what's up with __KERNEL as well? it still shows up
in a number of places in the kernel source. for example, see
include/linux/hdsmart.h:
...
#ifndef __KERNEL
...
#endif /* __KERNEL__ *
(and that missing trailing slash creeps me out, too.)
4)
any chance of replacing the current unifdef with sunifdef (son of
unifdef), which is smarter and can simplify some of the compound
logical preprocessor checks to get rid of more junk in those generated
headers?
http://www.sunifdef.strudl.org/
that should do for now.
rday
--
========================================================================
Robert P. J. Day
Linux Consulting, Training and Annoying Kernel Pedantry
Waterloo, Ontario, CANADA
Home page: http://crashcourse.ca
Fedora Cookbook: http://crashcourse.ca/wiki/index.php/Fedora_Cookbook
========================================================================
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
Powered by blists - more mailing lists