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
