From: Ben Greear It seems to be expected behaviour, and is seen fairly often in testing in adverse conditions, so make it a one-line log message instead of WARN splat. Signed-off-by: Ben Greear --- drivers/net/wireless/intel/iwlwifi/mld/agg.c | 12 +++++++++--- 1 file changed, 9 insertions(+), 3 deletions(-) diff --git a/drivers/net/wireless/intel/iwlwifi/mld/agg.c b/drivers/net/wireless/intel/iwlwifi/mld/agg.c index a757077b0a7a..23d55374ef8a 100644 --- a/drivers/net/wireless/intel/iwlwifi/mld/agg.c +++ b/drivers/net/wireless/intel/iwlwifi/mld/agg.c @@ -216,10 +216,16 @@ iwl_mld_reorder(struct iwl_mld *mld, struct napi_struct *napi, if (baid == IWL_RX_REORDER_DATA_INVALID_BAID) return IWL_MLD_PASS_SKB; - /* no sta yet */ - if (WARN_ONCE(!sta, - "Got valid BAID without a valid station assigned\n")) + /* no sta yet. This happens fairly often, don't WARN_ON about it. */ + if (!sta) { + static bool done_once; + + if (!done_once) { + IWL_ERR(mld, "Got valid BAID without a valid station assigned, will not log again.\n"); + done_once = true; + } return IWL_MLD_PASS_SKB; + } mld_sta = iwl_mld_sta_from_mac80211(sta); -- 2.42.0