[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <d5933562-b6d7-472f-97a7-3d72da3ff51f@davidwei.uk>
Date: Tue, 23 Sep 2025 09:08:03 -0700
From: David Wei <dw@...idwei.uk>
To: Jakub Kicinski <kuba@...nel.org>, Daniel Borkmann <daniel@...earbox.net>
Cc: netdev@...r.kernel.org, bpf@...r.kernel.org, davem@...emloft.net,
razor@...ckwall.org, pabeni@...hat.com, willemb@...gle.com, sdf@...ichev.me,
john.fastabend@...il.com, martin.lau@...nel.org, jordan@...fe.io,
maciej.fijalkowski@...el.com, magnus.karlsson@...el.com
Subject: Re: [PATCH net-next 06/20] net, ynl: Add peer info to queue-get
response
On 2025-09-22 18:32, Jakub Kicinski wrote:
> On Fri, 19 Sep 2025 23:31:39 +0200 Daniel Borkmann wrote:
>> + name: peer-info
>> + attributes:
>> + -
>> + name: id
>> + doc: Queue index of the netdevice to which the peer queue belongs.
>> + type: u32
>> + -
>> + name: ifindex
>> + doc: ifindex of the netdevice to which the peer queue belongs.
>> + type: u32
>
> Oh, we have an ifindex in the local netns. So the API is to bind a
> queue to one side of a netkit and then the other side of the netkit
> actually gets to use it? Should we not be "binding" to the device that
> is of interest rather than its peer?
We are binding from a netkit queue to a physical netdev queue of
interest.
Sorry, the terminology in this patchset is not consistent and confusing
clearly. Will address in v2.
Powered by blists - more mailing lists