| To: | Harald Welte <laforge@xxxxxxxxxxxx> |
|---|---|
| Subject: | Re: iproute2 and 2.6.9 kernel headers (was Re: [ANNOUNCE] iproute2 2.6.9-041019) |
| From: | David Woodhouse <dwmw2@xxxxxxxxxxxxx> |
| Date: | Wed, 20 Oct 2004 11:41:25 +0100 |
| Cc: | Jeff Chua <jeffchua@xxxxxxxxxxxxxxxxxxx>, Stephen Hemminger <shemminger@xxxxxxxx>, Linux Kernel <linux-kernel@xxxxxxxxxxxxxxx>, netdev@xxxxxxxxxxx, linux-net@xxxxxxxxxxxxxxx, LARTC@xxxxxxxxxxxxxxx |
| In-reply-to: | <20041020094123.GF19899@xxxxxxxxxxxxxxxxxxxxxxx> |
| References: | <41758014.4080502@xxxxxxxx> <Pine.LNX.4.61.0410200805110.8475@xxxxxxxxxxxxxxxxxxxxx> <20041020070017.GA19899@xxxxxxxxxxxxxxxxxxxxxxx> <20041020094123.GF19899@xxxxxxxxxxxxxxxxxxxxxxx> |
| Sender: | netdev-bounce@xxxxxxxxxxx |
On Wed, 2004-10-20 at 11:41 +0200, Harald Welte wrote:
> On Wed, Oct 20, 2004 at 09:00:17AM +0200, Harald Welte wrote:
> > I'll take care of this. sorry fort he inconvenience.
>
> I should actually read mails befor replying ;) I thought the bug was in
> lnstat - but apparently it wasn't.
>
> The include bug seems non-trivial to fix. (how do I hate kernel include
> from userspace issues):
>
> apparently __KERNEL_STRICT_NAMES is definde somewhere (glibc?) which
> prevents __le16, __le64 and others from being defined in linux/types.h.
>
> Just reietting it like this doesn't help much:
No, it wouldn't.
The time has come to fix it properly instead. Anything which these tools
actually need from the kernel headers should be moved into a separate
header file (still in the kernel source) which is usable from _both_
kernel and userspace. It should use standard types (like uint16_t etc)
instead of kernel-private types, and shouldn't have any #if{n,}def
__KERNEL__ in it. Ideally, it would be in a different directory too --
but we can worry about that later.
--
dwmw2
|
| <Prev in Thread] | Current Thread | [Next in Thread> |
|---|---|---|
| ||
| Previous by Date: | iproute2 and 2.6.9 kernel headers (was Re: [ANNOUNCE] iproute2 2.6.9-041019), Harald Welte |
|---|---|
| Next by Date: | Re: [PATCH] Change masquerade code to use first IP address of interface., Herbert Xu |
| Previous by Thread: | iproute2 and 2.6.9 kernel headers (was Re: [ANNOUNCE] iproute2 2.6.9-041019), Harald Welte |
| Next by Thread: | Re: iproute2 and 2.6.9 kernel headers (was Re: [ANNOUNCE] iproute2 2.6.9-041019), David Vrabel |
| Indexes: | [Date] [Thread] [Top] [All Lists] |