netdev - Re: [Intel-wired-lan] [PATCH iwl-next v4 0/4] igb: Add support for AF

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-ID: <3253130.2gtjKKCVsX@desktop>
Date: Thu, 27 Jun 2024 18:49:16 +0200
From: Benjamin Steinke <benjamin.steinke@...s-audio.com>
To: Sriram Yagnaraman <sriram.yagnaraman@....tech>
CC: <intel-wired-lan@...osl.org>, Maciej Fijalkowski
	<maciej.fijalkowski@...el.com>, Jesper Dangaard Brouer <hawk@...nel.org>,
	Daniel Borkmann <daniel@...earbox.net>, <netdev@...r.kernel.org>, Jonathan
 Lemon <jonathan.lemon@...il.com>, John Fastabend <john.fastabend@...il.com>,
	Alexei Starovoitov <ast@...nel.org>, Björn Töpel
	<bjorn@...nel.org>, Eric Dumazet <edumazet@...gle.com>, Sriram Yagnaraman
	<sriram.yagnaraman@....tech>, Tony Nguyen <anthony.l.nguyen@...el.com>, Jakub
 Kicinski <kuba@...nel.org>, <bpf@...r.kernel.org>, Paolo Abeni
	<pabeni@...hat.com>, "David S . Miller" <davem@...emloft.net>, Magnus
 Karlsson <magnus.karlsson@...el.com>, Kurt Kanzenbach <kurt@...utronix.de>
Subject: Re: [Intel-wired-lan] [PATCH iwl-next v4 0/4] igb: Add support for AF_XDP zero-copy

On Thursday, 27 June 2024, 09:07:55 CEST, Kurt Kanzenbach wrote:
> Hi Sriram,
> 
> On Fri Aug 04 2023, Sriram Yagnaraman wrote:
> > The first couple of patches adds helper funcctions to prepare for AF_XDP
> > zero-copy support which comes in the last couple of patches, one each
> > for Rx and TX paths.
> > 
> > As mentioned in v1 patchset [0], I don't have access to an actual IGB
> > device to provide correct performance numbers. I have used Intel 82576EB
> > emulator in QEMU [1] to test the changes to IGB driver.
> 
> I gave this patch series a try on a recent kernel and silicon
> (i210). There was one issue in igb_xmit_zc(). But other than that it
> worked very nicely.

Hi Kurt and Sriram,

I recently tried the patches on a 6.1 kernel. On two different devices i210 & 
i211 I couldn't see any packets being transmitted on the wire. Perhaps caused 
by the issue in igb_xmit_zc() you mentioned, Kurt? Can you share your findings, 
please?

RX seemed to work on first sight.

> It seems like it hasn't been merged yet. Do you have any plans for
> continuing to work on this?

I can offer to do testing and debugging on real hardware if this helps.

Thanks,
Benjamin