Re: [Intel-wired-lan] [iwl-next v1 03/14] ice: add basic devlink subfunctions support

2024-05-10 Thread Michal Swiatkowski
On Thu, May 09, 2024 at 01:06:52PM +0200, Jiri Pirko wrote: > Tue, May 07, 2024 at 01:45:04PM CEST, michal.swiatkow...@linux.intel.com > wrote: > >From: Piotr Raczynski > > > >Implement devlink port handlers responsible for ethernet type devlink > >subfunctions. Create subfunction devlink port an

Re: [Intel-wired-lan] [iwl-next v1 06/14] ice: base subfunction aux driver

2024-05-10 Thread Michal Swiatkowski
On Thu, May 09, 2024 at 01:13:55PM +0200, Jiri Pirko wrote: > Tue, May 07, 2024 at 01:45:07PM CEST, michal.swiatkow...@linux.intel.com > wrote: > >From: Piotr Raczynski > > > >Implement subfunction driver. It is probe when subfunction port is > >activated. > > > >VSI is already created. During th

Re: [Intel-wired-lan] [iwl-next v1 00/14] ice: support devlink subfunction

2024-05-10 Thread Michal Swiatkowski
On Thu, May 09, 2024 at 01:18:29PM +0200, Jiri Pirko wrote: > Tue, May 07, 2024 at 01:45:01PM CEST, michal.swiatkow...@linux.intel.com > wrote: > >Hi, > > > >Currently ice driver does not allow creating more than one networking > >device per physical function. The only way to have more hardware ba

Re: [Intel-wired-lan] [iwl-next v1 11/14] ice: netdevice ops for SF representor

2024-05-10 Thread Michal Swiatkowski
On Thu, May 09, 2024 at 01:17:26PM +0200, Jiri Pirko wrote: > Subject does not have verb. Please add it. > > Otherwise, the patch looks ok. Thanks, will add sth like "implement". > > Reviewed-by: Jiri Pirko > > > > Tue, May 07, 2024 at 01:45:12PM CEST, michal.swiatkow...@linux.intel.com >

Re: [Intel-wired-lan] [iwl-next v1 08/14] ice: create port representor for SF

2024-05-10 Thread Michal Swiatkowski
On Thu, May 09, 2024 at 01:16:05PM +0200, Jiri Pirko wrote: > Tue, May 07, 2024 at 01:45:09PM CEST, michal.swiatkow...@linux.intel.com > wrote: > >Store subfunction and VF pointer in port representor structure as an > >union. Add port representor type to distinguish between each of them. > > > >Ke

Re: [Intel-wired-lan] [iwl-next v1 14/14] ice: allow to activate and deactivate subfunction

2024-05-10 Thread Michal Swiatkowski
On Thu, May 09, 2024 at 01:36:13PM +0200, Jiri Pirko wrote: > Tue, May 07, 2024 at 01:45:15PM CEST, michal.swiatkow...@linux.intel.com > wrote: > >From: Piotr Raczynski > > > >Use previously implemented SF aux driver. It is probe during SF > >activation and remove after deactivation. > > > >Signe

Re: [Intel-wired-lan] [PATCH v2 2/2] e1000e: fix link fluctuations problem

2024-05-10 Thread Sasha Neftin
On 09/05/2024 16:46, Andrew Lunn wrote: On Thu, May 09, 2024 at 12:13:27PM +0300, Ruinskiy, Dima wrote: On 08/05/2024 8:05, Sasha Neftin wrote: On 07/05/2024 15:31, Andrew Lunn wrote: On Fri, May 03, 2024 at 06:18:36PM +0800, Ricky Wu wrote: As described in https://bugzilla.kernel.org/show_bu

Re: [Intel-wired-lan] [PATCH v2] e1000e: move force SMBUS near the end of enable_ulp function

2024-05-10 Thread Simon Horman
On Wed, May 08, 2024 at 08:06:04PM +0800, Hui Wang wrote: > The commit 861e8086029e ("e1000e: move force SMBUS from enable ulp > function to avoid PHY loss issue") introduces a regression on > CH_MTP_I219_LM18 (PCIID: 0x8086550A). Without the referred commit, the > ethernet works well after suspend

Re: [Intel-wired-lan] [iwl-next v1 03/14] ice: add basic devlink subfunctions support

