From mboxrd@z Thu Jan  1 00:00:00 1970
From: Ferruh Yigit <ferruh.yigit@intel.com>
Subject: Re: [PATCH v3 32/34] net/ice: support descriptor ops
Date: Thu, 13 Dec 2018 21:30:16 +0000
Message-ID: <1bafc5c8-8cac-e1ff-b9d6-a0233932489a@intel.com>
References: <1542956179-80951-1-git-send-email-wenzhuo.lu@intel.com>
 <1544598004-27099-1-git-send-email-wenzhuo.lu@intel.com>
 <1544598004-27099-33-git-send-email-wenzhuo.lu@intel.com>
Mime-Version: 1.0
Content-Type: text/plain; charset=utf-8
Content-Transfer-Encoding: 7bit
Cc: Qiming Yang <qiming.yang@intel.com>, Xiaoyun Li <xiaoyun.li@intel.com>,
 Jingjing Wu <jingjing.wu@intel.com>, Olivier MATZ <olivier.matz@6wind.com>
To: Wenzhuo Lu <wenzhuo.lu@intel.com>, dev@dpdk.org
Return-path: <dev-bounces@dpdk.org>
Received: from mga03.intel.com (mga03.intel.com [134.134.136.65])
 by dpdk.org (Postfix) with ESMTP id E0DF05B30
 for <dev@dpdk.org>; Thu, 13 Dec 2018 22:30:19 +0100 (CET)
In-Reply-To: <1544598004-27099-33-git-send-email-wenzhuo.lu@intel.com>
Content-Language: en-US
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
Sender: "dev" <dev-bounces@dpdk.org>

On 12/12/2018 7:00 AM, Wenzhuo Lu wrote:
> Add below ops,
> rx_descriptor_done
> rx_descriptor_status
> tx_descriptor_status

I guess this is our mistake to not clarify this, sorry about it, but
"rx_descriptor_status" replaces "rx_descriptor_done", it is and extended
version, cc'ed Olivier to correct me in case I am wrong.

So when "rx_descriptor_status" implemented, "rx_descriptor_done" can be dropped.

Please see commit log of:
Commit b1b700ce7d6f ("ethdev: add descriptor status API")

copy-paste related part:
"
    The descriptor_done() API, and probably the rx_queue_count() API could
    be replaced by this new API as soon as it is implemented on all PMDs.
"