lists.openwall.net | lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC | |
Open Source and information security mailing list archives
| ||
|
Date: Thu, 27 Feb 2014 11:08:23 +0300 From: Sergey Senozhatsky <sergey.senozhatsky@...il.com> To: Minchan Kim <minchan@...nel.org> Cc: Sergey Senozhatsky <sergey.senozhatsky@...il.com>, Andrew Morton <akpm@...ux-foundation.org>, Jerome Marchand <jmarchan@...hat.com>, Nitin Gupta <ngupta@...are.org>, linux-kernel@...r.kernel.org Subject: Re: [PATCHv8 0/6] add compressing abstraction and multi stream support Hello Minchan, On (02/27/14 11:16), Minchan Kim wrote: > Hello Sergey, > > Code looks good to me and I will queue it into ARM and x86 test tomorrow > and give the result to you with Acked-by. > > Thanks for your all works! > thanks a lot! -ss > On Wed, Feb 26, 2014 at 03:27:53PM +0300, Sergey Senozhatsky wrote: > > This patchset introduces zcomp compression backend abstraction > > adding ability to support compression algorithms other than LZO; > > support for multi compression streams, making parallel compressions > > possible. > > > > v7->v8 (reviewed by Minchan Kim): > > -- merge patches 'add multi stream functionality' and 'enable multi > > stream compression support in zram' > > -- return status code from set_max_streams knob and print message on > > error > > -- do not use atomic type for ->avail_strm > > -- return back: allocate by default only one stream for multi stream backend > > -- wake sleeping write in zcomp_strm_multi_put() only if we put stream > > to idle list > > -- minor code `nitpicks' > > > > v6->v7 (reviewed by Minchan Kim): > > -- enable multi and single stream support out of the box (drop > > ZRAM_MULTI_STREAM config option) > > -- add set_max_stream knob, so we can adjust max number of compression > > streams in runtime (for multi stream backend at the moment) > > -- minor code `nitpicks' > > > > v5->v6 (reviewed by Minchan Kim): > > -- handle single compression stream case separately, using mutex locking, > > to address perfomance regression > > -- handle multi compression stream using spin lock and wait_event()/wake_up() > > -- make multi compression stream support configurable (ZRAM_MULTI_STREAM > > config option) > > > > v4->v5 (reviewed by Minchan Kim): > > -- renamed zcomp buffer_lock; removed src len and dst len from > > compress() and decompress(); not using term `buffer' and > > `workmem' in code and documentation; define compress() and > > decompress() functions for LZO backend; not using goto's; > > do not put idle zcomp_strm to idle list tail. > > > > v3->v4 (reviewed by Minchan Kim): > > -- renamed compression backend and working memory structs as requested > > by Minchan Kim; fixed several issues noted by Minchan Kim. > > > > Sergey Senozhatsky (6): > > zram: introduce compressing backend abstraction > > zram: use zcomp compressing backends > > zram: factor out single stream compression > > zram: add multi stream functionality > > zram: add set_max_streams knob > > zram: document max_comp_streams > > > > Documentation/ABI/testing/sysfs-block-zram | 9 +- > > Documentation/blockdev/zram.txt | 31 ++- > > drivers/block/zram/Makefile | 2 +- > > drivers/block/zram/zcomp.c | 318 +++++++++++++++++++++++++++++ > > drivers/block/zram/zcomp.h | 59 ++++++ > > drivers/block/zram/zcomp_lzo.c | 48 +++++ > > drivers/block/zram/zram_drv.c | 98 +++++---- > > drivers/block/zram/zram_drv.h | 10 +- > > 8 files changed, 528 insertions(+), 47 deletions(-) > > create mode 100644 drivers/block/zram/zcomp.c > > create mode 100644 drivers/block/zram/zcomp.h > > create mode 100644 drivers/block/zram/zcomp_lzo.c > > > > -- > > 1.9.0.291.g027825b > > > > -- > > To unsubscribe from this list: send the line "unsubscribe linux-kernel" in > > the body of a message to majordomo@...r.kernel.org > > More majordomo info at http://vger.kernel.org/majordomo-info.html > > Please read the FAQ at http://www.tux.org/lkml/ > > -- > Kind regards, > Minchan Kim > -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@...r.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
Powered by blists - more mailing lists