mbox series

[v2,0/3] malloc related cleanups

Message ID 20240615160215.117401-1-stephen@networkplumber.org (mailing list archive)
Headers
Series malloc related cleanups |

Message

Stephen Hemminger June 15, 2024, 4 p.m. UTC 
The type parameter for malloc is only used for tracing.
Fix documentation and don't pass through heap routines.

Stephen Hemminger (3):
  rte_malloc: document that type is for tracing
  eal: remove type argument from internal routines
  event/sw: avoid snprintf truncation

v2 - keep usage for tracing
   - combine event malloc type patch

 drivers/event/sw/iq_chunk.h         |  2 --
 drivers/event/sw/sw_evdev.c         |  2 +-
 lib/eal/common/eal_common_memzone.c |  6 ++---
 lib/eal/common/malloc_heap.c        | 39 ++++++++++++-----------------
 lib/eal/common/malloc_heap.h        |  7 +++---
 lib/eal/common/rte_malloc.c         |  2 +-
 lib/eal/include/rte_malloc.h        | 24 +++++++++---------
 7 files changed, 35 insertions(+), 47 deletions(-)

Comments

Morten Brørup June 15, 2024, 9:40 p.m. UTC | #1 
> From: Stephen Hemminger [mailto:stephen@networkplumber.org]
> Sent: Saturday, 15 June 2024 18.00
> 
> The type parameter for malloc is only used for tracing.
> Fix documentation and don't pass through heap routines.
> 
> Stephen Hemminger (3):
>   rte_malloc: document that type is for tracing
>   eal: remove type argument from internal routines
>   event/sw: avoid snprintf truncation
> 
> v2 - keep usage for tracing
>    - combine event malloc type patch

For the series,
Acked-by: Morten Brørup <mb@smartsharesystems.com>
Patrick Robb June 17, 2024, 5:48 p.m. UTC | #2 
For CI Testing: re-applying to main and retesting because main was in
a bad state (now fixed by Maxime), causing a virtio_smoke fail on this
series.

https://git.dpdk.org/dpdk/commit/?id=6bdc14606724bc7fb3834d5ec59b1cccf98adf28

On Sat, Jun 15, 2024 at 12:02 PM Stephen Hemminger
<stephen@networkplumber.org> wrote:
>
> The type parameter for malloc is only used for tracing.
> Fix documentation and don't pass through heap routines.
>
> Stephen Hemminger (3):
>   rte_malloc: document that type is for tracing
>   eal: remove type argument from internal routines
>   event/sw: avoid snprintf truncation
>
> v2 - keep usage for tracing
>    - combine event malloc type patch
>
>  drivers/event/sw/iq_chunk.h         |  2 --
>  drivers/event/sw/sw_evdev.c         |  2 +-
>  lib/eal/common/eal_common_memzone.c |  6 ++---
>  lib/eal/common/malloc_heap.c        | 39 ++++++++++++-----------------
>  lib/eal/common/malloc_heap.h        |  7 +++---
>  lib/eal/common/rte_malloc.c         |  2 +-
>  lib/eal/include/rte_malloc.h        | 24 +++++++++---------
>  7 files changed, 35 insertions(+), 47 deletions(-)
>
> --
> 2.43.0
>