Re: [Intel-wired-lan] [PATCH iwl-next v5 01/15] devlink: add value check to devlink_info_version_put()

2025-03-03 Thread Kalesh Anakkur Purayil
On Fri, Feb 21, 2025 at 5:37 PM Jedrzej Jagielski wrote: > > Prevent from proceeding if there's nothing to print. > > Suggested-by: Przemek Kitszel > Reviewed-by: Jiri Pirko > Signed-off-by: Jedrzej Jagielski Reviewed-by: Kalesh AP -- Regards, Kalesh AP smime.p7s Description: S/MIME Crypt

Re: [Intel-wired-lan] [iwl-net v2 5/5] ice: fix using untrusted value of pkt_len in ice_vc_fdir_parse_raw()

2025-03-03 Thread Szapar-Mudlaw, Martyna
On 2/28/2025 6:09 PM, Simon Horman wrote: On Tue, Feb 25, 2025 at 10:08:49AM +0100, Martyna Szapar-Mudlaw wrote: From: Mateusz Polchlopek Fix using the untrusted value of proto->raw.pkt_len in function ice_vc_fdir_parse_raw() by verifying if it does not exceed the VIRTCHNL_MAX_SIZE_RAW_PACK

Re: [Intel-wired-lan] [iwl-net v2 5/5] ice: fix using untrusted value of pkt_len in ice_vc_fdir_parse_raw()

2025-03-03 Thread Szapar-Mudlaw, Martyna
On 3/3/2025 11:00 AM, Przemek Kitszel wrote: On 2/28/25 18:17, Simon Horman wrote: On Tue, Feb 25, 2025 at 10:08:49AM +0100, Martyna Szapar-Mudlaw wrote: From: Mateusz Polchlopek Fix using the untrusted value of proto->raw.pkt_len in function ice_vc_fdir_parse_raw() by verifying if it does

[Intel-wired-lan] [PATCH iwl-next v7 5/9] igc: Add support for frame preemption verification

2025-03-03 Thread Faizal Rahim
This patch implements the "ethtool --set-mm" callback to trigger the frame preemption verification handshake. Uses the MAC Merge Software Verification (mmsv) mechanism in ethtool to perform the verification handshake for igc. The structure fpe.mmsv is set by mmsv in ethtool and should remain read-

[Intel-wired-lan] [PATCH] e1000: Adicionado const para melhorar a segurança do código

2025-03-03 Thread joaomboni
Signed-off-by: joaomboni --- drivers/net/ethernet/intel/e1000/e1000_main.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/net/ethernet/intel/e1000/e1000_main.c b/drivers/net/ethernet/intel/e1000/e1000_main.c index 3f089c3d47b2..96bc85f09aaf 100644 --- a/drivers/net/e

[Intel-wired-lan] [PATCH iwl-next v3] igc: Change Tx mode for MQPRIO offloading

2025-03-03 Thread Kurt Kanzenbach
The current MQPRIO offload implementation uses the legacy TSN Tx mode. In this mode the hardware uses four packet buffers and considers queue priorities. In order to harmonize the TAPRIO implementation with MQPRIO, switch to the regular TSN Tx mode. This mode also uses four packet buffers and cons

[Intel-wired-lan] [PATCH iwl-next v7 2/9] igc: Rename xdp_get_tx_ring() for non-xdp usage

2025-03-03 Thread Faizal Rahim
Renamed xdp_get_tx_ring() function to a more generic name for use in upcoming frame preemption patches. Signed-off-by: Faizal Rahim --- drivers/net/ethernet/intel/igc/igc.h | 2 +- drivers/net/ethernet/intel/igc/igc_main.c | 9 - 2 files changed, 5 insertions(+), 6 deletions(-) dif

Re: [Intel-wired-lan] [PATCH iwl-next v5 07/15] ixgbe: read the netlist version information

2025-03-03 Thread R, Bharath
> -Original Message- > From: Intel-wired-lan On Behalf Of > Jedrzej Jagielski > Sent: Friday, February 21, 2025 5:21 PM > To: intel-wired-...@lists.osuosl.org > Cc: Nguyen, Anthony L ; > net...@vger.kernel.org; ho...@kernel.org; j...@nvidia.com; Mrozowicz, > SlawomirX ; Polchlopek, Mateusz

Re: [Intel-wired-lan] [PATCH iwl-next v3] igc: Change Tx mode for MQPRIO offloading

2025-03-03 Thread Abdul Rahim, Faizal
On 3/3/2025 5:16 pm, Kurt Kanzenbach wrote: The current MQPRIO offload implementation uses the legacy TSN Tx mode. In this mode the hardware uses four packet buffers and considers queue priorities. In order to harmonize the TAPRIO implementation with MQPRIO, switch to the regular TSN Tx mode.

