* [Intel-wired-lan] [PATCH net v1] i40e: Fix interface init with MSI interrupts (no MSI-X)
2022-02-15 13:35 [Intel-wired-lan] [PATCH net v1] i40e: Fix interface init with MSI interrupts (no MSI-X) Michal Maloszewski
@ 2022-02-15 12:08 ` Paul Menzel
0 siblings, 0 replies; 2+ messages in thread
From: Paul Menzel @ 2022-02-15 12:08 UTC (permalink / raw)
To: intel-wired-lan
Dear Michal,
Am 15.02.22 um 14:35 schrieb Michal Maloszewski:
Just a note, that the time is from the future again. (Some Intel
employee told me, the Intel firewall (in Poland offices?) blocks NTP,
and you have to configure an internal NTP server, which should now also
be documented your intranet.)
> Fix the inability to bring an interface up on a setup with
> only MSI interrupts enabled (no MSI-X).
Is there an error logged?
> Solution is to add a default number of QPs = 1. This is enough,
> since without MSI-X support driver enables only a basic feature set.
Please reflow for 75 characters per line (and maybe add a blank line
between paragraphs).
> Fixes: bc6d33c8d93f ("i40e: Fix the number of queues available to be mapped for use")
Wow, that went into 4.16-rc1. Any idea, why it hasn?t been reported
until now?
> Signed-off-by: Dawid Lukwinski <dawid.lukwinski@intel.com>
> Signed-off-by: Michal Maloszewski <michal.maloszewski@intel.com>
> ---
> drivers/net/ethernet/intel/i40e/i40e_main.c | 3 +++
> 1 file changed, 3 insertions(+)
>
> diff --git a/drivers/net/ethernet/intel/i40e/i40e_main.c b/drivers/net/ethernet/intel/i40e/i40e_main.c
> index 8572993972ef..dfaa56235b60 100644
> --- a/drivers/net/ethernet/intel/i40e/i40e_main.c
> +++ b/drivers/net/ethernet/intel/i40e/i40e_main.c
> @@ -1840,6 +1840,9 @@ static void i40e_vsi_setup_queue_map(struct i40e_vsi *vsi,
> vsi->num_queue_pairs = vsi->req_queue_pairs;
> else if (pf->flags & I40E_FLAG_MSIX_ENABLED)
> vsi->num_queue_pairs = pf->num_lan_msix;
> + else
> + /* We need at least one queue pair for the interface to be usable */
> + vsi->num_queue_pairs = 1;
Maybe you could rewrite the comment above the if statement (not in this
diff), and integrate the information about MSI and MSI-X.
> }
>
> /* Number of queues per enabled TC */
Kind regards,
Paul
^ permalink raw reply [flat|nested] 2+ messages in thread
* [Intel-wired-lan] [PATCH net v1] i40e: Fix interface init with MSI interrupts (no MSI-X)
@ 2022-02-15 13:35 Michal Maloszewski
2022-02-15 12:08 ` Paul Menzel
0 siblings, 1 reply; 2+ messages in thread
From: Michal Maloszewski @ 2022-02-15 13:35 UTC (permalink / raw)
To: intel-wired-lan
Fix the inability to bring an interface up on a setup with
only MSI interrupts enabled (no MSI-X).
Solution is to add a default number of QPs = 1. This is enough,
since without MSI-X support driver enables only a basic feature set.
Fixes: bc6d33c8d93f ("i40e: Fix the number of queues available to be mapped for use")
Signed-off-by: Dawid Lukwinski <dawid.lukwinski@intel.com>
Signed-off-by: Michal Maloszewski <michal.maloszewski@intel.com>
---
drivers/net/ethernet/intel/i40e/i40e_main.c | 3 +++
1 file changed, 3 insertions(+)
diff --git a/drivers/net/ethernet/intel/i40e/i40e_main.c b/drivers/net/ethernet/intel/i40e/i40e_main.c
index 8572993972ef..dfaa56235b60 100644
--- a/drivers/net/ethernet/intel/i40e/i40e_main.c
+++ b/drivers/net/ethernet/intel/i40e/i40e_main.c
@@ -1840,6 +1840,9 @@ static void i40e_vsi_setup_queue_map(struct i40e_vsi *vsi,
vsi->num_queue_pairs = vsi->req_queue_pairs;
else if (pf->flags & I40E_FLAG_MSIX_ENABLED)
vsi->num_queue_pairs = pf->num_lan_msix;
+ else
+ /* We need at least one queue pair for the interface to be usable */
+ vsi->num_queue_pairs = 1;
}
/* Number of queues per enabled TC */
--
2.27.0
^ permalink raw reply related [flat|nested] 2+ messages in thread
end of thread, other threads:[~2022-02-15 13:35 UTC | newest]
Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2022-02-15 13:35 [Intel-wired-lan] [PATCH net v1] i40e: Fix interface init with MSI interrupts (no MSI-X) Michal Maloszewski
2022-02-15 12:08 ` Paul Menzel
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.