linux-kernel.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
From: Maninder Singh <maninder1.s@samsung.com>
To: Maninder Singh <maninder1.s@samsung.com>,
	Vaneet Narang <v.narang@samsung.com>,
	Nick Terrell <terrelln@fb.com>,
	Sergey Senozhatsky <sergey.senozhatsky.work@gmail.com>,
	"herbert@gondor.apana.org.au" <herbert@gondor.apana.org.au>,
	Yann Collet <cyan@fb.com>
Cc: "davem@davemloft.net" <davem@davemloft.net>,
	"minchan@kernel.org" <minchan@kernel.org>,
	"ngupta@vflare.org" <ngupta@vflare.org>,
	Kees Cook <keescook@chromium.org>,
	"anton@enomsg.org" <anton@enomsg.org>,
	"ccross@android.com" <ccross@android.com>,
	"tony.luck@intel.com" <tony.luck@intel.com>,
	"akpm@linux-foundation.org" <akpm@linux-foundation.org>,
	"colin.king@canonical.com" <colin.king@canonical.com>,
	"linux-crypto@vger.kernel.org" <linux-crypto@vger.kernel.org>,
	"linux-kernel@vger.kernel.org" <linux-kernel@vger.kernel.org>,
	PANKAJ MISHRA <pankaj.m@samsung.com>,
	AMIT SAHRAWAT <a.sahrawat@samsung.com>,
	"gregkh@linuxfoundation.org" <gregkh@linuxfoundation.org>
Subject: Re: [PATCH 0/1] cover-letter/lz4: Implement lz4 with dynamic offset length.
Date: Mon, 16 Apr 2018 15:51:38 +0530	[thread overview]
Message-ID: <20180416102138epcms5p429e04b68b94dcf80cd84f7008467ed68@epcms5p4> (raw)
In-Reply-To: <20180329102613epcms5p6ffaa80d12115ded8f3c9ef01dd009744@epcms5p6>


 Hello Nick/ Yann,

Any inputs regarding LZ4 dyn results & lz4 dyn approach.

