netdev
[Top] [All Lists]

Re: iproute2 and 2.6.9 kernel headers (was Re: [ANNOUNCE] iproute2 2.6.9

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>