This series do following:
Patch 1 - Removes write-only flags field from i40e_veb structure and
from i40e_veb_setup() parameters
Patch 2 - Refactors parameter of i40e_notify_client_of_l2_param_changes()
and i40e_notify_client_of_netdev_close()
Patch 3 - Refactors parameter of i40
The field is initialized always to zero and it is never read.
Remove it.
Reviewed-by: Michal Schmidt
Reviewed-by: Aleksandr Loktionov
Reviewed-by: Kalesh AP
Reviewed-by: Tony Nguyen
Tested-by: Pucha Himasekhar Reddy
Signed-off-by: Ivan Vecera
---
drivers/net/ethernet/intel/i40e/i40e.h
Commit 0ef2d5afb12d ("i40e: KISS the client interface") simplified
the client interface so in practice it supports only one client
per i40e netdev. But we have still 2 notification functions that
uses as parameter a pointer to VSI of netdevice associated with
the client. After the mentioned commit
Commit 07d44190a389 ("i40e/i40evf: Detect and recover hung queue
scenario") changes i40e_detect_recover_hung() argument type from
i40e_pf* to i40e_vsi* to be shareable by both i40e and i40evf.
Because the i40evf does not exist anymore and the function is
exclusively used by i40e we can revert this
Add simple helper i40e_pf_get_main_vsi(pf) to access main VSI
that replaces pattern 'pf->vsi[pf->lan_vsi]'
Reviewed-by: Michal Schmidt
Reviewed-by: Tony Nguyen
Tested-by: Pucha Himasekhar Reddy
Signed-off-by: Ivan Vecera
---
drivers/net/ethernet/intel/i40e/i40e.h| 11 ++
drivers/net/
In the driver code there are 3 types of checks whether given
VSI is main or not:
1. vsi->type ==/!= I40E_VSI_MAIN
2. vsi ==/!= pf->vsi[pf->lan_vsi]
3. vsi->seid ==/!= pf->vsi[pf->lan_vsi]->seid
All of them are equivalent and can be consolidated. Convert cases
2 and 3 to case 1.
Reviewed-by: Micha
Add helper i40e_vsi_reconfig_tc(vsi) that configures TC
for given VSI using previously stored TC bitmap.
Effectively replaces open-coded patterns:
enabled_tc = vsi->tc_config.enabled_tc;
vsi->tc_config.enabled_tc = 0;
i40e_vsi_config_tc(vsi, enabled_tc);
Reviewed-by: Michal Schmidt
Reviewed-by:
Add a helper to access main VEB:
i40e_pf_get_main_veb(pf) replaces 'pf->veb[pf->lan_veb]'
Reviewed-by: Michal Schmidt
Reviewed-by: Aleksandr Loktionov
Reviewed-by: Tony Nguyen
Tested-by: Pucha Himasekhar Reddy
Signed-off-by: Ivan Vecera
---
drivers/net/ethernet/intel/i40e/i40e.h| 11
On Sat, Apr 27, 2024 at 10:54:23AM +, Artem S. Tashkinov wrote:
> Hello,
>
> This fix is still not queued in 6.8 stable:
What fix?
> https://git.kernel.org/pub/scm/linux/kernel/git/stable/stable-queue.git/tree/queue-6.8
>
> Why?
What is the git id of the change in Linus's tree?
thanks,
g
On Sat, Apr 27, 2024 at 09:26:02AM +0200, Ivan Vecera wrote:
> The field is initialized always to zero and it is never read.
> Remove it.
>
> Reviewed-by: Michal Schmidt
> Reviewed-by: Aleksandr Loktionov
> Reviewed-by: Kalesh AP
> Reviewed-by: Tony Nguyen
> Tested-by: Pucha Himasekhar Reddy
On Sat, Apr 27, 2024 at 09:26:03AM +0200, Ivan Vecera wrote:
> Commit 0ef2d5afb12d ("i40e: KISS the client interface") simplified
> the client interface so in practice it supports only one client
> per i40e netdev. But we have still 2 notification functions that
> uses as parameter a pointer to VSI
On Sat, Apr 27, 2024 at 09:26:04AM +0200, Ivan Vecera wrote:
> Commit 07d44190a389 ("i40e/i40evf: Detect and recover hung queue
> scenario") changes i40e_detect_recover_hung() argument type from
> i40e_pf* to i40e_vsi* to be shareable by both i40e and i40evf.
> Because the i40evf does not exist any
On Sat, Apr 27, 2024 at 09:26:06AM +0200, Ivan Vecera wrote:
> In the driver code there are 3 types of checks whether given
> VSI is main or not:
> 1. vsi->type ==/!= I40E_VSI_MAIN
> 2. vsi ==/!= pf->vsi[pf->lan_vsi]
> 3. vsi->seid ==/!= pf->vsi[pf->lan_vsi]->seid
>
> All of them are equivalent an
On Sat, Apr 27, 2024 at 09:26:05AM +0200, Ivan Vecera wrote:
> Add simple helper i40e_pf_get_main_vsi(pf) to access main VSI
> that replaces pattern 'pf->vsi[pf->lan_vsi]'
>
> Reviewed-by: Michal Schmidt
> Reviewed-by: Tony Nguyen
> Tested-by: Pucha Himasekhar Reddy
> Signed-off-by: Ivan Vecera
On Sat, Apr 27, 2024 at 09:26:07AM +0200, Ivan Vecera wrote:
> Add a helper to access main VEB:
>
> i40e_pf_get_main_veb(pf) replaces 'pf->veb[pf->lan_veb]'
>
> Reviewed-by: Michal Schmidt
> Reviewed-by: Aleksandr Loktionov
> Reviewed-by: Tony Nguyen
> Tested-by: Pucha Himasekhar Reddy
> Sign
On Sat, Apr 27, 2024 at 09:26:08AM +0200, Ivan Vecera wrote:
> Add helper i40e_vsi_reconfig_tc(vsi) that configures TC
> for given VSI using previously stored TC bitmap.
>
> Effectively replaces open-coded patterns:
>
> enabled_tc = vsi->tc_config.enabled_tc;
> vsi->tc_config.enabled_tc = 0;
> i4
allnoconfig gcc
arc allyesconfig gcc
arc defconfig gcc
arc randconfig-001-20240427 gcc
arc randconfig-002-20240427 gcc
arm allmodconfig gcc
allnoconfig gcc
arc allyesconfig gcc
arc defconfig gcc
arc randconfig-001-20240427 gcc
arc randconfig-002-20240427 gcc
arm
allnoconfig gcc
arc allyesconfig gcc
arc defconfig gcc
arc randconfig-001-20240427 gcc
arc randconfig-002-20240427 gcc
arm
nfig gcc
arc allnoconfig gcc
arc allyesconfig gcc
arc defconfig gcc
arc randconfig-001-20240427 gcc
arc randconfig-002-20240427 gcc
On 23/04/2024 17:40, Ilpo Järvinen wrote:
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/et
21 matches
Mail list logo
