mbox

[0/5] Fix use of RTE_LOGTYPE_USER1 in libraries.

Message ID 20230208044825.1682620-1-stephen@networkplumber.org (mailing list archive)
Headers

Message

Stephen Hemminger Feb. 8, 2023, 4:48 a.m. UTC 
  This is split out of other rte_log patchset.
The DPDK libraries and drivers should not be using the USER1 logtype.

Stephen Hemminger (5):
  ip_frag: use a dynamic logtype
  reorder: use a dynamic logtype
  latencystats: use dynamic logtype
  vhost: use logtype instead of RTE_LOGTYPE_USER1
  ipsec: fix usage of RTE_LOGTYPE_USER1

 drivers/crypto/ipsec_mb/ipsec_mb_ops.c |   3 +-
 lib/ip_frag/ip_frag_common.h           |  16 ++-
 lib/ip_frag/ip_frag_internal.c         | 174 ++++++++++++-------------
 lib/ip_frag/rte_ip_frag_common.c       |  13 +-
 lib/ip_frag/rte_ipv4_reassembly.c      |  14 +-
 lib/ip_frag/rte_ipv6_reassembly.c      |  59 ++++-----
 lib/latencystats/rte_latencystats.c    |  55 ++++----
 lib/reorder/rte_reorder.c              |  35 ++---
 lib/vhost/fd_man.c                     |  16 ++-
 lib/vhost/vhost_crypto.c               |  24 ++--
 10 files changed, 206 insertions(+), 203 deletions(-)

Comments

Morten Brørup Aug. 14, 2023, 5:41 p.m. UTC | #1 
> From: Stephen Hemminger [mailto:stephen@networkplumber.org]
> Sent: Monday, 14 August 2023 18.31
> 
> Libraries in DPDK should be using dynamic logtype instead
> of overloading the static logtype RTE_LOGTYPE_USER1
> 
> v8 - rebase and add power library
> 
> Stephen Hemminger (6):
>   ip_frag: use a dynamic logtype
>   reorder: use a dynamic logtype
>   latencystats: use dynamic logtype
>   vhost: use logtype instead of RTE_LOGTYPE_USER1
>   ipsec: fix usage of RTE_LOGTYPE_USER1
>   power: use a dynamic logtype for guest channel

Series-acked-by: Morten Brørup <mb@smartsharesystems.com>
David Marchand Dec. 4, 2023, 12:25 p.m. UTC | #2 
Hello Stephen,

On Mon, Aug 14, 2023 at 6:31 PM Stephen Hemminger
<stephen@networkplumber.org> wrote:
>
> Libraries in DPDK should be using dynamic logtype instead
> of overloading the static logtype RTE_LOGTYPE_USER1
>
> v8 - rebase and add power library
>
> Stephen Hemminger (6):
>   ip_frag: use a dynamic logtype
>   reorder: use a dynamic logtype
>   latencystats: use dynamic logtype
>   vhost: use logtype instead of RTE_LOGTYPE_USER1
>   ipsec: fix usage of RTE_LOGTYPE_USER1
>   power: use a dynamic logtype for guest channel
>
>  drivers/crypto/ipsec_mb/ipsec_mb_ops.c |  3 ++-
>  lib/ip_frag/ip_frag_common.h           |  5 ++++-
>  lib/ip_frag/rte_ip_frag_common.c       |  8 +++++---
>  lib/latencystats/rte_latencystats.c    |  3 ++-
>  lib/power/guest_channel.c              |  3 ++-
>  lib/reorder/rte_reorder.c              |  6 +++---
>  lib/vhost/fd_man.c                     |  4 ++--
>  lib/vhost/vhost_crypto.c               | 22 +++++++++++-----------
>  8 files changed, 31 insertions(+), 23 deletions(-)

Series applied, thanks.