Commit ae5ecbb0 authored by Rotem Saado's avatar Rotem Saado Committed by Gregory Greenman
Browse files

wifi: iwlwifi: dbg: use bit of DRAM alloc ID to store failed allocs 



The failed_alloc variable is used as a bitmask in the loop where we
check DRAM allocations.  But erroneously, we were clearing the DRAM
alloc IDs we removed as an integer.

Fix that by clearing them as bits instead.

Signed-off-by: default avatarRotem Saado <rotem.saado@intel.com>
Signed-off-by: default avatarGregory Greenman <gregory.greenman@intel.com>
Link: https://lore.kernel.org/r/20221102165239.688dec28b1d9.I470b8d29c28d16f25f4192773f075940de7ed33c@changeid
parent b79d2219

drivers/net/wireless/intel/iwlwifi/iwl-dbg-tlv.c

+1 −1
Original line number Original line Diff line number Diff line
@@ -1327,7 +1327,7 @@ static void iwl_dbg_tlv_init_cfg(struct iwl_fw_runtime *fwrt) 
			     "WRT: removing allocation id %d from region id %d\n",

			     "WRT: removing allocation id %d from region id %d\n",

			     le32_to_cpu(reg->dram_alloc_id), i);

			     le32_to_cpu(reg->dram_alloc_id), i);





		failed_alloc &= ~le32_to_cpu(reg->dram_alloc_id);

		failed_alloc &= ~BIT(le32_to_cpu(reg->dram_alloc_id));

		fwrt->trans->dbg.unsupported_region_msk |= BIT(i);

		fwrt->trans->dbg.unsupported_region_msk |= BIT(i);





		kfree(*active_reg);

		kfree(*active_reg);