[PATCH iwl-net] i40e: fix Jumbo Frame support after iPXE boot

[PATCH iwl-net] i40e: fix Jumbo Frame support after iPXE boot

[Jacob Keller]

The i40e hardware has multiple hardware settings which define the maximum
frame size of the physical port. The firmware has an AdminQ command
(0x0603) to configure all of these settings, but the i40e Linux driver
never issues this command.

In most cases this is no problem, as the NVM default value is to set it to
its maximum value of 9728. Unfortunately, since recent versions the intelxl
driver maintained within the iPXE network boot stack now issues the 0x0603
command to set the maximum frame size to a low value. This appears to have
occurred because the same intelxl driver is used for both the E700 and E800
series hardware, and both devices support the same 0x0603 AdminQ command.
The ice Linux PF driver already issues this command during probe.

Since commit 3a2c6ced90e1 ("i40e: Add a check to see if MFS is set"), the
driver does check the I40E_PRTGL_SAH register, but it only logs a warning.
This register also only covers received packets and not transmitted
packets. Additionally, a warning does not help users, as the larger MTU is
still not supported.

Instead, have the i40e driver issue the Set MAC Config AdminQ command
during boot in a similar fashion to the ice driver. Additionally, instead
of just checking I40E_PRTGL_SAH, read and update its Max Frame Size field
to the expected 9K value as well.

This ensures the driver restores the maximum frame size to its expected
value at probe, rather than assuming that no other driver has adjusted the
MAC config.

This is a better user experience, as we now fix the issues with larger MTU
instead of merely warning. It also aligns with the way the ice E800 series
driver works.

Fixes: 3a2c6ced90e1 ("i40e: Add a check to see if MFS is set")
Signed-off-by: Jacob Keller <jacob.e.keller@intel.com>
---
Strictly speaking, the "Fixes" tag isn't entirely accurate. The failure is
really the result of changes in the iPXE driver to support both E700 and E800
within the same intelxl driver. However, I think the warning added by that
commit was an insufficient solution and we should be restoring the value to
its expected default rather than merely issuing a warning to the kernel
log. Thus, this "fixes" the driver to better handle this case.
---
 drivers/net/ethernet/intel/i40e/i40e_prototype.h |  2 ++
 drivers/net/ethernet/intel/i40e/i40e_common.c    | 30 ++++++++++++++++++++++++
 drivers/net/ethernet/intel/i40e/i40e_main.c      | 17 +++++++++-----
 3 files changed, 43 insertions(+), 6 deletions(-)

diff --git a/drivers/net/ethernet/intel/i40e/i40e_prototype.h b/drivers/net/ethernet/intel/i40e/i40e_prototype.h
index aef5de53ce3b..26bb7bffe361 100644
--- a/drivers/net/ethernet/intel/i40e/i40e_prototype.h
+++ b/drivers/net/ethernet/intel/i40e/i40e_prototype.h
@@ -98,6 +98,8 @@ int i40e_aq_set_mac_loopback(struct i40e_hw *hw,
 			     struct i40e_asq_cmd_details *cmd_details);
 int i40e_aq_set_phy_int_mask(struct i40e_hw *hw, u16 mask,
 			     struct i40e_asq_cmd_details *cmd_details);
