From: David Woodhouse <dwmw2@infradead.org>
To: torvalds@osdl.org, akpm@osdl.org
Cc: linux-kernel@vger.kernel.org
Subject: Simple header cleanups
Date: Thu, 27 Apr 2006 03:13:42 +0100 [thread overview]
Message-ID: <1146104023.2885.15.camel@hades.cambridge.redhat.com> (raw)
Andrew (or preferably Linus since these are fairly simple and
unintrusive bug fixes), please pull from my tree at
git://git.infradead.org/hdrcleanup-2.6.git
(Gitweb at http://git.infradead.org/?p=hdrcleanup-2.6.git;a=summary)
This tree contains a number of simple fixes for kernel headers which are
currently exposing things to userspace that should not be present
outside #ifdef __KERNEL__.
Since ifdefs are horrid, these patches tend not to add new ones --
mostly it's a case of moving things inside existing ifdefs, rather than
adding new ones.
Most of the patches are small and self-contained, except for the one
which removes #include <linux/config.h> from everything under include/
The individual commits are as follows:
Remove user-visible references to PAGE_SIZE in include/asm-powerpc/elf.h
Include <linux/jiffies.h> from linux/acct.h only in kernel-private part.
Don't include agp_backend.h in user-visible part of agpgart.h
Use __KERNEL__ to hide kernel-private bits of linux/gameport.h
Export only the appropriate GS_xxx flags to userspace from generic_serial.h
Include various private files only from within __KERNEL__ in genhd.h
Sanitise linux/i2c-algo-ite.h for userspace consumption
Sanitise linux/i2c.h for userspace consumption
Don't include <linux/device.h> from user-visible part of linux/ipmi.h
Remove gratuitous inclusion of <linux/pci.h> from linux/isdn/tpam.h
Sanitise linux/mman.h for userspace consumption
Don't include private files from user-visible part of linux/ncp_fs.h
Don't include <linux/list.h> from user-visible part of linux/msg.h
Don't include <linux/stringify> from user-visible part of linux/net.h
Don't include private headers from user-visible parts of include/linux/nfs*.h
Don't include private headers from user-visible parts of linux/quota.h
Don't include <linux/list.h> from user-visible part of reiserfs_xattr.h
Partially sanitise linux/sched.h for userspace consumption
Don't include <asm/atomic.h> from user-visible part of linux/sem.h
Don't include private headers from user-visible part of linux/signal.h
Move comment in mtd-abi.h to stop confusing unifdef
Don't include <linux/spinlock.h> from user-visible part of linux/wanrouter.h
Don't export CONFIG_COMPAT stuff in linux/usbdevice_fs.h to userspace
Sanitise linux/sunrpc/debug.h for userspace consumption
Don't include private headers from user-visible part of linux/smb_fs.h
Don't include private headers from user-visible part of linux/ext2_fs.h
Don't include private headers from user-visible part of linux/ext3_fs.h
Don't include <linux/config.h> and <linux/linkage.h> from linux/socket.h
Don't include linux/config.h from anywhere else in include/
Sanitise linux/audit.h for userspace consumption, split elf-em.h from elf.h
Sanitise linux/sched.h for userspace consumption
Excluding the one-line files which have just <linux/config.h> removed,
the diffstat looks like this:
asm-m32r/mmu_context.h | 2
asm-powerpc/elf.h | 7 --
asm-ppc/page.h | 2
asm-sparc/system.h | 2
linux/acct.h | 3
linux/agpgart.h | 3
linux/audit.h | 4 -
linux/elf-em.h | 44 +++++++++++++
linux/elf.h | 59 -----------------
linux/ext2_fs.h | 2
linux/ext3_fs.h | 7 --
linux/gameport.h | 6 +
linux/generic_serial.h | 6 +
linux/genhd.h | 12 +--
linux/i2c-algo-ite.h | 7 +-
linux/i2c.h | 9 +-
linux/ipmi.h | 2
linux/mman.h | 12 ++-
linux/msg.h | 2
linux/ncp_fs.h | 5 -
linux/net.h | 3
linux/nfs.h | 8 +-
linux/nfs4.h | 6 -
linux/nfs_fs.h | 39 +++++------
linux/quota.h | 4 -
linux/reiserfs_xattr.h | 3
linux/sched.h | 90 +++++++++++++--------------
linux/sem.h | 2
linux/signal.h | 4 -
linux/smb_fs.h | 4 -
linux/socket.h | 2
linux/sunrpc/debug.h | 24 +++----
linux/usbdevice_fs.h | 2
linux/wanrouter.h | 4 -
mtd/mtd-abi.h | 5 -
864 files changed, 191 insertions(+), 1034 deletions(-)
--
dwmw2
next reply other threads:[~2006-04-27 2:13 UTC|newest]
Thread overview: 31+ messages / expand[flat|nested] mbox.gz Atom feed top
2006-04-27 2:13 David Woodhouse [this message]
2006-04-27 2:18 ` Linus Torvalds
2006-04-27 2:27 ` Gerrit Huizenga
2006-04-27 2:46 ` Linus Torvalds
2006-04-27 3:00 ` David Woodhouse
2006-04-27 4:32 ` Gerrit Huizenga
2006-04-27 2:37 ` David Woodhouse
2006-04-27 2:59 ` Linus Torvalds
2006-04-27 3:17 ` David Woodhouse
2006-04-27 3:31 ` Linus Torvalds
2006-04-27 3:41 ` David Woodhouse
2006-04-27 7:10 ` Denis Vlasenko
2006-04-27 7:23 ` Arjan van de Ven
2006-04-27 8:49 ` Krzysztof Halasa
2006-04-27 11:03 ` Arnd Bergmann
2006-04-27 21:37 ` Adrian Bunk
2006-04-27 22:00 ` Linus Torvalds
2006-04-27 23:11 ` David Woodhouse
2006-04-27 23:12 ` Adrian Bunk
2006-04-28 0:01 ` Linus Torvalds
2006-04-28 0:55 ` David Woodhouse
2006-04-28 15:32 ` Jan Engelhardt
2006-04-28 15:37 ` David Woodhouse
2006-04-28 15:59 ` Arjan van de Ven
2006-04-28 16:01 ` Kyle Moffett
2006-04-29 10:33 ` Krzysztof Halasa
2006-05-03 2:53 ` H. Peter Anvin
2006-04-30 9:38 ` Peter Chubb
2006-04-30 9:59 ` David Woodhouse
2006-04-30 21:48 ` Peter Chubb
2006-04-27 15:05 ` David Woodhouse
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=1146104023.2885.15.camel@hades.cambridge.redhat.com \
--to=dwmw2@infradead.org \
--cc=akpm@osdl.org \
--cc=linux-kernel@vger.kernel.org \
--cc=torvalds@osdl.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox
all inboxes | Powered by JetHome®