Re: [PATCH] Introduce compat_u64 and compat_s64 types

From: Robin Getz
Date: Fri Jun 15 2007 - 16:48:20 EST


On Fri 15 Jun 2007 08:54, Andi Kleen pondered:
> On Friday 15 June 2007 14:54:20 Alan Cox wrote:
>
> > > > > The network code requires unaligned accesses to work anyways so if
your
> > > > > architecture doesn't support them it is already remotely crashable.
> > > >
> > > > I thought we'd fixed all that.
> > >
> > > Did you audit the complete network stack?
> >
> > For the parts used by the processors in question yes
>
> That means? They're expected to run only a subset of the network stack?
> Is that expressed in Kconfig? Is it documented that the rest is dangerous?

For the architecture we use (Blackfin), it does not support unaligned
accesses, and we purposely never put in the trap/fixup code - we trap, and
printk("fix your source");

We have run into a few kernel issues (never networking) this way - but a fixup
of the source is normally the best solution - since it doesn't impose a
hidden performance issue by trapping everything all the time.

> > people have done
> > that work so using the types without unaligned.
>
> Very brave; we're talking about around half a million lines
> of non trivial source code here.

Is there something specific that you can think of that we should be testing?

We have done alot of testing, and people have shipped alot of systems
connected to a varity of networks, and have run all kinds of protocols on
them.

-Robin
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/