From: "Markus F.X.J. Oberhumer" <markus@oberhumer.com>
To: cover.1349621096.git.markus@oberhumer.co
Cc: Seth Jennings <sjenning@linux.vnet.ibm.com>,
Robert Jennings <rcj@linux.vnet.ibm.com>,
Andrew Morton <akpm@linux-foundation.org>,
LKML <linux-kernel@vger.kernel.org>,
Andi Kleen <andi@firstfloor.org>,
Johannes Stezenbach <js@sig21.net>,
Richard Weinberger <richard.weinberger@gmail.com>
Subject: Re: [PATCH 0/3] Update LZO compression
Date: Tue, 16 Oct 2012 01:45:04 +0200 [thread overview]
Message-ID: <507CA000.5040905@oberhumer.com> (raw)
In-Reply-To: <507C61B2.9060503@linux.vnet.ibm.com>
On 2012-10-15 21:19, Seth Jennings 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
>
> As this relates to my work on zcache, I just tested these patches on PPC64 and
> they cause the LZO crypto module to fail its self-test:
>
> [ 0.521137] alg: comp: Compression test 1 failed for lzo-generic: output len = 62
>
> I built the exact same kernel for x86_64 and all is fine. I suspect an endianness
> related bug, but I haven't looked at the code that closely yet.
>
> Any ideas? I'd be happy to test any potential fixes.
The crypto LZO test vectors had to be updated - this should land in linux-next
soon (or you can just pull from my branch).
BTW, this cannot have worked on x86_64 (or any other arch), so you probably
tested the wrong kernel.
Cheers,
Markus
> Seth
--
Markus Oberhumer, <markus@oberhumer.com>, http://www.oberhumer.com/
next prev parent reply other threads:[~2012-10-15 23:45 UTC|newest]
Thread overview: 10+ messages / expand[flat|nested] mbox.gz Atom feed top
2012-10-15 19:19 [PATCH 0/3] Update LZO compression Seth Jennings
2012-10-15 23:45 ` Markus F.X.J. Oberhumer [this message]
2012-10-16 16:50 ` Seth Jennings
-- strict thread matches above, loose matches on Subject: below --
2012-10-07 15:07 Markus F.X.J. Oberhumer
2012-10-09 19:26 ` 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
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=507CA000.5040905@oberhumer.com \
--to=markus@oberhumer.com \
--cc=akpm@linux-foundation.org \
--cc=andi@firstfloor.org \
--cc=cover.1349621096.git.markus@oberhumer.co \
--cc=js@sig21.net \
--cc=linux-kernel@vger.kernel.org \
--cc=rcj@linux.vnet.ibm.com \
--cc=richard.weinberger@gmail.com \
--cc=sjenning@linux.vnet.ibm.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.