From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mails.dpdk.org (mails.dpdk.org [217.70.189.124]) by inbox.dpdk.org (Postfix) with ESMTP id 8A45DA0C4A; Fri, 16 Jul 2021 05:04:41 +0200 (CEST) Received: from [217.70.189.124] (localhost [127.0.0.1]) by mails.dpdk.org (Postfix) with ESMTP id 067194014D; Fri, 16 Jul 2021 05:04:41 +0200 (CEST) Received: from szxga01-in.huawei.com (szxga01-in.huawei.com [45.249.212.187]) by mails.dpdk.org (Postfix) with ESMTP id A99CA40143 for ; Fri, 16 Jul 2021 05:04:39 +0200 (CEST) Received: from dggemv703-chm.china.huawei.com (unknown [172.30.72.56]) by szxga01-in.huawei.com (SkyGuard) with ESMTP id 4GQwx46GhFzZqP7; Fri, 16 Jul 2021 11:01:12 +0800 (CST) Received: from dggpeml500024.china.huawei.com (7.185.36.10) by dggemv703-chm.china.huawei.com (10.3.19.46) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_128_GCM_SHA256) id 15.1.2176.2; Fri, 16 Jul 2021 11:04:31 +0800 Received: from [10.40.190.165] (10.40.190.165) by dggpeml500024.china.huawei.com (7.185.36.10) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_128_GCM_SHA256) id 15.1.2176.2; Fri, 16 Jul 2021 11:04:31 +0800 To: Bruce Richardson CC: , , , , , , , , , , , , , , References: <1625231891-2963-1-git-send-email-fengchengwen@huawei.com> <1626363661-51103-1-git-send-email-fengchengwen@huawei.com> From: fengchengwen Message-ID: <0aa82990-566b-f471-129b-31fbc2410ecc@huawei.com> Date: Fri, 16 Jul 2021 11:04:30 +0800 User-Agent: Mozilla/5.0 (Windows NT 10.0; WOW64; rv:68.0) Gecko/20100101 Thunderbird/68.11.0 MIME-Version: 1.0 In-Reply-To: Content-Type: text/plain; charset="utf-8" Content-Language: en-US Content-Transfer-Encoding: 7bit X-Originating-IP: [10.40.190.165] X-ClientProxiedBy: dggems702-chm.china.huawei.com (10.3.19.179) To dggpeml500024.china.huawei.com (7.185.36.10) X-CFilter-Loop: Reflected Subject: Re: [dpdk-dev] [PATCH v4] dmadev: introduce DMA device library X-BeenThere: dev@dpdk.org X-Mailman-Version: 2.1.29 Precedence: list List-Id: DPDK patches and discussions List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: dev-bounces@dpdk.org Sender: "dev" On 2021/7/16 0:33, Bruce Richardson wrote: > On Fri, Jul 16, 2021 at 12:04:33AM +0800, fengchengwen wrote: >> @burce, jerin Some unmodified review comments are returned here: >> [snip] > >> 2. COMMENT: > + * @see struct rte_dmadev_info::dev_capa >>> + */ >> Drop this flag as unnecessary. All devices either always provide ordering >> guarantee - in which case it's a no-op - or else support the flag. >> >> REPLY: I prefer define it, it could let user know whether support fence. >> > I don't see it that way. The flag is pointless because the application > can't use it to make any decisions. If two operations require ordering the > application must use the fence flag because if the device doesn't guarantee > ordering it's necessary, and if the device does guarantee ordering it's > better and easier to just specify the flag than to put in code branches. > Having this as a capability is just going to confuse the user - better to > just say that if you need ordering, put in a fence. > If driver don't support fence, and application set the fence flag, What's driving behavior like? return error or implement fence at driver layer ? If expose the fence capability to application, then application could decide which option to use. e.g. commit the operations before the fence and make sure it completes, or use another collaborative approach. I think in this manner, the driver implementation can be simplified. [snip] >> 4. >> COMMENT: > + * @see RTE_DMA_DEV_TO_MEM >>> + * @see RTE_DMA_DEV_TO_DEV >> Since we can set of only one direction per vchan . Should be we make >> it as enum to >> make it clear. >> >> REPLY: May some devices support it. I think it's OK for future use. >> > +1 > That may need a capability flag though, to indicate if a device supports > multi-direction in a single vchannel. There are a lot of combinations, and I tend not to add a capability for multi-direction. Currently, no device supports multiple directions, So can we delay that definition? [snip] thanks