diff mbox series

[v1,08/11] test/bbdev: increase max burst size

Message ID 20230929181328.104311-9-hernan.vargas@intel.com (mailing list archive)
State Superseded, archived
Delegated to: Maxime Coquelin
Headers
Series test-bbdev changes for 23.11 |

Checks

Context Check Description
ci/checkpatch success coding style OK

Commit Message

Hernan Vargas Sept. 29, 2023, 6:13 p.m. UTC 
  Increase max burst size to 1023.

Signed-off-by: Hernan Vargas <hernan.vargas@intel.com>
---
 app/test-bbdev/main.h | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

Comments

Maxime Coquelin Oct. 17, 2023, 7:45 p.m. UTC | #1 
On 9/29/23 20:13, Hernan Vargas wrote:
> Increase max burst size to 1023.
> 
> Signed-off-by: Hernan Vargas <hernan.vargas@intel.com>
> ---
>   app/test-bbdev/main.h | 2 +-
>   1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/app/test-bbdev/main.h b/app/test-bbdev/main.h
> index dc10a505d141..ac0083ec6673 100644
> --- a/app/test-bbdev/main.h
> +++ b/app/test-bbdev/main.h
> @@ -16,7 +16,7 @@
>   #define TEST_FAILED     -1
>   #define TEST_SKIPPED    1
>   
> -#define MAX_BURST 512U
> +#define MAX_BURST 1023U

Why not 1024? It does not sound natural to change from a pow2 to a pow2
- 1, maybe add a comment to justify?

>   #define MAX_OPS 1024U
>   #define DEFAULT_BURST 32U
>   #define DEFAULT_OPS 64U
diff mbox series

Patch

diff --git a/app/test-bbdev/main.h b/app/test-bbdev/main.h
index dc10a505d141..ac0083ec6673 100644
--- a/app/test-bbdev/main.h
+++ b/app/test-bbdev/main.h
@@ -16,7 +16,7 @@ 
 #define TEST_FAILED     -1
 #define TEST_SKIPPED    1
 
-#define MAX_BURST 512U
+#define MAX_BURST 1023U
 #define MAX_OPS 1024U
 #define DEFAULT_BURST 32U
 #define DEFAULT_OPS 64U