>Hello Nick/Sergey,
> 
>Any suggestion or comments, so that we can change code and resend the patch?
> 
>> Hi Nick / Sergey,
>> 
>> 
>> We have compared LZ4 Dyn with Original LZ4 using some samples of realtime application data(4Kb)
>> compressed/decompressed by ZRAM. For comparison we have used lzbench (https://github.com/inikep/lzbench)
>> we have implemented dedicated LZ4 Dyn API & kept last literal length as 6 to avoid overhead 
>> of checks. It seems in average case there is a saving of 3~4% in compression ratio with almost same compression
>> speed and minor loss in decompression speed (~50MB/s) when compared with LZ4.
>> 
>> Comparison of Lz4 Dyn with LZO1x is also done as LZO1x is default compressor of ZRAM.
>> 
>> Original LZ4:
>> sh-3.2# ./lzbench  -r  -elz4  data/
>> lzbench 1.7.3 (32-bit Linux)   Assembled by P.Skibinski
>> Compressor name         Compress. Decompress. Compr. size  Ratio Filename
>> memcpy                   2205 MB/s  2217 MB/s        4096 100.00 data//data_1
>> lz4 1.8.0                 216 MB/s   761 MB/s        2433  59.40 data//data_1
>> lz4 1.8.0                 269 MB/s   877 MB/s        1873  45.73 data//data_2
>> lz4 1.8.0                 238 MB/s   575 MB/s        2060  50.29 data//data_3
>> lz4 1.8.0                 321 MB/s  1015 MB/s        1464  35.74 data//data_4
>> lz4 1.8.0                 464 MB/s  1090 MB/s         713  17.41 data//data_5
>> lz4 1.8.0                 296 MB/s   956 MB/s        1597  38.99 data//data_6
>> lz4 1.8.0                 338 MB/s   994 MB/s        2238  54.64 data//data_7
>> lz4 1.8.0                 705 MB/s  1172 MB/s         193   4.71 data//data_8
>> lz4 1.8.0                 404 MB/s  1150 MB/s        1097  26.78 data//data_9
>> lz4 1.8.0                 216 MB/s   921 MB/s        3183  77.71 data//data_10
>> lz4 1.8.0                 456 MB/s  1101 MB/s        1011  24.68 data//data_11
>> lz4 1.8.0                 867 MB/s  1202 MB/s          37   0.90 data//data_12
>> 
>> 
>> LZ4 Dynamic Offet:  
>> sh-3.2# ./lzbench  -r  -elz4_dyn  data/
>> lzbench 1.7.3 (32-bit Linux)   Assembled by P.Skibinski
>> Compressor name         Compress. Decompress. Compr. size  Ratio Filename
>> memcpy                   2203 MB/s  2218 MB/s        4096 100.00 data//data_1
>> lz4 1.8.0                 218 MB/s   693 MB/s        2228  54.39 data//data_1
>> lz4 1.8.0                 273 MB/s   851 MB/s        1739  42.46 data//data_2
>> lz4 1.8.0                 230 MB/s   526 MB/s        1800  43.95 data//data_3
>> lz4 1.8.0                 321 MB/s   952 MB/s        1357  33.13 data//data_4
>> lz4 1.8.0                 470 MB/s  1075 MB/s         664  16.21 data//data_5
>> lz4 1.8.0                 303 MB/s   964 MB/s        1455  35.52 data//data_6
>> lz4 1.8.0                 345 MB/s   951 MB/s        2126  51.90 data//data_7
>> lz4 1.8.0                 744 MB/s  1163 MB/s         177   4.32 data//data_8
>> lz4 1.8.0                 409 MB/s  1257 MB/s        1033  25.22 data//data_9
>> lz4 1.8.0                 220 MB/s   857 MB/s        3049  74.44 data//data_10
>> lz4 1.8.0                 464 MB/s  1105 MB/s         934  22.80 data//data_11
>> lz4 1.8.0                 874 MB/s  1194 MB/s          36   0.88 data//data_12
>> 
>> 
>> LZ4 Dynamic Offset with 32K data:
>> sh-3.2# ./lzbench -elz4_dyn data/data32k
>> lzbench 1.7.3 (32-bit Linux)   Assembled by P.Skibinski
>> Compressor name         Compress. Decompress. Compr. size  Ratio Filename
>> memcpy                   5285 MB/s  5283 MB/s       32768 100.00 data/data32k
>> lz4 1.8.0                 274 MB/s   995 MB/s       13435  41.00 data/data32k
>> done... (cIters=1 dIters=1 cTime=1.0 dTime=2.0 chunkSize=1706MB cSpeed=0MB)
>> 
>> Original LZ4 with 32K data:
>> sh-3.2# ./lzbench_orig -elz4 data/data32k
>> lzbench 1.7.3 (32-bit Linux)   Assembled by P.Skibinski
>> Compressor name         Compress. Decompress. Compr. size  Ratio Filename
>> memcpy                   4918 MB/s  5108 MB/s       32768 100.00 data/data32k
>> lz4 1.8.0                 276 MB/s  1045 MB/s       14492  44.23 data/data32k
>> 
>> LZO1x with 32K data (Default Compressor for ZRAM): 
>> sh-3.2# ./lzbench -elzo1x,1 data/data32k
>> lzbench 1.7.3 (32-bit Linux)   Assembled by P.Skibinski
>> Compressor name         Compress. Decompress. Compr. size  Ratio Filename
>> memcpy                   5273 MB/s  5320 MB/s       32768 100.00 data/data32k
>> lzo1x 2.09 -1             283 MB/s   465 MB/s       14292  43.62 data/data32k


Thanks,
Maninder Singh
 

  parent reply	other threads:[~2018-04-16 10:21 UTC|newest]

Thread overview: 23+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
     [not found] <CGME20180321044137epcas5p221e7ee4a0b7464eaa00dad8320f0251d@epcas5p2.samsung.com>
2018-03-21  4:40 ` [PATCH 0/1] cover-letter/lz4: Implement lz4 with dynamic offset length Maninder Singh
     [not found]   ` <CGME20180321044149epcas5p12cba1afa5fc0f493a47fe20c4a7394bd@epcas5p1.samsung.com>
2018-03-21  4:40     ` [PATCH 1/1] lz4: " Maninder Singh
2018-03-21  7:49       ` Sergey Senozhatsky
2018-03-21 19:59       ` Nick Terrell
     [not found]       ` <CGME20180321195916epcas4p1e89e1cc5fd242ee8c348f92f1d1740b0@epcms5p1>
2018-03-22  4:28         ` Maninder Singh
2018-03-22 23:09       ` kbuild test robot
2018-03-22 23:32       ` kbuild test robot
     [not found]       ` <CGME20180321195916epcas4p1e89e1cc5fd242ee8c348f92f1d1740b0@epcms5p8>
2018-03-23 13:21         ` Vaneet Narang
     [not found]       ` <CGME20180321044149epcas5p12cba1afa5fc0f493a47fe20c4a7394bd@epcms5p5>
2018-04-02  5:51         ` Maninder Singh
2018-04-03 12:26           ` Sergey Senozhatsky
     [not found]           ` <CGME20180321044149epcas5p12cba1afa5fc0f493a47fe20c4a7394bd@epcms5p7>
2018-04-03 13:43             ` Vaneet Narang
2018-04-04  1:40               ` Sergey Senozhatsky
2018-03-21  6:41   ` [PATCH 0/1] cover-letter/lz4: " Sergey Senozhatsky
2018-03-21  8:26   ` Sergey Senozhatsky
2018-03-21 19:56     ` Nick Terrell
2018-03-22  2:43       ` Sergey Senozhatsky
     [not found]     ` <CGME20180321044137epcas5p221e7ee4a0b7464eaa00dad8320f0251d@epcms5p6>
2018-03-23 13:43       ` Vaneet Narang
2018-03-29 10:26       ` Maninder Singh
2018-03-30  5:41         ` Sergey Senozhatsky
     [not found]     ` <CGME20180321044137epcas5p221e7ee4a0b7464eaa00dad8320f0251d@epcms5p4>
2018-04-16 10:21       ` Maninder Singh [this message]
2018-04-16 19:34         ` Yann Collet
2018-04-16 20:01           ` Eric Biggers
     [not found]   ` <CGME20180321044137epcas5p221e7ee4a0b7464eaa00dad8320f0251d@epcms5p3>
2018-04-02  6:03     ` Maninder Singh

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=20180416102138epcms5p429e04b68b94dcf80cd84f7008467ed68@epcms5p4 \
    --to=maninder1.s@samsung.com \
    --cc=a.sahrawat@samsung.com \
    --cc=akpm@linux-foundation.org \
    --cc=anton@enomsg.org \
    --cc=ccross@android.com \
    --cc=colin.king@canonical.com \
    --cc=cyan@fb.com \
    --cc=davem@davemloft.net \
    --cc=gregkh@linuxfoundation.org \
    --cc=herbert@gondor.apana.org.au \
    --cc=keescook@chromium.org \
    --cc=linux-crypto@vger.kernel.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=minchan@kernel.org \
    --cc=ngupta@vflare.org \
    --cc=pankaj.m@samsung.com \
    --cc=sergey.senozhatsky.work@gmail.com \
    --cc=terrelln@fb.com \
    --cc=tony.luck@intel.com \
    --cc=v.narang@samsung.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 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).