Re: [Intel-wired-lan] [PATCH 1/5] drivers/net/ethernet/intel-ice: ensure the copied buf is NULL terminated

2024-04-23 Thread Przemek Kitszel
On 4/22/24 18:41, Bui Quang Minh wrote: Currently, we allocate a count-sized kernel buffer and copy count bytes from userspace to that buffer. Later, we use sscanf on this buffer but we don't ensure that the string is terminated inside the buffer, this can lead to OOB read when using sscanf. Fix

[Intel-wired-lan] [PATCH] igb: cope with large MAX_SKB_FRAGS.

2024-04-23 Thread Corinna Vinschen
From: Paolo Abeni Sabrina reports that the igb driver does not cope well with large MAX_SKB_FRAG values: setting MAX_SKB_FRAG to 45 causes payload corruption on TX. The root cause of the issue is that the driver does not take into account properly the (possibly large) shared info size when selec

[Intel-wired-lan] [PATCH] igc: fix a log entry using uninitialized netdev

2024-04-23 Thread Corinna Vinschen
During successful probe, igc logs this: [5.133667] igc :01:00.0 (unnamed net_device) (uninitialized): PHC added The reason is that igc_ptp_init() is called very early, even before register_netdev() has been called. So the

Re: [Intel-wired-lan] [PATCH 0/5] Ensure the copied buf is NULL terminated

2024-04-23 Thread Marcin Szycik
On 22.04.2024 18:41, Bui Quang Minh wrote: > Hi everyone, > > I found that some drivers contains an out-of-bound read pattern like this > > kern_buf = memdup_user(user_buf, count); > ... > sscanf(kern_buf, ...); > > The sscanf can be replaced by some other string-related fun

Re: [Intel-wired-lan] [PATCH 0/5] Ensure the copied buf is NULL terminated

2024-04-23 Thread Przemek Kitszel
On 4/23/24 13:10, Marcin Szycik wrote: On 22.04.2024 18:41, Bui Quang Minh wrote: Hi everyone, I found that some drivers contains an out-of-bound read pattern like this kern_buf = memdup_user(user_buf, count); ... sscanf(kern_buf, ...); The sscanf can be replaced by

Re: [Intel-wired-lan] [PATCH] igb: cope with large MAX_SKB_FRAGS.

2024-04-23 Thread Jiri Pirko
Tue, Apr 23, 2024 at 12:24:46PM CEST, vinsc...@redhat.com wrote: >From: Paolo Abeni > >Sabrina reports that the igb driver does not cope well with large >MAX_SKB_FRAG values: setting MAX_SKB_FRAG to 45 causes payload >corruption on TX. > >The root cause of the issue is that the driver does not tak

Re: [Intel-wired-lan] [PATCH] igb: cope with large MAX_SKB_FRAGS.

2024-04-23 Thread Jiri Pirko
Tue, Apr 23, 2024 at 12:24:46PM CEST, vinsc...@redhat.com wrote: >From: Paolo Abeni > >Sabrina reports that the igb driver does not cope well with large >MAX_SKB_FRAG values: setting MAX_SKB_FRAG to 45 causes payload >corruption on TX. > >The root cause of the issue is that the driver does not tak

Re: [Intel-wired-lan] [PATCH iwl-next v2] ice: Extend auxbus device naming

2024-04-23 Thread Jiri Pirko
Tue, Apr 23, 2024 at 11:14:59AM CEST, sergey.temerkha...@intel.com wrote: >Include segment/domain number in the device name to distinguish >between PCI devices located on different root complexes in >multi-segment configurations. Naming is changed from >ptp___clk to ptpclk I don't understand

Re: [Intel-wired-lan] [PATCH] igb: cope with large MAX_SKB_FRAGS.

2024-04-23 Thread Paul Menzel
Dear Corinna, Thank you for the patch. Am 23.04.24 um 12:24 schrieb Corinna Vinschen: From: Paolo Abeni It’d be great if you removed the trailing dot/period in the commit message summary. Sabrina reports that the igb driver does not cope well with large MAX_SKB_FRAG values: setting MAX

