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 <pablo.de.lara.gua...@intel.com>; Pattan, Reshma > <reshma.pat...@intel.com>; Chaitanya Babu, TalluriX > <tallurix.chaitanya.b...@intel.com> > Subject: [PATCH] crypto/aesni_gcm: add dynamic logging to aesni_gcm > > From: tchaitax <tallurix.chaitanya.b...@intel.com> > > 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 <tallurix.chaitanya.b...@intel.com> > Reviewed-by: Reshma Pattan <reshma.pat...@intel.com>
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