Re: [Intel-wired-lan] [PATCH iwl-next v5 03/15] ixgbe: add initial devlink support

2025-03-03 Thread R, Bharath
> -Original Message- > From: Intel-wired-lan On Behalf Of > Jedrzej Jagielski > Sent: Friday, February 21, 2025 5:21 PM > To: intel-wired-...@lists.osuosl.org > Cc: Nguyen, Anthony L ; > net...@vger.kernel.org; ho...@kernel.org; j...@nvidia.com; Jagielski, Jedrzej > ; Polchlopek, Mateusz >

Re: [Intel-wired-lan] [PATCH iwl-next v5 11/15] ixgbe: add device flash update via devlink

2025-03-03 Thread R, Bharath
> -Original Message- > From: Intel-wired-lan On Behalf Of > Jedrzej Jagielski > Sent: Friday, February 21, 2025 5:21 PM > To: intel-wired-...@lists.osuosl.org > Cc: Nguyen, Anthony L ; > net...@vger.kernel.org; ho...@kernel.org; j...@nvidia.com; Jagielski, Jedrzej > ; Keller, Jacob E ; > M

Re: [Intel-wired-lan] [PATCH iwl-next v5 12/15] ixgbe: add support for devlink reload

2025-03-03 Thread R, Bharath
> -Original Message- > From: Intel-wired-lan On Behalf Of > Jedrzej Jagielski > Sent: Friday, February 21, 2025 5:21 PM > To: intel-wired-...@lists.osuosl.org > Cc: Nguyen, Anthony L ; > net...@vger.kernel.org; ho...@kernel.org; j...@nvidia.com; Jagielski, Jedrzej > ; Polchlopek, Mateusz >

Re: [Intel-wired-lan] [PATCH iwl-next v1] ice: refactor the Tx scheduler feature

2025-03-03 Thread Simon Horman
On Wed, Feb 26, 2025 at 12:33:56PM +0100, Mateusz Polchlopek wrote: > Embed ice_get_tx_topo_user_sel() inside the only caller: > ice_devlink_tx_sched_layers_get(). > Instead of jump from the wrapper to the function that does "get" operation > it does "get" itself. > > Remove unnecessary comment an

Re: [Intel-wired-lan] [iwl-net v2 5/5] ice: fix using untrusted value of pkt_len in ice_vc_fdir_parse_raw()

2025-03-03 Thread Przemek Kitszel
On 2/28/25 18:17, Simon Horman wrote: On Tue, Feb 25, 2025 at 10:08:49AM +0100, Martyna Szapar-Mudlaw wrote: From: Mateusz Polchlopek Fix using the untrusted value of proto->raw.pkt_len in function ice_vc_fdir_parse_raw() by verifying if it does not exceed the VIRTCHNL_MAX_SIZE_RAW_PACKET valu

[Intel-wired-lan] [PATCH iwl-next v7 0/9] igc: Add support for Frame Preemption feature in IGC

2025-03-03 Thread Faizal Rahim
Introduces support for the FPE feature in the IGC driver. The patches aligns with the upstream FPE API: https://patchwork.kernel.org/project/netdevbpf/cover/20230220122343.1156614-1-vladimir.olt...@nxp.com/ https://patchwork.kernel.org/project/netdevbpf/cover/20230119122705.73054-1-vladimir.olt...

[Intel-wired-lan] [PATCH iwl-next v7 3/9] igc: Optimize the TX packet buffer utilization

2025-03-03 Thread Faizal Rahim
Packet buffers (RX + TX) total 64KB. Neither RX or TX buffers can be larger than 34KB. So divide the buffer equally, 32KB for each. Co-developed-by: Vinicius Costa Gomes Signed-off-by: Vinicius Costa Gomes Signed-off-by: Faizal Rahim --- drivers/net/ethernet/intel/igc/igc_defines.h | 3 ++- 1

[Intel-wired-lan] [PATCH iwl-next v7 6/9] igc: Add support to set tx-min-frag-size

2025-03-03 Thread Faizal Rahim
Add support to set tx-min-frag-size via set_mm callback in igc. Increase the max limit of tx-ming-frag-size in ethtool from 252 to 256 since i225/6 value range is 64, 128, 192 and 256. Co-developed-by: Vinicius Costa Gomes Signed-off-by: Vinicius Costa Gomes Signed-off-by: Faizal Rahim --- dri

[Intel-wired-lan] [PATCH iwl-next v7 8/9] igc: Add support to get MAC Merge data via ethtool

