mbox series

[v4,0/4] compressdev: add LZ4 support

Message ID 20230214174029.2397125-1-michaelba@nvidia.com (mailing list archive)
Headers
Series compressdev: add LZ4 support |

Message

Michael Baum Feb. 14, 2023, 5:40 p.m. UTC 
  Add support for LZ4 compression algorithm for both API and app.

In addition, enable app to test de/comp only. This option already
exists in both app doc and arguments but isn't implemented inside the
application.

LZ4: https://github.com/lz4/lz4
RFC:
https://patchwork.dpdk.org/project/dpdk/patch/20220410182622.8828-1-rzidane@nvidia.com/

v2:
 - Add macros for all LZ4 flags in the standard.
 - Rebase.

v3:
 - Rebase.

v4:
 - Change "allow test single" commit to fix and add "Cc: stable@dpdk.org".
 - Take all typo fixes into separate commit.
 - Squash "add LZ4 algo" commit into "add algo option" commit.
 - Remove unnecessary release notes updates.

Michael Baum (4):
  compressdev: add LZ4 algorithm support
  app/test-compress-perf: fix some typos
  app/test-compress-perf: fix missing test single compress op
  app/test-compress-perf: add algo option

 app/test-compress-perf/comp_perf_options.h    |   9 +-
 .../comp_perf_options_parse.c                 |  83 +++++++++++-
 .../comp_perf_test_common.c                   | 126 +++++++++++++-----
 .../comp_perf_test_cyclecount.c               |  82 +++++++-----
 .../comp_perf_test_throughput.c               |  75 +++++++----
 .../comp_perf_test_verify.c                   |  75 +++++++----
 app/test-compress-perf/main.c                 |  63 +++++++--
 doc/guides/compressdevs/features/default.ini  |   7 +
 doc/guides/rel_notes/release_23_03.rst        |   7 +
 doc/guides/tools/comp_perf.rst                |   5 +
 lib/compressdev/rte_comp.c                    |  12 ++
 lib/compressdev/rte_comp.h                    |  79 ++++++++++-
 12 files changed, 479 insertions(+), 144 deletions(-)

Comments

Akhil Goyal Feb. 16, 2023, 8:01 a.m. UTC | #1 
> Add support for LZ4 compression algorithm for both API and app.
> 
> In addition, enable app to test de/comp only. This option already
> exists in both app doc and arguments but isn't implemented inside the
> application.
> 
> LZ4: https://urldefense.proofpoint.com/v2/url?u=https-
> 3A__github.com_lz4_lz4&d=DwIDAg&c=nKjWec2b6R0mOyPaz7xtfQ&r=DnL7Si2
> wl_PRwpZ9TWey3eu68gBzn7DkPwuqhd6WNyo&m=zuvFwq-
> wCfo4E5qcsXuVW4Tg-xq4cnKWlMdx8BrlcwM9-
> mjeKpQgAejfzaUzSwyP&s=89JL5ZLAeh2gshZ3Gqohsr3JZomgK9pjaCnLRHpnG0w
> &e=
> RFC:
> https://urldefense.proofpoint.com/v2/url?u=https-
> 3A__patchwork.dpdk.org_project_dpdk_patch_20220410182622.8828-2D1-
> 2Drzidane-
> 40nvidia.com_&d=DwIDAg&c=nKjWec2b6R0mOyPaz7xtfQ&r=DnL7Si2wl_PRwpZ
> 9TWey3eu68gBzn7DkPwuqhd6WNyo&m=zuvFwq-wCfo4E5qcsXuVW4Tg-
> xq4cnKWlMdx8BrlcwM9-
> mjeKpQgAejfzaUzSwyP&s=HhD8pa6ivCYKy89RMuLN2OhehLXxbluIyVWxF0QdjXE
> &e=
> 
> v2:
>  - Add macros for all LZ4 flags in the standard.
>  - Rebase.
> 
> v3:
>  - Rebase.
> 
> v4:
>  - Change "allow test single" commit to fix and add "Cc: stable@dpdk.org".
>  - Take all typo fixes into separate commit.
>  - Squash "add LZ4 algo" commit into "add algo option" commit.
>  - Remove unnecessary release notes updates.
Applied to dpdk-next-crypto

Thanks