Re: [Intel-wired-lan] [PATCH iwl-next v2] ice: Extend auxbus device naming

2024-04-23 Thread Jiri Pirko
Tue, Apr 23, 2024 at 01:56:55PM CEST, sergey.temerkha...@intel.com wrote: > > >> -Original Message- >> From: Jiri Pirko >> Sent: Tuesday, April 23, 2024 1:36 PM >> To: Temerkhanov, Sergey >> Cc: intel-wired-...@lists.osuosl.org; net...@vger.kernel.org; Kitszel, >> Przemyslaw >> Subject:

[Intel-wired-lan] [PATCH net v2] igb: cope with large MAX_SKB_FRAGS

2024-04-23 Thread Corinna Vinschen
From: Paolo Abeni Sabrina reports that the igb driver does not cope well with large MAX_SKB_FRAG values: setting MAX_SKB_FRAG to 45 causes payload corruption on TX. An easy reproducer is to run ssh to connect to the machine. With MAX_SKB_FRAGS=17 it works, with MAX_SKB_FRAGS=45 it fails. The r

Re: [Intel-wired-lan] [PATCH] igb: cope with large MAX_SKB_FRAGS.

2024-04-23 Thread Corinna Vinschen
Hi Paul, On Apr 23 13:52, Paul Menzel wrote: > Dear Corinna, > > > Thank you for the patch. > > > Am 23.04.24 um 12:24 schrieb Corinna Vinschen: > > From: Paolo Abeni > > It’d be great if you removed the trailing dot/period in the commit message > summary. > > > Sabrina reports that the igb

[Intel-wired-lan] [tnguy-next-queue:dev-queue] BUILD SUCCESS 00d9951472721c4285593d6aa0ab03eabdc933e1

2024-04-23 Thread kernel test robot
allmodconfig gcc arc allnoconfig gcc arc allyesconfig gcc arc defconfig gcc arc randconfig-001-20240423 gcc arc randconfig-002-20240423 gcc arm

Re: [Intel-wired-lan] [PATCH] igb: cope with large MAX_SKB_FRAGS.

2024-04-23 Thread Corinna Vinschen
Hi Jiri, On Apr 23 13:26, Jiri Pirko wrote: > Tue, Apr 23, 2024 at 12:24:46PM CEST, vinsc...@redhat.com wrote: > >From: Paolo Abeni > > > >Sabrina reports that the igb driver does not cope well with large > >MAX_SKB_FRAG values: setting MAX_SKB_FRAG to 45 causes payload > >corruption on TX. > > >

Re: [Intel-wired-lan] [PATCH net v2] igb: cope with large MAX_SKB_FRAGS

2024-04-23 Thread Eric Dumazet
On Tue, Apr 23, 2024 at 3:47 PM Corinna Vinschen wrote: > > From: Paolo Abeni > > Sabrina reports that the igb driver does not cope well with large > MAX_SKB_FRAG values: setting MAX_SKB_FRAG to 45 causes payload > corruption on TX. > > An easy reproducer is to run ssh to connect to the machine.

[Intel-wired-lan] [PATCH iwl-next] ice: remove correct filters during eswitch release

2024-04-23 Thread Marcin Szycik
From: Michal Swiatkowski ice_clear_dflt_vsi() is only removing default rule. Both default RX and TX rule should be removed during release. If it isn't switching to switchdev, second time results in error, because TX filter is already there. Fix it by removing the correct set of rules. Fixes: 5

[Intel-wired-lan] [PATCH 1/1] net: e1000e & ixgbe: Remove PCI_HEADER_TYPE_MFD duplicates

2024-04-23 Thread Ilpo Järvinen
PCI_HEADER_TYPE_MULTIFUNC is define by e1000e and ixgbe and both are unused. There is already PCI_HEADER_TYPE_MFD in pci_regs.h anyway which should be used instead so remove the duplicated defines of it. Signed-off-by: Ilpo Järvinen --- drivers/net/ethernet/intel/e1000e/defines.h | 2 -- drive

