Re: [GIT PULL v2] Update LZO compression
CC'in akpm. On Wed, Oct 3, 2012 at 12:34 PM, Markus F.X.J. Oberhumer mar...@oberhumer.com wrote: Hi all, is there some official maintainer who feels responsible for pushing the LZO update to Linus? Otherwise I also could issue a pull-request on github, but somehow I don't think that this is the preferred workflow. Cheers, Markus On 2012-08-21 17:21, Markus F.X.J. Oberhumer wrote: Hi all, as suggested on the mailing list I have converted the updated LZO code into git, so please pull my lzo-update branch from git://github.com/markus-oberhumer/linux.git lzo-update You can browse the branch at https://github.com/markus-oberhumer/linux/compare/lzo-update I'd ask some official kernel maintainer for review and to push this into linux-next so that it will hopefully land in the 3.7 release. Share and enjoy, Markus Signed-off-by: Markus F.X.J. Oberhumer mar...@oberhumer.com [ Changes in v2: Optimize code for CPUs with inefficient unaligned access = significant speed increase on ARM ] On 2012-07-16 20:30, Markus F.X.J. Oberhumer wrote: Hi all, I finally have prepared a small package that updates the LZO version in the Linux kernel. Please get it from: http://www.oberhumer.com/opensource/lzo/download/Testing/linux-kernel-lzo-20120716.tar.gz As stated in the README this version is significantly faster (typically more than 2 times faster!) than the current version, has been thoroughly tested on x86_64/i386/powerpc platforms and is intended to get included into the official Linux 3.6 or 3.7 release. I encourage all compression users to test and benchmark this new version, and I also would ask some official LZO maintainer to convert the updated source files into a GIT commit and possibly push it to Linus or linux-next. Share and enjoy, Markus Signed-off-by: Markus F.X.J. Oberhumer mar...@oberhumer.com -- Markus Oberhumer, mar...@oberhumer.com, http://www.oberhumer.com/ -- Thanks, //richard -- To unsubscribe from this list: send the line unsubscribe linux-btrfs in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
Re: [GIT PULL v2] Update LZO compression
Hi all, is there some official maintainer who feels responsible for pushing the LZO update to Linus? Otherwise I also could issue a pull-request on github, but somehow I don't think that this is the preferred workflow. Cheers, Markus On 2012-08-21 17:21, Markus F.X.J. Oberhumer wrote: Hi all, as suggested on the mailing list I have converted the updated LZO code into git, so please pull my lzo-update branch from git://github.com/markus-oberhumer/linux.git lzo-update You can browse the branch at https://github.com/markus-oberhumer/linux/compare/lzo-update I'd ask some official kernel maintainer for review and to push this into linux-next so that it will hopefully land in the 3.7 release. Share and enjoy, Markus Signed-off-by: Markus F.X.J. Oberhumer mar...@oberhumer.com [ Changes in v2: Optimize code for CPUs with inefficient unaligned access = significant speed increase on ARM ] On 2012-07-16 20:30, Markus F.X.J. Oberhumer wrote: Hi all, I finally have prepared a small package that updates the LZO version in the Linux kernel. Please get it from: http://www.oberhumer.com/opensource/lzo/download/Testing/linux-kernel-lzo-20120716.tar.gz As stated in the README this version is significantly faster (typically more than 2 times faster!) than the current version, has been thoroughly tested on x86_64/i386/powerpc platforms and is intended to get included into the official Linux 3.6 or 3.7 release. I encourage all compression users to test and benchmark this new version, and I also would ask some official LZO maintainer to convert the updated source files into a GIT commit and possibly push it to Linus or linux-next. Share and enjoy, Markus Signed-off-by: Markus F.X.J. Oberhumer mar...@oberhumer.com -- Markus Oberhumer, mar...@oberhumer.com, http://www.oberhumer.com/ -- To unsubscribe from this list: send the line unsubscribe linux-btrfs in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
Re: [GIT PULL v2] Update LZO compression
On Wed, 3 Oct 2012 12:48:46 +0200 richard -rw- weinberger richard.weinber...@gmail.com wrote: CC'in akpm. Thanks. Hi all, I finally have prepared a small package that updates the LZO version in the Linux kernel. Please get it from: http://www.oberhumer.com/opensource/lzo/download/Testing/linux-kernel-lzo-20120716.tar.gz As stated in the README this version is significantly faster (typically more than 2 times faster!) than the current version, has been thoroughly tested on x86_64/i386/powerpc platforms and is intended to get included into the official Linux 3.6 or 3.7 release. I encourage all compression users to test and benchmark this new version, and I also would ask some official LZO maintainer to convert the updated source files into a GIT commit and possibly push it to Linus or linux-next. No, lib/lzo has no identifiable maintainer. I suggest you proceed as follows: - Post the entire patch series to lkml for review (I'd like a cc please) - After that process has played out, ask Stephen to add this git tree to linux-next. - After that process has played out, ask Linus to pull the tree during a merge window. I haven't actually looked at the patches yet, but if they are as extensive as they sound, it would be appropriate for you become the formal maintainer of lib/lzo. -- To unsubscribe from this list: send the line unsubscribe linux-btrfs in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
Re: [GIT PULL v2] Update LZO compression
- Post the entire patch series to lkml for review (I'd like a cc please) Already happened, multiple people reviewed and tested. um, I would not consider Looks ok to me from a quick look. and I couldn't tell from the github view, but I assume you follow standard coding style. to indicate a rigorous code review! Let's put it like this. LZO is mature widely used production software and Markus wrote and maintains it. Essentially this is a release from him. The main problem before with using his version directly was that it did not follow Linux coding standards, but that seems to be fixed now. It's also relatively simple and at least I didn't spot anything bad when I looked. It was runtime tested by several people on different architectures. If it was fuzzed so we can assume reasonably good security. -Andi -- a...@linux.intel.com -- Speaking for myself only. -- To unsubscribe from this list: send the line unsubscribe linux-btrfs in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
Re: [GIT PULL v2] Update LZO compression
On Tue, Aug 21, 2012 at 05:21:50PM +0200, Markus F.X.J. Oberhumer wrote: as suggested on the mailing list I have converted the updated LZO code into git, so please pull my lzo-update branch from ... [ Changes in v2: Optimize code for CPUs with inefficient unaligned access = significant speed increase on ARM ] I can confirm that this new code runs at the same speed as the current lzo code in the Linux kernel on my ARM926EJ-S based platform. I only tested decompression, using the attached hacky userspace code. # time ./lzo-bench/old/unlzop lzoimage /dev/null real0m 0.29s # time ./lzo-bench/new/unlzop lzoimage /dev/null real0m 0.29s (where lzoimage is a Linux Image compressed with lzop) So, from my side there are no more objections. Thanks for doing this work, Markus. Johannes lzo-bench.tar.gz Description: Binary data
[GIT PULL v2] Update LZO compression
Hi all, as suggested on the mailing list I have converted the updated LZO code into git, so please pull my lzo-update branch from git://github.com/markus-oberhumer/linux.git lzo-update You can browse the branch at https://github.com/markus-oberhumer/linux/compare/lzo-update I'd ask some official kernel maintainer for review and to push this into linux-next so that it will hopefully land in the 3.7 release. Share and enjoy, Markus Signed-off-by: Markus F.X.J. Oberhumer mar...@oberhumer.com [ Changes in v2: Optimize code for CPUs with inefficient unaligned access = significant speed increase on ARM ] On 2012-07-16 20:30, Markus F.X.J. Oberhumer wrote: Hi all, I finally have prepared a small package that updates the LZO version in the Linux kernel. Please get it from: http://www.oberhumer.com/opensource/lzo/download/Testing/linux-kernel-lzo-20120716.tar.gz As stated in the README this version is significantly faster (typically more than 2 times faster!) than the current version, has been thoroughly tested on x86_64/i386/powerpc platforms and is intended to get included into the official Linux 3.6 or 3.7 release. I encourage all compression users to test and benchmark this new version, and I also would ask some official LZO maintainer to convert the updated source files into a GIT commit and possibly push it to Linus or linux-next. Share and enjoy, Markus Signed-off-by: Markus F.X.J. Oberhumer mar...@oberhumer.com -- Markus Oberhumer, mar...@oberhumer.com, http://www.oberhumer.com/ -- To unsubscribe from this list: send the line unsubscribe linux-btrfs in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
