[dpdk-dev] [PATCH] crypto/null: add dynamic logging to null driver

De Lara Guarch, Pablo pablo.de.lara.guarch at intel.com
Wed Jun 27 18:27:36 CEST 2018


Hi Jananee/Pallantla,

> -----Original Message-----
> From: Parthasarathy, JananeeX M
> Sent: Friday, June 15, 2018 11:44 AM
> To: dev at dpdk.org
> Cc: De Lara Guarch, Pablo <pablo.de.lara.guarch at intel.com>; Pattan, Reshma
> <reshma.pattan at intel.com>; Parthasarathy, JananeeX M
> <jananeex.m.parthasarathy at intel.com>; Poornima, PallantlaX
> <pallantlax.poornima at intel.com>
> Subject: [PATCH] crypto/null: add dynamic logging to null driver
> 
> 1.added new logtype for null driver.
> 2.registered new logtype.
> 3.NULL_CRYPTO_LOG_ERR, RTE_LOG_ERR and CDEV_LOG_ERR are replaced
> with new logtype name NULL_PMD_LOG.
> 
> Signed-off-by: Pallantla Poornima <pallantlax.poornima at intel.com>
> Reviewed-by: Reshma Pattan <reshma.pattan at intel.com>

Could you make the following changes?

- Change the macro to NULL_LOG, to make it shorter.
- Change title to: crypto/null: add dynamic logging
- There is a mismatch between the author name and the name in the Signed-off line.
  Could you fix it?
- Set log level of the new type to INFO (see isal_compress_pmd.c)

Thanks,
Pablo


More information about the dev mailing list