Hi Anoob, I have a question about the doc. Do you think I should add a folder in the doc/guides/ just like doc/guides/dma_perf?
Thanks, Cheng > -----Original Message----- > From: Jiang, Cheng1 > Sent: Friday, June 16, 2023 11:16 PM > To: Anoob Joseph <[email protected]> > Cc: [email protected]; Hu, Jiayu <[email protected]>; Ding, Xuan > <[email protected]>; [email protected]; Richardson, Bruce > <[email protected]>; [email protected]; Xia, Chenbo > <[email protected]>; Amit Prakash Shukla > <[email protected]>; Ma, WenwuX <[email protected]>; > Wang, YuanX <[email protected]>; He, Xingguang > <[email protected]> > Subject: RE: [EXT] [PATCH v6] app/dma-perf: introduce dma-perf application > > Hi Anoob, > > Replies are inline. > > Thanks, > Cheng > > > -----Original Message----- > > From: Anoob Joseph <[email protected]> > > Sent: Friday, June 16, 2023 6:53 PM > > To: Jiang, Cheng1 <[email protected]> > > Cc: [email protected]; Hu, Jiayu <[email protected]>; Ding, Xuan > > <[email protected]>; [email protected]; Richardson, Bruce > > <[email protected]>; [email protected]; Xia, Chenbo > > <[email protected]>; Amit Prakash Shukla > > <[email protected]>; Ma, WenwuX <[email protected]>; > Wang, > > YuanX <[email protected]>; He, Xingguang <[email protected]> > > 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 <[email protected]> > > > Sent: Friday, June 16, 2023 3:19 PM > > > To: Jiang, Cheng1 <[email protected]> > > > Cc: [email protected]; Hu, Jiayu <[email protected]>; Ding, Xuan > > > <[email protected]>; [email protected]; Richardson, Bruce > > > <[email protected]>; [email protected]; Xia, > Chenbo > > > <[email protected]>; Amit Prakash Shukla > > > <[email protected]>; Ma, WenwuX <[email protected]>; > > Wang, > > > YuanX <[email protected]>; He, Xingguang > <[email protected]> > > > 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
