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

2024-01-25 Thread Wojciech Drewek
Recent changes to the devlink reload (commit 9b2348e2d6c9 ("devlink: warn about existing entities during reload-reinit")) force the drivers to destroy devlink ports during reinit. Adjust ice driver to this requirement, unregister netdvice, destroy devlink port. ice_init_eth() was removed and all th

Re: [Intel-wired-lan] [PATCH v7 iwl-next 7/7] ice: stop destroying and reinitalizing Tx tracker during reset

2024-01-25 Thread Pucha, HimasekharX Reddy
> -Original Message- > From: Intel-wired-lan On Behalf Of Karol > Kolacinski > Sent: Tuesday, January 23, 2024 4:22 PM > To: intel-wired-...@lists.osuosl.org > Cc: Keller, Jacob E ; net...@vger.kernel.org; > Kolacinski, Karol ; Nguyen, Anthony L > ; Brandeburg, Jesse > Subject: [Intel

Re: [Intel-wired-lan] [PATCH v7 iwl-next 6/7] ice: factor out ice_ptp_rebuild_owner()

2024-01-25 Thread Pucha, HimasekharX Reddy
> -Original Message- > From: Intel-wired-lan On Behalf Of Karol > Kolacinski > Sent: Tuesday, January 23, 2024 4:22 PM > To: intel-wired-...@lists.osuosl.org > Cc: Keller, Jacob E ; net...@vger.kernel.org; > Kolacinski, Karol ; Nguyen, Anthony L > ; Brandeburg, Jesse > Subject: [Intel-

Re: [Intel-wired-lan] [PATCH v7 iwl-next 5/7] ice: rename ice_ptp_tx_cfg_intr

2024-01-25 Thread Pucha, HimasekharX Reddy
> -Original Message- > From: Intel-wired-lan On Behalf Of Karol > Kolacinski > Sent: Tuesday, January 23, 2024 4:21 PM > To: intel-wired-...@lists.osuosl.org > Cc: Keller, Jacob E ; net...@vger.kernel.org; > Kolacinski, Karol ; Nguyen, Anthony L > ; Brandeburg, Jesse > Subject: [Intel-

Re: [Intel-wired-lan] [PATCH v7 iwl-next 4/7] ice: don't check has_ready_bitmap in E810 functions

2024-01-25 Thread Pucha, HimasekharX Reddy
> -Original Message- > From: Intel-wired-lan On Behalf Of Karol > Kolacinski > Sent: Tuesday, January 23, 2024 4:21 PM > To: intel-wired-...@lists.osuosl.org > Cc: Keller, Jacob E ; net...@vger.kernel.org; > Kolacinski, Karol ; Nguyen, Anthony L > ; Brandeburg, Jesse > Subject: [Intel-

Re: [Intel-wired-lan] [PATCH v7 iwl-next 2/7] ice: pass reset type to PTP reset functions

2024-01-25 Thread Pucha, HimasekharX Reddy
> -Original Message- > From: Intel-wired-lan On Behalf Of Karol > Kolacinski > Sent: Tuesday, January 23, 2024 4:21 PM > To: intel-wired-...@lists.osuosl.org > Cc: Keller, Jacob E ; net...@vger.kernel.org; > Kolacinski, Karol ; Nguyen, Anthony L > ; Brandeburg, Jesse > Subject: [Intel-

Re: [Intel-wired-lan] [PATCH v7 iwl-next 1/7] ice: introduce PTP state machine

2024-01-25 Thread Pucha, HimasekharX Reddy
> -Original Message- > From: Intel-wired-lan On Behalf Of Karol > Kolacinski > Sent: Tuesday, January 23, 2024 4:21 PM > To: intel-wired-...@lists.osuosl.org > Cc: Keller, Jacob E ; net...@vger.kernel.org; > Kolacinski, Karol ; Nguyen, Anthony L > ; Brandeburg, Jesse > Subject: [Intel-

Re: [Intel-wired-lan] [PATCH v7 iwl-next 3/7] ice: rename verify_cached to has_ready_bitmap

2024-01-25 Thread Pucha, HimasekharX Reddy
> -Original Message- > From: Intel-wired-lan On Behalf Of Karol > Kolacinski > Sent: Tuesday, January 23, 2024 4:21 PM > To: intel-wired-...@lists.osuosl.org > Cc: Keller, Jacob E ; net...@vger.kernel.org; > Kolacinski, Karol ; Nguyen, Anthony L > ; Brandeburg, Jesse > Subject: [Intel-

Re: [Intel-wired-lan] [PATCH iwl-next v3 1/3] ixgbe: Convert ret val type from s32 to int

2024-01-25 Thread Jagielski, Jedrzej
From: Nguyen, Anthony L Sent: Monday, January 22, 2024 10:19 PM >On 1/18/2024 5:43 AM, Jedrzej Jagielski wrote: >> Currently big amount of the functions returning standard error codes >> are of type s32. Convert them to regular ints as typdefs here are not >> necessary to return standard error c

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

2024-01-25 Thread Simon Horman
On Tue, Jan 23, 2024 at 12:18:49PM +0100, Wojciech Drewek wrote: > Recent changes to the devlink reload (commit 9b2348e2d6c9 > ("devlink: warn about existing entities during reload-reinit")) > force the drivers to destroy devlink ports during reinit. > Adjust ice driver to this requirement, unregis

Re: [Intel-wired-lan] [PATCH v1 iwl-next] igc: Add support for LEDs on i225/i226

2024-01-25 Thread Simon Horman
+ Dan Carpenter On Thu, Jan 25, 2024 at 08:20:40AM +0100, Kurt Kanzenbach wrote: > On Wed Jan 24 2024, Simon Horman wrote: > > On Wed, Jan 24, 2024 at 09:24:08AM +0100, Kurt Kanzenbach wrote: > > > > ... > > > >> +static int igc_led_hw_control_set(struct led_classdev *led_cdev, > >> +

Re: [Intel-wired-lan] [PATCH v2 iwl-next 1/3] igc: Use reverse xmas tree

2024-01-25 Thread Simon Horman
On Wed, Jan 24, 2024 at 09:55:30AM +0100, Kurt Kanzenbach wrote: > Use reverse xmas tree coding style convention in igc_add_flex_filter(). > > Signed-off-by: Kurt Kanzenbach > Acked-by: Vinicius Costa Gomes Reviewed-by: Simon Horman

Re: [Intel-wired-lan] [PATCH v2 iwl-next 2/3] igc: Use netdev printing functions for flex filters

2024-01-25 Thread Simon Horman
On Wed, Jan 24, 2024 at 09:55:31AM +0100, Kurt Kanzenbach wrote: > All igc filter implementations use netdev_*() printing functions except for > the flex filters. Unify it. > > Signed-off-by: Kurt Kanzenbach > Acked-by: Vinicius Costa Gomes Reviewed-by: Simon Horman

Re: [Intel-wired-lan] [PATCH v2 iwl-next 3/3] igc: Unify filtering rule fields

2024-01-25 Thread Simon Horman
On Wed, Jan 24, 2024 at 09:55:32AM +0100, Kurt Kanzenbach wrote: > All filtering parameters such as EtherType and VLAN TCI are stored in host > byte order except for the VLAN EtherType. Unify it. > > Signed-off-by: Kurt Kanzenbach > Acked-by: Vinicius Costa Gomes Reviewed-by: Simon Horman

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

2024-01-25 Thread Vadim Fedorenko
On 25/01/2024 08:54, Wojciech Drewek wrote: Recent changes to the devlink reload (commit 9b2348e2d6c9 ("devlink: warn about existing entities during reload-reinit")) force the drivers to destroy devlink ports during reinit. Adjust ice driver to this requirement, unregister netdvice, destroy devli

[Intel-wired-lan] [iwl-next v1 0/8] ice: use less resources in switchdev

2024-01-25 Thread Michal Swiatkowski
Hi, Switchdev is using one queue per created port representor. This can quickly lead to Rx queue shortage, as with subfunction support user can create high number of PRs. Save one MSI-X and 'number of PRs' * 1 queues. Refactor switchdev slow-path to use less resources (even no additional resource

[Intel-wired-lan] [iwl-next v1 1/8] ice: remove eswitch changing queues algorithm

2024-01-25 Thread Michal Swiatkowski
Changing queues used by eswitch will be done through PF netdev. There is no need to reserve queues if the number of used queues is known. Reviewed-by: Wojciech Drewek Reviewed-by: Marcin Szycik Signed-off-by: Michal Swiatkowski --- drivers/net/ethernet/intel/ice/ice.h | 6 driver

[Intel-wired-lan] [iwl-next v1 2/8] ice: do Tx through PF netdev in slow-path

2024-01-25 Thread Michal Swiatkowski
Tx can be done using PF netdev. Checks before Tx are unnecessary. Checking if switchdev mode is set seems too defensive (there is no PR netdev in legacy mode). If corresponding VF is disabled or during reset, PR netdev also should be down. Reviewed-by: Marcin Szycik Signed-off-by: Michal Swiatko

[Intel-wired-lan] [iwl-next v1 3/8] ice: default Tx rule instead of to queue

2024-01-25 Thread Michal Swiatkowski
Steer all packets that miss other rules to PF VSI. Previously in switchdev mode, PF VSI received missed packets, but only ones marked as Rx. Now it is receiving all missed packets. To queue rule per PR isn't needed, because we use PF VSI instead of control VSI now, and it's already correctly confi

[Intel-wired-lan] [iwl-next v1 4/8] ice: control default Tx rule in lag

2024-01-25 Thread Michal Swiatkowski
Tx rule in switchdev was changed to use PF instead of additional control plane VSI. Because of that during lag we should control it. Control means to add and remove the default Tx rule during lag active/inactive switching. It can be done the same way as default Rx rule. Reviewed-by: Wojciech Drew

[Intel-wired-lan] [iwl-next v1 5/8] ice: remove switchdev control plane VSI

2024-01-25 Thread Michal Swiatkowski
For slow-path Rx and Tx PF VSI is used. There is no need to have control plane VSI. Remove all code related to it. Eswitch rebuild can't fail without rebuilding control plane VSI. Return void from ice_eswitch_rebuild(). Reviewed-by: Marcin Szycik Signed-off-by: Michal Swiatkowski --- drivers/n

[Intel-wired-lan] [iwl-next v1 6/8] ice: change repr::id values

2024-01-25 Thread Michal Swiatkowski
Instead of getting repr::id from xa_alloc() value, set it to the src_vsi::num_vsi value. It is unique for each PR. Reviewed-by: Przemek Kitszel Reviewed-by: Marcin Szycik Signed-off-by: Michal Swiatkowski --- drivers/net/ethernet/intel/ice/ice_eswitch.c | 5 ++--- drivers/net/ethernet/intel/ic

[Intel-wired-lan] [iwl-next v1 7/8] ice: do switchdev slow-path Rx using PF VSI

2024-01-25 Thread Michal Swiatkowski
Add an ICE_RX_FLAG_MULTIDEV flag to Rx ring. If it is set try to find correct port representor. Do it based on src_vsi value stored in flex descriptor. Ids of representor pointers stored in xarray are equal to corresponding src_vsi value. Thanks to that we can directly get correct representor if w

[Intel-wired-lan] [iwl-next v1 8/8] ice: count representor stats

2024-01-25 Thread Michal Swiatkowski
Removing control plane VSI result in no information about slow-path statistic. In current solution statistics need to be counted in driver. Patch is based on similar implementation done by Simon Horman in nfp: commit eadfa4c3be99 ("nfp: add stats and xmit helpers for representors") Add const modi

[Intel-wired-lan] [PATCH iwl-net v1] ice: fix connection state of DPLL and out pin

2024-01-25 Thread Yochai Hagvi
Fix the connection state between source DPLL and output pin, updating the attribute 'state' of 'parent_device'. Previously, the connection state was broken, and didn't reflect the correct state. When 'state_on_dpll_set' is called with the value 'DPLL_PIN_STATE_CONNECTED' (1), the output pin will s

Re: [Intel-wired-lan] [PATCH v1 iwl-next] igc: Add support for LEDs on i225/i226

2024-01-25 Thread Andrew Lunn
On Thu, Jan 25, 2024 at 08:31:54AM +0100, Kurt Kanzenbach wrote: > On Wed Jan 24 2024, Andrew Lunn wrote: > > On Wed, Jan 24, 2024 at 09:24:08AM +0100, Kurt Kanzenbach wrote: > >> Add support for LEDs on i225/i226. The LEDs can be controlled via sysfs > >> from user space using the netdev trigger.

Re: [Intel-wired-lan] [PATCH 4/6 iwl-next] idpf: refactor remaining virtchnl messages

2024-01-25 Thread Brady, Alan
On 1/24/2024 7:17 PM, Willem de Bruijn wrote: Alan Brady wrote: This takes care of RSS/SRIOV/MAC and other misc virtchnl messages. This again is mostly mechanical. There's some added functionality with MAC filters which makes sure we remove bad filters now that we can better handle asynchronous

Re: [Intel-wired-lan] [PATCH iwl-next] ice: remove duplicate comment

2024-01-25 Thread Tony Nguyen
On 1/24/2024 1:26 PM, Simon Horman wrote: On Mon, Jan 22, 2024 at 10:24:17AM -0800, Tony Nguyen wrote: From: Paul M Stillwell Jr Remove a comment that was not correct; this structure has nothing to do with FW alignment. Reviewed-by: Przemek Kitszel Signed-off-by: Paul M Stillwell Jr Su

Re: [Intel-wired-lan] [PATCH iwl-next v3 1/3] ixgbe: Convert ret val type from s32 to int

2024-01-25 Thread Tony Nguyen
On 1/25/2024 2:16 AM, Jagielski, Jedrzej wrote: From: Nguyen, Anthony L Sent: Monday, January 22, 2024 10:19 PM On 1/18/2024 5:43 AM, Jedrzej Jagielski wrote: Currently big amount of the functions returning standard error codes are of type s32. Convert them to regular ints as typdefs here

[Intel-wired-lan] [PATCH net-next v3] ethtool: ice: Support for RSS settings to GTP from ethtool

2024-01-25 Thread Takeru Hayasaka
This is a patch that enables RSS functionality for GTP packets using ethtool. A user can include her TEID and make RSS work for GTP-U over IPv4 by doing the following: `ethtool -N ens3 rx-flow-hash gtpu4 sde` In addition to gtpu(4|6), we now support gtpc(4|6),gtpc(4|6)t,gtpu(4|6)e, gtpu(4|6)u, and

[Intel-wired-lan] [PATCH net-next v3] ethtool: ice: Support for RSS settings to GTP from ethtool

2024-01-25 Thread Takeru Hayasaka
This is a patch that enables RSS functionality for GTP packets using ethtool. A user can include her TEID and make RSS work for GTP-U over IPv4 by doing the following: `ethtool -N ens3 rx-flow-hash gtpu4 sde` In addition to gtpu(4|6), we now support gtpc(4|6),gtpc(4|6)t,gtpu(4|6)e, gtpu(4|6)u, and

[Intel-wired-lan] [PATCH net-next v3] ethtool: ice: Support for RSS settings to GTP from ethtool

2024-01-25 Thread Takeru Hayasaka
This is a patch that enables RSS functionality for GTP packets using ethtool. A user can include her TEID and make RSS work for GTP-U over IPv4 by doing the following: `ethtool -N ens3 rx-flow-hash gtpu4 sde` In addition to gtpu(4|6), we now support gtpc(4|6),gtpc(4|6)t,gtpu(4|6)e, gtpu(4|6)u, and

[Intel-wired-lan] [PATCH net-next v3] ethtool: ice: Support for RSS settings to GTP from ethtool

2024-01-25 Thread Takeru Hayasaka
This is a patch that enables RSS functionality for GTP packets using ethtool. A user can include her TEID and make RSS work for GTP-U over IPv4 by doing the following: `ethtool -N ens3 rx-flow-hash gtpu4 sde` In addition to gtpu(4|6), we now support gtpc(4|6),gtpc(4|6)t,gtpu(4|6)e, gtpu(4|6)u, and

[Intel-wired-lan] [PATCH net-next v3] ethtool: ice: Support for RSS settings to GTP from ethtool

2024-01-25 Thread Takeru Hayasaka
This is a patch that enables RSS functionality for GTP packets using ethtool. A user can include her TEID and make RSS work for GTP-U over IPv4 by doing the following: `ethtool -N ens3 rx-flow-hash gtpu4 sde` In addition to gtpu(4|6), we now support gtpc(4|6),gtpc(4|6)t,gtpu(4|6)e, gtpu(4|6)u, and

[Intel-wired-lan] [PATCH v2 0/7 iwl-next] idpf: refactor virtchnl messages

2024-01-25 Thread Alan Brady
The motivation for this series has two primary goals. We want to enable support of multiple simultaneous messages and make the channel more robust. The way it works right now, the driver can only send and receive a single message at a time and if something goes really wrong, it can lead to data cor

[Intel-wired-lan] [PATCH v2 1/7 iwl-next] idpf: implement virtchnl transaction manager

2024-01-25 Thread Alan Brady
This starts refactoring how virtchnl messages are handled by adding a transaction manager (idpf_vc_xn_manager). There are two primary motivations here which are to enable handling of multiple messages at once and to make it more robust in general. As it is right now, the driver may only have one p

[Intel-wired-lan] [PATCH v2 2/7 iwl-next] idpf: refactor vport virtchnl messages

2024-01-25 Thread Alan Brady
This reworks the way vport related virtchnl messages work to take advantage of the added transaction API. It is fairly mechanical as, to use the transaction API, the function just needs to fill out an appropriate idpf_vc_xn_params struct to pass to idpf_vc_xn_exec which will take care of the actual

[Intel-wired-lan] [PATCH v2 3/7 iwl-next] idpf: refactor queue related virtchnl messages

2024-01-25 Thread Alan Brady
This reworks queue specific virtchnl messages to use the added transaction API. It is fairly mechanical and generally makes the functions using it more simple. Functions using transaction API no longer need to take the vc_buf_lock since it's not using it anymore. After filling out an idpf_vc_xn_pa

[Intel-wired-lan] [PATCH v2 4/7 iwl-next] idpf: refactor remaining virtchnl messages

2024-01-25 Thread Alan Brady
This takes care of RSS/SRIOV/MAC and other misc virtchnl messages. This again is mostly mechanical. In absence of an async_handler for MAC filters, this will simply generically report any errors from idpf_vc_xn_forward_async. This maintains the existing behavior. Follow up patch will add an async

[Intel-wired-lan] [PATCH v2 5/7 iwl-next] idpf: add async_handler for MAC filter messages

2024-01-25 Thread Alan Brady
There are situations where the driver needs to add a MAC filter but we're explicitly not allowed to sleep so we can wait for a virtchnl message to complete. This adds an async_handler for asynchronously sent virtchnl messages for MAC filters so that we can better handle if there's an error of some

[Intel-wired-lan] [PATCH v2 6/7 iwl-next] idpf: refactor idpf_recv_mb_msg

2024-01-25 Thread Alan Brady
Now that all the messages are using the transaction API, we can rework idpf_recv_mb_msg quite a lot to simplify it. Due to this, we remove idpf_find_vport as no longer used and alter idpf_recv_event_msg slightly. Reviewed-by: Przemek Kitszel Reviewed-by: Igor Bagnucki Signed-off-by: Alan Brady

[Intel-wired-lan] [PATCH v2 7/7 iwl-next] idpf: cleanup virtchnl cruft

2024-01-25 Thread Alan Brady
We can now remove a bunch of gross code we don't need anymore like the vc state bits and vc_buf_lock since everything is using transaction API now. Reviewed-by: Przemek Kitszel Reviewed-by: Igor Bagnucki Signed-off-by: Alan Brady --- drivers/net/ethernet/intel/idpf/idpf.h| 86 -