Re: [PATCH] Add noinline attribute

From: Linus Torvalds
Date: Thu Jan 15 2004 - 18:24:35 EST




On Wed, 15 Jan 2004, Andi Kleen wrote:
>
> Actually you would get a non booting system because the broken mount
> ABI does a stress test of EFAULT on every boot.

There is somethinglike _three_ exceptions that get any kind of testing:
the WP bit, the mount interface, and the FP fault check.

That's three out of several thousand, so coverage really sucks. That's why
I'd rather just sort the damn thing once and for all, and not have that
issue pop up every once in a while.

> That's fine for me. In fact I did this some time ago on x86-64 when I
> ran into similar problems. Here's a port of the x86-64 sort function.

Ugh. Can't we just make this be generic code (and that means calling it in
the module loading code too..)?

As to bubble sort (which is fine for something that is 99% sorted anyway),
isn't it better to continue pushing the entry down when you find something
out of order? That way you don't have to repeat the whole scan, you just
continue with the next entry once the unsorted entry has percolated to its
place (ie keep entries "0..n-1" sorted at all times). That should make the
code cleaner too.

Linus
-
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/