From: "Loftus, Ciara" <ciara.loftus@intel.com>
To: "Tahhan, Maryam" <mtahhan@redhat.com>,
"ferruh.yigit@amd.com" <ferruh.yigit@amd.com>,
"stephen@networkplumber.org" <stephen@networkplumber.org>,
"lihuisong@huawei.com" <lihuisong@huawei.com>,
"fengchengwen@huawei.com" <fengchengwen@huawei.com>,
"liuyonglong@huawei.com" <liuyonglong@huawei.com>,
"Marchand, David" <david.marchand@redhat.com>,
"Koikkara Reeny, Shibin" <shibin.koikkara.reeny@intel.com>
Cc: "dev@dpdk.org" <dev@dpdk.org>, "Tahhan, Maryam" <mtahhan@redhat.com>
Subject: RE: [v14 0/3] net/af_xdp: fix multi interface support for K8s
Date: Mon, 8 Apr 2024 13:19:31 +0000 [thread overview]
Message-ID: <MW4PR11MB58722C71B7D5704A045649B48E002@MW4PR11MB5872.namprd11.prod.outlook.com> (raw)
In-Reply-To: <20240408130924.232154-1-mtahhan@redhat.com>
>
> The original `use_cni` implementation was limited to
> supporting only a single netdev in a DPDK pod. This patchset
> aims to fix this limitation transparently to the end user.
> It will also enable compatibility with the latest AF_XDP
> Device Plugin.
>
> Signed-off-by: Maryam Tahhan <mtahhan@redhat.com>
Thanks Maryam.
For the series,
Acked-by: Ciara Loftus <ciara.loftus@intel.com>
> ---
> v14:
> * Fixup bpf_map_update_elem() in compat.h to use xsk fd as the
> third argument.
>
> v13:
> * Fixup checkpatch issues.
>
> v12:
> * Ensure backwards compability with libbpf versions that don't support
> xsk_socket__update_xskmap().
>
> v11:
> * Fixed up typos picked up by checkpatch.
>
> v10:
> * Add UDS acronym
> * Update `use_cni` in docs with ``use_cni``
> * Remove reference to limitations and simply document behaviour
> before and after DPDK 23.11.
>
> v9:
> * Fixup checkpatch issues.
>
> v8:
> * Go back to using `use_cni` vdev argument
> * Introduce `use_map_pinning` vdev param.
> * Rename `uds_path` to `dp_path` so that it can be used
> with map pinning as well as `use_cni`.
> * Set `dp_path` internally in the AF_XDP PMD if it's
> not configured by the user.
> * Clean up the original `use_cni` documentation separately
> to coding changes.
>
> v7:
> * Give a more descriptive commit msg headline.
> * Fixup typos in documentation.
>
> v6:
> * Add link to PR 81 in commit message
> * Add release notes changes to this patchset
>
> v5:
> * Fix alignment for ETH_AF_XDP_USE_DP_UDS_PATH_ARG
> * Remove use_cni references in af_xdp.rst
>
> v4:
> * Rename af_xdp_cni.rst to af_xdp_dp.rst
> * Removed all incorrect references to CNI throughout af_xdp
> PMD file.
> * Fixed Typos in af_xdp_dp.rst
>
> v3:
> * Remove `use_cni` vdev argument as it's no longer needed.
> * Update incorrect CNI references for the AF_XDP DP in the
> documentation.
> * Update the documentation to run a simple example with the
> AF_XDP DP plugin in K8s.
>
> v2:
> * Rename sock_path to uds_path.
> * Update documentation to reflect when CAP_BPF is needed.
> * Fix testpmd arguments in the provided example for Pods.
> * Use AF_XDP API to update the xskmap entry.
> ---
>
> Maryam Tahhan (3):
> docs: AF_XDP Device Plugin
> net/af_xdp: fix multi interface support for K8s
> net/af_xdp: support AF_XDP DP pinned maps
>
> doc/guides/howto/af_xdp_cni.rst | 253 ------------------
> doc/guides/howto/af_xdp_dp.rst | 340 +++++++++++++++++++++++++
> doc/guides/howto/index.rst | 2 +-
> doc/guides/nics/af_xdp.rst | 44 +++-
> doc/guides/rel_notes/release_24_07.rst | 17 ++
> drivers/net/af_xdp/compat.h | 15 ++
> drivers/net/af_xdp/meson.build | 4 +
> drivers/net/af_xdp/rte_eth_af_xdp.c | 170 +++++++++----
> 8 files changed, 543 insertions(+), 302 deletions(-)
> delete mode 100644 doc/guides/howto/af_xdp_cni.rst
> create mode 100644 doc/guides/howto/af_xdp_dp.rst
>
> --
> 2.41.0
next prev parent reply other threads:[~2024-04-08 13:19 UTC|newest]
Thread overview: 10+ messages / expand[flat|nested] mbox.gz Atom feed top
2024-04-08 13:09 Maryam Tahhan
2024-04-08 13:09 ` [v14 1/3] docs: AF_XDP Device Plugin Maryam Tahhan
2024-04-19 10:13 ` Ferruh Yigit
2024-04-22 14:53 ` Maryam Tahhan
2024-04-22 16:14 ` Ferruh Yigit
2024-04-08 13:09 ` [v14 2/3] net/af_xdp: fix multi interface support for K8s Maryam Tahhan
2024-04-08 13:09 ` [v14 3/3] net/af_xdp: support AF_XDP DP pinned maps Maryam Tahhan
2024-04-08 13:19 ` Loftus, Ciara [this message]
2024-04-19 11:00 ` [v14 0/3] net/af_xdp: fix multi interface support for K8s Ferruh Yigit
2024-04-18 21:52 ` 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=MW4PR11MB58722C71B7D5704A045649B48E002@MW4PR11MB5872.namprd11.prod.outlook.com \
--to=ciara.loftus@intel.com \
--cc=david.marchand@redhat.com \
--cc=dev@dpdk.org \
--cc=fengchengwen@huawei.com \
--cc=ferruh.yigit@amd.com \
--cc=lihuisong@huawei.com \
--cc=liuyonglong@huawei.com \
--cc=mtahhan@redhat.com \
--cc=shibin.koikkara.reeny@intel.com \
--cc=stephen@networkplumber.org \
/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).