From: Ferruh Yigit <ferruh.yigit@intel.com>
To: David Marchand <david.marchand@redhat.com>,
Xiaolong Ye <xiaolong.ye@intel.com>
Cc: dev <dev@dpdk.org>, Qi Zhang <qi.z.zhang@intel.com>,
Karlsson Magnus <magnus.karlsson@intel.com>,
Topel Bjorn <bjorn.topel@intel.com>
Subject: Re: [dpdk-dev] [PATCH v4 0/4] some fixes for AF_XDP pmd
Date: Wed, 17 Apr 2019 17:38:47 +0100 [thread overview]
Message-ID: <d8423095-8a54-df2e-e2d5-b4e16691cb8b@intel.com> (raw)
Message-ID: <20190417163847.BUZ9SYAkxckoy5EVrT9181QWS_dfETAOtRlxMoRwD-c@z> (raw)
In-Reply-To: <CAJFAV8yYK0QeTOwzuoju45ukTzMikkdz8FVZBA8X_3-3NpKN0w@mail.gmail.com>
On 4/17/2019 3:02 PM, David Marchand wrote:
> On Wed, Apr 17, 2019 at 3:55 PM Xiaolong Ye <xiaolong.ye@intel.com
> <mailto:xiaolong.ye@intel.com>> wrote:
>
> This patchset provides some fixes to af_xdp pmd, at first, I just added
> a simple error handling when Tx queue allocation fails, then David
> suggested a better way to do it and pointed out the inconsistent issue
> of reserve/submit ops (for Tx queue) and peek/release ops (for Rx
> queue), the third patch addressed this.
>
> v4 changes:
>
> - remove unnecessary mtu valid check in eth_dev_mtu_set
> - add Reported-by and Reviewed-by tags of David
>
> v3 changes:
>
> - address David's review comments
> - add one patch to specify the mtu range
> - add one fix patch for typo
>
> v2 changes:
>
> - adopt David's suggestion to refactor the code
>
> Xiaolong Ye (4):
> net/af_xdp: enqueue buf ring when allocate Tx queue fails
> net/af_xdp: specify minimal and maximal MTU
> net/af_xdp: make reserve/submit peek/release consistent
> net/af_xdp: fix typos in Rx function
>
> drivers/net/af_xdp/rte_eth_af_xdp.c | 87 +++++++++++++++--------------
> 1 file changed, 46 insertions(+), 41 deletions(-)
>
> --
> 2.17.1
>
>
> LGTM.
> Thanks Xiaolong.
Series applied to dpdk-next-net/master, thanks.
next prev parent reply other threads:[~2019-04-17 16:38 UTC|newest]
Thread overview: 82+ messages / expand[flat|nested] mbox.gz Atom feed top
2019-04-09 8:21 [dpdk-dev] [PATCH] net/af_xdp: free mbuf when allocate Tx queue fails Xiaolong Ye
2019-04-09 8:21 ` Xiaolong Ye
2019-04-09 8:34 ` David Marchand
2019-04-09 8:34 ` David Marchand
2019-04-09 14:48 ` Ye Xiaolong
2019-04-09 14:48 ` Ye Xiaolong
2019-04-09 15:19 ` [dpdk-dev] [PATCH] net/af_xdp: enqueue buf ring " Xiaolong Ye
2019-04-09 15:19 ` Xiaolong Ye
2019-04-10 8:23 ` David Marchand
2019-04-10 8:23 ` David Marchand
2019-04-10 10:22 ` Ye Xiaolong
2019-04-10 10:22 ` Ye Xiaolong
2019-04-10 10:53 ` [dpdk-dev] [PATCH] net/af_xdp: submit valid count to Tx queue Xiaolong Ye
2019-04-10 10:53 ` Xiaolong Ye
2019-04-10 11:30 ` David Marchand
2019-04-10 11:30 ` David Marchand
2019-04-11 2:24 ` Ye Xiaolong
2019-04-11 2:24 ` Ye Xiaolong
2019-04-11 7:20 ` David Marchand
2019-04-11 7:20 ` David Marchand
2019-04-11 7:27 ` Ye Xiaolong
2019-04-11 7:27 ` Ye Xiaolong
2019-04-12 14:48 ` [dpdk-dev] [PATCH 1/2] net/af_xdp: enqueue buf ring when allocate Tx queue fails Xiaolong Ye
2019-04-12 14:48 ` Xiaolong Ye
2019-04-12 14:48 ` [dpdk-dev] [PATCH 2/2] net/af_xdp: make reserve/submit peek/release consistent Xiaolong Ye
2019-04-12 14:48 ` Xiaolong Ye
2019-04-15 8:19 ` David Marchand
2019-04-15 8:19 ` David Marchand
2019-04-15 14:42 ` Ye Xiaolong
2019-04-15 14:42 ` Ye Xiaolong
2019-04-16 15:03 ` [dpdk-dev] [PATCH v2 0/2] some fixes Xiaolong Ye
2019-04-16 15:03 ` Xiaolong Ye
2019-04-16 15:03 ` [dpdk-dev] [PATCH v2 1/2] net/af_xdp: enqueue buf ring when allocate Tx queue fails Xiaolong Ye
2019-04-16 15:03 ` Xiaolong Ye
2019-04-17 7:45 ` David Marchand
2019-04-17 7:45 ` David Marchand
2019-04-16 15:03 ` [dpdk-dev] [PATCH v2 2/2] net/af_xdp: make reserve/submit peek/release consistent Xiaolong Ye
2019-04-16 15:03 ` Xiaolong Ye
2019-04-17 7:45 ` David Marchand
2019-04-17 7:45 ` David Marchand
2019-04-17 7:53 ` Ye Xiaolong
2019-04-17 7:53 ` Ye Xiaolong
2019-04-17 8:56 ` [dpdk-dev] [PATCH v3 0/4] some fixes for AF_XDP pmd Xiaolong Ye
2019-04-17 8:56 ` Xiaolong Ye
2019-04-17 8:56 ` [dpdk-dev] [PATCH v3 1/4] net/af_xdp: enqueue buf ring when allocate Tx queue fails Xiaolong Ye
2019-04-17 8:56 ` Xiaolong Ye
2019-04-17 9:15 ` David Marchand
2019-04-17 9:15 ` David Marchand
2019-04-17 13:26 ` Ye Xiaolong
2019-04-17 13:26 ` Ye Xiaolong
2019-04-17 8:56 ` [dpdk-dev] [PATCH v3 2/4] net/af_xdp: specify minimal and maximal MTU Xiaolong Ye
2019-04-17 8:56 ` Xiaolong Ye
2019-04-17 9:38 ` David Marchand
2019-04-17 9:38 ` David Marchand
2019-04-17 13:25 ` Ye Xiaolong
2019-04-17 13:25 ` Ye Xiaolong
2019-04-17 8:56 ` [dpdk-dev] [PATCH v3 3/4] net/af_xdp: make reserve/submit peek/release consistent Xiaolong Ye
2019-04-17 8:56 ` Xiaolong Ye
2019-04-17 9:25 ` David Marchand
2019-04-17 9:25 ` David Marchand
2019-04-17 8:56 ` [dpdk-dev] [PATCH v3 4/4] net/af_xdp: fix typos in Rx function Xiaolong Ye
2019-04-17 8:56 ` Xiaolong Ye
2019-04-17 9:25 ` David Marchand
2019-04-17 9:25 ` David Marchand
2019-04-17 13:49 ` [dpdk-dev] [PATCH v4 0/4] some fixes for AF_XDP pmd Xiaolong Ye
2019-04-17 13:49 ` Xiaolong Ye
2019-04-17 13:49 ` [dpdk-dev] [PATCH v4 1/4] net/af_xdp: enqueue buf ring when allocate Tx queue fails Xiaolong Ye
2019-04-17 13:49 ` Xiaolong Ye
2019-04-17 13:49 ` [dpdk-dev] [PATCH v4 2/4] net/af_xdp: specify minimal and maximal MTU Xiaolong Ye
2019-04-17 13:49 ` Xiaolong Ye
2019-04-17 13:49 ` [dpdk-dev] [PATCH v4 3/4] net/af_xdp: make reserve/submit peek/release consistent Xiaolong Ye
2019-04-17 13:49 ` Xiaolong Ye
2019-04-17 13:49 ` [dpdk-dev] [PATCH v4 4/4] net/af_xdp: fix typos in Rx function Xiaolong Ye
2019-04-17 13:49 ` Xiaolong Ye
2019-04-17 15:31 ` Rami Rosen
2019-04-17 15:31 ` Rami Rosen
2019-04-17 14:02 ` [dpdk-dev] [PATCH v4 0/4] some fixes for AF_XDP pmd David Marchand
2019-04-17 14:02 ` David Marchand
2019-04-17 15:27 ` Ye Xiaolong
2019-04-17 15:27 ` Ye Xiaolong
2019-04-17 16:38 ` Ferruh Yigit [this message]
2019-04-17 16:38 ` Ferruh Yigit
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=d8423095-8a54-df2e-e2d5-b4e16691cb8b@intel.com \
--to=ferruh.yigit@intel.com \
--cc=bjorn.topel@intel.com \
--cc=david.marchand@redhat.com \
--cc=dev@dpdk.org \
--cc=magnus.karlsson@intel.com \
--cc=qi.z.zhang@intel.com \
--cc=xiaolong.ye@intel.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).