[Intel-wired-lan] [PATCH net-next] idpf: Slightly simplify memory management in idpf_add_del_mac_filters()

2024-08-22 Thread Christophe JAILLET
In idpf_add_del_mac_filters(), filters are chunked up into multiple messages to avoid sending a control queue message buffer that is too large. Each chunk has up to IDPF_NUM_FILTERS_PER_MSG entries. So except for the last iteration which can be smaller, space for exactly IDPF_NUM_FILTERS_PER_MSG e

[Intel-wired-lan] [PATCH][next] idpf: make read-only arrays tx_itr and rx_itr static const

2024-08-22 Thread Colin Ian King
Don't populate the read-only arrays tx_itr and rx_itr on the stack at run time, instead make them static const. Signed-off-by: Colin Ian King --- drivers/net/ethernet/intel/idpf/idpf_virtchnl.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/drivers/net/ethernet/intel/idp

Re: [Intel-wired-lan] [PATCH net-next v2 2/2] ice: add callbacks for Embedded SYNC enablement on dpll pins

2024-08-22 Thread Kubalewski, Arkadiusz
>From: Jiri Pirko >Sent: Thursday, August 22, 2024 12:29 PM > >Wed, Aug 21, 2024 at 11:32:18PM CEST, arkadiusz.kubalew...@intel.com wrote: >>Allow the user to get and set configuration of Embedded SYNC feature >>on the ice driver dpll pins. >> >>Reviewed-by: Aleksandr Loktionov >>Signed-off-by: A

Re: [Intel-wired-lan] [PATCH net-next v2 1/2] dpll: add Embedded SYNC feature for a pin

2024-08-22 Thread Kubalewski, Arkadiusz
>From: Jiri Pirko >Sent: Thursday, August 22, 2024 12:22 PM > >Wed, Aug 21, 2024 at 11:32:17PM CEST, arkadiusz.kubalew...@intel.com wrote: >>Implement and document new pin attributes for providing Embedded SYNC >>capabilities to the DPLL subsystem users through a netlink pin-get >>do/dump messages

[Intel-wired-lan] [PATCH net-next v3 2/2] ice: add callbacks for Embedded SYNC enablement on dpll pins

2024-08-22 Thread Arkadiusz Kubalewski
Allow the user to get and set configuration of Embedded SYNC feature on the ice driver dpll pins. Reviewed-by: Aleksandr Loktionov Signed-off-by: Arkadiusz Kubalewski --- v3: - remove redundant extack error - rename esync_freq -> freq drivers/net/ethernet/intel/ice/ice_dpll.c | 223 +++

[Intel-wired-lan] [PATCH net-next v3 1/2] dpll: add Embedded SYNC feature for a pin

2024-08-22 Thread Arkadiusz Kubalewski
Implement and document new pin attributes for providing Embedded SYNC capabilities to the DPLL subsystem users through a netlink pin-get do/dump messages. Allow the user to set Embedded SYNC frequency with pin-set do netlink message. Reviewed-by: Aleksandr Loktionov Signed-off-by: Arkadiusz Kubal

[Intel-wired-lan] [PATCH net-next v3 0/2] Add Embedded SYNC feature for a dpll's pin

2024-08-22 Thread Arkadiusz Kubalewski
Introduce and allow DPLL subsystem users to get/set capabilities of Embedded SYNC on a dpll's pin. Signed-off-by: Arkadiusz Kubalewski Arkadiusz Kubalewski (2): dpll: add Embedded SYNC feature for a pin ice: add callbacks for Embedded SYNC enablement on dpll pins Documentation/driver-api/d

Re: [Intel-wired-lan] [PATCH iwl-net v3 4/6] ice: check ICE_VSI_DOWN under rtnl_lock when preparing for reset

2024-08-22 Thread Larysa Zaremba
On Thu, Aug 22, 2024 at 04:42:44PM +0200, Maciej Fijalkowski wrote: > On Thu, Aug 22, 2024 at 02:56:50PM +0200, Larysa Zaremba wrote: > > On Thu, Aug 22, 2024 at 01:34:33PM +0200, Maciej Fijalkowski wrote: > > > On Mon, Aug 19, 2024 at 12:05:41PM +0200, Larysa Zaremba wrote: > > > > Consider the fo

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

2024-08-22 Thread Alexander Lobakin
From: Wojciech Drewek Date: Wed, 21 Aug 2024 14:15:39 +0200 > From: Jacob Keller > > Add support for receive timestamps to the Rx hotpath. This support only > works when using the flexible descriptor format, so make sure that we > request this format by default if we have receive timestamp supp

Re: [Intel-wired-lan] [PATCH iwl-next v10 13/14] iavf: handle set and get timestamps ops

