Re: [RFC] LZO de/compression support - take 6

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Tue, May 29, 2007 at 09:08:27AM +0100, Michael-Luke Jones wrote:
> On 28 May 2007, at 18:11, Adrian Bunk wrote:
>
>> I have not seen any explanations:
>> - Why did the upstream author write the code that way?
>
> Apparently due to his requirement for extreme portability. The original 
> code was designed to work on everything from 16-bit DOS through CRAY 
> supercomputers through Windows, Unices and Linux.

Sure, this could be the reason in some or all cases.

The upstream author knows the code best, and discussing such issues with 
him will in many cases be a win:

It could be that there was in some cases no good reason, and the 
upstream code that gets used by many other projects could become faster.

Or there was a good reason that applies also to the in-kernel version 
and a change breaks some corner case.

> The author has stated on the thread that it's a good idea to remove 
> unnecessary ifdefs when porting the code into the kernel, given that the 
> portability requirements are obviously no longer needed.

"remove unnecessary ifdefs" implies "generated code is identical".

That's quite different from "code is 10% faster".

> Michael-Luke

cu
Adrian

-- 

       "Is there not promise of rain?" Ling Tan asked suddenly out
        of the darkness. There had been need of rain for many days.
       "Only a promise," Lao Er said.
                                       Pearl S. Buck - Dragon Seed

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

[Index of Archives]     [Kernel Newbies]     [Netfilter]     [Bugtraq]     [Photo]     [Stuff]     [Gimp]     [Yosemite News]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux RAID]     [Video 4 Linux]     [Linux for the blind]     [Linux Resources]
  Powered by Linux