Hi Anoob, I've looked into the memory footprint issue, and I didn't find any. So could you please help to share the config file you are using? Maybe it can help me to accelerate the debug process.
Thanks a lot, Cheng > -----Original Message----- > From: Jiang, Cheng1 > Sent: Friday, June 16, 2023 11:16 PM > To: Anoob Joseph <ano...@marvell.com> > Cc: dev@dpdk.org; Hu, Jiayu <jiayu...@intel.com>; Ding, Xuan > <xuan.d...@intel.com>; tho...@monjalon.net; Richardson, Bruce > <bruce.richard...@intel.com>; m...@smartsharesystems.com; Xia, Chenbo > <chenbo....@intel.com>; Amit Prakash Shukla > <amitpraka...@marvell.com>; Ma, WenwuX <wenwux...@intel.com>; > Wang, YuanX <yuanx.w...@intel.com>; He, Xingguang > <xingguang...@intel.com> > Subject: RE: [EXT] [PATCH v6] app/dma-perf: introduce dma-perf application > > Hi Anoob, > > Replies are inline. > > Thanks, > Cheng > > > -----Original Message----- > > From: Anoob Joseph <ano...@marvell.com> > > Sent: Friday, June 16, 2023 6:53 PM > > To: Jiang, Cheng1 <cheng1.ji...@intel.com> > > Cc: dev@dpdk.org; Hu, Jiayu <jiayu...@intel.com>; Ding, Xuan > > <xuan.d...@intel.com>; tho...@monjalon.net; Richardson, Bruce > > <bruce.richard...@intel.com>; m...@smartsharesystems.com; Xia, Chenbo > > <chenbo....@intel.com>; Amit Prakash Shukla > > <amitpraka...@marvell.com>; Ma, WenwuX <wenwux...@intel.com>; > Wang, > > YuanX <yuanx.w...@intel.com>; He, Xingguang <xingguang...@intel.com> > > Subject: RE: [EXT] [PATCH v6] app/dma-perf: introduce dma-perf > > application > > > > Hi Cheng, > > > > I think there is a bug in EAL parsing. Please check the below diff and > > see if we should do something similar. > > > > diff --git a/app/test-dma-perf/main.c b/app/test-dma-perf/main.c index > > d65655b87b..7fcaa5acf6 100644 > > --- a/app/test-dma-perf/main.c > > +++ b/app/test-dma-perf/main.c > > @@ -432,7 +432,7 @@ append_eal_args(int argc, char **argv, const char > > *eal_args, char **new_argv) > > i++; > > continue; > > } > > - strlcpy(new_argv[new_argc], argv[i], > > sizeof(new_argv[new_argc])); > > + strlcpy(new_argv[new_argc], argv[i], > > + MAX_EAL_PARAM_LEN); > > new_argc++; > > } > > > > Thanks, > > Anoob > > [Cheng] yes there is an issue in it. And I have the same fix. I'll submit it > later, > thanks. > > > > > > -----Original Message----- > > > From: Anoob Joseph <ano...@marvell.com> > > > Sent: Friday, June 16, 2023 3:19 PM > > > To: Jiang, Cheng1 <cheng1.ji...@intel.com> > > > Cc: dev@dpdk.org; Hu, Jiayu <jiayu...@intel.com>; Ding, Xuan > > > <xuan.d...@intel.com>; tho...@monjalon.net; Richardson, Bruce > > > <bruce.richard...@intel.com>; m...@smartsharesystems.com; Xia, > Chenbo > > > <chenbo....@intel.com>; Amit Prakash Shukla > > > <amitpraka...@marvell.com>; Ma, WenwuX <wenwux...@intel.com>; > > Wang, > > > YuanX <yuanx.w...@intel.com>; He, Xingguang > <xingguang...@intel.com> > > > Subject: RE: [EXT] [PATCH v6] app/dma-perf: introduce dma-perf > > > application > > > > > > Hi Cheng, > > > > > > > [Cheng] sure, no problem. Feel free to contact me if you have any > > > > more questions. > > > > > > No further comments from my side. Can you submit next version? > > [Cheng] Glad to know. Yes, the next version is almost ready, I'll submit it > this > weekend. > > > > > > > Top level, I think you may need to add a doc file for the new app. > > [Cheng] sure, I was thinking about this, and I'll add a doc for this, thanks. > > > > > > > Few issues that we are also checking in our end, 1. The app is > > > having significant memory footprint. Need to see where we can improve. > > > 2. EAL args passing doesn't seem to be working. Are you able to > > > enable specific devices by making using of '-a' arg? > > [Cheng] I'll take a look at the significant memory footprint issue later, > thanks. > As for the eal args process, yes there is an issue, Dengdui's comments also > mentioned it, and I have fixed it in the v7 patch. > > > > > > > Thanks, > > > Anoob
