Hi Ravi,

> -----Original Message-----
> From: Ravi Kumar [mailto:ravi1.ku...@amd.com]
> Sent: Wednesday, May 9, 2018 1:46 PM
> To: dev@dpdk.org
> Cc: De Lara Guarch, Pablo <pablo.de.lara.gua...@intel.com>;
> tho...@monjalon.net; hemant.agra...@nxp.com
> Subject: [PATCH v2] crypto/ccp: support command line option to enable CPU
> based auth
> 
> CCP PMD supports authentication offload to either of CCP or CPU.
> The earlier version of patch provides this option as compile time.
> This patch changes this option from compile time to run time.
> User can pass "ccp_auth_opt=1" as an additional arguments to vdev
> parameter to enable authentication operations on CPU.
> 
> Signed-off-by: Ravi Kumar <ravi1.ku...@amd.com>

...

>  RTE_PMD_REGISTER_VDEV(CRYPTODEV_NAME_CCP_PMD,
> cryptodev_ccp_pmd_drv);
>  RTE_PMD_REGISTER_PARAM_STRING(CRYPTODEV_NAME_CCP_PMD,
> -     "max_nb_queue_pairs=<int> max_nb_sessions=<int> socket_id=<int>");
> +     "max_nb_queue_pairs=<int> "
> +     "max_nb_sessions=<int> "
> +     "socket_id=<int> "
> +     "ccp_auth_opt=<int>");


Die_num is not in this list. I was going to add it myself,
but then I realized that it is not actually used anywhere.
Am I missing something?

Also, I think it is worth updating the documentation of the PMD with these 
parameters.

Thanks,
Pablo

Reply via email to