[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