On Fri, 2007-07-27 at 16:35 +0400, Edward Shishkin wrote:
quoted text
> Sorry, guys, I am not happy with the modified LZO:
> the compressor tries to test bytes which are out of bounds.
> 
> The attached module testlzo.c causes an oops in the second pass:
> AFAIK, both, @m and @m_pos should be in [wrkmem, wrkmem + 64K);
> I have attached trace.txt with their actual values.
> 
> Not ready to migrate to this library.
> 
> Any ideas?

I can confirm its broken, it give an oops on ARM 2.6.23-rc1 too in the
first pass.

I guess the only real solution now is to compare the byte code from the
new and original and find out where this problem has crept in from and
check there are no others. I'll look into it but its going to take time.
There goes my weekend :-(

Richard

-
unsubscribe notice
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/