RE: [EXTERNAL] [PATCH v5 1/2] compress/uadk: use async mode to replace sync mode

[Akhil Goyal <gakhil@marvell.com>]

> On Wed, 9 Jul 2025 at 18:03, Akhil Goyal <gakhil@marvell.com> wrote:
> >
> > > On Wed, 9 Jul 2025 at 15:19, Akhil Goyal <gakhil@marvell.com> wrote:
> > > >
> > > > > To get better performance, using async mode to replace sync mode
> > > > >
> > > > > Signed-off-by: Zhangfei Gao <zhangfei.gao@linaro.org>
> > > > > ---
> > > > >  doc/guides/compressdevs/uadk.rst              | 15 +++
> > > > >  drivers/compress/uadk/meson.build             |  4 +-
> > > > >  drivers/compress/uadk/uadk_compress_pmd.c     | 94 ++++++++++++++--
> ---
> > > > >  .../compress/uadk/uadk_compress_pmd_private.h |  2 +-
> > > > >  4 files changed, 86 insertions(+), 29 deletions(-)
> > > > >
> > > > > diff --git a/doc/guides/compressdevs/uadk.rst
> > > > > b/doc/guides/compressdevs/uadk.rst
> > > > > index a29043e0c0..8812fd180a 100644
> > > > > --- a/doc/guides/compressdevs/uadk.rst
> > > > > +++ b/doc/guides/compressdevs/uadk.rst
> > > > > @@ -96,3 +96,18 @@ Test steps
> > > > >       sudo dpdk-test --vdev=compress_uadk
> > > > >       RTE>>compressdev_autotest
> > > > >       RTE>>quit
> > > > > +
> > > > > +
> > > > > +As a reference, the following table shows a mapping between the past
> DPDK
> > > > > versions
> > > > > +and the UADK library version supported by them:
> > > > > +
> > > > > +.. _table_uadk_compress_pmd_versions:
> > > > > +
> > > > > +.. table:: DPDK and external UADK library version compatibility
> > > > > +
> > > > > +   ==============  ============================
> > > > > +   DPDK version    UADK library version
> > > > > +   ==============  ============================
> > > > > +   24.07 - 25.03   2.4  - 2.8
> > > > > +   25.07+          2.9.1
> > > > > +   ==============  ============================
> > > > > diff --git a/drivers/compress/uadk/meson.build
> > > > > b/drivers/compress/uadk/meson.build
> > > > > index 34113cd08c..32cadd02ea 100644
> > > > > --- a/drivers/compress/uadk/meson.build
> > > > > +++ b/drivers/compress/uadk/meson.build
> > > > > @@ -13,7 +13,7 @@ sources = files(
> > > > >  )
> > > > >
> > > > >  deps += 'bus_vdev'
> > > > > -dep = dependency('libwd_comp', required: false, method: 'pkg-config')
> > > > > +dep = dependency('libwd_comp', version: '>=2.9.1', required: false,
> method:
> > > > > 'pkg-config')
> > > > >  if not dep.found()
> > > > >      build = false
> > > > >      reason = 'missing dependency, "libwd_comp"'
> > > > > @@ -21,7 +21,7 @@ else
> > > > >      ext_deps += dep
> > > > >  endif
> > > > >
> > > > > -dep = dependency('libwd', required: false, method: 'pkg-config')
> > > > > +dep = dependency('libwd', version: '>=2.9.1', required: false, method:
> 'pkg-
> > > > > config')
> > > > >  if not dep.found()
> > > > >      build = false
> > > > >      reason = 'missing dependency, "libwd"'
> > > >
> > > > I am seeing this now
> > > > Dependency libwd_comp found: NO. Found 2.9 but need: '>=2.9.1'
> > >
> > > Hi, Akhil
> > >
> > > The master has merged a release patch 80adefa this afternoon
> > > https://urldefense.proofpoint.com/v2/url?u=https-
> > <https://urldefense.proofpoint.com/v2/url?u=https-  >> >
> 3A__github.com_Linaro_uadk_commits_master_&d=DwIBaQ&c=nKjWec2b6R0m
> > >
> OyPaz7xtfQ&r=DnL7Si2wl_PRwpZ9TWey3eu68gBzn7DkPwuqhd6WNyo&m=zu_lC
> > >
> 2WIV6KqBhSa15SOw4sYvI6W3SromCy_Z_A19mliFmfbx6fQsi8SVmdY7OVv&s=oYs
> > > SVDbopcr5aRNW6_Z36Jup6ccbKTve3V1Gp6jUtPo&e=
> > > And the https://urldefense.proofpoint.com/v2/url?u=https-
> > <https://urldefense.proofpoint.com/v2/url?u=https-  >> >
> 3A__github.com_Linaro_uadk_tree_v2.9.1&d=DwIBaQ&c=nKjWec2b6R0mOyPaz
> > >
> 7xtfQ&r=DnL7Si2wl_PRwpZ9TWey3eu68gBzn7DkPwuqhd6WNyo&m=zu_lC2WIV
> > >
> 6KqBhSa15SOw4sYvI6W3SromCy_Z_A19mliFmfbx6fQsi8SVmdY7OVv&s=Yt0A8F8
> > > o96plEAsy4VfmELvDkjGPwh3YkcpcXzWtbaY&e= has updated to the
> > > release 2.9.1
> > >
> > > Would you mind re-check.
> > >
> > Still getting this
> >
> > Dependency libwd_comp found: YES 2.9.1 (cached)
> > Dependency libwd found: NO found 2.9 but need: '>=2.9.1' (cached)
> 
> it may be because we ret-tag v2.9.1 to the 80adefa, removing the old
> tag may be required.
> 
> git tag -d v2.9.1
> Deleted tag 'v2.9.1' (was 999f79b)
> 
> git fetch origin
> From https://urldefense.proofpoint.com/v2/url?u=https-
> 3A__github.com_Linaro_uadk&d=DwIBaQ&c=nKjWec2b6R0mOyPaz7xtfQ&r=DnL
> 7Si2wl_PRwpZ9TWey3eu68gBzn7DkPwuqhd6WNyo&m=Gx53tufwHvZ7dBhZujOj
> Uv-FiwfOMDX5eblHHsIcKFCK0srJJA1IbPqi-
> NvC0hDP&s=MWvXtIabRZ2W7MokxZ9dFzMcgW1ZJvn8HnqBRpoqkfY&e=
>  * [new tag]         v2.9.1     -> v2.9.1
> 
> 
> git fetch origin
> git checkout -b 2.9.1 v2.9.1
> git log --oneline
> 80adefa Merge pull request #681 from gaozhangfei/rel-2.9.1
> 0f75d0f uadk: release 2.9.1
> 51915ff Merge pull request #680 from gaozhangfei/check_x86
> 34df8cf uadk: check whether host cpu is aarch64
> d593554 uadk: fix the compilation error on x86
> 
> sudo rm /usr/local/lib/libwd*
> 
> ./cleanup.sh;
> ./autogen.sh;
> ./configure
> make -j128
> sudo make install
> 
>  ls .libs/

Clean build worked.
Thanks.
Series Applied to dpdk-next-crypto