On Fri, 2017-09-29 at 15:03 +0300, Luca Coelho wrote: > From: Shaul Triebitz <shaul.triebitz@xxxxxxxxx> > > In monitor mode we are not expected to decrypt encrypted > packets (not having the keys). > Hence we are expected to get an unknown rx security status. > Keeping the print in monitor mode causes a print for each > captured packet flooding the dmesg. [] > diff --git a/drivers/net/wireless/intel/iwlwifi/mvm/mvm.h b/drivers/net/wireless/intel/iwlwifi/mvm/mvm.h [] > @@ -1015,6 +1015,9 @@ struct iwl_mvm { > bool drop_bcn_ap_mode; > > struct delayed_work cs_tx_unblock_dwork; > + > + /* does a monitor vif exist (only one can exist hence bool) */ > + bool monitor_on; It would be slightly more space efficient to move this above struct delayed_work cs_tx_unblock_dwork adjacent to bool drop_bcn_ap_mode