Re: [Intel-wired-lan] [PATCH net-next 1/2] stmmac: xsk: fix underflow of budget in zerocopy mode

2025-07-22 Thread Willem de Bruijn
Jason Xing wrote: > On Tue, Jul 22, 2025 at 10:16 PM Willem de Bruijn > wrote: > > > > Jason Xing wrote: > > > Hi Paul, > > > > > > On Mon, Jul 21, 2025 at 4:56 PM Paul Menzel wrote: > > > > > > > > Dear Jason, > > >

Re: [Intel-wired-lan] [PATCH net-next 1/2] stmmac: xsk: fix underflow of budget in zerocopy mode

2025-07-22 Thread Willem de Bruijn
Jason Xing wrote: > Hi Paul, > > On Mon, Jul 21, 2025 at 4:56 PM Paul Menzel wrote: > > > > Dear Jason, > > > > > > Thank you for your patch. > > Thanks for your quick response and review :) > > > > > Am 21.07.25 um 10:33 schrieb Jason Xing: > > > From: Jason Xing > > > > > > The issue can hap

Re: [Intel-wired-lan] [PATCH iwl-next] idpf: preserve coalescing settings across resets

2025-06-26 Thread Willem de Bruijn
w fields to struct idpf_vport_user_config_data to save the user > settings and re-apply them after reset. > > Reviewed-by: Madhu Chittim > Signed-off-by: Ahmed Zaki Reviewed-by: Willem de Bruijn

Re: [Intel-wired-lan] [PATCH iwl-next] idpf: add cross timestamping

2025-05-24 Thread Willem de Bruijn
both ARM and x86 archs. > > Signed-off-by: Milena Olech > Reviewed-by: Karol Kolacinski Reviewed-by: Willem de Bruijn

Re: [Intel-wired-lan] [PATCH v3 iwl-next 08/10] idpf: add Tx timestamp flows

2025-01-14 Thread Willem de Bruijn
Olech, Milena wrote: > On 01/10/2025 10:34PM Willem de Bruijn wrote: > > > > > > Add functions to request Tx timestamp for the PTP packets, read the Tx > > > > > timestamp when the completion tag for that packet is being received, > > > > >

Re: [Intel-wired-lan] [PATCH v3 iwl-next 08/10] idpf: add Tx timestamp flows

2025-01-10 Thread Willem de Bruijn
Olech, Milena wrote: > On 01/07/2025 3:55PM Willem de Bruijn wrote: > > > > Add functions to request Tx timestamp for the PTP packets, read the Tx > > > timestamp when the completion tag for that packet is being received, > > > extend the Tx timestamp value a

Re: [Intel-wired-lan] [PATCH v3 iwl-next 08/10] idpf: add Tx timestamp flows

2025-01-07 Thread Willem de Bruijn
Milena Olech wrote: > Add functions to request Tx timestamp for the PTP packets, read the Tx > timestamp when the completion tag for that packet is being received, > extend the Tx timestamp value and set the supported timestamping modes. > > Tx timestamp is requested for the PTP packets by setting

Re: [Intel-wired-lan] [PATCH v3 iwl-next 07/10] idpf: add Tx timestamp capabilities negotiation

2024-12-19 Thread Willem de Bruijn
t the Tx timestamp capabilities and parse the uplink > vport flag. > > Reviewed-by: Alexander Lobakin > Co-developed-by: Emil Tantilov > Signed-off-by: Emil Tantilov > Co-developed-by: Pavan Kumar Linga > Signed-off-by: Pavan Kumar Linga > Signed-off-by: Milena Olech Reviewed-by: Willem de Bruijn

Re: [Intel-wired-lan] [PATCH v3 iwl-next 09/10] idpf: add support for Rx timestamping

2024-12-19 Thread Willem de Bruijn
ned-off-by: Milena Olech Reviewed-by: Willem de Bruijn

Re: [Intel-wired-lan] [PATCH v2 iwl-next 09/10] idpf: add support for Rx timestamping

2024-11-29 Thread Willem de Bruijn
Milena Olech wrote: > Add Rx timestamp function when the Rx timestamp value is read directly > from the Rx descriptor. In order to extend the Rx timestamp value to 64 > bit in hot path, the PHC time is cached in the receive groups. > Add supported Rx timestamp modes. > > Reviewed-by: Alexander Lob

Re: [Intel-wired-lan] [PATCH v2 iwl-next 08/10] idpf: add Tx timestamp flows

2024-11-29 Thread Willem de Bruijn
Milena Olech wrote: > Add functions to request Tx timestamp for the PTP packets, read the Tx > timestamp when the completion tag for that packet is being received, > extend the Tx timestamp value and set the supported timestamping modes. > > Tx timestamp is requested for the PTP packets by setting

