On Wed, Sep 15, 2021 at 4:15 PM Thomas Monjalon <[email protected]> wrote: > > The macro PMD_INIT_LOG includes already the line feed character. > Redundant \n are removed. > > Signed-off-by: Thomas Monjalon <[email protected]>
Ah.. I did a similar patch cleaning logs in net/virtio this morning. Applying and diffing with mine, the differences are: - calls to PMD_DRV_LOG with the same issue, - calls to RTE_LOG(xx, PMD, ) (maybe worth a separate patch with a Fixes: + backport), This could be done in a followup, or directly as a v2 of this cleanup. In any case, the current patch lgtm. Reviewed-by: David Marchand <[email protected]> -- David Marchand