2025-03-03 Thread Faizal Rahim
Implement "ethtool --show-mm" callback for IGC. Tested with command: $ ethtool --show-mm enp1s0. MAC Merge layer state for enp1s0: pMAC enabled: on TX enabled: on TX active: on TX minimum fragment size: 64 RX minimum fragment size: 60 Verify enabled: on Verify time: 128 Max verif

[Intel-wired-lan] [PATCH iwl-next v7 4/9] igc: Set the RX packet buffer size for TSN mode

2025-03-03 Thread Faizal Rahim
In preparation for supporting frame preemption, when entering TSN mode set the receive packet buffer to 16KB for the Express MAC, 16KB for the Preemptible MAC and 2KB for the BMC, according to the datasheet section 7.1.3.2. Co-developed-by: Vinicius Costa Gomes Signed-off-by: Vinicius Costa Gomes

[Intel-wired-lan] [PATCH iwl-next v7 9/9] igc: Add support to get frame preemption statistics via ethtool

2025-03-03 Thread Faizal Rahim
Implemented "ethtool --include-statistics --show-mm" callback for IGC. Tested preemption scenario to check preemption statistics: 1) Trigger verification handshake on both boards: $ sudo ethtool --set-mm enp1s0 pmac-enabled on $ sudo ethtool --set-mm enp1s0 tx-enabled on $ sudo ethtool

[Intel-wired-lan] [PATCH iwl-next v7 7/9] igc: Block setting preemptible traffic class in taprio

2025-03-03 Thread Faizal Rahim
Since preemptible tc implementation is not ready yet, block it from being set in taprio. The existing code already blocks it in mqprio. Signed-off-by: Faizal Rahim --- drivers/net/ethernet/intel/igc/igc_main.c | 4 1 file changed, 4 insertions(+) diff --git a/drivers/net/ethernet/intel/igc

[Intel-wired-lan] [PATCH iwl-next v7 1/9] net: ethtool: mm: extract stmmac verification logic into common library

2025-03-03 Thread Faizal Rahim
From: Vladimir Oltean It appears that stmmac is not the only hardware which requires a software-driven verification state machine for the MAC Merge layer. While on the one hand it's good to encourage hardware implementations, on the other hand it's quite difficult to tolerate multiple drivers im

Re: [Intel-wired-lan] [PATCH iwl-next v5 01/15] devlink: add value check to devlink_info_version_put()

2025-03-03 Thread R, Bharath
> -Original Message- > From: Intel-wired-lan On Behalf Of > Jedrzej Jagielski > Sent: Friday, February 21, 2025 5:21 PM > To: intel-wired-...@lists.osuosl.org > Cc: Nguyen, Anthony L ; > net...@vger.kernel.org; ho...@kernel.org; j...@nvidia.com; Jagielski, Jedrzej > ; Kitszel, Przemyslaw >

Re: [Intel-wired-lan] [PATCH iwl-next v5 05/15] ixgbe: add E610 functions for acquiring flash data

2025-03-03 Thread R, Bharath
> -Original Message- > From: Intel-wired-lan On Behalf Of > Jedrzej Jagielski > Sent: Friday, February 21, 2025 5:21 PM > To: intel-wired-...@lists.osuosl.org > Cc: Nguyen, Anthony L ; > net...@vger.kernel.org; ho...@kernel.org; j...@nvidia.com; Mrozowicz, > SlawomirX ; Kitszel, Przemyslaw

Re: [Intel-wired-lan] [PATCH iwl-next v2 1/4] ixgbe: create E610 specific ethtool_ops structure

2025-03-03 Thread Jagielski, Jedrzej
From: Paul Menzel Sent: Friday, February 28, 2025 11:17 AM >Dear Jedrzej, > > >Thank you for your patch. > >Am 28.02.25 um 09:37 schrieb Jedrzej Jagielski: >> E610's implementation of various ethtool ops is different than >> the ones corresponding to ixgbe legacy products. Therefore create >> se

[Intel-wired-lan] [PATCH iwl-next v3 1/4] ixgbe: create E610 specific ethtool_ops structure

2025-03-03 Thread Jedrzej Jagielski
E610's implementation of various ethtool ops is different than the ones corresponding to ixgbe legacy products. Therefore create separate E610 ethtool_ops struct which will be filled out in the forthcoming patches. Add adequate ops struct basing on MAC type. This step requires changing a bit the f

[Intel-wired-lan] [PATCH iwl-next v3 4/4] ixgbe: add E610 .set_phys_id() callback implementation

2025-03-03 Thread Jedrzej Jagielski
Legacy implementation of .set_phys_id() ethtool callback is not applicable for E610 device. Add new implementation which uses 0x06E9 command by calling ixgbe_aci_set_port_id_led(). Reviewed-by: Aleksandr Loktionov Reviewed-by: Michal Swiatkowski Signed-off-by: Jedrzej Jagielski --- v3: move th

[Intel-wired-lan] [PATCH iwl-next v3 0/4] ixgbe: add ethtool support for E610

2025-03-03 Thread Jedrzej Jagielski
As E610 adapter introduces different approach to communicate with FW than the previous adapters using the ixgbe driver, there is a need to adjust some of the ethtool callbacks implementations to the new pattern. Some of the features are now set using Admin Command Interface (eg. controlling LEDs) w

[Intel-wired-lan] [PATCH iwl-next v3 2/4] ixgbe: add support for ACPI WOL for E610

2025-03-03 Thread Jedrzej Jagielski
Currently only APM (Advanced Power Management) is supported by the ixgbe driver. It works for magic packets only, as for different sources of wake-up E610 adapter utilizes different feature. Add E610 specific implementation of ixgbe_set_wol() callback. When any of broadcast/multicast/unicast wake-

Re: [Intel-wired-lan] [PATCH] e1000: Adicionado const para melhorar a segurança do código

2025-03-03 Thread Andrew Lunn
On Mon, Mar 03, 2025 at 10:11:55AM -0300, joaomboni wrote: > Signed-off-by: joaomboni Sorry, but the Subject: line needs to be in English. And the name in Signed-off-by: needs to be a real name. The code change itself looks sensible. Andrew --- pw-bot: cr

[Intel-wired-lan] [PATCH iwl-next v3 3/4] ixgbe: apply different rules for setting FC on E610

2025-03-03 Thread Jedrzej Jagielski
E610 device doesn't support disabling FC autonegotiation. Create dedicated E610 .set_pauseparam() implementation and assign it to ixgbe_ethtool_ops_e610. Reviewed-by: Aleksandr Loktionov Signed-off-by: Jedrzej Jagielski --- .../net/ethernet/intel/ixgbe/ixgbe_ethtool.c | 57 ---

[Intel-wired-lan] [PATCH v2] e1000: The 'const' qualifier has been added where applicable to enhance code safety and prevent unintended modifications.

2025-03-03 Thread joaomboni
Signed-off-by: Joao Bonifacio --- drivers/net/ethernet/intel/e1000/e1000_main.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/net/ethernet/intel/e1000/e1000_main.c b/drivers/net/ethernet/intel/e1000/e1000_main.c index 3f089c3d47b2..96bc85f09aaf 100644 --- a/drivers/

[Intel-wired-lan] MMIO write access to an invalid page in i40e_clear_hw()

2025-03-03 Thread Kyungwook Boo
Hello, It seems that there are invalid page MMIO write access in i40e_clear_hw() due to an integer underflow from num_pf_int(also num_vf_int seems possible). The following is a sample code in i40e_clear_hw(): val = rd32(hw, I40E_GLPCI_CNF2); // (1) num_pf_int = FIELD_GET(I40E_GLPCI_CNF2_MSI_X_PF

Re: [Intel-wired-lan] [PATCH net-next v9 2/6] net: ena: use napi's aRFS rmap notifers

2025-03-03 Thread Ahmed Zaki
On 2025-03-03 10:11 a.m., Arinzon, David wrote: Use the core's rmap notifiers and delete our own. Acked-by: David Arinzon Signed-off-by: Ahmed Zaki --- drivers/net/ethernet/amazon/ena/ena_netdev.c | 43 +--- 1 file changed, 1 insertion(+), 42 deletions(-) diff --git a/dr

Re: [Intel-wired-lan] [PATCH] e1000: Adicionado const para melhorar a segurança do código

2025-03-03 Thread Jakub Kicinski
On Mon, 3 Mar 2025 14:48:46 +0100 Andrew Lunn wrote: > Sorry, but the Subject: line needs to be in English. > > And the name in Signed-off-by: needs to be a real name. > > The code change itself looks sensible. If only we were also LLMs, not just humans, we could read commits in any language!

Re: [Intel-wired-lan] [PATCH net-next v9 2/6] net: ena: use napi's aRFS rmap notifers

2025-03-03 Thread Arinzon, David
> Use the core's rmap notifiers and delete our own. > > Acked-by: David Arinzon > Signed-off-by: Ahmed Zaki > --- > drivers/net/ethernet/amazon/ena/ena_netdev.c | 43 +--- > 1 file changed, 1 insertion(+), 42 deletions(-) > > diff --git a/drivers/net/ethernet/amazon/ena/ena_net