Re: [Intel-wired-lan] [PATCH iwl-net 1/2] ice: fix NULL pointer dereference in ice_unplug_aux_dev() on reset

2025-06-24 Thread Brett Creeley
On 6/24/2025 7:26 AM, Emil Tantilov wrote: Caution: This message originated from an External Source. Use proper caution when opening attachments, clicking links, or responding. Issuing a reset when the driver is loaded without RDMA support, will results in a crash as it attempts to remove R

Re: [Intel-wired-lan] [PATCH iwl-next, v3 1/2] igc: Relocate RSS field definitions to igc_defines.h

2025-06-23 Thread Brett Creeley
/ethernet/intel/igc/igc.h | 4 drivers/net/ethernet/intel/igc/igc_defines.h | 3 +++ 2 files changed, 3 insertions(+), 4 deletions(-) LGTM. Reviewed-by: Brett Creeley diff --git a/drivers/net/ethernet/intel/igc/igc.h b/drivers/net/ethernet/intel/igc/igc.h index 1525ae25fd3e

Re: [Intel-wired-lan] [PATCH iwl-next, v2 1/1] igc: Add wildcard rule support to ethtool NFC using Default Queue

2025-06-19 Thread Brett Creeley
On 6/19/2025 8:37 AM, Song Yoong Siang wrote: Caution: This message originated from an External Source. Use proper caution when opening attachments, clicking links, or responding. Introduce support for a lowest priority wildcard (catch-all) rule in ethtool's Network Flow Classification (NFC) f

Re: [Intel-wired-lan] [PATCH iwl-net v3 1/2] ice: Fix entering Safe Mode

