> -----Original Message-----
> From: Tal Shnaiderman <tal...@nvidia.com>
> Sent: Sunday, April 11, 2021 4:00 AM
> To: Xueming(Steven) Li <xuemi...@nvidia.com>; NBU-Contact-Thomas Monjalon
> <tho...@monjalon.net>; Gaetan Rivet
> <gaet...@nvidia.com>
> Cc: dev@dpdk.org; Xueming(Steven) Li <xuemi...@nvidia.com>; Asaf Penso
> <as...@nvidia.com>; Wenzhuo Lu
> <wenzhuo...@intel.com>; Beilei Xing <beilei.x...@intel.com>; Bernard
> Iremonger <bernard.iremon...@intel.com>; Gaetan Rivet
> <gr...@u256.net>; Anatoly Burakov <anatoly.bura...@intel.com>; Dmitry Kozlyuk
> <dmitry.kozl...@gmail.com>; Narcisa Ana Maria
> Vasile <navas...@linux.microsoft.com>; Dmitry Malloy <dmit...@microsoft.com>;
> Pallavi Kadam <pallavi.ka...@intel.com>; Ray
> Kinsella <m...@ashroe.eu>; Neil Horman <nhor...@tuxdriver.com>; Ferruh Yigit
> <ferruh.yi...@intel.com>; Andrew Rybchenko
> <andrew.rybche...@oktetlabs.ru>
> Subject: RE: [dpdk-dev] [PATCH v4 1/5] devargs: unify scratch buffer storage
>
> > Subject: [dpdk-dev] [PATCH v4 1/5] devargs: unify scratch buffer
> > storage
> >
> > In current design, legacy parser rte_devargs_parse() saved scratch
> > buffer to devargs.args while new parser rte_devargs_layers_parse()
> > saved to devargs.data. Code using devargs had to know the difference
> > and cleaned up memory accordingly - error prone.
> >
> > This patch unifies scratch buffer to data field, introduces
> > rte_devargs_reset() function to wrap the memory clean up logic.
> >
> > Signed-off-by: Xueming Li <xuemi...@nvidia.com>
> > Acked-by: Ray Kinsella <m...@ashroe.eu>
> > Reviewed-by: Gaetan Rivet <gr...@u256.net>
> > ---
> > app/test-pmd/config.c | 3 +-
> > app/test-pmd/testpmd.c | 5 +--
> > drivers/bus/vdev/vdev.c | 9 +++---
> > drivers/net/failsafe/failsafe_args.c | 3 +-
> > drivers/net/failsafe/failsafe_eal.c | 2 +-
> > examples/multi_process/hotplug_mp/commands.c | 6 ++--
> > lib/librte_eal/common/eal_common_dev.c | 9 +++---
> > lib/librte_eal/common/eal_common_devargs.c | 34 +++++++++++---------
> > lib/librte_eal/common/hotplug_mp.c | 6 ++--
> > lib/librte_eal/include/rte_devargs.h | 18 ++++++++---
> > lib/librte_eal/rte_eal_exports.def | 1 +
>
> rte_eal_exports.def was united with version.map and removed thus the
> modification above is unneeded.
Thanks, I'll rebase my code :)
>
> > lib/librte_eal/version.map | 1 +
> > lib/librte_ethdev/rte_ethdev.c | 8 ++---
> > 13 files changed, 59 insertions(+), 46 deletions(-)