DPDK patches and discussions
 help / color / mirror / Atom feed
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


  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).