2024-05-10 Thread Jiri Pirko
Fri, May 10, 2024 at 09:13:51AM CEST, michal.swiatkow...@linux.intel.com wrote: >On Thu, May 09, 2024 at 01:06:52PM +0200, Jiri Pirko wrote: >> Tue, May 07, 2024 at 01:45:04PM CEST, michal.swiatkow...@linux.intel.com >> wrote: >> >From: Piotr Raczynski >> > >> >Implement devlink port handlers res

Re: [Intel-wired-lan] [iwl-next v1 06/14] ice: base subfunction aux driver

2024-05-10 Thread Jiri Pirko
Fri, May 10, 2024 at 09:20:51AM CEST, michal.swiatkow...@linux.intel.com wrote: >On Thu, May 09, 2024 at 01:13:55PM +0200, Jiri Pirko wrote: >> Tue, May 07, 2024 at 01:45:07PM CEST, michal.swiatkow...@linux.intel.com >> wrote: >> >From: Piotr Raczynski >> > >> >Implement subfunction driver. It is

Re: [Intel-wired-lan] [iwl-next v1 08/14] ice: create port representor for SF

2024-05-10 Thread Jiri Pirko
Fri, May 10, 2024 at 09:31:15AM CEST, michal.swiatkow...@linux.intel.com wrote: >On Thu, May 09, 2024 at 01:16:05PM +0200, Jiri Pirko wrote: >> Tue, May 07, 2024 at 01:45:09PM CEST, michal.swiatkow...@linux.intel.com >> wrote: >> >Store subfunction and VF pointer in port representor structure as a

Re: [Intel-wired-lan] [iwl-next v1 14/14] ice: allow to activate and deactivate subfunction

2024-05-10 Thread Jiri Pirko
Fri, May 10, 2024 at 09:33:54AM CEST, michal.swiatkow...@linux.intel.com wrote: >On Thu, May 09, 2024 at 01:36:13PM +0200, Jiri Pirko wrote: >> Tue, May 07, 2024 at 01:45:15PM CEST, michal.swiatkow...@linux.intel.com >> wrote: >> >From: Piotr Raczynski >> > >> >Use previously implemented SF aux d

Re: [Intel-wired-lan] [iwl-next v1 00/14] ice: support devlink subfunction

2024-05-10 Thread Jiri Pirko
Fri, May 10, 2024 at 09:24:48AM CEST, michal.swiatkow...@linux.intel.com wrote: >On Thu, May 09, 2024 at 01:18:29PM +0200, Jiri Pirko wrote: >> Tue, May 07, 2024 at 01:45:01PM CEST, michal.swiatkow...@linux.intel.com >> wrote: >> >Hi, >> > >> >Currently ice driver does not allow creating more than

Re: [Intel-wired-lan] [iwl-next v1 03/14] ice: add basic devlink subfunctions support

2024-05-10 Thread Michal Swiatkowski
On Fri, May 10, 2024 at 01:05:33PM +0200, Jiri Pirko wrote: > Fri, May 10, 2024 at 09:13:51AM CEST, michal.swiatkow...@linux.intel.com > wrote: > >On Thu, May 09, 2024 at 01:06:52PM +0200, Jiri Pirko wrote: > >> Tue, May 07, 2024 at 01:45:04PM CEST, michal.swiatkow...@linux.intel.com > >> wrote:

Re: [Intel-wired-lan] [iwl-next v1 08/14] ice: create port representor for SF

2024-05-10 Thread Michal Swiatkowski
On Fri, May 10, 2024 at 01:07:44PM +0200, Jiri Pirko wrote: > Fri, May 10, 2024 at 09:31:15AM CEST, michal.swiatkow...@linux.intel.com > wrote: > >On Thu, May 09, 2024 at 01:16:05PM +0200, Jiri Pirko wrote: > >> Tue, May 07, 2024 at 01:45:09PM CEST, michal.swiatkow...@linux.intel.com > >> wrote:

Re: [Intel-wired-lan] [iwl-next v1 00/14] ice: support devlink subfunction

2024-05-10 Thread Michal Swiatkowski
On Fri, May 10, 2024 at 01:09:20PM +0200, Jiri Pirko wrote: > Fri, May 10, 2024 at 09:24:48AM CEST, michal.swiatkow...@linux.intel.com > wrote: > >On Thu, May 09, 2024 at 01:18:29PM +0200, Jiri Pirko wrote: > >> Tue, May 07, 2024 at 01:45:01PM CEST, michal.swiatkow...@linux.intel.com > >> wrote:

Re: [Intel-wired-lan] [PATCH v2 2/2] e1000e: fix link fluctuations problem

2024-05-10 Thread Andrew Lunn
> > It would be interesting to see what the link partner sees. What does > > it think the I219-LM is advertising? Is it advertising 1000BaseT_Half? > > i219 parts come with LSI PHY. 1000BASE-T half-duplex is not supported. > 1000BASET half-duplex not advertised in IEEE 1000BASE-T Control Register

[Intel-wired-lan] [tnguy-next-queue:10GbE] BUILD SUCCESS e7073830cc8b52ef3df7dd150e4dac7706e0e104

2024-05-10 Thread kernel test robot
defconfig clang i386 allmodconfig gcc i386 allnoconfig gcc i386 allyesconfig gcc i386 buildonly-randconfig-001-20240510 clang i386 buildonly-randconfig-002

Re: [Intel-wired-lan] [PATCH v7 10/12] pps: generators: Add PPS Generator TIO Driver

2024-05-10 Thread Andy Shevchenko
On Thu, May 09, 2024 at 04:38:49AM +, D, Lakshmi Sowjanya wrote: > Will update as suggested. Just a side note: Since the series most likely missed v6.10, don't forget to bump dates and versions in ABI documentation in the next version. -- With Best Regards, Andy Shevchenko

[Intel-wired-lan] [PATCH iwl-next v2 0/3] ice:Support to dump PHY config, FEC

2024-05-10 Thread Anil Samal
Implementation to dump PHY configuration and FEC statistics to facilitate link level debugging of customer issues. Implementation has two parts a. Serdes equalization # ethtool -d eth0 Output: Offset Values -- -- 0x:

[Intel-wired-lan] [PATCH iwl-next v2 1/3] ice: Extend Sideband Queue command to support dynamic flag

2024-05-10 Thread Anil Samal
Current driver implementation for Sideband Queue supports a fixed flag (ICE_AQ_FLAG_RD). To retrieve FEC statistics from firmware, Sideband Queue command is used with a different flag. Extend API for Sideband Queue command to use 'flag' as input argument. Reviewed-by: Anthony L Nguyen Reviewed-b

[Intel-wired-lan] [PATCH iwl-next v2 2/3] ice: Implement driver functionality to dump fec statistics

2024-05-10 Thread Anil Samal
To debug link issues in the field, it is paramount to dump fec corrected/uncorrected block counts from firmware. Firmware requires PCS quad number and PCS port number to read FEC statistics. Current driver implementation does not maintain above physical properties of a port. Add new driver API to

[Intel-wired-lan] [PATCH iwl-next v2 3/3] ice: Implement driver functionality to dump serdes equalizer values

2024-05-10 Thread Anil Samal
To debug link issues in the field, serdes Tx/Rx equalizer values help to determine the health of serdes lane. Extend 'ethtool -d' option to dump serdes Tx/Rx equalizer. The following list of equalizer param is supported a. rx_equalization_pre2 b. rx_equalization_pre1 c. rx_equalization

[Intel-wired-lan] [PATCH RFC iwl-next 00/12] idpf: XDP chapter I: convert Rx to libeth

2024-05-10 Thread Alexander Lobakin
Applies on top of "idpf: don't enable NAPI and interrupts prior to allocating Rx buffers" from Tony's tree. Sent as RFC as we're at the end of the development cycle and several kdocs are messed up. I'll fix them when sending non-RFC after the window opens. XDP for idpf is currently 5 chapters: * c

[Intel-wired-lan] [PATCH RFC iwl-next 01/12] libeth: add cacheline / struct alignment helpers

2024-05-10 Thread Alexander Lobakin
Following the latest netdev trend, i.e. effective and usage-based field cacheline placement, add helpers to group and then assert struct fields by cachelines. For 64-bit with 64-byte cachelines, the assertions are more strict as the size can then be easily predicted. For the rest, just make sure th

[Intel-wired-lan] [PATCH RFC iwl-next 02/12] idpf: stop using macros for accessing queue descriptors

2024-05-10 Thread Alexander Lobakin
In C, we have structures and unions. Casting `void *` via macros is not only error-prone, but also looks confusing and awful in general. In preparation for splitting the queue structs, replace it with a union and direct array dereferences. Signed-off-by: Alexander Lobakin --- drivers/net/etherne