Re: [Intel-wired-lan] [PATCH v2 iwl-next 07/10] idpf: add Tx timestamp capabilities negotiation

2024-11-29 Thread Willem de Bruijn
Milena Olech wrote: > Tx timestamp capabilities are negotiated for the uplink Vport. > Driver receives information about the number of available Tx timestamp > latches, the size of Tx timestamp value and the set of indexes used > for Tx timestamping. > > Add function to get the Tx timestamp capabi

Re: [Intel-wired-lan] [PATCH v2 iwl-next 04/10] idpf: negotiate PTP capabilities and get PTP clock

2024-11-27 Thread Willem de Bruijn
to the stack. > > Reviewed-by: Alexander Lobakin > Tested-by: Willem de Bruijn > Signed-off-by: Milena Olech Reviewed-by: Willem de Bruijn > --- > v1 -> v2: change the size of access fields in ptp struct also removes dependency on CONFIG_PCIE_PTM

Re: [Intel-wired-lan] [PATCH iwl-net 09/10] idpf: add support for Rx timestamping

2024-11-18 Thread Willem de Bruijn
Olech, Milena wrote: > On 11/14/2024 9:54 PM, Willem de Bruijn wrote: > > > Milena Olech wrote: > > > Add Rx timestamp function when the Rx timestamp value is read directly > > > from the Rx descriptor. Add supported Rx timestamp modes. > > > > > &

Re: [Intel-wired-lan] [PATCH iwl-net 08/10] idpf: add Tx timestamp flows

2024-11-18 Thread Willem de Bruijn
Olech, Milena wrote: > On 11/15/2024 12:20 AM, Willem de Bruijn wrote: > > > Milena Olech wrote: > > > Add functions to request Tx timestamp for the PTP packets, read the Tx > > > timestamp when the completion tag for that packet is being received, > > > ex

Re: [Intel-wired-lan] [PATCH iwl-net 04/10] idpf: negotiate PTP capabilies and get PTP clock

2024-11-18 Thread Willem de Bruijn
Olech, Milena wrote: > On 11/14/2024 9:20 PM, Willem de Bruijn wrote: > > > Milena Olech wrote: > > > PTP capabilities are negotiated using virtchnl command. Add get > > > capabilities function, direct access to read the PTP clock time and > > > direct acce

Re: [Intel-wired-lan] [PATCH iwl-net 04/10] idpf: negotiate PTP capabilies and get PTP clock

2024-11-14 Thread Willem de Bruijn
to the stack. > > Reviewed-by: Alexander Lobakin > Signed-off-by: Milena Olech Tested-by: Willem de Bruijn - Brought up a device - Verified the virtchannel capability negotiation - Verified /dev/ptp0 becomes availabe - Read the device clock using clock_gettime(FD_TO_CLOCKID(fd), &ts) - Verifi

Re: [Intel-wired-lan] [PATCH iwl-net 08/10] idpf: add Tx timestamp flows

2024-11-14 Thread Willem de Bruijn
Milena Olech wrote: > Add functions to request Tx timestamp for the PTP packets, read the Tx > timestamp when the completion tag for that packet is being received, > extend the Tx timestamp value and set the supported timestamping modes. > > Tx timestamp is requested for the PTP packets by setting

Re: [Intel-wired-lan] [PATCH iwl-net 04/10] idpf: negotiate PTP capabilies and get PTP clock

