Re: [PATCH] only use 48-bit lba when necessary

From: Timothy Miller (miller@techsource.com)
Date: Thu Apr 17 2003 - 13:49:13 EST


Matt Mackall wrote:

>
>
>>>
>>>
>>What's ugly about them?
>>
>>
>
>It doesn't pass the test of "would I use it if I didn't think it was
>faster?"
>
>As I pointed out, your variant is not faster with a reasonable
>compiler, only less obvious. And none of this sort of optimization
>will ever be measurably better in the IO path anyway. But every one of
>these false optimizations is a barrier to the understanding that will
>allow real cleanups to make fundamental improvements.
>
>
>
Agreed, but in this case, it's simply a matter of culture. If some of
these things were more common, then they would pass the test, like the
use of "!!".

One thing for your side of the argument is that if something works,
don't mess with it and risk breaking it, especially if it has no
practical impact on performance.

If we really wanted to improve readability, we could start doing
something like Windows developers are so fond of doing and use Hungarian
Notation. I know it's ugly, and we eschew anything to do with
Microsoft, but once you get used to it, it can be very helpful in
keeping things straight.

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



This archive was generated by hypermail 2b29 : Wed Apr 23 2003 - 22:00:22 EST