2024-08-22 Thread Alexander Lobakin
From: Wojciech Drewek Date: Wed, 21 Aug 2024 14:15:38 +0200 > From: Jacob Keller > > Add handlers for the .ndo_hwtstamp_get and .ndo_hwtstamp_set ops which > allow userspace to request timestamp enablement for the device. This > support allows standard Linux applications to request the timestam

Re: [Intel-wired-lan] [PATCH iwl-next v10 12/14] iavf: Implement checking DD desc field

2024-08-22 Thread Alexander Lobakin
From: Wojciech Drewek Date: Wed, 21 Aug 2024 14:15:37 +0200 > From: Mateusz Polchlopek > > Rx timestamping introduced in PF driver caused the need of refactoring > the VF driver mechanism to check packet fields. [...] > +static bool iavf_is_descriptor_done(struct iavf_ring *rx_ring, const >

Re: [Intel-wired-lan] [PATCH iwl-next v10 11/14] iavf: refactor iavf_clean_rx_irq to support legacy and flex descriptors

2024-08-22 Thread Alexander Lobakin
From: Wojciech Drewek Date: Wed, 21 Aug 2024 14:15:36 +0200 > From: Jacob Keller > > Using VIRTCHNL_VF_OFFLOAD_FLEX_DESC, the iAVF driver is capable of > negotiating to enable the advanced flexible descriptor layout. Add the > flexible NIC layout (RXDID=2) as a member of the Rx descriptor union

Re: [Intel-wired-lan] [PATCH iwl-next v10 10/14] iavf: define Rx descriptors as qwords

2024-08-22 Thread Alexander Lobakin
From: Wojciech Drewek Date: Wed, 21 Aug 2024 14:15:35 +0200 > From: Mateusz Polchlopek > > The union iavf_32byte_rx_desc consists of two unnamed structs defined > inside. One of them represents legacy 32 byte descriptor and second the > 16 byte descriptor (extended to 32 byte). Each of them con

Re: [Intel-wired-lan] [PATCH iwl-net v3 4/6] ice: check ICE_VSI_DOWN under rtnl_lock when preparing for reset

2024-08-22 Thread Maciej Fijalkowski
On Thu, Aug 22, 2024 at 02:56:50PM +0200, Larysa Zaremba wrote: > On Thu, Aug 22, 2024 at 01:34:33PM +0200, Maciej Fijalkowski wrote: > > On Mon, Aug 19, 2024 at 12:05:41PM +0200, Larysa Zaremba wrote: > > > Consider the following scenario: > > > > > > .ndo_bpf()| ice_prepare_for_r

Re: [Intel-wired-lan] [iwl-next v1 1/1] igc: Add Energy Efficient Ethernet ability

2024-08-22 Thread Dahan, AvigailX
On 7/13/2024 10:02 PM, Sasha Neftin wrote: According to the IEEE standard report the EEE ability (registers 7.60 and 7.62) and the EEE Link Partner ability (registers 7.61 and 7.63). Use the kernel's 'ethtool_keee' structure and report EEE link modes. Example: ethtool --show-eee Before: Advert

Re: [Intel-wired-lan] [PATCH iwl-net v3 5/6] ice: remove ICE_CFG_BUSY locking from AF_XDP code

2024-08-22 Thread Larysa Zaremba
On Thu, Aug 22, 2024 at 01:43:50PM +0200, Maciej Fijalkowski wrote: > On Mon, Aug 19, 2024 at 12:05:42PM +0200, Larysa Zaremba wrote: > > Locking used in ice_qp_ena() and ice_qp_dis() does pretty much nothing, > > because ICE_CFG_BUSY is a state flag that is supposed to be set in a PF > > state, no

Re: [Intel-wired-lan] [PATCH iwl-net v3 2/6] ice: protect XDP configuration with a mutex

2024-08-22 Thread Larysa Zaremba
On Thu, Aug 22, 2024 at 01:39:15PM +0200, Maciej Fijalkowski wrote: > On Mon, Aug 19, 2024 at 12:05:39PM +0200, Larysa Zaremba wrote: > > The main threat to data consistency in ice_xdp() is a possible asynchronous > > PF reset. It can be triggered by a user or by TX timeout handler. > > > > XDP se

Re: [Intel-wired-lan] [PATCH iwl-net v3] igb: Fix not clearing TimeSync interrupts for 82580

2024-08-22 Thread Pucha, HimasekharX Reddy
> -Original Message- > From: Intel-wired-lan On Behalf Of > Daiwei Li > Sent: Wednesday, August 14, 2024 10:26 AM > To: intel-wired-...@lists.osuosl.org > Cc: Daiwei Li ; Neftin, Sasha ; > Gomes, Vinicius ; net...@vger.kernel.org; > richardcoch...@gmail.com; k...@linutronix.de; linux-ke

Re: [Intel-wired-lan] [PATCH iwl-next v3] i40e: Add Energy Efficient Ethernet ability for X710 Base-T/KR/KX cards

2024-08-22 Thread Pucha, HimasekharX Reddy
> -Original Message- > From: Intel-wired-lan On Behalf Of > Aleksandr Loktionov > Sent: Monday, August 19, 2024 2:58 PM > To: intel-wired-...@lists.osuosl.org; Nguyen, Anthony L > ; Loktionov, Aleksandr > > Cc: net...@vger.kernel.org; Kubalewski, Arkadiusz > > Subject: [Intel-wired-l

Re: [Intel-wired-lan] [PATCH iwl-net v3 4/6] ice: check ICE_VSI_DOWN under rtnl_lock when preparing for reset

2024-08-22 Thread Larysa Zaremba
On Thu, Aug 22, 2024 at 01:34:33PM +0200, Maciej Fijalkowski wrote: > On Mon, Aug 19, 2024 at 12:05:41PM +0200, Larysa Zaremba wrote: > > Consider the following scenario: > > > > .ndo_bpf() | ice_prepare_for_reset() | > > |

Re: [Intel-wired-lan] [PATCH iwl-net] igb: Always call igb_xdp_ring_update_tail() under Tx lock

2024-08-22 Thread Maciej Fijalkowski
*tx_ring = igb_xdp_tx_queue_mapping(adapter); > > + nq = txring_txq(tx_ring); > + __netif_tx_lock(nq, cpu); > igb_xdp_ring_update_tail(tx_ring); > + __netif_tx_unlock(nq); > } > > u64_stats_update_begin(&rx_ring->rx_syncp); > > --- > base-commit: a0b4a80ed6ce2cf8140fe926303ba609884b5d9b > change-id: 20240822-igb_xdp_tx_lock-b6846fddc758 > > Best regards, > -- > Kurt Kanzenbach >

Re: [Intel-wired-lan] [PATCH iwl-net v3 5/6] ice: remove ICE_CFG_BUSY locking from AF_XDP code

2024-08-22 Thread Maciej Fijalkowski
On Mon, Aug 19, 2024 at 12:05:42PM +0200, Larysa Zaremba wrote: > Locking used in ice_qp_ena() and ice_qp_dis() does pretty much nothing, > because ICE_CFG_BUSY is a state flag that is supposed to be set in a PF > state, not VSI one. Therefore it does not protect the queue pair from > e.g. reset. >

Re: [Intel-wired-lan] [PATCH iwl-net v3 2/6] ice: protect XDP configuration with a mutex

2024-08-22 Thread Maciej Fijalkowski
On Mon, Aug 19, 2024 at 12:05:39PM +0200, Larysa Zaremba wrote: > The main threat to data consistency in ice_xdp() is a possible asynchronous > PF reset. It can be triggered by a user or by TX timeout handler. > > XDP setup and PF reset code access the same resources in the following > sections: >

Re: [Intel-wired-lan] [PATCH iwl-net v3 3/6] ice: check for XDP rings instead of bpf program when unconfiguring

2024-08-22 Thread Maciej Fijalkowski
On Mon, Aug 19, 2024 at 12:05:40PM +0200, Larysa Zaremba wrote: > If VSI rebuild is pending, .ndo_bpf() can attach/detach the XDP program on > VSI without applying new ring configuration. When unconfiguring the VSI, we > can encounter the state in which there is an XDP program but no XDP rings > to

Re: [Intel-wired-lan] [PATCH iwl-net v3 6/6] ice: do not bring the VSI up, if it was down before the XDP setup

2024-08-22 Thread Maciej Fijalkowski
On Mon, Aug 19, 2024 at 12:05:43PM +0200, Larysa Zaremba wrote: > After XDP configuration is completed, we bring the interface up > unconditionally, regardless of its state before the call to .ndo_bpf(). > > Preserve the information whether the interface had to be brought down and > later bring it

Re: [Intel-wired-lan] [PATCH iwl-net v3 4/6] ice: check ICE_VSI_DOWN under rtnl_lock when preparing for reset

2024-08-22 Thread Maciej Fijalkowski
On Mon, Aug 19, 2024 at 12:05:41PM +0200, Larysa Zaremba wrote: > Consider the following scenario: > > .ndo_bpf()| ice_prepare_for_reset() | > |___| > rtnl_lock() |

Re: [Intel-wired-lan] [PATCH iwl-net v2 1/1] e1000e: change I219 (19) devices to ADP

2024-08-22 Thread Paul Menzel
Dear Vitaly, Thank you very much for the improved version with the updated commit message. Am 22.08.24 um 12:33 schrieb Vitaly Lifshits: Sporadic issues have been observed on I219 (19) devices. It’d be great if you could be specific and describe them, despite they are fixed with the patch

Re: [Intel-wired-lan] [PATCH iwl-next v3 3/6] devlink: add devlink_fmsg_dump_skb() function

2024-08-22 Thread Simon Horman
On Wed, Aug 21, 2024 at 03:37:11PM +0200, Przemek Kitszel wrote: > From: Mateusz Polchlopek > > Add devlink_fmsg_dump_skb() function that adds some diagnostic > information about skb (like length, pkt type, MAC, etc) to devlink > fmsg mechanism using bunch of devlink_fmsg_put() function calls. >

[Intel-wired-lan] [PATCH iwl-net v2 1/1] e1000e: change I219 (19) devices to ADP

2024-08-22 Thread Vitaly Lifshits
Sporadic issues have been observed on I219 (19) devices. It was found that these devices have hardware more closely related to ADP than MTP and the issues were caused by taking MTP-specific flows. Change the MAC and board types of these devices from MTP to ADP to correctly reflect the LAN hardware

Re: [Intel-wired-lan] [PATCH net-next v2 2/2] ice: add callbacks for Embedded SYNC enablement on dpll pins

2024-08-22 Thread Jiri Pirko
Wed, Aug 21, 2024 at 11:32:18PM CEST, arkadiusz.kubalew...@intel.com wrote: >Allow the user to get and set configuration of Embedded SYNC feature >on the ice driver dpll pins. > >Reviewed-by: Aleksandr Loktionov >Signed-off-by: Arkadiusz Kubalewski >--- >v2: >- align to v2 changes of "dpll: add E

Re: [Intel-wired-lan] ACPI IRQ storm with 6.10

2024-08-22 Thread Petr Valenta
Dne 22. 08. 24 v 11:18 Vitaly Lifshits napsal(a): On 8/22/2024 11:33 AM, Petr Valenta wrote: Dne 22. 08. 24 v 9:44 Petr Valenta napsal(a): Dne 21. 08. 24 v 17:17 Vitaly Lifshits napsal(a): On 8/21/2024 5:59 PM, Bjorn Helgaas wrote: [+to Dima, Vitaly, Hui; beginning of thread at https

Re: [Intel-wired-lan] [PATCH net-next v2 1/2] dpll: add Embedded SYNC feature for a pin

2024-08-22 Thread Jiri Pirko
Wed, Aug 21, 2024 at 11:32:17PM CEST, arkadiusz.kubalew...@intel.com wrote: >Implement and document new pin attributes for providing Embedded SYNC >capabilities to the DPLL subsystem users through a netlink pin-get >do/dump messages. Allow the user to set Embedded SYNC frequency with >pin-set do ne

Re: [Intel-wired-lan] ACPI IRQ storm with 6.10

2024-08-22 Thread Vitaly Lifshits
On 8/22/2024 11:33 AM, Petr Valenta wrote: Dne 22. 08. 24 v 9:44 Petr Valenta napsal(a): Dne 21. 08. 24 v 17:17 Vitaly Lifshits napsal(a): On 8/21/2024 5:59 PM, Bjorn Helgaas wrote: [+to Dima, Vitaly, Hui; beginning of thread at https://lore.kernel.org/r/60ac8988-ace4-4cf0-8c44-028ca741

Re: [Intel-wired-lan] ACPI IRQ storm with 6.10

2024-08-22 Thread Petr Valenta
Dne 22. 08. 24 v 9:44 Petr Valenta napsal(a): Dne 21. 08. 24 v 17:17 Vitaly Lifshits napsal(a): On 8/21/2024 5:59 PM, Bjorn Helgaas wrote: [+to Dima, Vitaly, Hui; beginning of thread at https://lore.kernel.org/r/60ac8988-ace4-4cf0-8c44-028ca741c...@kernel.org] On Wed, Aug 21, 2024 at 01:

Re: [Intel-wired-lan] ACPI IRQ storm with 6.10

2024-08-22 Thread Petr Valenta
Dne 21. 08. 24 v 17:17 Vitaly Lifshits napsal(a): On 8/21/2024 5:59 PM, Bjorn Helgaas wrote: [+to Dima, Vitaly, Hui; beginning of thread at https://lore.kernel.org/r/60ac8988-ace4-4cf0-8c44-028ca741c...@kernel.org] On Wed, Aug 21, 2024 at 01:39:11PM +0200, Petr Valenta wrote: Dne 20. 08. 2

[Intel-wired-lan] [PATCH iwl-net] igb: Always call igb_xdp_ring_update_tail() under Tx lock

2024-08-22 Thread Kurt Kanzenbach
igb_xdp_ring_update_tail(tx_ring); + __netif_tx_unlock(nq); } u64_stats_update_begin(&rx_ring->rx_syncp); --- base-commit: a0b4a80ed6ce2cf8140fe926303ba609884b5d9b change-id: 20240822-igb_xdp_tx_lock-b6846fddc758 Best regards, -- Kurt Kanzenbach