+int i40e_aq_set_mac_config(struct i40e_hw *hw, u16 max_frame_size,
+			   struct i40e_asq_cmd_details *cmd_details);
 int i40e_aq_clear_pxe_mode(struct i40e_hw *hw,
 			   struct i40e_asq_cmd_details *cmd_details);
 int i40e_aq_set_link_restart_an(struct i40e_hw *hw,
diff --git a/drivers/net/ethernet/intel/i40e/i40e_common.c b/drivers/net/ethernet/intel/i40e/i40e_common.c
index 270e7e8cf9cf..f6b6a4925b27 100644
--- a/drivers/net/ethernet/intel/i40e/i40e_common.c
+++ b/drivers/net/ethernet/intel/i40e/i40e_common.c
@@ -1189,6 +1189,36 @@ int i40e_set_fc(struct i40e_hw *hw, u8 *aq_failures,
 	return status;
 }
 
+/**
+ * i40e_aq_set_mac_config
+ * @hw: pointer to the hw struct
+ * @max_frame_size: Maximum Frame Size to be supported by the port
+ * @cmd_details: pointer to command details structure or NULL
+ *
+ * Configure MAC settings for frame size (0x0603).
+ *
+ * Return: 0 on success, or a negative error code on failure.
+ **/
+int i40e_aq_set_mac_config(struct i40e_hw *hw, u16 max_frame_size,
+			   struct i40e_asq_cmd_details *cmd_details)
+{
+	struct i40e_aq_set_mac_config *cmd;
+	struct libie_aq_desc desc;
+
+	if (max_frame_size == 0)
+		return -EINVAL;
+
+	i40e_fill_default_direct_cmd_desc(&desc, i40e_aqc_opc_set_mac_config);
+
+	cmd->max_frame_size = cpu_to_le16(max_frame_size);
+
+#define I40E_AQ_SET_MAC_CONFIG_FC_DEFAULT_THRESHOLD	0x7FFF
+	cmd->fc_refresh_threshold =
+		cpu_to_le16(I40E_AQ_SET_MAC_CONFIG_FC_DEFAULT_THRESHOLD);
+
+	return i40e_asq_send_command(hw, &desc, NULL, 0, cmd_details);
+}
+
 /**
  * i40e_aq_clear_pxe_mode
  * @hw: pointer to the hw struct
diff --git a/drivers/net/ethernet/intel/i40e/i40e_main.c b/drivers/net/ethernet/intel/i40e/i40e_main.c
index b83f823e4917..4796fdd0b966 100644
--- a/drivers/net/ethernet/intel/i40e/i40e_main.c
+++ b/drivers/net/ethernet/intel/i40e/i40e_main.c
@@ -16045,13 +16045,18 @@ static int i40e_probe(struct pci_dev *pdev, const struct pci_device_id *ent)
 		dev_dbg(&pf->pdev->dev, "get supported phy types ret =  %pe last_status =  %s\n",
 			ERR_PTR(err), libie_aq_str(pf->hw.aq.asq_last_status));
 
-	/* make sure the MFS hasn't been set lower than the default */
 #define MAX_FRAME_SIZE_DEFAULT 0x2600
-	val = FIELD_GET(I40E_PRTGL_SAH_MFS_MASK,
-			rd32(&pf->hw, I40E_PRTGL_SAH));
-	if (val < MAX_FRAME_SIZE_DEFAULT)
-		dev_warn(&pdev->dev, "MFS for port %x (%d) has been set below the default (%d)\n",
-			 pf->hw.port, val, MAX_FRAME_SIZE_DEFAULT);
+
+	err = i40e_aq_set_mac_config(hw, MAX_FRAME_SIZE_DEFAULT, NULL);
+	if (err) {
+		dev_warn(&pdev->dev, "set mac config ret =  %pe last_status =  %s\n",
+			 ERR_PTR(err), libie_aq_str(pf->hw.aq.asq_last_status));
+	}
+
+	/* Make sure the MFS is set to the expected value */
+	val = rd32(hw, I40E_PRTGL_SAH);
+	FIELD_MODIFY(I40E_PRTGL_SAH_MFS_MASK, &val, MAX_FRAME_SIZE_DEFAULT);
+	wr32(hw, I40E_PRTGL_SAH, val);
 
 	/* Add a filter to drop all Flow control frames from any VSI from being
 	 * transmitted. By doing so we stop a malicious VF from sending out

---
base-commit: d7e82594a45c5cb270940ac469846e8026c7db0f
change-id: 20250813-jk-fix-i40e-ice-pxe-9k-mtu-2b6d03621cd9

Best regards,
--  
Jacob Keller <jacob.e.keller@intel.com>

Re: [Intel-wired-lan] [PATCH iwl-net] i40e: fix Jumbo Frame support after iPXE boot

[Paul Menzel]

Dear Jacob,


Thank you for your patch.

Am 14.08.25 um 23:21 schrieb Jacob Keller:
> The i40e hardware has multiple hardware settings which define the maximum
> frame size of the physical port. The firmware has an AdminQ command
> (0x0603) to configure all of these settings, but the i40e Linux driver
> never issues this command.
> 
> In most cases this is no problem, as the NVM default value is to set it to
> its maximum value of 9728. Unfortunately, since recent versions the intelxl
> driver maintained within the iPXE network boot stack now issues the 0x0603
> command to set the maximum frame size to a low value. This appears to have

Maybe add (MFS) so the abbreviation is clear.

> occurred because the same intelxl driver is used for both the E700 and E800
> series hardware, and both devices support the same 0x0603 AdminQ command.

Do you have a link to the intelxl change?

> The ice Linux PF driver already issues this command during probe.
> 
> Since commit 3a2c6ced90e1 ("i40e: Add a check to see if MFS is set"), the
> driver does check the I40E_PRTGL_SAH register, but it only logs a warning.

… if MFS is set lower than the default.

> This register also only covers received packets and not transmitted
> packets. Additionally, a warning does not help users, as the larger MTU is
> still not supported.
> 
> Instead, have the i40e driver issue the Set MAC Config AdminQ command
> during boot in a similar fashion to the ice driver. Additionally, instead
> of just checking I40E_PRTGL_SAH, read and update its Max Frame Size field
> to the expected 9K value as well.
> 
> This ensures the driver restores the maximum frame size to its expected
> value at probe, rather than assuming that no other driver has adjusted the
> MAC config.
> 
> This is a better user experience, as we now fix the issues with larger MTU
> instead of merely warning. It also aligns with the way the ice E800 series
> driver works.

Is there a regression potential, that users won’t be able to access 
their systems over the network, because there are faulty switches or such?

To save people search for it, how can the MFS be read out from the 
command line?

> Fixes: 3a2c6ced90e1 ("i40e: Add a check to see if MFS is set")
> Signed-off-by: Jacob Keller <jacob.e.keller@intel.com>
> ---
> Strictly speaking, the "Fixes" tag isn't entirely accurate. The failure is
> really the result of changes in the iPXE driver to support both E700 and E800
> within the same intelxl driver. However, I think the warning added by that
> commit was an insufficient solution and we should be restoring the value to
> its expected default rather than merely issuing a warning to the kernel
> log. Thus, this "fixes" the driver to better handle this case.
> ---
>   drivers/net/ethernet/intel/i40e/i40e_prototype.h |  2 ++
>   drivers/net/ethernet/intel/i40e/i40e_common.c    | 30 ++++++++++++++++++++++++
>   drivers/net/ethernet/intel/i40e/i40e_main.c      | 17 +++++++++-----
>   3 files changed, 43 insertions(+), 6 deletions(-)
> 
> diff --git a/drivers/net/ethernet/intel/i40e/i40e_prototype.h b/drivers/net/ethernet/intel/i40e/i40e_prototype.h
> index aef5de53ce3b..26bb7bffe361 100644
> --- a/drivers/net/ethernet/intel/i40e/i40e_prototype.h
> +++ b/drivers/net/ethernet/intel/i40e/i40e_prototype.h
> @@ -98,6 +98,8 @@ int i40e_aq_set_mac_loopback(struct i40e_hw *hw,
>   			     struct i40e_asq_cmd_details *cmd_details);
>   int i40e_aq_set_phy_int_mask(struct i40e_hw *hw, u16 mask,
>   			     struct i40e_asq_cmd_details *cmd_details);
> +int i40e_aq_set_mac_config(struct i40e_hw *hw, u16 max_frame_size,
> +			   struct i40e_asq_cmd_details *cmd_details);
>   int i40e_aq_clear_pxe_mode(struct i40e_hw *hw,
>   			   struct i40e_asq_cmd_details *cmd_details);
>   int i40e_aq_set_link_restart_an(struct i40e_hw *hw,
> diff --git a/drivers/net/ethernet/intel/i40e/i40e_common.c b/drivers/net/ethernet/intel/i40e/i40e_common.c
> index 270e7e8cf9cf..f6b6a4925b27 100644
> --- a/drivers/net/ethernet/intel/i40e/i40e_common.c
> +++ b/drivers/net/ethernet/intel/i40e/i40e_common.c
> @@ -1189,6 +1189,36 @@ int i40e_set_fc(struct i40e_hw *hw, u8 *aq_failures,
>   	return status;
>   }
>   
> +/**
> + * i40e_aq_set_mac_config
> + * @hw: pointer to the hw struct
> + * @max_frame_size: Maximum Frame Size to be supported by the port

Mention that it needs to be positive (or non-0)?

> + * @cmd_details: pointer to command details structure or NULL
> + *
> + * Configure MAC settings for frame size (0x0603).
> + *
> + * Return: 0 on success, or a negative error code on failure.
> + **/
> +int i40e_aq_set_mac_config(struct i40e_hw *hw, u16 max_frame_size,
> +			   struct i40e_asq_cmd_details *cmd_details)
> +{
> +	struct i40e_aq_set_mac_config *cmd;
> +	struct libie_aq_desc desc;
> +
> +	if (max_frame_size == 0)
> +		return -EINVAL;
> +
> +	i40e_fill_default_direct_cmd_desc(&desc, i40e_aqc_opc_set_mac_config);
> +
> +	cmd->max_frame_size = cpu_to_le16(max_frame_size);
> +
> +#define I40E_AQ_SET_MAC_CONFIG_FC_DEFAULT_THRESHOLD	0x7FFF
> +	cmd->fc_refresh_threshold =
> +		cpu_to_le16(I40E_AQ_SET_MAC_CONFIG_FC_DEFAULT_THRESHOLD);
> +
> +	return i40e_asq_send_command(hw, &desc, NULL, 0, cmd_details);
> +}
> +
>   /**
>    * i40e_aq_clear_pxe_mode
>    * @hw: pointer to the hw struct
> diff --git a/drivers/net/ethernet/intel/i40e/i40e_main.c b/drivers/net/ethernet/intel/i40e/i40e_main.c
> index b83f823e4917..4796fdd0b966 100644
> --- a/drivers/net/ethernet/intel/i40e/i40e_main.c
> +++ b/drivers/net/ethernet/intel/i40e/i40e_main.c
> @@ -16045,13 +16045,18 @@ static int i40e_probe(struct pci_dev *pdev, const struct pci_device_id *ent)
>   		dev_dbg(&pf->pdev->dev, "get supported phy types ret =  %pe last_status =  %s\n",
>   			ERR_PTR(err), libie_aq_str(pf->hw.aq.asq_last_status));
>   
> -	/* make sure the MFS hasn't been set lower than the default */
>   #define MAX_FRAME_SIZE_DEFAULT 0x2600
> -	val = FIELD_GET(I40E_PRTGL_SAH_MFS_MASK,
> -			rd32(&pf->hw, I40E_PRTGL_SAH));
> -	if (val < MAX_FRAME_SIZE_DEFAULT)
> -		dev_warn(&pdev->dev, "MFS for port %x (%d) has been set below the default (%d)\n",
> -			 pf->hw.port, val, MAX_FRAME_SIZE_DEFAULT);
> +
> +	err = i40e_aq_set_mac_config(hw, MAX_FRAME_SIZE_DEFAULT, NULL);
> +	if (err) {
> +		dev_warn(&pdev->dev, "set mac config ret =  %pe last_status =  %s\n",
> +			 ERR_PTR(err), libie_aq_str(pf->hw.aq.asq_last_status));
> +	}
> +
> +	/* Make sure the MFS is set to the expected value */
> +	val = rd32(hw, I40E_PRTGL_SAH);
> +	FIELD_MODIFY(I40E_PRTGL_SAH_MFS_MASK, &val, MAX_FRAME_SIZE_DEFAULT);
> +	wr32(hw, I40E_PRTGL_SAH, val);
>   
>   	/* Add a filter to drop all Flow control frames from any VSI from being
>   	 * transmitted. By doing so we stop a malicious VF from sending out
The diff looks good:

Reviewed-by: Paul Menzel <pmenzel@molgen.mpg.de>


Kind regards,

Paul

Re: [Intel-wired-lan] [PATCH iwl-net] i40e: fix Jumbo Frame support after iPXE boot

[Jacob Keller]

[-- Attachment #1.1: Type: text/plain, Size: 9622 bytes --]



On 8/15/2025 5:07 AM, Paul Menzel wrote:
> Dear Jacob,
> 
> 
> Thank you for your patch.
> 
> Am 14.08.25 um 23:21 schrieb Jacob Keller:
>> The i40e hardware has multiple hardware settings which define the maximum
>> frame size of the physical port. The firmware has an AdminQ command
>> (0x0603) to configure all of these settings, but the i40e Linux driver
>> never issues this command.
>>
>> In most cases this is no problem, as the NVM default value is to set it to
>> its maximum value of 9728. Unfortunately, since recent versions the intelxl
>> driver maintained within the iPXE network boot stack now issues the 0x0603
>> command to set the maximum frame size to a low value. This appears to have
> 
> Maybe add (MFS) so the abbreviation is clear.
> 

Can fix if there is a substantial need for a v2.

>> occurred because the same intelxl driver is used for both the E700 and E800
>> series hardware, and both devices support the same 0x0603 AdminQ command.
> 
> Do you have a link to the intelxl change?
> 

It comes as part of commit 6871a7de705b ("[intelxl] Use admin queue to
set port MAC address and maximum frame size") which can be viewed here:

https://github.com/ipxe/ipxe/commit/6871a7de705b6f6a4046f0d19da9bcd689c3bc8e

This is a prerequisite of the commit to add 100G support:

cad1cc6b449b ("[intelxl] Add driver for Intel 100 Gigabit Ethernet NICs")

https://github.com/ipxe/ipxe/commit/cad1cc6b449b63415ffdad8e12f13df4256106fb

Strictly speaking, the 100G support is part of a separate driver, but
they do share substantial code with the 40G intelxl driver, including
the AQ command to set the maximum frame size.

The problem is that the addition of this AQ command did not play well
with the standard i40e drivers such as the one in Linux.


I will update the commit message to better clarify these details.

>> The ice Linux PF driver already issues this command during probe.
>>
>> Since commit 3a2c6ced90e1 ("i40e: Add a check to see if MFS is set"), the
>> driver does check the I40E_PRTGL_SAH register, but it only logs a warning.
> 
> … if MFS is set lower than the default.
> 

True, this could be spelled out in full.

>> This register also only covers received packets and not transmitted
>> packets. Additionally, a warning does not help users, as the larger MTU is
>> still not supported.
>>
>> Instead, have the i40e driver issue the Set MAC Config AdminQ command
>> during boot in a similar fashion to the ice driver. Additionally, instead
>> of just checking I40E_PRTGL_SAH, read and update its Max Frame Size field
>> to the expected 9K value as well.
>>
>> This ensures the driver restores the maximum frame size to its expected
>> value at probe, rather than assuming that no other driver has adjusted the
>> MAC config.
>>
>> This is a better user experience, as we now fix the issues with larger MTU
>> instead of merely warning. It also aligns with the way the ice E800 series
>> driver works.
> 
> Is there a regression potential, that users won’t be able to access 
> their systems over the network, because there are faulty switches or such?
> 

Yes, but only for situations where larger frame sizes are used. I do not
know exactly what the ipxe driver sets the maximum frame size to, but it
is lower than the default 9728 for 9K Jumbo Frames.

> To save people search for it, how can the MFS be read out from the 
> command line?

As far as I know, you can't. We can read the PRTGL_SAH register in i40e,
but I don't think anything actually reports this. The i40e driver
previously expected this to just always be set to 9728. We limit the
size of transmission with other per-VSI settings such as setting the
VSI's maximum allowed sizes. The MFS is a port global value which is
independent of the VSI settings.

Worse, the PRTGL_SAH register is only one of the things impacted by the
AQ command. Previously the intelxl driver set the PRTGL_SAH register,
but did not send the AQ command. I believe the warning Todd introduced
was intended to at least catch this with the warning message.

However, with the addition of the AQ command, other device configuration
is set (Unfortunately, I don't fully understand what all gets changed by
the firmware...), and as a result changing PRTGL_SAH on its own is not
sufficient to restore functionality.

Thus, I believe the best fix is to do the same as what ice does for
E800: issue this AQ command to set it to the maximum 9K value.

> 
>> Fixes: 3a2c6ced90e1 ("i40e: Add a check to see if MFS is set")
>> Signed-off-by: Jacob Keller <jacob.e.keller@intel.com>
>> ---
>> Strictly speaking, the "Fixes" tag isn't entirely accurate. The failure is
>> really the result of changes in the iPXE driver to support both E700 and E800
>> within the same intelxl driver. However, I think the warning added by that
>> commit was an insufficient solution and we should be restoring the value to
>> its expected default rather than merely issuing a warning to the kernel
>> log. Thus, this "fixes" the driver to better handle this case.
>> ---
>>   drivers/net/ethernet/intel/i40e/i40e_prototype.h |  2 ++
>>   drivers/net/ethernet/intel/i40e/i40e_common.c    | 30 ++++++++++++++++++++++++
>>   drivers/net/ethernet/intel/i40e/i40e_main.c      | 17 +++++++++-----
>>   3 files changed, 43 insertions(+), 6 deletions(-)
>>
>> diff --git a/drivers/net/ethernet/intel/i40e/i40e_prototype.h b/drivers/net/ethernet/intel/i40e/i40e_prototype.h
>> index aef5de53ce3b..26bb7bffe361 100644
>> --- a/drivers/net/ethernet/intel/i40e/i40e_prototype.h
>> +++ b/drivers/net/ethernet/intel/i40e/i40e_prototype.h
>> @@ -98,6 +98,8 @@ int i40e_aq_set_mac_loopback(struct i40e_hw *hw,
>>   			     struct i40e_asq_cmd_details *cmd_details);
>>   int i40e_aq_set_phy_int_mask(struct i40e_hw *hw, u16 mask,
>>   			     struct i40e_asq_cmd_details *cmd_details);
>> +int i40e_aq_set_mac_config(struct i40e_hw *hw, u16 max_frame_size,
>> +			   struct i40e_asq_cmd_details *cmd_details);
>>   int i40e_aq_clear_pxe_mode(struct i40e_hw *hw,
>>   			   struct i40e_asq_cmd_details *cmd_details);
>>   int i40e_aq_set_link_restart_an(struct i40e_hw *hw,
>> diff --git a/drivers/net/ethernet/intel/i40e/i40e_common.c b/drivers/net/ethernet/intel/i40e/i40e_common.c
>> index 270e7e8cf9cf..f6b6a4925b27 100644
>> --- a/drivers/net/ethernet/intel/i40e/i40e_common.c
>> +++ b/drivers/net/ethernet/intel/i40e/i40e_common.c
>> @@ -1189,6 +1189,36 @@ int i40e_set_fc(struct i40e_hw *hw, u8 *aq_failures,
>>   	return status;
>>   }
>>   
>> +/**
>> + * i40e_aq_set_mac_config
>> + * @hw: pointer to the hw struct
>> + * @max_frame_size: Maximum Frame Size to be supported by the port
> 
> Mention that it needs to be positive (or non-0)?
> 
>> + * @cmd_details: pointer to command details structure or NULL
>> + *
>> + * Configure MAC settings for frame size (0x0603).
>> + *
>> + * Return: 0 on success, or a negative error code on failure.
>> + **/
>> +int i40e_aq_set_mac_config(struct i40e_hw *hw, u16 max_frame_size,
>> +			   struct i40e_asq_cmd_details *cmd_details)
>> +{
>> +	struct i40e_aq_set_mac_config *cmd;
>> +	struct libie_aq_desc desc;
>> +
>> +	if (max_frame_size == 0)
>> +		return -EINVAL;
>> +
>> +	i40e_fill_default_direct_cmd_desc(&desc, i40e_aqc_opc_set_mac_config);
>> +
>> +	cmd->max_frame_size = cpu_to_le16(max_frame_size);

One of the bots complained about possible uninitialized access here, so
I'll need to check what I did wrong. I think its because of the libie AQ
conversion.
>> +
>> +#define I40E_AQ_SET_MAC_CONFIG_FC_DEFAULT_THRESHOLD	0x7FFF
>> +	cmd->fc_refresh_threshold =
>> +		cpu_to_le16(I40E_AQ_SET_MAC_CONFIG_FC_DEFAULT_THRESHOLD);
>> +
>> +	return i40e_asq_send_command(hw, &desc, NULL, 0, cmd_details);
>> +}
>> +
>>   /**
>>    * i40e_aq_clear_pxe_mode
>>    * @hw: pointer to the hw struct
>> diff --git a/drivers/net/ethernet/intel/i40e/i40e_main.c b/drivers/net/ethernet/intel/i40e/i40e_main.c
>> index b83f823e4917..4796fdd0b966 100644
>> --- a/drivers/net/ethernet/intel/i40e/i40e_main.c
>> +++ b/drivers/net/ethernet/intel/i40e/i40e_main.c
>> @@ -16045,13 +16045,18 @@ static int i40e_probe(struct pci_dev *pdev, const struct pci_device_id *ent)
>>   		dev_dbg(&pf->pdev->dev, "get supported phy types ret =  %pe last_status =  %s\n",
>>   			ERR_PTR(err), libie_aq_str(pf->hw.aq.asq_last_status));
>>   
>> -	/* make sure the MFS hasn't been set lower than the default */
>>   #define MAX_FRAME_SIZE_DEFAULT 0x2600
>> -	val = FIELD_GET(I40E_PRTGL_SAH_MFS_MASK,
>> -			rd32(&pf->hw, I40E_PRTGL_SAH));
>> -	if (val < MAX_FRAME_SIZE_DEFAULT)
>> -		dev_warn(&pdev->dev, "MFS for port %x (%d) has been set below the default (%d)\n",
>> -			 pf->hw.port, val, MAX_FRAME_SIZE_DEFAULT);
>> +
>> +	err = i40e_aq_set_mac_config(hw, MAX_FRAME_SIZE_DEFAULT, NULL);
>> +	if (err) {
>> +		dev_warn(&pdev->dev, "set mac config ret =  %pe last_status =  %s\n",
>> +			 ERR_PTR(err), libie_aq_str(pf->hw.aq.asq_last_status));
>> +	}
>> +
>> +	/* Make sure the MFS is set to the expected value */
>> +	val = rd32(hw, I40E_PRTGL_SAH);
>> +	FIELD_MODIFY(I40E_PRTGL_SAH_MFS_MASK, &val, MAX_FRAME_SIZE_DEFAULT);
>> +	wr32(hw, I40E_PRTGL_SAH, val);
>>   
>>   	/* Add a filter to drop all Flow control frames from any VSI from being
>>   	 * transmitted. By doing so we stop a malicious VF from sending out
> The diff looks good:
> 
> Reviewed-by: Paul Menzel <pmenzel@molgen.mpg.de>
> 
> 
> Kind regards,
> 
> Paul


[-- Attachment #2: OpenPGP digital signature --]
[-- Type: application/pgp-signature, Size: 236 bytes --]