* Re: [PATCH 0/3] Update LZO compression
[not found] <cover.1349621096.git.markus@oberhumer.com>
@ 2012-10-09 19:26 ` Andrew Morton
2012-10-09 19:54 ` Markus F.X.J. Oberhumer
0 siblings, 1 reply; 6+ messages in thread
From: Andrew Morton @ 2012-10-09 19:26 UTC (permalink / raw)
To: Markus F.X.J. Oberhumer
Cc: linux-kernel, Andi Kleen, Johannes Stezenbach,
richard -rw- weinberger, linux-btrfs, linux-crypto,
Artem Bityutskiy, Adrian Hunter, David Woodhouse, Phillip Lougher,
Dan Magenheimer, Dan Carpenter, Stephen Rothwell
On Sun, 7 Oct 2012 17:07:55 +0200
"Markus F.X.J. Oberhumer" <markus@oberhumer.com> wrote:
> As requested by akpm I am sending my "lzo-update" branch at
>
> git://github.com/markus-oberhumer/linux.git lzo-update
>
> to lkml as a patch series created by "git format-patch -M v3.5..lzo-update".
>
> You can also browse the branch at
>
> https://github.com/markus-oberhumer/linux/compare/lzo-update
>
> and review the three patches at
>
> https://github.com/markus-oberhumer/linux/commit/7c979cebc0f93dc692b734c12665a6824d219c20
> https://github.com/markus-oberhumer/linux/commit/10f6781c8591fe5fe4c8c733131915e5ae057826
> https://github.com/markus-oberhumer/linux/commit/5f702781f158cb59075cfa97e5c21f52275057f1
The changes look OK to me. Please ask Stephen to include the tree in
linux-next, for a 3.7 merge.
The changelog for patch 2/3 says:
: This commit updates the kernel LZO code to the current upsteam version
: which features a significant speed improvement - benchmarking the Calgary
: and Silesia test corpora typically shows a doubled performance in
: both compression and decompression on modern i386/x86_64/powerpc machines.
There are significant clients of the LZO library - crypto, btrfs,
jffs2, ubifs, squashfs and zcache. So let's give all those people a cc
and ask that they test the LZO changes once they land in linux-next.
For correctness and performance, please.
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH 0/3] Update LZO compression
2012-10-09 19:26 ` [PATCH 0/3] Update LZO compression Andrew Morton
@ 2012-10-09 19:54 ` Markus F.X.J. Oberhumer
2012-10-09 22:43 ` Stephen Rothwell
2012-10-11 11:41 ` Arnd Bergmann
0 siblings, 2 replies; 6+ messages in thread
From: Markus F.X.J. Oberhumer @ 2012-10-09 19:54 UTC (permalink / raw)
To: Stephen Rothwell
Cc: Andrew Morton, linux-kernel, Andi Kleen, Johannes Stezenbach,
richard -rw- weinberger, linux-btrfs, linux-crypto,
Artem Bityutskiy, Adrian Hunter, David Woodhouse, Phillip Lougher,
Dan Magenheimer, Dan Carpenter
Hi Stephen,
On 2012-10-09 21:26, Andrew Morton wrote:
> On Sun, 7 Oct 2012 17:07:55 +0200
> "Markus F.X.J. Oberhumer" <markus@oberhumer.com> wrote:
>
>> As requested by akpm I am sending my "lzo-update" branch at
>>
>> git://github.com/markus-oberhumer/linux.git lzo-update
>>
>> to lkml as a patch series created by "git format-patch -M v3.5..lzo-update".
>>
>> You can also browse the branch at
>>
>> https://github.com/markus-oberhumer/linux/compare/lzo-update
>>
>> and review the three patches at
>>
>> https://github.com/markus-oberhumer/linux/commit/7c979cebc0f93dc692b734c12665a6824d219c20
>> https://github.com/markus-oberhumer/linux/commit/10f6781c8591fe5fe4c8c733131915e5ae057826
>> https://github.com/markus-oberhumer/linux/commit/5f702781f158cb59075cfa97e5c21f52275057f1
>
> The changes look OK to me. Please ask Stephen to include the tree in
> linux-next, for a 3.7 merge.
I'd ask you to include my "lzo-update" branch in linux-next:
git://github.com/markus-oberhumer/linux.git lzo-update
> The changelog for patch 2/3 says:
>
> : This commit updates the kernel LZO code to the current upsteam version
> : which features a significant speed improvement - benchmarking the Calgary
> : and Silesia test corpora typically shows a doubled performance in
> : both compression and decompression on modern i386/x86_64/powerpc machines.
>
> There are significant clients of the LZO library - crypto, btrfs,
> jffs2, ubifs, squashfs and zcache. So let's give all those people a cc
> and ask that they test the LZO changes once they land in linux-next.
> For correctness and performance, please.
The core compression and decompression code has been thoroughly tested, so I
do not expect major problems.
Good testing after the merge and feedback about build or performance issues
(and improvements!) is highly appreciated.
Many thanks,
Markus
--
Markus Oberhumer, <markus@oberhumer.com>, http://www.oberhumer.com/
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH 0/3] Update LZO compression
2012-10-09 19:54 ` Markus F.X.J. Oberhumer
@ 2012-10-09 22:43 ` Stephen Rothwell
2012-10-11 11:41 ` Arnd Bergmann
1 sibling, 0 replies; 6+ messages in thread
From: Stephen Rothwell @ 2012-10-09 22:43 UTC (permalink / raw)
To: Markus F.X.J. Oberhumer
Cc: Andrew Morton, linux-kernel, Andi Kleen, Johannes Stezenbach,
richard -rw- weinberger, linux-btrfs, linux-crypto,
Artem Bityutskiy, Adrian Hunter, David Woodhouse, Phillip Lougher,
Dan Magenheimer, Dan Carpenter
[-- Attachment #1: Type: text/plain, Size: 2772 bytes --]
Hi Markus,
On Tue, 09 Oct 2012 21:54:59 +0200 "Markus F.X.J. Oberhumer" <markus@oberhumer.com> wrote:
>
> On 2012-10-09 21:26, Andrew Morton wrote:
> > On Sun, 7 Oct 2012 17:07:55 +0200
> > "Markus F.X.J. Oberhumer" <markus@oberhumer.com> wrote:
> >
> >> As requested by akpm I am sending my "lzo-update" branch at
> >>
> >> git://github.com/markus-oberhumer/linux.git lzo-update
> >>
> >> to lkml as a patch series created by "git format-patch -M v3.5..lzo-update".
> >>
> >> You can also browse the branch at
> >>
> >> https://github.com/markus-oberhumer/linux/compare/lzo-update
> >>
> >> and review the three patches at
> >>
> >> https://github.com/markus-oberhumer/linux/commit/7c979cebc0f93dc692b734c12665a6824d219c20
> >> https://github.com/markus-oberhumer/linux/commit/10f6781c8591fe5fe4c8c733131915e5ae057826
> >> https://github.com/markus-oberhumer/linux/commit/5f702781f158cb59075cfa97e5c21f52275057f1
> >
> > The changes look OK to me. Please ask Stephen to include the tree in
> > linux-next, for a 3.7 merge.
>
> I'd ask you to include my "lzo-update" branch in linux-next:
>
> git://github.com/markus-oberhumer/linux.git lzo-update
I have added this from today.
Thanks for adding your subsystem tree as a participant of linux-next. As
you may know, this is not a judgment of your code. The purpose of
linux-next is for integration testing and to lower the impact of
conflicts between subsystems in the next merge window.
You will need to ensure that the patches/commits in your tree/series have
been:
* submitted under GPL v2 (or later) and include the Contributor's
Signed-off-by,
* posted to the relevant mailing list,
* reviewed by you (or another maintainer of your subsystem tree),
* successfully unit tested, and
* destined for the current or next Linux merge window.
Basically, this should be just what you would send to Linus (or ask him
to fetch). It is allowed to be rebased if you deem it necessary.
--
Cheers,
Stephen Rothwell
sfr@canb.auug.org.au
Legal Stuff:
By participating in linux-next, your subsystem tree contributions are
public and will be included in the linux-next trees. You may be sent
e-mail messages indicating errors or other issues when the
patches/commits from your subsystem tree are merged and tested in
linux-next. These messages may also be cross-posted to the linux-next
mailing list, the linux-kernel mailing list, etc. The linux-next tree
project and IBM (my employer) make no warranties regarding the linux-next
project, the testing procedures, the results, the e-mails, etc. If you
don't agree to these ground rules, let me know and I'll remove your tree
from participation in linux-next.
[-- Attachment #2: Type: application/pgp-signature, Size: 836 bytes --]
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH 0/3] Update LZO compression
2012-10-09 19:54 ` Markus F.X.J. Oberhumer
2012-10-09 22:43 ` Stephen Rothwell
@ 2012-10-11 11:41 ` Arnd Bergmann
2012-10-11 16:28 ` Markus F.X.J. Oberhumer
1 sibling, 1 reply; 6+ messages in thread
From: Arnd Bergmann @ 2012-10-11 11:41 UTC (permalink / raw)
To: Markus F.X.J. Oberhumer
Cc: Stephen Rothwell, Andrew Morton, linux-kernel, Andi Kleen,
Johannes Stezenbach, richard -rw- weinberger, linux-btrfs,
linux-crypto, Artem Bityutskiy, Adrian Hunter, David Woodhouse,
Phillip Lougher, Dan Magenheimer, Dan Carpenter
On Tuesday 09 October 2012, Markus F.X.J. Oberhumer wrote:
> >
> > : This commit updates the kernel LZO code to the current upsteam version
> > : which features a significant speed improvement - benchmarking the Calgary
> > : and Silesia test corpora typically shows a doubled performance in
> > : both compression and decompression on modern i386/x86_64/powerpc machines.
> >
> > There are significant clients of the LZO library - crypto, btrfs,
> > jffs2, ubifs, squashfs and zcache. So let's give all those people a cc
> > and ask that they test the LZO changes once they land in linux-next.
> > For correctness and performance, please.
>
> The core compression and decompression code has been thoroughly tested, so I
> do not expect major problems.
>
> Good testing after the merge and feedback about build or performance issues
> (and improvements!) is highly appreciated.
The addition of the lzo tree to linux-next caused this problem for ARM
imx_v6_v7_defconfig:
In file included from /home/arnd/linux-arm/arch/arm/boot/compressed/decompress.c:40:0:
/home/arnd/linux-arm/arch/arm/boot/compressed/../../../../lib/decompress_unlzo.c:34:34: fatal error: lzo/lzo1x_decompress.c: No such file or directory
Since the file was renamed, anything including it needs to be updated to the
new file name.
Signed-off-by: Arnd Bergmann <arnd@arndb.de>
diff --git a/lib/decompress_unlzo.c b/lib/decompress_unlzo.c
index 4531294..960183d 100644
--- a/lib/decompress_unlzo.c
+++ b/lib/decompress_unlzo.c
@@ -31,7 +31,7 @@
*/
#ifdef STATIC
-#include "lzo/lzo1x_decompress.c"
+#include "lzo/lzo1x_decompress_safe.c"
#else
#include <linux/decompress/unlzo.h>
#endif
^ permalink raw reply related [flat|nested] 6+ messages in thread
* Re: [PATCH 0/3] Update LZO compression
2012-10-11 11:41 ` Arnd Bergmann
@ 2012-10-11 16:28 ` Markus F.X.J. Oberhumer
2012-12-21 2:03 ` Dan Magenheimer
0 siblings, 1 reply; 6+ messages in thread
From: Markus F.X.J. Oberhumer @ 2012-10-11 16:28 UTC (permalink / raw)
To: Arnd Bergmann
Cc: Stephen Rothwell, Andrew Morton, linux-kernel, Andi Kleen,
Johannes Stezenbach, richard -rw- weinberger, linux-btrfs,
linux-crypto, Artem Bityutskiy, Adrian Hunter, David Woodhouse,
Phillip Lougher, Dan Magenheimer, Dan Carpenter
Thanks Arnd,
On 2012-10-11 13:41, Arnd Bergmann wrote:
> On Tuesday 09 October 2012, Markus F.X.J. Oberhumer wrote:
>>>
>>> : This commit updates the kernel LZO code to the current upsteam version
>>> : which features a significant speed improvement - benchmarking the Calgary
>>> : and Silesia test corpora typically shows a doubled performance in
>>> : both compression and decompression on modern i386/x86_64/powerpc machines.
>>>
>>> There are significant clients of the LZO library - crypto, btrfs,
>>> jffs2, ubifs, squashfs and zcache. So let's give all those people a cc
>>> and ask that they test the LZO changes once they land in linux-next.
>>> For correctness and performance, please.
>>
>> The core compression and decompression code has been thoroughly tested, so I
>> do not expect major problems.
>>
>> Good testing after the merge and feedback about build or performance issues
>> (and improvements!) is highly appreciated.
>
> The addition of the lzo tree to linux-next caused this problem for ARM
> imx_v6_v7_defconfig:
>
> In file included from /home/arnd/linux-arm/arch/arm/boot/compressed/decompress.c:40:0:
> /home/arnd/linux-arm/arch/arm/boot/compressed/../../../../lib/decompress_unlzo.c:34:34: fatal error: lzo/lzo1x_decompress.c: No such file or directory
>
> Since the file was renamed, anything including it needs to be updated to the
> new file name.
I will add that patch to my tree.
Cheers,
Markus
>
> Signed-off-by: Arnd Bergmann <arnd@arndb.de>
>
> diff --git a/lib/decompress_unlzo.c b/lib/decompress_unlzo.c
> index 4531294..960183d 100644
> --- a/lib/decompress_unlzo.c
> +++ b/lib/decompress_unlzo.c
> @@ -31,7 +31,7 @@
> */
>
> #ifdef STATIC
> -#include "lzo/lzo1x_decompress.c"
> +#include "lzo/lzo1x_decompress_safe.c"
> #else
> #include <linux/decompress/unlzo.h>
> #endif
--
Markus Oberhumer, <markus@oberhumer.com>, http://www.oberhumer.com/
^ permalink raw reply [flat|nested] 6+ messages in thread
* RE: [PATCH 0/3] Update LZO compression
2012-10-11 16:28 ` Markus F.X.J. Oberhumer
@ 2012-12-21 2:03 ` Dan Magenheimer
0 siblings, 0 replies; 6+ messages in thread
From: Dan Magenheimer @ 2012-12-21 2:03 UTC (permalink / raw)
To: Markus F.X.J. Oberhumer, Arnd Bergmann
Cc: Stephen Rothwell, Andrew Morton, linux-kernel, Andi Kleen,
Johannes Stezenbach, richard -rw- weinberger, linux-btrfs,
linux-crypto, Artem Bityutskiy, Adrian Hunter, David Woodhouse,
Phillip Lougher, Dan Carpenter
> From: Markus F.X.J. Oberhumer [mailto:markus@oberhumer.com]
> Subject: Re: [PATCH 0/3] Update LZO compression
>
> Thanks Arnd,
>
> On 2012-10-11 13:41, Arnd Bergmann wrote:
> > On Tuesday 09 October 2012, Markus F.X.J. Oberhumer wrote:
> >>>
> >>> : This commit updates the kernel LZO code to the current upsteam version
> >>> : which features a significant speed improvement - benchmarking the Calgary
> >>> : and Silesia test corpora typically shows a doubled performance in
> >>> : both compression and decompression on modern i386/x86_64/powerpc machines.
> >>>
> >>> There are significant clients of the LZO library - crypto, btrfs,
> >>> jffs2, ubifs, squashfs and zcache. So let's give all those people a cc
> >>> and ask that they test the LZO changes once they land in linux-next.
> >>> For correctness and performance, please.
> >>
> >> The core compression and decompression code has been thoroughly tested, so I
> >> do not expect major problems.
> >>
> >> Good testing after the merge and feedback about build or performance issues
> >> (and improvements!) is highly appreciated.
> >
> > The addition of the lzo tree to linux-next caused this problem for ARM
> > imx_v6_v7_defconfig:
> >
> > In file included from /home/arnd/linux-arm/arch/arm/boot/compressed/decompress.c:40:0:
> > /home/arnd/linux-arm/arch/arm/boot/compressed/../../../../lib/decompress_unlzo.c:34:34: fatal error:
> lzo/lzo1x_decompress.c: No such file or directory
> >
> > Since the file was renamed, anything including it needs to be updated to the
> > new file name.
>
> I will add that patch to my tree.
>
> Cheers,
> Markus
Sorry if I missed it (bad connectivity this week), but is someone going
to send a pull request to get this LZO update from linux-next into
Linus's tree? The window is closing soon isn't it?
Dan
^ permalink raw reply [flat|nested] 6+ messages in thread
end of thread, other threads:[~2012-12-21 2:04 UTC | newest]
Thread overview: 6+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
[not found] <cover.1349621096.git.markus@oberhumer.com>
2012-10-09 19:26 ` [PATCH 0/3] Update LZO compression Andrew Morton
2012-10-09 19:54 ` Markus F.X.J. Oberhumer
2012-10-09 22:43 ` Stephen Rothwell
2012-10-11 11:41 ` Arnd Bergmann
2012-10-11 16:28 ` Markus F.X.J. Oberhumer
2012-12-21 2:03 ` Dan Magenheimer
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).