Message ID | cover.1709210551.git.gmuthukrishn@marvell.com (mailing list archive) |
---|---|
Headers |
Return-Path: <dev-bounces@dpdk.org> X-Original-To: patchwork@inbox.dpdk.org Delivered-To: patchwork@inbox.dpdk.org Received: from mails.dpdk.org (mails.dpdk.org [217.70.189.124]) by inbox.dpdk.org (Postfix) with ESMTP id 4E61043C3B; Thu, 29 Feb 2024 14:48:20 +0100 (CET) Received: from mails.dpdk.org (localhost [127.0.0.1]) by mails.dpdk.org (Postfix) with ESMTP id 3E559402CD; Thu, 29 Feb 2024 14:48:20 +0100 (CET) Received: from mx0b-0016f401.pphosted.com (mx0b-0016f401.pphosted.com [67.231.156.173]) by mails.dpdk.org (Postfix) with ESMTP id 3E5EF41109 for <dev@dpdk.org>; Thu, 29 Feb 2024 14:48:19 +0100 (CET) Received: from pps.filterd (m0045851.ppops.net [127.0.0.1]) by mx0b-0016f401.pphosted.com (8.17.1.24/8.17.1.24) with ESMTP id 41T7OAiI029089; Thu, 29 Feb 2024 05:48:18 -0800 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=marvell.com; h= from:to:cc:subject:date:message-id:in-reply-to:references :mime-version:content-transfer-encoding:content-type; s= pfpt0220; bh=p0tT1MZPWLJTG0zCQZQ44jc3mDa0W/AATGvHkO3Velo=; b=VSu PXOoUBrUM9XvMEMWCEEKB/LvWJNgconEg2Uv+MGQMPru2kujeoQQRJtCOAUDKDtr ircbsRdIDwZSKF2VeVlGihLQUEr5ttBBhuRLUZH8BphllMMgGLPz7zn9iuE8WWKd kCOM6rCaFMoF/0VY18DK1nAIV6kUMlykjEdrwaOPu45KK27T2/QyoVrbhGicHbzt mV51wuhGKo+olDYoYzN/WDdO5Q/kYxkwLrMWG4xoWW8aqLyUlVwJ0EE2hARB052G Gt3LhIzsu2vP26VhCpsr4sICOvurn7ALDfVxae4X6q8PNjVQCY7hTSXu7+CxMXPA z254rboFNmbXiLgBP5Q== Received: from dc6wp-exch02.marvell.com ([4.21.29.225]) by mx0b-0016f401.pphosted.com (PPS) with ESMTPS id 3wjfay2xcu-1 (version=TLSv1.2 cipher=ECDHE-RSA-AES256-GCM-SHA384 bits=256 verify=NOT); Thu, 29 Feb 2024 05:48:18 -0800 (PST) Received: from DC6WP-EXCH02.marvell.com (10.76.176.209) by DC6WP-EXCH02.marvell.com (10.76.176.209) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.2.1258.12; Thu, 29 Feb 2024 05:48:17 -0800 Received: from maili.marvell.com (10.69.176.80) by DC6WP-EXCH02.marvell.com (10.76.176.209) with Microsoft SMTP Server id 15.2.1258.12 via Frontend Transport; Thu, 29 Feb 2024 05:48:17 -0800 Received: from BG-LT91401.marvell.com (unknown [10.193.66.108]) by maili.marvell.com (Postfix) with ESMTP id EF1603F7253; Thu, 29 Feb 2024 05:48:13 -0800 (PST) From: Gowrishankar Muthukrishnan <gmuthukrishn@marvell.com> To: <dev@dpdk.org> CC: <anoobj@marvell.com>, Cheng Jiang <honest.jiang@foxmail.com>, Kevin Laatz <kevin.laatz@intel.com>, Bruce Richardson <bruce.richardson@intel.com>, "Pavan Nikhilesh" <pbhagavatula@marvell.com>, Amit Prakash Shukla <amitprakashs@marvell.com>, Chengwen Feng <fengchengwen@huawei.com>, Jerin <jerinj@marvell.com>, Gowrishankar Muthukrishnan <gmuthukrishn@marvell.com> Subject: [v11 0/4] PCI Dev and SG copy support Date: Thu, 29 Feb 2024 19:18:06 +0530 Message-ID: <cover.1709210551.git.gmuthukrishn@marvell.com> X-Mailer: git-send-email 2.21.0 In-Reply-To: <20240227183555.3932711-1-amitprakashs@marvell.com> References: <20240227183555.3932711-1-amitprakashs@marvell.com> MIME-Version: 1.0 Content-Transfer-Encoding: 8bit Content-Type: text/plain X-Proofpoint-ORIG-GUID: E5PkerG-cY3yAv4NfOdGxcLRHgmwreT2 X-Proofpoint-GUID: E5PkerG-cY3yAv4NfOdGxcLRHgmwreT2 X-Proofpoint-Virus-Version: vendor=baseguard engine=ICAP:2.0.272,Aquarius:18.0.1011,Hydra:6.0.619,FMLib:17.11.176.26 definitions=2024-02-29_02,2024-02-29_01,2023-05-22_02 X-BeenThere: dev@dpdk.org X-Mailman-Version: 2.1.29 Precedence: list List-Id: DPDK patches and discussions <dev.dpdk.org> List-Unsubscribe: <https://mails.dpdk.org/options/dev>, <mailto:dev-request@dpdk.org?subject=unsubscribe> List-Archive: <http://mails.dpdk.org/archives/dev/> List-Post: <mailto:dev@dpdk.org> List-Help: <mailto:dev-request@dpdk.org?subject=help> List-Subscribe: <https://mails.dpdk.org/listinfo/dev>, <mailto:dev-request@dpdk.org?subject=subscribe> Errors-To: dev-bounces@dpdk.org |
Series | PCI Dev and SG copy support | |
Message
Gowrishankar Muthukrishnan
Feb. 29, 2024, 1:48 p.m. UTC
Improve dma-perf application to support PCI dev and SG copy, along with additional supports as below: - validate copied memory - skip tests if not opted. v11: - Review suggestions. Gowrishankar Muthukrishnan (4): app/dma-perf: add skip support app/dma-perf: add PCI device support app/dma-perf: validate copied memory app/dma-perf: add SG copy support app/test-dma-perf/benchmark.c | 413 ++++++++++++++++++++++++++++++---- app/test-dma-perf/config.ini | 56 +++++ app/test-dma-perf/main.c | 178 ++++++++++++--- app/test-dma-perf/main.h | 13 +- 4 files changed, 595 insertions(+), 65 deletions(-)
Comments
29/02/2024 14:48, Gowrishankar Muthukrishnan: > Improve dma-perf application to support PCI dev and SG copy, > along with additional supports as below: > - validate copied memory > - skip tests if not opted. > > v11: > - Review suggestions. > > Gowrishankar Muthukrishnan (4): > app/dma-perf: add skip support > app/dma-perf: add PCI device support > app/dma-perf: validate copied memory > app/dma-perf: add SG copy support Waiting for a confirmation that this series is good to go. Recheck-request: iol-unit-amd64-testing
Hi Thomas and Gowrishankar, On 2024/3/7 3:50, Thomas Monjalon wrote: > 29/02/2024 14:48, Gowrishankar Muthukrishnan: >> Improve dma-perf application to support PCI dev and SG copy, >> along with additional supports as below: >> - validate copied memory >> - skip tests if not opted. >> >> v11: >> - Review suggestions. >> >> Gowrishankar Muthukrishnan (4): >> app/dma-perf: add skip support >> app/dma-perf: add PCI device support >> app/dma-perf: validate copied memory >> app/dma-perf: add SG copy support > > Waiting for a confirmation that this series is good to go. In the discuss of thread [1], I hope this patchset continue take a step forward (means new version) to support bi-direction test just by modify config.ini file. [1] [PATCH v2] app/dma-perf: support bi-directional transfer Thanks > > Recheck-request: iol-unit-amd64-testing > > > . >
Hi Thomas, > 29/02/2024 14:48, Gowrishankar Muthukrishnan: > > Improve dma-perf application to support PCI dev and SG copy, along > > with additional supports as below: > > - validate copied memory > > - skip tests if not opted. > > > > v11: > > - Review suggestions. > > > > Gowrishankar Muthukrishnan (4): > > app/dma-perf: add skip support > > app/dma-perf: add PCI device support > > app/dma-perf: validate copied memory > > app/dma-perf: add SG copy support > > Waiting for a confirmation that this series is good to go. > Patches are already acked by maintainer and is ready for merge. Thanks, Gowrishankar > Recheck-request: iol-unit-amd64-testing >
Hi Fengchengwen, > > Waiting for a confirmation that this series is good to go. > > In the discuss of thread [1], I hope this patchset continue take a step forward > (means new version) to support bi-direction test just by modify config.ini file. > This patch set already exposes all configuration via config.ini. I didn't follow what is missing. For bi-direction, we can better continue discussing on that patch. Thanks, Gowrishankar > [1] [PATCH v2] app/dma-perf: support bi-directional transfer > > Thanks > > > > > Recheck-request: iol-unit-amd64-testing > > > > > > . > >
07/03/2024 14:55, Gowrishankar Muthukrishnan: > Hi Fengchengwen, > > > > Waiting for a confirmation that this series is good to go. > > > > In the discuss of thread [1], I hope this patchset continue take a step forward > > (means new version) to support bi-direction test just by modify config.ini file. > > > > This patch set already exposes all configuration via config.ini. I didn't follow what is missing. For bi-direction, we can better continue discussing on that patch. Chengwen, please can you confirm whether you require a new version? Which change exactly is missing?
Hi Thomas, On 2024/3/12 17:15, Thomas Monjalon wrote: > 07/03/2024 14:55, Gowrishankar Muthukrishnan: >> Hi Fengchengwen, >> >>>> Waiting for a confirmation that this series is good to go. >>> >>> In the discuss of thread [1], I hope this patchset continue take a step forward >>> (means new version) to support bi-direction test just by modify config.ini file. >>> >> >> This patch set already exposes all configuration via config.ini. I didn't follow what is missing. For bi-direction, we can better continue discussing on that patch. > > Chengwen, please can you confirm whether you require a new version? > Which change exactly is missing? This patchset is OK with one sub-test only tackle one DMA direction. But there is a later patch [1] which will support multiple DMA directions within one sub-test. it will add a entry "xfer_mode", but I think it complicate the test, I prefer we do more in this patchset to support some like bi-direction just by modify config.ini, some like this: 1. extend lcore_dma: current lcore_dma is: lcore10@0000:00:04.2 extend it support: lcore10@0000:00:04.2,dir=m2d,coreid=1,pfid=2,vfid=3,raddr=0xXXXX 2. to fix one entry can't hold too many dma device, support entrys: lcore_dma_1, lcore_dma_2 which value is same with lcore_dma. So for bi-direction, we just define config.ini as: lcore_dma=lcore10@0000:00:04.2,dir=m2d,coreid=1,pfid=2,vfid=3,raddr=0xXXXX, lcore10@0000:00:04.2,dir=d2m,coreid=1,pfid=2,vfid=3,raddr=0xXXXX [1] https://patches.dpdk.org/project/dpdk/patch/20240229141426.4188428-1-amitprakashs@marvell.com/ >
Hi Fengchengwen > > Hi Thomas, > > On 2024/3/12 17:15, Thomas Monjalon wrote: > > 07/03/2024 14:55, Gowrishankar Muthukrishnan: > >> Hi Fengchengwen, > >> > >>>> Waiting for a confirmation that this series is good to go. > >>> > >>> In the discuss of thread [1], I hope this patchset continue take a > >>> step forward (means new version) to support bi-direction test just by > modify config.ini file. > >>> > >> > >> This patch set already exposes all configuration via config.ini. I didn't follow > what is missing. For bi-direction, we can better continue discussing on that > patch. > > > > Chengwen, please can you confirm whether you require a new version? > > Which change exactly is missing? > > This patchset is OK with one sub-test only tackle one DMA direction. > Thanks for the confirmation. > But there is a later patch [1] which will support multiple DMA directions within > one sub-test. > it will add a entry "xfer_mode", but I think it complicate the test, I prefer we do > more in this patchset to support some like bi-direction just by modify > config.ini, some like this: > I think we should discuss about that in bi-directional patch series. This series is self-contained and there is no need to add bi-directional as part of this series. As far as this patch set is concerned, all the options are exposed via config.ini. Can you comment if there is anything missing, assuming that we are taking bi-directional support as a separate feature addition. Thanks, Gowrishankar > 1. extend lcore_dma: > current lcore_dma is: lcore10@0000:00:04.2 > extend it support: > lcore10@0000:00:04.2,dir=m2d,coreid=1,pfid=2,vfid=3,raddr=0xXXXX > 2. to fix one entry can't hold too many dma device, support entrys: > lcore_dma_1, lcore_dma_2 > which value is same with lcore_dma. > > So for bi-direction, we just define config.ini as: > lcore_dma=lcore10@0000:00:04.2,dir=m2d,coreid=1,pfid=2,vfid=3,raddr=0x > XXXX, lcore10@0000:00:04.2,dir=d2m,coreid=1,pfid=2,vfid=3,raddr=0xXXXX > > [1] https://urldefense.proofpoint.com/v2/url?u=https- > 3A__patches.dpdk.org_project_dpdk_patch_20240229141426.4188428- > 2D1-2Damitprakashs- > 40marvell.com_&d=DwICaQ&c=nKjWec2b6R0mOyPaz7xtfQ&r=EAtr- > g7yUFhtOio8r2Rtm13Aqe4WVp_S_gHpcu6KFVo&m=DUaL_AJR1zqM0T2yw3 > aV44EObB90uqw5weFzSm- > w39citSeGozNdEe4kzicss_KG&s=UTAcoZx5DjSJHyzxyLMxXz1bPqfPXQM7feDx > ZdC6Jgk&e= > > >
Hi Gowrishankar, On 2024/3/12 20:24, Gowrishankar Muthukrishnan wrote: > Hi Fengchengwen > >> >> Hi Thomas, >> >> On 2024/3/12 17:15, Thomas Monjalon wrote: >>> 07/03/2024 14:55, Gowrishankar Muthukrishnan: >>>> Hi Fengchengwen, >>>> >>>>>> Waiting for a confirmation that this series is good to go. >>>>> >>>>> In the discuss of thread [1], I hope this patchset continue take a >>>>> step forward (means new version) to support bi-direction test just by >> modify config.ini file. >>>>> >>>> >>>> This patch set already exposes all configuration via config.ini. I didn't follow >> what is missing. For bi-direction, we can better continue discussing on that >> patch. >>> >>> Chengwen, please can you confirm whether you require a new version? >>> Which change exactly is missing? >> >> This patchset is OK with one sub-test only tackle one DMA direction. >> > Thanks for the confirmation. > >> But there is a later patch [1] which will support multiple DMA directions within >> one sub-test. >> it will add a entry "xfer_mode", but I think it complicate the test, I prefer we do >> more in this patchset to support some like bi-direction just by modify >> config.ini, some like this: >> > I think we should discuss about that in bi-directional patch series. This series is self-contained and there is no need to add bi-directional as part of this series. As far as this patch set is concerned, all the options are exposed via config.ini. Can you comment if there is anything missing, assuming that we are taking bi-directional support as a separate feature addition. I have identified some improvements to the dma-perf app, and I plan to do it in 24.07, so if you don't mind, I will incorporate your commits (keeping your signed-off-by) and modify to the one that I described above, and then send to community (also with my improvements commits). Thanks > > Thanks, > Gowrishankar > >> 1. extend lcore_dma: >> current lcore_dma is: lcore10@0000:00:04.2 >> extend it support: >> lcore10@0000:00:04.2,dir=m2d,coreid=1,pfid=2,vfid=3,raddr=0xXXXX >> 2. to fix one entry can't hold too many dma device, support entrys: >> lcore_dma_1, lcore_dma_2 >> which value is same with lcore_dma. >> >> So for bi-direction, we just define config.ini as: >> lcore_dma=lcore10@0000:00:04.2,dir=m2d,coreid=1,pfid=2,vfid=3,raddr=0x >> XXXX, lcore10@0000:00:04.2,dir=d2m,coreid=1,pfid=2,vfid=3,raddr=0xXXXX >> >> [1] https://urldefense.proofpoint.com/v2/url?u=https- >> 3A__patches.dpdk.org_project_dpdk_patch_20240229141426.4188428- >> 2D1-2Damitprakashs- >> 40marvell.com_&d=DwICaQ&c=nKjWec2b6R0mOyPaz7xtfQ&r=EAtr- >> g7yUFhtOio8r2Rtm13Aqe4WVp_S_gHpcu6KFVo&m=DUaL_AJR1zqM0T2yw3 >> aV44EObB90uqw5weFzSm- >> w39citSeGozNdEe4kzicss_KG&s=UTAcoZx5DjSJHyzxyLMxXz1bPqfPXQM7feDx >> ZdC6Jgk&e= >> >>>
Hi Fengchengwen > Hi Gowrishankar, > > On 2024/3/12 20:24, Gowrishankar Muthukrishnan wrote: > > Hi Fengchengwen > > > >> > >> Hi Thomas, > >> > >> On 2024/3/12 17:15, Thomas Monjalon wrote: > >>> 07/03/2024 14:55, Gowrishankar Muthukrishnan: > >>>> Hi Fengchengwen, > >>>> > >>>>>> Waiting for a confirmation that this series is good to go. > >>>>> > >>>>> In the discuss of thread [1], I hope this patchset continue take a > >>>>> step forward (means new version) to support bi-direction test just > >>>>> by > >> modify config.ini file. > >>>>> > >>>> > >>>> This patch set already exposes all configuration via config.ini. I > >>>> didn't follow > >> what is missing. For bi-direction, we can better continue discussing > >> on that patch. > >>> > >>> Chengwen, please can you confirm whether you require a new version? > >>> Which change exactly is missing? > >> > >> This patchset is OK with one sub-test only tackle one DMA direction. > >> > > Thanks for the confirmation. > > > >> But there is a later patch [1] which will support multiple DMA > >> directions within one sub-test. > >> it will add a entry "xfer_mode", but I think it complicate the test, > >> I prefer we do more in this patchset to support some like > >> bi-direction just by modify config.ini, some like this: > >> > > I think we should discuss about that in bi-directional patch series. This series > is self-contained and there is no need to add bi-directional as part of this > series. As far as this patch set is concerned, all the options are exposed via > config.ini. Can you comment if there is anything missing, assuming that we are > taking bi-directional support as a separate feature addition. > > I have identified some improvements to the dma-perf app, and I plan to do it It is unclear at this point what is the issue that you have with the app or this patch set. This series was first submitted on Aug 10 2023. You had acked v8 on Jan 25 2024. After the patches were acked, there were still review comments on variable renames etc, which were all addressed. The patches had been under review for more than 8 months with very slow progress. > in 24.07, so if you don't mind, I will incorporate your commits (keeping your > signed-off-by) and modify to the one that I described above, and then send to > community (also with my improvements commits). I would like to have this series merged first and not pulled into another series. We do have few other features that we would like to add on top. I would assume that you can also add your changes on top. To make contribution easier, isn't it better to accept at least this patch set (as you acked earlier) and then you can continue working on the improvements? Thanks, Gowrishankar > > Thanks > > > > > Thanks, > > Gowrishankar > > > >> 1. extend lcore_dma: > >> current lcore_dma is: lcore10@0000:00:04.2 > >> extend it support: > >> lcore10@0000:00:04.2,dir=m2d,coreid=1,pfid=2,vfid=3,raddr=0xXXXX > >> 2. to fix one entry can't hold too many dma device, support entrys: > >> lcore_dma_1, lcore_dma_2 > >> which value is same with lcore_dma. > >> > >> So for bi-direction, we just define config.ini as: > >> > lcore_dma=lcore10@0000:00:04.2,dir=m2d,coreid=1,pfid=2,vfid=3,raddr=0 > >> x XXXX, > >> lcore10@0000:00:04.2,dir=d2m,coreid=1,pfid=2,vfid=3,raddr=0xXXXX > >> > >> [1] https://urldefense.proofpoint.com/v2/url?u=https- > >> 3A__patches.dpdk.org_project_dpdk_patch_20240229141426.4188428- > >> 2D1-2Damitprakashs- > >> 40marvell.com_&d=DwICaQ&c=nKjWec2b6R0mOyPaz7xtfQ&r=EAtr- > >> > g7yUFhtOio8r2Rtm13Aqe4WVp_S_gHpcu6KFVo&m=DUaL_AJR1zqM0T2yw3 > >> aV44EObB90uqw5weFzSm- > >> > w39citSeGozNdEe4kzicss_KG&s=UTAcoZx5DjSJHyzxyLMxXz1bPqfPXQM7feDx > >> ZdC6Jgk&e= > >> > >>>
Hi Thomas, > > Hi Fengchengwen > > > Hi Gowrishankar, > > > > On 2024/3/12 20:24, Gowrishankar Muthukrishnan wrote: > > > Hi Fengchengwen > > > > > >> > > >> Hi Thomas, > > >> > > >> On 2024/3/12 17:15, Thomas Monjalon wrote: > > >>> 07/03/2024 14:55, Gowrishankar Muthukrishnan: > > >>>> Hi Fengchengwen, > > >>>> > > >>>>>> Waiting for a confirmation that this series is good to go. > > >>>>> > > >>>>> In the discuss of thread [1], I hope this patchset continue take > > >>>>> a step forward (means new version) to support bi-direction test > > >>>>> just by > > >> modify config.ini file. > > >>>>> > > >>>> > > >>>> This patch set already exposes all configuration via config.ini. > > >>>> I didn't follow > > >> what is missing. For bi-direction, we can better continue > > >> discussing on that patch. > > >>> > > >>> Chengwen, please can you confirm whether you require a new version? > > >>> Which change exactly is missing? > > >> > > >> This patchset is OK with one sub-test only tackle one DMA direction. > > >> > > > Thanks for the confirmation. > > > > > >> But there is a later patch [1] which will support multiple DMA > > >> directions within one sub-test. > > >> it will add a entry "xfer_mode", but I think it complicate the > > >> test, I prefer we do more in this patchset to support some like > > >> bi-direction just by modify config.ini, some like this: > > >> > > > I think we should discuss about that in bi-directional patch series. > > > This series > > is self-contained and there is no need to add bi-directional as part > > of this series. As far as this patch set is concerned, all the options > > are exposed via config.ini. Can you comment if there is anything > > missing, assuming that we are taking bi-directional support as a separate > feature addition. > > > > I have identified some improvements to the dma-perf app, and I plan to > > do it > > It is unclear at this point what is the issue that you have with the app or this > patch set. This series was first submitted on Aug 10 2023. You had acked v8 on > Jan 25 2024. After the patches were acked, there were still review comments > on variable renames etc, which were all addressed. The patches had been > under review for more than 8 months with very slow progress. > > > in 24.07, so if you don't mind, I will incorporate your commits > > (keeping your > > signed-off-by) and modify to the one that I described above, and then > > send to community (also with my improvements commits). > > I would like to have this series merged first and not pulled into another series. > We do have few other features that we would like to add on top. I would > assume that you can also add your changes on top. To make contribution > easier, isn't it better to accept at least this patch set (as you acked earlier) and > then you can continue working on the improvements? > Can this series be merged ? Thanks, Gowrishankar > Thanks, > Gowrishankar >
13/03/2024 09:22, Gowrishankar Muthukrishnan: > Hi Fengchengwen > > > Hi Gowrishankar, > > > > On 2024/3/12 20:24, Gowrishankar Muthukrishnan wrote: > > > Hi Fengchengwen > > > > > >> > > >> Hi Thomas, > > >> > > >> On 2024/3/12 17:15, Thomas Monjalon wrote: > > >>> 07/03/2024 14:55, Gowrishankar Muthukrishnan: > > >>>> Hi Fengchengwen, > > >>>> > > >>>>>> Waiting for a confirmation that this series is good to go. > > >>>>> > > >>>>> In the discuss of thread [1], I hope this patchset continue take a > > >>>>> step forward (means new version) to support bi-direction test just > > >>>>> by > > >> modify config.ini file. > > >>>>> > > >>>> > > >>>> This patch set already exposes all configuration via config.ini. I > > >>>> didn't follow > > >> what is missing. For bi-direction, we can better continue discussing > > >> on that patch. > > >>> > > >>> Chengwen, please can you confirm whether you require a new version? > > >>> Which change exactly is missing? > > >> > > >> This patchset is OK with one sub-test only tackle one DMA direction. > > >> > > > Thanks for the confirmation. > > > > > >> But there is a later patch [1] which will support multiple DMA > > >> directions within one sub-test. > > >> it will add a entry "xfer_mode", but I think it complicate the test, > > >> I prefer we do more in this patchset to support some like > > >> bi-direction just by modify config.ini, some like this: > > >> > > > I think we should discuss about that in bi-directional patch series. This series > > is self-contained and there is no need to add bi-directional as part of this > > series. As far as this patch set is concerned, all the options are exposed via > > config.ini. Can you comment if there is anything missing, assuming that we are > > taking bi-directional support as a separate feature addition. > > > > I have identified some improvements to the dma-perf app, and I plan to do it > > It is unclear at this point what is the issue that you have with the app or this patch set. This series was first submitted on Aug 10 2023. You had acked v8 on Jan 25 2024. After the patches were acked, there were still review comments on variable renames etc, which were all addressed. The patches had been under review for more than 8 months with very slow progress. > > > in 24.07, so if you don't mind, I will incorporate your commits (keeping your > > signed-off-by) and modify to the one that I described above, and then send to > > community (also with my improvements commits). > > I would like to have this series merged first and not pulled into another series. We do have few other features that we would like to add on top. I would assume that you can also add your changes on top. To make contribution easier, isn't it better to accept at least this patch set (as you acked earlier) and then you can continue working on the improvements? OK, one feature at a time. Let's work on top of this patchset applied.
Hi Thomas and Fengchengwen, > > 13/03/2024 09:22, Gowrishankar Muthukrishnan: > > Hi Fengchengwen > > > > > Hi Gowrishankar, > > > > > > On 2024/3/12 20:24, Gowrishankar Muthukrishnan wrote: > > > > Hi Fengchengwen > > > > > > > >> > > > >> Hi Thomas, > > > >> > > > >> On 2024/3/12 17:15, Thomas Monjalon wrote: > > > >>> 07/03/2024 14:55, Gowrishankar Muthukrishnan: > > > >>>> Hi Fengchengwen, > > > >>>> > > > >>>>>> Waiting for a confirmation that this series is good to go. > > > >>>>> > > > >>>>> In the discuss of thread [1], I hope this patchset continue > > > >>>>> take a step forward (means new version) to support > > > >>>>> bi-direction test just by > > > >> modify config.ini file. > > > >>>>> > > > >>>> > > > >>>> This patch set already exposes all configuration via > > > >>>> config.ini. I didn't follow > > > >> what is missing. For bi-direction, we can better continue > > > >> discussing on that patch. > > > >>> > > > >>> Chengwen, please can you confirm whether you require a new > version? > > > >>> Which change exactly is missing? > > > >> > > > >> This patchset is OK with one sub-test only tackle one DMA direction. > > > >> > > > > Thanks for the confirmation. > > > > > > > >> But there is a later patch [1] which will support multiple DMA > > > >> directions within one sub-test. > > > >> it will add a entry "xfer_mode", but I think it complicate the > > > >> test, I prefer we do more in this patchset to support some like > > > >> bi-direction just by modify config.ini, some like this: > > > >> > > > > I think we should discuss about that in bi-directional patch > > > > series. This series > > > is self-contained and there is no need to add bi-directional as part > > > of this series. As far as this patch set is concerned, all the > > > options are exposed via config.ini. Can you comment if there is > > > anything missing, assuming that we are taking bi-directional support as a > separate feature addition. > > > > > > I have identified some improvements to the dma-perf app, and I plan > > > to do it > > > > It is unclear at this point what is the issue that you have with the app or this > patch set. This series was first submitted on Aug 10 2023. You had acked v8 on > Jan 25 2024. After the patches were acked, there were still review comments > on variable renames etc, which were all addressed. The patches had been > under review for more than 8 months with very slow progress. > > > > > in 24.07, so if you don't mind, I will incorporate your commits > > > (keeping your > > > signed-off-by) and modify to the one that I described above, and > > > then send to community (also with my improvements commits). > > > > I would like to have this series merged first and not pulled into another > series. We do have few other features that we would like to add on top. I > would assume that you can also add your changes on top. To make > contribution easier, isn't it better to accept at least this patch set (as you acked > earlier) and then you can continue working on the improvements? > > OK, one feature at a time. > Let's work on top of this patchset applied. > Thank you both for reviewing this series and accepting it in RC3. Regards, Gowrishankar