[dpdk-dev] [PATCH] app/bbdev: update test vectors
De Lara Guarch, Pablo
pablo.de.lara.guarch at intel.com
Tue Apr 24 16:56:51 CEST 2018
> -----Original Message-----
> From: dev [mailto:dev-bounces at dpdk.org] On Behalf Of KamilX Chalupnik
> Sent: Wednesday, April 4, 2018 3:06 PM
> To: dev at dpdk.org
> Cc: Mokhtar, Amr <amr.mokhtar at intel.com>; Chalupnik, KamilX
> <kamilx.chalupnik at intel.com>
> Subject: [dpdk-dev] [PATCH] app/bbdev: update test vectors
>
> From: "Chalupnik, KamilX" <kamilx.chalupnik at intel.com>
>
> Update test vectors directory for Wireless Baseband Device:
> - creation of new vectors to test and validate BBDevice capabilities
> - python script used for tests execution updated
>
> Update the test app documentation:
> - vector test names updated
> - description of tests added
> - usage of test app updated
>
> Signed-off-by: KamilX Chalupnik <kamilx.chalupnik at intel.com>
Could you split this patch into multiple patches? You are doing several things here, so it is better to split each thing into a different patch.
Make sure that the functionality and compilation is not broken for each patch.
I think you can have a patch updating the names of current tests (updating the documentation in the same patch),
another patch adding the new tests (also updating the documentation) and a last patch with the rest of the
documentation changes.
This way, it will be easier to review.
Also, another comment below.
> --- a/doc/guides/tools/testbbdev.rst
> +++ b/doc/guides/tools/testbbdev.rst
...
>
> ./test-bbdev.py -p ../../x86_64-native-linuxapp-icc/app/testbbdev
> - -v ./test_vectors/bbdev_vector_td_default.data
> + -v turbo_dec_default.data
Shouldn't this be still under test_vectors?
More information about the dev
mailing list