tree: https://git.kernel.org/pub/scm/linux/kernel/git/jberg/mac80211.git master head: 8b194febe111c5cc47595749a766d24ca33dd95a commit: 8b194febe111c5cc47595749a766d24ca33dd95a [9/9] mac80211: 160MHz with extended NSS BW in CSA config: i386-randconfig-s002-20210118 (attached as .config) compiler: gcc-9 (Debian 9.3.0-15) 9.3.0 reproduce: # apt-get install sparse # sparse version: v0.6.3-208-g46a52ca4-dirty # https://git.kernel.org/pub/scm/linux/kernel/git/jberg/mac80211.git/commit/?id=8b194febe111c5cc47595749a766d24ca33dd95a git remote add mac80211 https://git.kernel.org/pub/scm/linux/kernel/git/jberg/mac80211.git git fetch --no-tags mac80211 master git checkout 8b194febe111c5cc47595749a766d24ca33dd95a # save the attached .config to linux build tree make W=1 C=1 CF='-fdiagnostic-prefix -D__CHECK_ENDIAN__' ARCH=i386 If you fix the issue, kindly add following tag as appropriate Reported-by: kernel test robot <lkp@xxxxxxxxx> "sparse warnings: (new ones prefixed by >>)" >> net/mac80211/spectmgmt.c:147:42: sparse: sparse: incorrect type in initializer (different base types) @@ expected restricted __le16 [usertype] operation_mode @@ got int @@ net/mac80211/spectmgmt.c:147:42: sparse: expected restricted __le16 [usertype] operation_mode net/mac80211/spectmgmt.c:147:42: sparse: got int vim +147 net/mac80211/spectmgmt.c 21 22 int ieee80211_parse_ch_switch_ie(struct ieee80211_sub_if_data *sdata, 23 struct ieee802_11_elems *elems, 24 enum nl80211_band current_band, 25 u32 vht_cap_info, 26 u32 sta_flags, u8 *bssid, 27 struct ieee80211_csa_ie *csa_ie) 28 { 29 enum nl80211_band new_band = current_band; 30 int new_freq; 31 u8 new_chan_no; 32 struct ieee80211_channel *new_chan; 33 struct cfg80211_chan_def new_vht_chandef = {}; 34 const struct ieee80211_sec_chan_offs_ie *sec_chan_offs; 35 const struct ieee80211_wide_bw_chansw_ie *wide_bw_chansw_ie; 36 int secondary_channel_offset = -1; 37 38 memset(csa_ie, 0, sizeof(*csa_ie)); 39 40 sec_chan_offs = elems->sec_chan_offs; 41 wide_bw_chansw_ie = elems->wide_bw_chansw_ie; 42 43 if (sta_flags & (IEEE80211_STA_DISABLE_HT | 44 IEEE80211_STA_DISABLE_40MHZ)) { 45 sec_chan_offs = NULL; 46 wide_bw_chansw_ie = NULL; 47 } 48 49 if (sta_flags & IEEE80211_STA_DISABLE_VHT) 50 wide_bw_chansw_ie = NULL; 51 52 if (elems->ext_chansw_ie) { 53 if (!ieee80211_operating_class_to_band( 54 elems->ext_chansw_ie->new_operating_class, 55 &new_band)) { 56 sdata_info(sdata, 57 "cannot understand ECSA IE operating class, %d, ignoring\n", 58 elems->ext_chansw_ie->new_operating_class); 59 } 60 new_chan_no = elems->ext_chansw_ie->new_ch_num; 61 csa_ie->count = elems->ext_chansw_ie->count; 62 csa_ie->mode = elems->ext_chansw_ie->mode; 63 } else if (elems->ch_switch_ie) { 64 new_chan_no = elems->ch_switch_ie->new_ch_num; 65 csa_ie->count = elems->ch_switch_ie->count; 66 csa_ie->mode = elems->ch_switch_ie->mode; 67 } else { 68 /* nothing here we understand */ 69 return 1; 70 } 71 72 /* Mesh Channel Switch Parameters Element */ 73 if (elems->mesh_chansw_params_ie) { 74 csa_ie->ttl = elems->mesh_chansw_params_ie->mesh_ttl; 75 csa_ie->mode = elems->mesh_chansw_params_ie->mesh_flags; 76 csa_ie->pre_value = le16_to_cpu( 77 elems->mesh_chansw_params_ie->mesh_pre_value); 78 79 if (elems->mesh_chansw_params_ie->mesh_flags & 80 WLAN_EID_CHAN_SWITCH_PARAM_REASON) 81 csa_ie->reason_code = le16_to_cpu( 82 elems->mesh_chansw_params_ie->mesh_reason); 83 } 84 85 new_freq = ieee80211_channel_to_frequency(new_chan_no, new_band); 86 new_chan = ieee80211_get_channel(sdata->local->hw.wiphy, new_freq); 87 if (!new_chan || new_chan->flags & IEEE80211_CHAN_DISABLED) { 88 sdata_info(sdata, 89 "BSS %pM switches to unsupported channel (%d MHz), disconnecting\n", 90 bssid, new_freq); 91 return -EINVAL; 92 } 93 94 if (sec_chan_offs) { 95 secondary_channel_offset = sec_chan_offs->sec_chan_offs; 96 } else if (!(sta_flags & IEEE80211_STA_DISABLE_HT)) { 97 /* If the secondary channel offset IE is not present, 98 * we can't know what's the post-CSA offset, so the 99 * best we can do is use 20MHz. 100 */ 101 secondary_channel_offset = IEEE80211_HT_PARAM_CHA_SEC_NONE; 102 } 103 104 switch (secondary_channel_offset) { 105 default: 106 /* secondary_channel_offset was present but is invalid */ 107 case IEEE80211_HT_PARAM_CHA_SEC_NONE: 108 cfg80211_chandef_create(&csa_ie->chandef, new_chan, 109 NL80211_CHAN_HT20); 110 break; 111 case IEEE80211_HT_PARAM_CHA_SEC_ABOVE: 112 cfg80211_chandef_create(&csa_ie->chandef, new_chan, 113 NL80211_CHAN_HT40PLUS); 114 break; 115 case IEEE80211_HT_PARAM_CHA_SEC_BELOW: 116 cfg80211_chandef_create(&csa_ie->chandef, new_chan, 117 NL80211_CHAN_HT40MINUS); 118 break; 119 case -1: 120 cfg80211_chandef_create(&csa_ie->chandef, new_chan, 121 NL80211_CHAN_NO_HT); 122 /* keep width for 5/10 MHz channels */ 123 switch (sdata->vif.bss_conf.chandef.width) { 124 case NL80211_CHAN_WIDTH_5: 125 case NL80211_CHAN_WIDTH_10: 126 csa_ie->chandef.width = 127 sdata->vif.bss_conf.chandef.width; 128 break; 129 default: 130 break; 131 } 132 break; 133 } 134 135 if (wide_bw_chansw_ie) { 136 u8 new_seg1 = wide_bw_chansw_ie->new_center_freq_seg1; 137 struct ieee80211_vht_operation vht_oper = { 138 .chan_width = 139 wide_bw_chansw_ie->new_channel_width, 140 .center_freq_seg0_idx = 141 wide_bw_chansw_ie->new_center_freq_seg0, 142 .center_freq_seg1_idx = new_seg1, 143 /* .basic_mcs_set doesn't matter */ 144 }; 145 struct ieee80211_ht_operation ht_oper = { 146 .operation_mode = > 147 new_seg1 << IEEE80211_HT_OP_MODE_CCFS2_SHIFT, 148 }; 149 150 /* default, for the case of IEEE80211_VHT_CHANWIDTH_USE_HT, 151 * to the previously parsed chandef 152 */ 153 new_vht_chandef = csa_ie->chandef; 154 155 /* ignore if parsing fails */ 156 if (!ieee80211_chandef_vht_oper(&sdata->local->hw, 157 vht_cap_info, 158 &vht_oper, &ht_oper, 159 &new_vht_chandef)) 160 new_vht_chandef.chan = NULL; 161 162 if (sta_flags & IEEE80211_STA_DISABLE_80P80MHZ && 163 new_vht_chandef.width == NL80211_CHAN_WIDTH_80P80) 164 ieee80211_chandef_downgrade(&new_vht_chandef); 165 if (sta_flags & IEEE80211_STA_DISABLE_160MHZ && 166 new_vht_chandef.width == NL80211_CHAN_WIDTH_160) 167 ieee80211_chandef_downgrade(&new_vht_chandef); 168 } 169 170 /* if VHT data is there validate & use it */ 171 if (new_vht_chandef.chan) { 172 if (!cfg80211_chandef_compatible(&new_vht_chandef, 173 &csa_ie->chandef)) { 174 sdata_info(sdata, 175 "BSS %pM: CSA has inconsistent channel data, disconnecting\n", 176 bssid); 177 return -EINVAL; 178 } 179 csa_ie->chandef = new_vht_chandef; 180 } 181 182 if (elems->max_channel_switch_time) 183 csa_ie->max_switch_time = 184 (elems->max_channel_switch_time[0] << 0) | 185 (elems->max_channel_switch_time[1] << 8) | 186 (elems->max_channel_switch_time[2] << 16); 187 188 return 0; 189 } 190 --- 0-DAY CI Kernel Test Service, Intel Corporation https://lists.01.org/hyperkitty/list/kbuild-all@xxxxxxxxxxxx
Attachment:
.config.gz
Description: application/gzip
