Re: [PATCH] crypto/aesni_mb: add dynamic logging to aesni_mb
From: De Lara Guarch, Pablo <hidden>
Date: 2018-06-27 14:56:59
From: De Lara Guarch, Pablo <hidden>
Date: 2018-06-27 14:56:59
Hi Naga,
-----Original Message----- From: Somarowthu, Naga SureshX Sent: Tuesday, June 12, 2018 7:24 AM To: dev@dpdk.org Cc: De Lara Guarch, Pablo <redacted>; Pattan, Reshma [off-list ref]; Somarowthu, Naga SureshX [off-list ref] Subject: [PATCH] crypto/aesni_mb: add dynamic logging to aesni_mb From: Naga Suresh Somarowthu <redacted> 1.added new logtype for aesni_mb driver. 2.registered new logtype. 3.MB_LOG_ERR and CDEV_LOG_ERR are replaced with new logtype name AESNI_MB_PMD_LOG. Signed-off-by: Naga Suresh Somarowthu [off-list ref] Reviewed-by: Reshma Pattan <redacted>
Could you make the following changes? - Change the macro to AESNI_MB_LOG, to make it shorter. - Change title to: crypto/aesni_mb: add dynamic logging - Remove unused "CONFIG_RTE_LIBRTE_PMD_AESNI_MB_DEBUG=n" from config file. - Set log level of the new type to INFO (see isal_compress_pmd.c) Thanks, Pablo