[Accel-config] Re: [PATCH v4 00/10] Separate common codes from DSA source code file

On 4/9/2022 7:14 PM, Li Zhang wrote:
> For sharing codes between DSA and IAA test code, move all
> reused code into common files, merge structs, rename symbols
> for universal meaning.

For the series

Reviewed-by: Dave Jiang <dave.jiang(a)intel.com>


>
> Changelog:
> v2:
> - Change the prefix actst_ to acctest_
> - For avoiding using macro to conditionally compile DSA and IAA source
>    code, merge DSA and IAA structs.
> v3:
> - Add compl_size in struct accfg_device and struct acctest_context,
>    modify accel config and DSA code for using compl_size to allocate
>    and dump completion record.
> v4:
> - Split compl_size relate patch into 3 patches,
>    The 1st: Add completion record size setup for accfg_device.
>    The 2nd: Export API to retrieve completion record size for the device.
>    The 3rd: Use compl_size in DSA test code.
>
> Li Zhang (10):
>    accel-config/test: DSA functions are renamed universal name
>    accel-config/test: Macros are renamed universal name
>    accel-config: Add completion record size setup for accfg_device
>    accel-config: Export API to retrieve completion record size for the
>      device
>    accel-config/test: Use compl_size in DSA test code
>    accel-config/test: Modify structs for reused by DSA and IAA
>    accel-config/test: Modify name of structs for universal meaning
>    accel-config/test: Create new files for placing reused code
>    accel-config/test: Rename prep.c to dsa_prep.c
>    accel-config/test: Move common code into common files
>
>   accfg/idxd.h                  | 104 ++--
>   accfg/lib/libaccel-config.sym |   1 +
>   accfg/lib/libaccfg.c          |  23 +
>   accfg/lib/private.h           |   1 +
>   accfg/libaccel_config.h       |   6 +
>   test/Makefile.am              |   2 +-
>   test/accel_test.c             | 515 ++++++++++++++++++++
>   test/accel_test.h             | 245 ++++++++++
>   test/dsa.c                    | 878 ++++++++--------------------------
>   test/dsa.h                    | 301 ++----------
>   test/{prep.c => dsa_prep.c}   | 305 ++++++------
>   test/dsa_test.c               | 187 ++++----
>   12 files changed, 1335 insertions(+), 1233 deletions(-)
>   create mode 100644 test/accel_test.c
>   create mode 100644 test/accel_test.h
>   rename test/{prep.c => dsa_prep.c} (68%)
>