2024-11-14 Thread Willem de Bruijn
Vadim Fedorenko wrote: > > +/** > > + * idpf_ptp_read_src_clk_reg_direct - Read directly the main timer value > > + * @adapter: Driver specific private structure > > + * @sts: Optional parameter for holding a pair of system timestamps from > > + * the system clock. Will be ignored when NULL is gi

Re: [Intel-wired-lan] [PATCH iwl-net 09/10] idpf: add support for Rx timestamping

2024-11-14 Thread Willem de Bruijn
Milena Olech wrote: > Add Rx timestamp function when the Rx timestamp value is read directly > from the Rx descriptor. Add supported Rx timestamp modes. > > Reviewed-by: Alexander Lobakin > Signed-off-by: Milena Olech > --- > drivers/net/ethernet/intel/idpf/idpf_ptp.c | 74

Re: [Intel-wired-lan] [PATCH iwl-net 07/10] idpf: add Tx timestamp capabilities negotiation

2024-11-14 Thread Willem de Bruijn
Milena Olech wrote: > Tx timestamp capabilities are negotiated for the uplink Vport. > Driver receives information about the number of available Tx timestamp > latches, the size of Tx timestamp value and the set of indexes used > for Tx timestamping. > > Add function to get the Tx timestamp capabi

Re: [Intel-wired-lan] [PATCH iwl-net 06/10] idpf: add PTP clock configuration

2024-11-14 Thread Willem de Bruijn
and add functions to enable these actions using dedicated virtchnl > messages. > > Reviewed-by: Alexander Lobakin > Signed-off-by: Milena Olech Reviewed-by: Willem de Bruijn

Re: [Intel-wired-lan] [PATCH iwl-net 05/10] idpf: add mailbox access to read PTP clock time

2024-11-14 Thread Willem de Bruijn
negotiation. > > Add functions to recognize PTP messages, move them to dedicated secondary > mailbox, read the PTP clock time and cross timestamp using mailbox > messages. > > Reviewed-by: Alexander Lobakin > Signed-off-by: Milena Olech Reviewed-by: Willem de Bruijn

Re: [Intel-wired-lan] [PATCH iwl-net 04/10] idpf: negotiate PTP capabilies and get PTP clock

2024-11-14 Thread Willem de Bruijn
to the stack. > > Reviewed-by: Alexander Lobakin > Signed-off-by: Milena Olech Tested-by: Willem de Bruijn > /** > * struct idpf_ptp - PTP parameters > * @info: structure defining PTP hardware capabilities > * @clock: pointer to registered PTP clock device > * @adapter: ba

Re: [Intel-wired-lan] [PATCH iwl-net 03/10] idpf: move virtchnl structures to the header file

2024-11-14 Thread Willem de Bruijn
Milena Olech wrote: > Move virtchnl strucutres to the header file to expose them for the PTP > virtchnl file. Minor: s/strucutres/structures > Reviewed-by: Alexander Lobakin > Signed-off-by: Milena Olech Reviewed-by: Willem de Bruijn

Re: [Intel-wired-lan] [PATCH iwl-net 02/10] virtchnl: add PTP virtchnl definitions

2024-11-14 Thread Willem de Bruijn
- maximum > adjustment of the clock and the basic increment value. > > Additionally, API allows to configure the secondary mailbox, dedicated > exclusively for PTP purposes. > > Reviewed-by: Alexander Lobakin > Signed-off-by: Milena Olech minor issue, with that addressed R

Re: [Intel-wired-lan] [PATCH iwl-net 01/10] idpf: initial PTP support

2024-11-14 Thread Willem de Bruijn
Milena Olech wrote: > PTP feature is supported if the VIRTCHNL2_CAP_PTP is negotiated during the > capabilities recognition. Initial PTP support includes PTP initialization > and registration of the clock. > > Reviewed-by: Alexander Lobakin > Signed-off-by: Milena Olech Revi

Re: [Intel-wired-lan] [PATCH iwl-net] selftests/net: Fix csum test for short packets

2024-08-21 Thread Willem de Bruijn
Krzysztof Galazka wrote: > For IPv4 and IPv6 packets shorter than minimum Ethernet > frame payload, recvmsg returns lenght including padding. nit: length > Use length from header for checksum verification to avoid > csum test failing on correct packets. > > Fixes: 1d0dc857b5d8 (selftests: drv-ne

Re: [Intel-wired-lan] [PATCH v1 5/5] idpf: warn on possible ctlq overflow

2024-08-13 Thread Willem de Bruijn
Manoj Vishwanathan wrote: > From: Willem de Bruijn > > The virtchannel control queue is lossy to avoid deadlock. Ensure that > no losses occur in practice. Detect a full queue, when overflows may > have happened. > > In practice, virtchnl is synchronous currenty and messag

Re: [Intel-wired-lan] [PATCH iwl-next 11/12] idpf: convert header split mode to libeth + napi_build_skb()

2024-06-20 Thread Willem de Bruijn
Alexander Lobakin wrote: > From: Willem De Bruijn > Date: Mon, 17 Jun 2024 14:13:07 -0400 > > > Alexander Lobakin wrote: > >> From: Willem De Bruijn > >> Date: Thu, 30 May 2024 09:46:46 -0400 > >> > >>> Alexander Lobakin wrote: > >

Re: [Intel-wired-lan] [PATCH iwl-next 11/12] idpf: convert header split mode to libeth + napi_build_skb()

2024-06-17 Thread Willem de Bruijn
Alexander Lobakin wrote: > From: Willem De Bruijn > Date: Thu, 30 May 2024 09:46:46 -0400 > > > Alexander Lobakin wrote: > >> Currently, idpf uses the following model for the header buffers: > >> > >> * buffers are allocated via dma_alloc_coherent();

Re: [Intel-wired-lan] [PATCH iwl-next 11/12] idpf: convert header split mode to libeth + napi_build_skb()

2024-05-30 Thread Willem de Bruijn
Alexander Lobakin wrote: > Currently, idpf uses the following model for the header buffers: > > * buffers are allocated via dma_alloc_coherent(); > * when receiving, napi_alloc_skb() is called and then the header is > copied to the newly allocated linear part. > > This is far from optimal as DM

Re: [Intel-wired-lan] [PATCH 4/6 iwl-next] idpf: refactor remaining virtchnl messages

2024-01-24 Thread Willem de Bruijn
Alan Brady wrote: > This takes care of RSS/SRIOV/MAC and other misc virtchnl messages. This > again is mostly mechanical. There's some added functionality with MAC > filters which makes sure we remove bad filters now that we can better > handle asynchronous messages. Can you split the part that is

Re: [Intel-wired-lan] [PATCH 0/6 iwl-next] idpf: refactor virtchnl messages

2024-01-24 Thread Willem de Bruijn
Alan Brady wrote: > The motivation for this series has two primary goals. We want to enable > support of multiple simultaneous messages and make the channel more > robust. The way it works right now, the driver can only send and receive > a single message at a time and if something goes really wron

Re: [Intel-wired-lan] [PATCH RFC net-next 05/34] idpf: convert header split mode to libie + napi_build_skb()

2024-01-09 Thread Willem de Bruijn
Alexander Lobakin wrote: > From: Willem De Bruijn > Date: Wed, 27 Dec 2023 10:30:48 -0500 > > > Alexander Lobakin wrote: > >> Currently, idpf uses the following model for the header buffers: > >> > >> * buffers are allocated via dma_alloc_coherent();

Re: [Intel-wired-lan] [PATCH RFC net-next 00/34] Christmas 3-serie XDP for idpf (+generic stuff)

2024-01-08 Thread Willem de Bruijn
Alexander Lobakin wrote: > From: Willem De Bruijn > Date: Tue, 26 Dec 2023 15:23:41 -0500 > > > Alexander Lobakin wrote: > >> I was highly asked to send this WIP before the holidays to trigger > >> some discussions at least for the generic parts. > >> &

Re: [Intel-wired-lan] [PATCH RFC net-next 01/34] idpf: reuse libie's definitions of parsed ptype structures

2023-12-27 Thread Willem de Bruijn
Alexander Lobakin wrote: > idpf's in-kernel parsed ptype structure is almost identical to the one > used in the previous Intel drivers, which means it can be converted to > use libie's definitions and even helpers. The only difference is that > it doesn't use a constant table, rather than one obtai

Re: [Intel-wired-lan] [PATCH RFC net-next 05/34] idpf: convert header split mode to libie + napi_build_skb()

2023-12-27 Thread Willem de Bruijn
Alexander Lobakin wrote: > Currently, idpf uses the following model for the header buffers: > > * buffers are allocated via dma_alloc_coherent(); > * when receiving, napi_alloc_skb() is called and then the header is > copied to the newly allocated linear part. > > This is far from optimal as DM

Re: [Intel-wired-lan] [PATCH RFC net-next 00/34] Christmas 3-serie XDP for idpf (+generic stuff)

2023-12-26 Thread Willem de Bruijn
Alexander Lobakin wrote: > I was highly asked to send this WIP before the holidays to trigger > some discussions at least for the generic parts. > > This all depends on libie[0] and WB-on-ITR fix[1]. The RFC does not > guarantee to work perfectly, but at least regular XDP seems to work > for me...

Re: [Intel-wired-lan] [PATCH net-next v3 1/6] net: ethtool: allow symmetric-xor RSS hash for any flow type

2023-10-11 Thread Willem de Bruijn
On Tue, Oct 10, 2023 at 5:34 PM Ahmed Zaki wrote: > > > On 2023-10-10 14:40, Willem de Bruijn wrote: > > On Tue, Oct 10, 2023 at 4:05 PM Ahmed Zaki wrote: > > Symmetric RSS hash functions are beneficial in applications that monitor > both Tx and Rx packets of the

Re: [Intel-wired-lan] [PATCH net-next v3 1/6] net: ethtool: allow symmetric-xor RSS hash for any flow type

2023-10-10 Thread Willem de Bruijn
On Tue, Oct 10, 2023 at 4:05 PM Ahmed Zaki wrote: > > Symmetric RSS hash functions are beneficial in applications that monitor > both Tx and Rx packets of the same flow (IDS, software firewalls, ..etc). > Getting all traffic of the same flow on the same RX queue results in > higher CPU cache effic

Re: [Intel-wired-lan] [PATCH net-next v2 1/6] net: ethtool: allow symmetric RSS hash for any flow type

2023-10-07 Thread Willem de Bruijn
On Fri, Oct 6, 2023 at 7:22 PM Jakub Kicinski wrote: > > On Fri, 6 Oct 2023 16:47:21 -0600 Ahmed Zaki wrote: > > Symmetric RSS hash functions are beneficial in applications that monitor > > both Tx and Rx packets of the same flow (IDS, software firewalls, ..etc). > > Getting all traffic of the sa