2024-09-24 Thread Brett Creeley
(pf)) { + if (err || ice_is_safe_mode(pf)) { LGTM. Reviewed-by: Brett Creeley /* we already got function/device capabilities but these don't * reflect what the driver needs to do in safe mode. Instead of * adding conditional logi

Re: [Intel-wired-lan] [PATCH iwl-net v3 2/2] ice: Fix netif_is_ice() in Safe Mode

2024-09-24 Thread Brett Creeley
;& (dev->netdev_ops == &ice_netdev_ops); + return dev && (dev->netdev_ops == &ice_netdev_ops || + dev->netdev_ops == &ice_netdev_safe_mode_ops); } LGTM. Reviewed-by: Brett Creeley /** -- 2.45.0

Re: [Intel-wired-lan] [PATCH iwl-net v2 1/2] ice: Fix entering Safe Mode

2024-09-20 Thread Brett Creeley
On 9/20/2024 9:59 AM, Marcin Szycik wrote: Caution: This message originated from an External Source. Use proper caution when opening attachments, clicking links, or responding. If DDP package is missing or corrupted, the driver should enter Safe Mode. Instead, an error is returned and probe

Re: [Intel-wired-lan] [PATCH iwl-net 1/2] ice: Fix entering Safe Mode

2024-09-20 Thread Brett Creeley
On 9/20/2024 5:03 AM, Maciej Fijalkowski wrote: Caution: This message originated from an External Source. Use proper caution when opening attachments, clicking links, or responding. On Fri, Sep 20, 2024 at 01:55:09PM +0200, Marcin Szycik wrote: If DDP package is missing or corrupted, the d

Re: [Intel-wired-lan] [PATCH iwl-next v2] igc: Get rid of spurious interrupts

2024-07-24 Thread Brett Creeley
On 7/24/2024 12:26 AM, Kurt Kanzenbach wrote: On Tue Jul 23 2024, Brett Creeley wrote: @@ -5811,11 +5815,29 @@ static void igc_watchdog_task(struct work_struct *work) if (adapter->flags & IGC_FLAG_HAS_MSIX) { u32 eics = 0; - for (i = 0; i &l

Re: [Intel-wired-lan] [PATCH iwl-next v2] i40e: Add support for fw health report

2024-07-23 Thread Brett Creeley
On 7/18/2024 11:13 AM, Kamal Heib wrote: Caution: This message originated from an External Source. Use proper caution when opening attachments, clicking links, or responding. Add support for reporting fw status via the devlink health report. Example: # devlink health show pci/:02:00.

Re: [Intel-wired-lan] [PATCH iwl-next v2] igc: Get rid of spurious interrupts

2024-07-23 Thread Brett Creeley
On 6/20/2024 11:56 PM, Kurt Kanzenbach wrote: Caution: This message originated from an External Source. Use proper caution when opening attachments, clicking links, or responding. When running the igc with XDP/ZC in busy polling mode with deferral of hard interrupts, interrupts still happen

Re: [Intel-wired-lan] [PATCH iwl-net v2] ice: implement AQ download pkg retry

2024-05-20 Thread Brett Creeley
++-- 1 file changed, 17 insertions(+), 2 deletions(-) LGTM. Reviewed-by: Brett Creeley diff --git a/drivers/net/ethernet/intel/ice/ice_ddp.c b/drivers/net/ethernet/intel/ice/ice_ddp.c index ce5034ed2b24..77b81e5a5a44 100644 --- a/drivers/net/ethernet/intel/ice/ice_ddp.c +++ b/

Re: [Intel-wired-lan] [PATCH iwl-next v11] ice: Add get/set hw address for VFs using devlink commands

2024-05-20 Thread Brett Creeley
insertions(+), 10 deletions(-) LGTM. Reviewed-by: Brett Creeley diff --git a/drivers/net/ethernet/intel/ice/devlink/devlink_port.c b/drivers/net/ethernet/intel/ice/devlink/devlink_port.c index c9fbeebf7fb9..00fed5a61d62 100644 --- a/drivers/net/ethernet/intel/ice/devlink/devlink_port.c +++ b/drivers

Re: [Intel-wired-lan] [PATCH iwl-net] ice: implement AQ download pkg retry

2024-05-16 Thread Brett Creeley
On 5/16/2024 7:04 AM, Wojciech Drewek wrote: Caution: This message originated from an External Source. Use proper caution when opening attachments, clicking links, or responding. ice_aqc_opc_download_pkg (0x0C40) AQ sporadically returns error due to FW issue. Fix this by retrying five times

Re: [Intel-wired-lan] [PATCH iwl-next v9] ice: Add get/set hw address for VFs using devlink commands

2024-05-16 Thread Brett Creeley
On 5/15/2024 2:21 PM, Keller, Jacob E wrote: Caution: This message originated from an External Source. Use proper caution when opening attachments, clicking links, or responding. -Original Message- From: Brett Creeley Sent: Wednesday, May 15, 2024 9:53 AM To: Karthik Sundaravel

Re: [Intel-wired-lan] [PATCH iwl-next v9] ice: Add get/set hw address for VFs using devlink commands

2024-05-15 Thread Brett Creeley
On 5/15/2024 7:22 AM, Karthik Sundaravel wrote: Caution: This message originated from an External Source. Use proper caution when opening attachments, clicking links, or responding. Changing the MAC address of the VFs is currently unsupported via devlink. Add the function handlers to set an

Re: [Intel-wired-lan] [PATCH v4 iwl-net] i40e: Prevent setting MTU if greater than MFS

2024-03-15 Thread Brett Creeley
On 3/15/2024 2:17 AM, Erwan Velu wrote: Caution: This message originated from an External Source. Use proper caution when opening attachments, clicking links, or responding. Le 14/03/2024 à 21:31, Tony Nguyen a écrit : [..] Setting the mfs size to max values during init and reset would bet

Re: [Intel-wired-lan] [PATCH v4 iwl-net] i40e: Prevent setting MTU if greater than MFS

2024-03-14 Thread Brett Creeley
On 3/14/2024 10:10 AM, Erwan Velu wrote: Caution: This message originated from an External Source. Use proper caution when opening attachments, clicking links, or responding. Le 14/03/2024 à 17:10, Brett Creeley a écrit : [...] If this is how the max_mtu is determined, does it make sense

Re: [Intel-wired-lan] [PATCH v4 iwl-net] i40e: Prevent setting MTU if greater than MFS

2024-03-14 Thread Brett Creeley
On 3/13/2024 2:07 AM, Erwan Velu wrote: Caution: This message originated from an External Source. Use proper caution when opening attachments, clicking links, or responding. Commit 6871a7de705 ("[intelxl] Use admin queue to set port MAC address and maximum frame size") from iPXE project set

Re: [Intel-wired-lan] [PATCH net] i40e: Fix VF MAC filter removal

2024-03-14 Thread Brett Creeley
er_addr_equal(addr, vf->default_lan_addr.addr)) { + if (i40e_can_vf_change_mac(vf)) + was_unimac_deleted = true; + else + continue; + } Seems okay to me. Reviewed-by: Brett Creeley

Re: [Intel-wired-lan] [PATCH iwl-next v5 2/2] ice: Fix debugfs with devlink reload

2024-02-08 Thread Brett Creeley
ned-off-by: Wojciech Drewek LGTM. Reviewed-by: Brett Creeley --- drivers/net/ethernet/intel/ice/ice.h | 1 + drivers/net/ethernet/intel/ice/ice_debugfs.c | 10 ++ drivers/net/ethernet/intel/ice/ice_fwlog.c | 2 ++ drivers/net/ethernet/intel/ice/ice_main.c| 3 +--

Re: [Intel-wired-lan] [PATCH iwl-next v3] ice: Remove and readd netdev during devlink reload

2024-01-29 Thread Brett Creeley
e ice_debugfs_exit() call to ice_module_exit(). Nit, but the function is no longer ice_debugfs_fwlog_deinit() as it changed from v2->v3. Other than that, LGTM. Reviewed-by: Brett Creeley Suggested-by: Jiri Pirko Reviewed-by: Przemek Kitszel Reviewed-by: Vadim Fedorenko Reviewed-by:

Re: [Intel-wired-lan] [PATCH iwl-next v2] ice: Remove and readd netdev during devlink reload

2024-01-26 Thread Brett Creeley
On 1/25/2024 12:54 AM, Wojciech Drewek wrote: Caution: This message originated from an External Source. Use proper caution when opening attachments, clicking links, or responding. Recent changes to the devlink reload (commit 9b2348e2d6c9 ("devlink: warn about existing entities during reload

Re: [Intel-wired-lan] [PATCH iwl-next v4 2/2] ice: Implement 'flow-type ether' rules

2024-01-24 Thread Brett Creeley
ing prints. Reviewed-by: Brett Creeley [...]

Re: [Intel-wired-lan] [PATCH v4 iwl-next 0/6] ice: fix timestamping in reset process

2023-12-21 Thread Brett Creeley
), 106 deletions(-) For the series: Reviewed-by: Brett Creeley base-commit: 67b40ee196fd2fd6d9b7f9b58912587c837bdc39 -- 2.40.1 ___ Intel-wired-lan mailing list Intel-wired-lan@osuosl.org https://lists.osuosl.org/mailman/listinfo/intel-wired-lan

Re: [Intel-wired-lan] [PATCH iwl-next v3 2/2] ice: Implement 'flow-type ether' rules

2023-12-20 Thread Brett Creeley
On 12/20/2023 6:19 AM, Plachno, Lukasz wrote: Caution: This message originated from an External Source. Use proper caution when opening attachments, clicking links, or responding. On 12/19/2023 6:35 PM, Brett Creeley wrote: On 12/13/2023 8:34 PM, Lukasz Plachno wrote: Caution: This

Re: [Intel-wired-lan] [PATCH iwl-next v3 2/2] ice: Implement 'flow-type ether' rules

2023-12-19 Thread Brett Creeley
On 12/13/2023 8:34 PM, Lukasz Plachno wrote: Caution: This message originated from an External Source. Use proper caution when opening attachments, clicking links, or responding. From: Jakub Buchocki Add support for 'flow-type ether' Flow Director rules via ethtool. Rules not containing

Re: [Intel-wired-lan] [PATCH iwl-net v2] idpf: enable WB_ON_ITR

2023-12-15 Thread Brett Creeley
| 2 ++ 5 files changed, 41 insertions(+), 2 deletions(-) [...] Reviewed-by: Brett Creeley ___ Intel-wired-lan mailing list Intel-wired-lan@osuosl.org https://lists.osuosl.org/mailman/listinfo/intel-wired-lan

Re: [Intel-wired-lan] [PATCH iwl-net] idpf: enable WB_ON_ITR

2023-12-15 Thread Brett Creeley
On 12/12/2023 6:55 AM, Michal Kubiak wrote: Caution: This message originated from an External Source. Use proper caution when opening attachments, clicking links, or responding. From: Joshua Hay Tell hardware to writeback completed descriptors even when interrupts are disabled. Otherwise,

Re: [Intel-wired-lan] [PATCH iwl-next v6] ice: Add support for packet mirroring using hardware in switchdev mode

2023-12-15 Thread Brett Creeley
On 12/12/2023 4:51 AM, Andrii Staikov wrote: Caution: This message originated from an External Source. Use proper caution when opening attachments, clicking links, or responding. Switchdev mode allows to add mirroring rules to mirror incoming and outgoing packets to the interface's port rep

Re: [Intel-wired-lan] [net PATCH] i40e: fix use-after-free in i40e_aqc_add_filters()

2023-12-15 Thread Brett Creeley
On 12/13/2023 2:49 AM, Ke Xiao wrote: Caution: This message originated from an External Source. Use proper caution when opening attachments, clicking links, or responding. Commit 3116f59c12bd ("i40e: fix use-after-free in i40e_sync_filters_subtask()") avoided use-after-free issues, by increasi

Re: [Intel-wired-lan] [PATCH iwl-next v4 03/12] ice: Introduce VF state ICE_VF_STATE_REPLAYING_VC for migration

2023-12-08 Thread Brett Creeley
On 11/20/2023 6:51 PM, Yahui Cao wrote: Caution: This message originated from an External Source. Use proper caution when opening attachments, clicking links, or responding. From: Lingyu Liu During migration device resume stage, part of device state is loaded by replaying logged virtual chan

Re: [Intel-wired-lan] [PATCH iwl-next v4 02/12] ice: Add function to get and set TX queue context

2023-12-08 Thread Brett Creeley
On 11/20/2023 6:51 PM, Yahui Cao wrote: Caution: This message originated from an External Source. Use proper caution when opening attachments, clicking links, or responding. Export TX queue context get and set function which is consumed by linux live migration driver to save and load device st

Re: [Intel-wired-lan] [PATCH iwl-next v4 01/12] ice: Add function to get RX queue context

2023-12-08 Thread Brett Creeley
On 11/20/2023 6:51 PM, Yahui Cao wrote: Caution: This message originated from an External Source. Use proper caution when opening attachments, clicking links, or responding. Export RX queue context get function which is consumed by linux live migration driver to save and load device state. N

Re: [Intel-wired-lan] [PATCH iwl-next v4 05/12] ice: Log virtual channel messages in PF

2023-12-07 Thread Brett Creeley
On 11/20/2023 6:51 PM, Yahui Cao wrote: Caution: This message originated from an External Source. Use proper caution when opening attachments, clicking links, or responding. From: Lingyu Liu Save the virtual channel messages sent by VF on the source side during runtime. The logged virtchn