> From: Don Wallwork [mailto:d...@xsightlabs.com]
> Sent: Friday, 29 April 2022 22.01
> 
> Add support for using hugepages for worker lcore stack memory.  The
> intent is to improve performance by reducing stack memory related TLB
> misses and also by using memory local to the NUMA node of each lcore.
> 
> EAL option '--huge-worker-stack [stack-size-kbytes]' is added to allow
> the feature to be enabled at runtime.  If the size is not specified,
> the system pthread stack size will be used.

It would be nice if DPDK EAL could parse size parameter values provided as "1M" 
or "128k"; but it is clearly not a requirement for this patch. Just mentioning 
it.

> 
> Signed-off-by: Don Wallwork <d...@xsightlabs.com>
> ---

>  /**
>   * internal configuration
>   */
> @@ -102,6 +105,7 @@ struct internal_config {
>       unsigned int no_telemetry; /**< true to disable Telemetry */
>       struct simd_bitwidth max_simd_bitwidth;
>       /**< max simd bitwidth path to use */
> +     size_t huge_worker_stack_size; /**< worker thread stack size in
> kbytes */

The command line parameter value has been converted from kbytes to bytes here, 
so this comment is wrong.

Acked-by: Morten Brørup <m...@smartsharesystems.com>

Reply via email to