[EXT] [PATCH v6] app/dma-perf: introduce dma-perf application
Anoob Joseph
anoobj at marvell.com
Fri Jun 16 12:52:56 CEST 2023
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
> -----Original Message-----
> From: Anoob Joseph <anoobj at marvell.com>
> Sent: Friday, June 16, 2023 3:19 PM
> To: Jiang, Cheng1 <cheng1.jiang at intel.com>
> Cc: dev at dpdk.org; Hu, Jiayu <jiayu.hu at intel.com>; Ding, Xuan
> <xuan.ding at intel.com>; thomas at monjalon.net; Richardson, Bruce
> <bruce.richardson at intel.com>; mb at smartsharesystems.com; Xia, Chenbo
> <chenbo.xia at intel.com>; Amit Prakash Shukla
> <amitprakashs at marvell.com>; Ma, WenwuX <wenwux.ma at intel.com>;
> Wang, YuanX <yuanx.wang at intel.com>; He, Xingguang
> <xingguang.he at 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?
>
> Top level, I think you may need to add a doc file for the new app.
>
> 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?
>
> Thanks,
> Anoob
More information about the dev
mailing list