[Intel-wired-lan] [PATCH RFC iwl-next 04/12] idpf: avoid bloating &idpf_q_vector with big %NR_CPUS

2024-05-10 Thread Alexander Lobakin
With CONFIG_MAXSMP, sizeof(cpumask_t) is 1 Kb. The queue vector structure has them embedded, which means 1 additional Kb of not really hotpath data. We have cpumask_var_t, which is either an embedded cpumask or a pointer for allocating it dynamically when it's big. Use it instead of plain cpumasks

[Intel-wired-lan] [PATCH RFC iwl-next 05/12] idpf: strictly assert cachelines of queue and queue vector structures

2024-05-10 Thread Alexander Lobakin
Now that the queue and queue vector structures are separated and layed out optimally, group the fields as read-mostly, read-write, and cold cachelines and add size assertions to make sure new features won't push something out of its place and provoke perf regression. Despite looking innocent, this

intel-wired-lan@osuosl.org

2024-05-10 Thread Alexander Lobakin
It makes no sense to have a second &net_device_ops struct (800 bytes of rodata) with only one difference in .ndo_start_xmit, which can easily be just one `if`. This `if` is a drop in the ocean and you won't see any difference. Define unified idpf_xmit_start(). The preparation for sending is the sam

[Intel-wired-lan] [PATCH RFC iwl-next 07/12] idpf: compile singleq code only under default-n CONFIG_IDPF_SINGLEQ

2024-05-10 Thread Alexander Lobakin
Currently, there's no HW supporting idpf in the singleq model. Still, this dead code is supported by the driver and often times add hotpath branches and redundant cacheline accesses. While it can't currently be removed, add CONFIG_IDPF_SINGLEQ and build the singleq code only when it's enabled manua

[Intel-wired-lan] [PATCH RFC iwl-next 09/12] idpf: remove legacy Page Pool Ethtool stats

2024-05-10 Thread Alexander Lobakin
Page Pool Ethtool stats are deprecated since the Netlink Page Pool interface introduction. idpf receives big changes in Rx buffer management, including &page_pool layout, so keeping these deprecated stats does only harm, not speaking of that CONFIG_IDPF selects CONFIG_PAGE_POOL_STATS unconditionall

[Intel-wired-lan] [PATCH RFC iwl-next 08/12] idpf: reuse libeth's definitions of parsed ptype structures

2024-05-10 Thread Alexander Lobakin
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 libeth's definitions and even helpers. The only difference is that it doesn't use a constant table (libie), rather than one obtained from the device. Rem

[Intel-wired-lan] [PATCH RFC iwl-next 10/12] libeth: support different types of buffers for Rx

2024-05-10 Thread Alexander Lobakin
Unlike previous generations, idpf requires more buffer types for optimal performance. This includes: header buffers, short buffers, and no-overhead buffers (w/o headroom and tailroom, for TCP zerocopy when the header split is enabled). Introduce libeth Rx buffer type and calculate page_pool params

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

2024-05-10 Thread Alexander Lobakin
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 DMA coherent zone is slow on many systems

[Intel-wired-lan] [PATCH RFC iwl-next 12/12] idpf: use libeth Rx buffer management for payload buffer

2024-05-10 Thread Alexander Lobakin
idpf uses Page Pool for data buffers with hardcoded buffer lengths of 4k for "classic" buffers and 2k for "short" ones. This is not flexible and does not ensure optimal memory usage. Why would you need 4k buffers when the MTU is 1500? Use libeth for the data buffers and don't hardcode any buffer si

[Intel-wired-lan] [PATCH RFC iwl-next 03/12] idpf: split &idpf_queue into 4 strictly-typed queue structures

2024-05-10 Thread Alexander Lobakin
Currently, sizeof(struct idpf_queue) is 32 Kb. This is due to the 12-bit hashtable declaration at the end of the queue. This HT is needed only for Tx queues when the flow scheduling mode is enabled. But &idpf_queue is unified for all of the queue types, provoking excessive memory usage. The unified

Re: [Intel-wired-lan] [PATCH RFC iwl-next 08/12] idpf: reuse libeth's definitions of parsed ptype structures

2024-05-10 Thread Mina Almasry
On Fri, May 10, 2024 at 8:30 AM 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 libeth's definitions and even helpers. The only difference is that > it doesn't use a c