Intel-Wired-Lan Archive on lore.kernel.org
 help / color / mirror / Atom feed
* [Intel-wired-lan] [PATCH] fixup! ice: add support for sideband messages
@ 2021-05-26 20:23 Jacob Keller
  0 siblings, 0 replies; only message in thread
From: Jacob Keller @ 2021-05-26 20:23 UTC (permalink / raw)
  To: intel-wired-lan

The check in ice_clean_sbq_subtask for whether to exit early
accidentally checked for if the sbq was supported. This check should be
negated, so that we only try to clean the sideband queue on devices
which support it. Otherwise without this fix, on devices that do not
support the sideband queue, the following message may get spammed into
the log:

ice 0000:af:00.0: Sideband Receive Queue event error ICE_ERR_AQ_EMPTY

Additionally, on devices which do support the sideband queue, the
receive portion will never be properly cleaned.

Signed-off-by: Jacob Keller <jacob.e.keller@intel.com>
---

Tony, if you wouldn't mind squashing this into the noted patch in the
description before we send this to netdev.

 drivers/net/ethernet/intel/ice/ice_main.c | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

diff --git a/drivers/net/ethernet/intel/ice/ice_main.c b/drivers/net/ethernet/intel/ice/ice_main.c
index 3cc1237a2ebe..ca80b858ef20 100644
--- a/drivers/net/ethernet/intel/ice/ice_main.c
+++ b/drivers/net/ethernet/intel/ice/ice_main.c
@@ -1430,7 +1430,8 @@ static void ice_clean_sbq_subtask(struct ice_pf *pf)
 {
 	struct ice_hw *hw = &pf->hw;
 
-	if (ice_is_sbq_supported(hw)) {
+	/* Nothing to do here if sideband queue is not supported */
+	if (!ice_is_sbq_supported(hw)) {
 		clear_bit(ICE_SIDEBANDQ_EVENT_PENDING, pf->state);
 		return;
 	}

base-commit: 80e0251735737d53ea5dc85e2cf5596df721ddc5
-- 
2.31.1.331.gb0c09ab8796f


^ permalink raw reply related	[flat|nested] only message in thread

only message in thread, other threads:[~2021-05-26 20:23 UTC | newest]

Thread overview: (only message) (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2021-05-26 20:23 [Intel-wired-lan] [PATCH] fixup! ice: add support for sideband messages Jacob Keller

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox