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

2025-03-04 Thread Przemek Kitszel
On 3/3/25 21:47, joaomboni wrote: Signed-off-by: Joao Bonifacio it will be good to use imperative mood in the Subject, and add one more paragraph, like: Subject: e1000: mark global variables const where possible Next paragraph: Mark global variables const, so unintended modification would no

Re: [Intel-wired-lan] [PATCH iwl-next v5 09/15] ixgbe: add E610 functions getting PBA and FW ver info

2025-03-04 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] [iwl-net v3 1/5] virtchnl: make proto and filter action count unsigned

2025-03-04 Thread Szapar-Mudlaw, Martyna
On 3/4/2025 12:15 PM, Paul Menzel wrote: Dear Jan, dear Martina, Thank you for the patch. Am 04.03.25 um 12:08 schrieb Martyna Szapar-Mudlaw: From: Jan Glaza The count field in virtchnl_proto_hdrs and virtchnl_filter_action_set should never be negative while still being valid. Changing i

Re: [Intel-wired-lan] [iwl-net v3 1/5] virtchnl: make proto and filter action count unsigned

2025-03-04 Thread Paul Menzel
Dear Martyna, Thank you for your quick reply. Am 04.03.25 um 12:45 schrieb Szapar-Mudlaw, Martyna: On 3/4/2025 12:15 PM, Paul Menzel wrote: Am 04.03.25 um 12:08 schrieb Martyna Szapar-Mudlaw: From: Jan Glaza The count field in virtchnl_proto_hdrs and virtchnl_filter_action_set should ne

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

2025-03-04 Thread Vladimir Oltean
On Mon, Mar 03, 2025 at 05:26:54AM -0500, Faizal Rahim wrote: > +static inline bool igc_fpe_is_verify_or_response(union igc_adv_rx_desc > *rx_desc, > + unsigned int size) > +{ > + u32 status_error = le32_to_cpu(rx_desc->wb.upper.status_error); > +

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

2025-03-04 Thread Szapar-Mudlaw, Martyna
On 3/3/2025 10:54 AM, Simon Horman wrote: 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" it

Re: [Intel-wired-lan] [iwl-net v3 1/5] virtchnl: make proto and filter action count unsigned

2025-03-04 Thread Szapar-Mudlaw, Martyna
On 3/4/2025 12:51 PM, Paul Menzel wrote: Dear Martyna, Thank you for your quick reply. Am 04.03.25 um 12:45 schrieb Szapar-Mudlaw, Martyna: On 3/4/2025 12:15 PM, Paul Menzel wrote: Am 04.03.25 um 12:08 schrieb Martyna Szapar-Mudlaw: From: Jan Glaza The count field in virtchnl_proto_

Re: [Intel-wired-lan] [PATCH] e1000e: Link flap workaround option for false IRP events

2025-03-04 Thread Andrew Lunn
> > > However, that does not really help explain how this helps prevent an > > > interrupt. I assume playing with EEE settings was also played > > > with. Not that is register appears to have anything to do with EEE! > > > > > I don't think we did tried those - it was never suggested that I can re

Re: [Intel-wired-lan] [PATCH iwl-next v1 1/3] ice: remove SW side band access workaround for E825

2025-03-04 Thread Nitka, Grzegorz
> -Original Message- > From: Paul Menzel > Sent: Friday, February 21, 2025 10:16 PM > To: Nitka, Grzegorz ; Kolacinski, Karol > > Cc: intel-wired-...@lists.osuosl.org; net...@vger.kernel.org; Kitszel, > Przemyslaw ; Michal Swiatkowski > > Subject: Re: [Intel-wired-lan] [PATCH iwl-next v1

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

2025-03-04 Thread Ahmed Zaki
[RE-SEND] I just realized I sent this only to iwl, sorry for spamming. 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 +

Re: [Intel-wired-lan] [PATCH] e1000e: Link flap workaround option for false IRP events

2025-03-04 Thread Mark Pearson
Thanks Vitaly, On Tue, Mar 4, 2025, at 5:48 AM, Lifshits, Vitaly wrote: > On 3/3/2025 5:34 AM, Mark Pearson wrote: >> Hi Andrew, >> >> On Sun, Mar 2, 2025, at 11:13 AM, Andrew Lunn wrote: >>> On Sun, Mar 02, 2025 at 03:09:35PM +0200, Lifshits, Vitaly wrote: Hi Mark, > Hi A

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

2025-03-04 Thread Simon Horman
On Mon, Mar 03, 2025 at 10:16:33AM +0100, 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 t

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

2025-03-04 Thread Arinzon, David
> [RE-SEND] I just realized I sent this only to iwl, sorry for spamming. > > > 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/

Re: [Intel-wired-lan] [PATCH] e1000e: Link flap workaround option for false IRP events

2025-03-04 Thread Lifshits, Vitaly
On 3/3/2025 5:34 AM, Mark Pearson wrote: Hi Andrew, On Sun, Mar 2, 2025, at 11:13 AM, Andrew Lunn wrote: On Sun, Mar 02, 2025 at 03:09:35PM +0200, Lifshits, Vitaly wrote: Hi Mark, Hi Andrew On Thu, Feb 27, 2025, at 11:07 AM, Andrew Lunn wrote: + e1e_rphy(hw, PHY_

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

2025-03-04 Thread Loktionov, Aleksandr
> -Original Message- > From: Intel-wired-lan On Behalf Of > Przemek Kitszel > Sent: Tuesday, March 4, 2025 9:28 AM > To: joaomboni > Cc: Nguyen, Anthony L ; > andrew+net...@lunn.ch; da...@davemloft.net; eduma...@google.com; > k...@kernel.org; pab...@redhat.com; intel-wired-...@lists.osu

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

2025-03-04 Thread Loktionov, Aleksandr
> -Original Message- > From: Intel-wired-lan On Behalf Of > joaomboni > Sent: Monday, March 3, 2025 9:48 PM > To: Nguyen, Anthony L ; Kitszel, Przemyslaw > ; andrew+net...@lunn.ch; > da...@davemloft.net; eduma...@google.com; k...@kernel.org; > pab...@redhat.com > Cc: intel-wired-...@lis

[Intel-wired-lan] [iwl-net v3 2/5] ice: stop truncating queue ids when checking

2025-03-04 Thread Martyna Szapar-Mudlaw
From: Jan Glaza Queue IDs can be up to 4096, fix invalid check to stop truncating IDs to 8 bits. Fixes: bf93bf791cec8 ("ice: introduce ice_virtchnl.c and ice_virtchnl.h") Reviewed-by: Aleksandr Loktionov Reviewed-by: Jedrzej Jagielski Reviewed-by: Simon Horman Signed-off-by: Jan Glaza Signed

[Intel-wired-lan] [iwl-net v3 3/5] ice: validate queue quanta parameters to prevent OOB access

2025-03-04 Thread Martyna Szapar-Mudlaw
From: Jan Glaza Add queue wraparound prevention in quanta configuration. Ensure end_qid does not overflow by validating start_qid and num_queues. Fixes: 015307754a19 ("ice: Support VF queue rate limit and quanta size configuration") Reviewed-by: Jedrzej Jagielski Reviewed-by: Simon Horman Sig

[Intel-wired-lan] [iwl-net v3 1/5] virtchnl: make proto and filter action count unsigned

2025-03-04 Thread Martyna Szapar-Mudlaw
From: Jan Glaza The count field in virtchnl_proto_hdrs and virtchnl_filter_action_set should never be negative while still being valid. Changing it from int to u32 ensures proper handling of values in virtchnl messages in driverrs and prevents unintended behavior. In its current signed form, a ne

[Intel-wired-lan] [iwl-net v3 0/5] ice: fix validation issues in virtchnl parameters

2025-03-04 Thread Martyna Szapar-Mudlaw
This patch series addresses validation issues in the virtchnl interface of the ice driver. These fixes correct improper value checking, ensuring that the driver can properly handle and reject invalid inputs from potentially malicious VFs. By fixing validation mechanisms, these patches strictly enfo

[Intel-wired-lan] [iwl-net v3 4/5] ice: fix input validation for virtchnl BW

2025-03-04 Thread Martyna Szapar-Mudlaw
From: Lukasz Czapnik Add missing validation of tc and queue id values sent by a VF in ice_vc_cfg_q_bw(). Additionally fixed logged value in the warning message, where max_tx_rate was incorrectly referenced instead of min_tx_rate. Also correct error handling in this function by properly exiting wh

Re: [Intel-wired-lan] [iwl-net v3 1/5] virtchnl: make proto and filter action count unsigned

2025-03-04 Thread Paul Menzel
Dear Jan, dear Martina, Thank you for the patch. Am 04.03.25 um 12:08 schrieb Martyna Szapar-Mudlaw: From: Jan Glaza The count field in virtchnl_proto_hdrs and virtchnl_filter_action_set should never be negative while still being valid. Changing it from int to u32 ensures proper handling of

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

2025-03-04 Thread Martyna Szapar-Mudlaw
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 value. Fixes: 99f419df8a5c ("ice: enable FDIR filters from raw binary patterns for VFs") Reviewed-by: Przemek Kits

Re: [Intel-wired-lan] e1000e/I219-LM: Speed negotiation problems

2025-03-04 Thread Lifshits, Vitaly
On 3/3/2025 5:30 PM, Paul Menzel wrote: Dear Linux folks, On a Dell OptiPlex 7000/0F1HC1, BIOS 1.8.2 12/08/2022 with     00:1f.6 Ethernet controller [0200]: Intel Corporation Ethernet Connection (17) I219-LM [8086:1a1c] (rev 11) and Dell OptiPlex 7071/097YXY, BIOS 1.1.2 08/29/2019 with

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

2025-03-04 Thread Vladimir Oltean
On Mon, Mar 03, 2025 at 05:26:50AM -0500, Faizal Rahim wrote: > +/** > + * ethtool_mmsv_link_state_handle() - Inform MAC Merge Software Verification > + * of link state changes > + * @mmsv: MAC Merge Software Verification state > + * @up: True if device carrier is up

Re: [Intel-wired-lan] [PATCH] e1000e: Link flap workaround option for false IRP events

2025-03-04 Thread Andrew Lunn
> > I suggest to try replacing the register read for a short delay or > > reading the PHY STATUS register instead. > > > > Ack - we'll try that, and collect some other debug registers in the process. > Will update with findings - this may take a while :) Please be careful with which register you

Re: [Intel-wired-lan] [PATCH iwl-next v1 3/3] ice: enable timesync operation on 2xNAC E825 devices

2025-03-04 Thread Nitka, Grzegorz
> -Original Message- > From: Simon Horman > Sent: Tuesday, February 25, 2025 4:05 PM > To: Nitka, Grzegorz > Cc: intel-wired-...@lists.osuosl.org; net...@vger.kernel.org; Kolacinski, > Karol ; Kitszel, Przemyslaw > > Subject: Re: [PATCH iwl-next v1 3/3] ice: enable timesync operation on