On Wed, Sep 15, 2021 at 4:15 PM Thomas Monjalon <tho...@monjalon.net> wrote: > > The macro PMD_INIT_LOG includes already the line feed character. > Redundant \n are removed. > > Signed-off-by: Thomas Monjalon <tho...@monjalon.net>
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 <david.march...@redhat.com> -- David Marchand