[Intel-wired-lan] [tnguy-next-queue:100GbE] BUILD SUCCESS 9afff0de30db149a1bf440db26a3ddd6a4f260d8

2024-04-23 Thread kernel test robot
allnoconfig gcc arc allyesconfig gcc arc defconfig gcc arc randconfig-001-20240423 gcc arc randconfig-002-20240423 gcc arm

[Intel-wired-lan] [PATCH] net/i40e: Fix repeated EEH reports in MSI domain

2024-04-23 Thread Thinh Tran
The patch fixes an issue when repeated EEH reports with a single error on the bus of Intel X710 4-port 10G Base-T adapter, in the MSI domain causing the devices to be permanently disabled. It fully resets and restart the devices when handling the PCI EEH error. Two new functions, i40e_io_suspend(

Re: [Intel-wired-lan] [PATCH 5/5] drivers/s390/cio: ensure the copied buf is NULL terminated

2024-04-23 Thread Heiko Carstens
On Mon, Apr 22, 2024 at 11:41:40PM +0700, Bui Quang Minh wrote: > Currently, we allocate a lbuf-sized kernel buffer and copy lbuf from > userspace to that buffer. Later, we use scanf on this buffer but we don't > ensure that the string is terminated inside the buffer, this can lead to > OOB read wh

[Intel-wired-lan] [PATCH iwl-next v2] ice: Extend auxbus device naming

2024-04-23 Thread Sergey Temerkhanov
Include segment/domain number in the device name to distinguish between PCI devices located on different root complexes in multi-segment configurations. Naming is changed from ptp___clk to ptpclk v1->v2 Rebase on top of the latest changes Signed-off-by: Sergey Temerkhanov Reviewed-by: Przem

Re: [Intel-wired-lan] [PATCH iwl-next v5 09/12] iavf: refactor iavf_clean_rx_irq to support legacy and flex descriptors

2024-04-23 Thread Sunil Kovvuri Goutham
> -Original Message- > From: Mateusz Polchlopek > Sent: Thursday, April 18, 2024 10:55 AM > To: intel-wired-...@lists.osuosl.org > Cc: net...@vger.kernel.org; ho...@kernel.org; anthony.l.ngu...@intel.com; > Jacob Keller ; Wojciech Drewek > ; Mateusz Polchlopek > > Subject: [EXTERNAL] [

Re: [Intel-wired-lan] [PATCH iwl-next v5 12/12] iavf: add support for Rx timestamps to hotpath

2024-04-23 Thread Sunil Kovvuri Goutham
> -Original Message- > From: Mateusz Polchlopek > Sent: Thursday, April 18, 2024 10:55 AM > To: intel-wired-...@lists.osuosl.org > Cc: net...@vger.kernel.org; ho...@kernel.org; anthony.l.ngu...@intel.com; > Jacob Keller ; Wojciech Drewek > ; Mateusz Polchlopek > > Subject: [EXTERNAL] [

[Intel-wired-lan] [PATCH] igc: fix a log entry using uninitialized netdev

2024-04-23 Thread Hariprasad Kelam
> During successful probe, igc logs this: > > [5.133667] igc :01:00.0 (unnamed net_device) (uninitialized): PHC > added > > The reason is that igc_ptp_init() is called very early, even before > register_netdev() has be

[Intel-wired-lan] [PATCH iwl-net v3] ice: Do not get coalesce settings while in reset

2024-04-23 Thread Dawid Osuchowski
From: Ngai-Mint Kwan Getting coalesce settings while reset is in progress can cause NULL pointer deference bug. If under reset, abort get coalesce for ethtool. Fixes: 67fe64d78c43 ("ice: Implement getting and setting ethtool coalesce") Signed-off-by: Ngai-Mint Kwan Reviewed-by: Mateusz Polchlop

Re: [Intel-wired-lan] [PATCH iwl-next v2] ice: Extend auxbus device naming

2024-04-23 Thread Temerkhanov, Sergey
> -Original Message- > From: Jiri Pirko > Sent: Tuesday, April 23, 2024 1:36 PM > To: Temerkhanov, Sergey > Cc: intel-wired-...@lists.osuosl.org; net...@vger.kernel.org; Kitszel, > Przemyslaw > Subject: Re: [PATCH iwl-next v2] ice: Extend auxbus device naming > > Tue, Apr 23, 2024 at

Re: [Intel-wired-lan] [PATCH 5/5] drivers/s390/cio: ensure the copied buf is NULL terminated

2024-04-23 Thread Bui Quang Minh
On 4/23/24 13:50, Heiko Carstens wrote: On Mon, Apr 22, 2024 at 11:41:40PM +0700, Bui Quang Minh wrote: Currently, we allocate a lbuf-sized kernel buffer and copy lbuf from userspace to that buffer. Later, we use scanf on this buffer but we don't ensure that the string is terminated inside the b

Re: [Intel-wired-lan] [PATCH] igb: cope with large MAX_SKB_FRAGS.

2024-04-23 Thread Jiri Pirko
Tue, Apr 23, 2024 at 03:54:05PM CEST, vinsc...@redhat.com wrote: >Hi Jiri, > >On Apr 23 13:26, Jiri Pirko wrote: >> Tue, Apr 23, 2024 at 12:24:46PM CEST, vinsc...@redhat.com wrote: >> >From: Paolo Abeni >> > >> >Sabrina reports that the igb driver does not cope well with large >> >MAX_SKB_FRAG val

Re: [Intel-wired-lan] [BUG] e1000e, scheduling while atomic (stable)

2024-04-23 Thread Greg Kroah-Hartman
On Fri, Apr 19, 2024 at 12:20:05PM -0400, Jérôme Carretero wrote: > Hi Sasha, > > > Thank you, sorry for the delay but I coudln't reboot. > > Adding Greg KH because I don't know if stable will receive my e-mail > (not subscribed) but the regression was integrated in stable: > commit 0a4e3c2d976

[Intel-wired-lan] [tnguy-net-queue:dev-queue] BUILD SUCCESS d6dc36ca8d435c9dbe04f20ecb388fd31dab4d99

2024-04-23 Thread kernel test robot
allnoconfig gcc arc allyesconfig gcc arc defconfig gcc arc randconfig-001-20240423 gcc arc randconfig-002-20240423 gcc arm

[Intel-wired-lan] [tnguy-net-queue:1GbE] BUILD SUCCESS ddab364b39698e97b156200d1e8dcc791a1ee15a

2024-04-23 Thread kernel test robot
allmodconfig gcc arc allnoconfig gcc arc allyesconfig gcc arc defconfig gcc arc randconfig-001-20240423 gcc arc randconfig-002

Re: [Intel-wired-lan] [PATCH iwl-next v2] ice: Extend auxbus device naming

2024-04-23 Thread Jacob Keller
On 4/23/2024 6:14 AM, Jiri Pirko wrote: > Tue, Apr 23, 2024 at 01:56:55PM CEST, sergey.temerkha...@intel.com wrote: >> >> >>> -Original Message- >>> From: Jiri Pirko >>> Sent: Tuesday, April 23, 2024 1:36 PM >>> To: Temerkhanov, Sergey >>> Cc: intel-wired-...@lists.osuosl.org; net...@v

Re: [Intel-wired-lan] [PATCH iwl-net v1 1/1] e1000e: change usleep_range to udelay in PHY mdic access

2024-04-23 Thread Ruinskiy, Dima
On 17/04/2024 22:03, Lifshits, Vitaly wrote: This is a partial revert of commit 6dbdd4de0362 ("e1000e: Workaround for sporadic MDI error on Meteor Lake systems"). The referenced commit introduced an issue on vPro systems, where disconnecting and reconnecting the LAN cable might result in a kernel