From: Maryam Tahhan <mtahhan@redhat.com>
To: Ferruh Yigit <ferruh.yigit@amd.com>,
stephen@networkplumber.org, lihuisong@huawei.com,
fengchengwen@huawei.com, liuyonglong@huawei.com,
david.marchand@redhat.com
Cc: dev@dpdk.org, Ciara Loftus <ciara.loftus@intel.com>,
Shibin Koikkara Reeny <shibin.koikkara.reeny@intel.com>
Subject: Re: [v7 1/1] net/af_xdp: fix multi interface support for K8s
Date: Thu, 11 Jan 2024 09:01:19 +0000 [thread overview]
Message-ID: <a9e88269-0642-4945-a5dd-e65076a7f82c@redhat.com> (raw)
In-Reply-To: <6cd6ab2d-56ab-4afe-b0cd-05ab6b017469@amd.com>
On 10/01/2024 15:21, Ferruh Yigit wrote:
> On 1/10/2024 2:58 PM, Maryam Tahhan wrote:
>> Hi folks
>>
>> Just wondering if there's any other comments re this patch after all the
>> review comments were addressed?
>>
> Hi Maryam,
>
> There was a request from David, to the previous version, that fixes
> (either to code or document) should be separated into its own patch, so
> that they can be merged to our LTS (say 21.11.x) releases, to make sure
> LTS release has correct info/code. On top of it your changes can be
> another patch.
>
> As document completely changed, it is not easy for me to say, can you
> please confirm that there is no fix to the documentation unrelated with
> your code change?
>
Hey Ferruh
Yes - I should've clarified. I took David's comments into consideration.
The old use_cni implementation is broken (so if a DPDK pod wants
multiple AF_XDP interfaces it will not work). So IMHO it doesn't make
sense to patch documentation for something that is broken. The whole
change would need to be back ported to LTS.
However, I have a second change to apply on top of this - so I will send
a series. So please hold off merging for now.
BR
Maryam
next prev parent reply other threads:[~2024-01-11 9:01 UTC|newest]
Thread overview: 13+ messages / expand[flat|nested] mbox.gz Atom feed top
2023-12-22 11:04 Maryam Tahhan
2024-01-10 14:58 ` Maryam Tahhan
2024-01-10 15:21 ` Ferruh Yigit
2024-01-11 9:01 ` Maryam Tahhan [this message]
2024-01-11 11:35 ` Ferruh Yigit
2024-01-11 12:21 ` Maryam Tahhan
2024-01-11 13:28 ` Kevin Traynor
2024-01-11 14:21 ` Ferruh Yigit
2024-02-07 23:24 ` Ferruh Yigit
2024-02-09 12:40 ` Loftus, Ciara
2024-02-09 16:52 ` Maryam Tahhan
2024-02-09 18:07 ` Ferruh Yigit
2024-02-09 17:02 ` Maryam Tahhan
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=a9e88269-0642-4945-a5dd-e65076a7f82c@redhat.com \
--to=mtahhan@redhat.com \
--cc=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=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).