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