Re: [PATCH 0/6] lib/lzo: performance improvements

2018-11-22 Thread Sergey Senozhatsky
On (11/21/18 12:06), Dave Rodgman wrote: > > Overall, performance is improved by around 1.1 - 4.8x (data-dependent: data > with many zero runs shows higher improvement). Under real-world testing with > zram, time spent in (de)compression during swapping is reduced by around 27%. Impressive. I th

RE: [PATCH 0/6] lib/lzo: performance improvements

2018-11-21 Thread Matt Sealey
Hi Markus. > Hi Dave, > > thanks for your patch set. Just some initial comments: > > > I think the three patches > > [PATCH 2/6] lib/lzo: enable 64-bit CTZ on Arm > [PATCH 3/6] lib/lzo: 64-bit CTZ on Arm aarch64 > [PATCH 4/6] lib/lzo: fast 8-byte copy on arm64 > > should be applied in a

Re: [PATCH 0/6] lib/lzo: performance improvements

2018-11-21 Thread Dave Rodgman
On 21/11/2018 1:44 pm, Markus F.X.J. Oberhumer wrote: > I think the three patches > >[PATCH 2/6] lib/lzo: enable 64-bit CTZ on Arm >[PATCH 3/6] lib/lzo: 64-bit CTZ on Arm aarch64 >[PATCH 4/6] lib/lzo: fast 8-byte copy on arm64 > > should be applied in any case - could you please make

Re: [PATCH 0/6] lib/lzo: performance improvements

2018-11-21 Thread Markus F.X.J. Oberhumer
Hi Dave, thanks for your patch set. Just some initial comments: I think the three patches [PATCH 2/6] lib/lzo: enable 64-bit CTZ on Arm [PATCH 3/6] lib/lzo: 64-bit CTZ on Arm aarch64 [PATCH 4/6] lib/lzo: fast 8-byte copy on arm64 should be applied in any case - could you please make an e