Emmanuel Grumbach <emmanuel.grumbach@xxxxxxxxx> wrote: > The purpose of this was to keep all the queues updated with > the Rx sequence numbers because unlikely yet possible > situations where queues can't understand if a specific > packet needs to be dropped or not. > > Unfortunately, it was reported that this caused issues in > our DMA engine. We don't fully understand how this is related, > but this is being currently debugged. For now, just don't send > this notification to the Rx queues. This de-facto reverts my > commit 3c514bf831ac12356b695ff054bef641b9e99593: > > iwlwifi: mvm: add a loose synchronization of the NSSN across Rx queues > > This issue was reported here: > https://bugzilla.kernel.org/show_bug.cgi?id=204873 > https://bugzilla.kernel.org/show_bug.cgi?id=205001 > and others maybe. > > Fixes: 3c514bf831ac ("iwlwifi: mvm: add a loose synchronization of the NSSN across Rx queues") > CC: <stable@xxxxxxxxxxxxxxx> # 5.3+ > Signed-off-by: Emmanuel Grumbach <emmanuel.grumbach@xxxxxxxxx> New warning: drivers/net/wireless/intel/iwlwifi/mvm/rxmq.c: In function 'iwl_mvm_sync_nssn': drivers/net/wireless/intel/iwlwifi/mvm/rxmq.c:517:32: warning: unused variable 'notif' [-Wunused-variable] struct iwl_mvm_rss_sync_notif notif = { ^~~~~ Patch set to Changes Requested. -- https://patchwork.kernel.org/patch/11253817/ https://wireless.wiki.kernel.org/en/developers/documentation/submittingpatches
