Re: [PATCH RFC 0/4] Add XDP RX queue index metadata via kfuncs

Mehdi Ben Hadj Khelifa
2025-09-25T08:57:37Z

On 9/24/25 11:31 PM, Stanislav Fomichev wrote:
> On 09/24, Mehdi Ben Hadj Khelifa wrote:
>> On 9/23/25 9:45 PM, Stanislav Fomichev wrote:
>>> On 09/23, Mehdi Ben Hadj Khelifa wrote:
>>>> ---
>>>> Mehdi Ben Hadj Khelifa (4):
>>>> netlink: specs: Add XDP RX queue index to XDP metadata
>>>> net: xdp: Add xmo_rx_queue_index callback
>>>> uapi: netdev: Add XDP RX queue index metadata flags
>>>> net: veth: Implement RX queue index XDP hint
>>>>
>>>> Documentation/netlink/specs/netdev.yaml | 5 +++++
>>>> drivers/net/veth.c | 12 ++++++++++++
>>>> include/net/xdp.h | 5 +++++
>>>> include/uapi/linux/netdev.h | 3 +++
>>>> net/core/xdp.c | 15 +++++++++++++++
>>>> tools/include/uapi/linux/netdev.h | 3 +++
>>>> 6 files changed, 43 insertions(+)
>>>> ---
>>>> base-commit: 07e27ad16399afcd693be20211b0dfae63e0615f
>>>> this is the commit of tag: v6.17-rc7 on the mainline.
>>>> This patch series is intended to make a base for setting
>>>> queue_index in the xdp_rxq_info struct in bpf/cpumap.c to
>>>> the right index. Although that part I still didn't figure
>>>> out yet,I m searching for my guidance to do that as well
>>>> as for the correctness of the patches in this series.
>>
>>>
> I don't really understand what queue_index means for the cpu map. It is
> a kernel thread doing work, there is no queue. Maybe whoever added
> the todo can clarify?

Hi Lorenzo,
Can you help us clarify the todo added in cpu_map_bpf_prog_run_xdp() in
this commit:
github.com/torvalds/linux/commit/9216477449f33cdbc9c9a99d49f500b7fbb81702 ?

Regards,
Mehdi