Re: [PATCH] some Alpha cleanups

Alan Cox (alan@lxorguk.ukuu.org.uk)
Sun, 24 Jan 1999 00:41:51 +0000 (GMT)


> *(int *)&id->cur_capacity0 = id->lba_capacity;
>
> in ide_disk.c. Such code is terrible.

That one is

> Nobody knows the order of the two shorts in an int.
> And everyone who does know will be sorry.
> On some machines this will work, on others it won't.

It works on them all for now - not that it excuses this

What I mean is for a lot of the unaligned trap cases that arent bugs you
can use the egcs stuff

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