[Top] [All Lists]

Re: ipt_physdev.c alignment problems on parisc64

To: "David S. Miller" <davem@xxxxxxxxxx>
Subject: Re: ipt_physdev.c alignment problems on parisc64
From: Bart De Schuymer <bdschuym@xxxxxxxxxx>
Date: Tue, 16 Sep 2003 08:05:07 +0200
Cc: laforge@xxxxxxxxxxxxx, acme@xxxxxxxxxxxxxxxx, netfilter-devel@xxxxxxxxxxxxxxxxxxx, netdev@xxxxxxxxxxx
In-reply-to: <20030915155903.12a3f95d.davem@xxxxxxxxxx>
References: <200309022116.41697.bdschuym@xxxxxxxxxx> <200309132159.37834.bdschuym@xxxxxxxxxx> <20030915155903.12a3f95d.davem@xxxxxxxxxx>
Sender: netdev-bounce@xxxxxxxxxxx
User-agent: KMail/1.5
On Tuesday 16 September 2003 00:59, David S. Miller wrote:
> > -   char physindev[IFNAMSIZ];
> > +   char physindev[IFNAMSIZ] __attribute__ ((aligned (__alignof__(unsigned
> > int))));
> I have to reject the fix for this problem again.
> You can't change the struct exported to userspace in any way
> without breaking the tools.  I tried to explain this in the
> first revision of your change.

On Thursday 04 September 2003 05:05, you wrote:
As I mentioned in another email, this patch can't be used, it
breaks in mixed 32/64 bit environments.

I guess I'm a bit on the slow side but I didn't quite read the above text like 
Anyway, this rule about not changing structs exported to userspace truly 
sucks, IMHO. If someone decides to compile her own kernel this person should 
not expect every userspace tool to keep working, again IMHO. Adding a new 
module to the kernel results in the user having to upgrade her tools, 
recompiling won't even do. Why on earth do you allow adding modules.
But I'm sure some nice volunteer will come along and downgrade that code 


<Prev in Thread] Current Thread [Next in Thread>