This one is sort of a false positive. The code works. I feel like it
should be written:
memcpy(data, ath10k_gstrings_stats, sizeof(ath10k_gstrings_stats));
But that's a style issue rather than a bug.
regards,
dan carpenter
On Thu, Nov 13, 2014 at 10:04:33PM +0800, kbuild test robot wrote:
> TO: Ben Greear <[email protected]>
> CC: Kalle Valo <[email protected]>
>
> tree: git://git.kernel.org/pub/scm/linux/kernel/git/next/linux-next.git
> master
> head: bbdef57970d5e1887de755474ff1562baa17ef11
> commit: 6cddcc7ac215a3e988ec2cbf303fb0a39220dc3b [200/6487] ath10k: support
> ethtool stats
> :::::: branch date: 7 hours ago
> :::::: commit date: 6 weeks ago
>
> drivers/net/wireless/ath/ath10k/debug.c:1195 ath10k_debug_get_et_strings()
> error: __builtin_memcpy() '*ath10k_gstrings_stats' too small (32 vs 1472)
>
> git remote add next
> git://git.kernel.org/pub/scm/linux/kernel/git/next/linux-next.git
> git remote update next
> git checkout 6cddcc7ac215a3e988ec2cbf303fb0a39220dc3b
> vim +1195 drivers/net/wireless/ath/ath10k/debug.c
>
> 6cddcc7a Ben Greear 2014-09-29 1179 "d_rx_mpdu_stack",
> 6cddcc7a Ben Greear 2014-09-29 1180 "d_rx_phy_err",
> 6cddcc7a Ben Greear 2014-09-29 1181 "d_rx_phy_err_drops",
> 6cddcc7a Ben Greear 2014-09-29 1182 "d_rx_mpdu_errors", /* FCS,
> MIC, ENC */
> 6cddcc7a Ben Greear 2014-09-29 1183 "d_fw_crash_count",
> 6cddcc7a Ben Greear 2014-09-29 1184 "d_fw_warm_reset_count",
> 6cddcc7a Ben Greear 2014-09-29 1185 "d_fw_cold_reset_count",
> 6cddcc7a Ben Greear 2014-09-29 1186 };
> 6cddcc7a Ben Greear 2014-09-29 1187
> 6cddcc7a Ben Greear 2014-09-29 1188 #define ATH10K_SSTATS_LEN
> ARRAY_SIZE(ath10k_gstrings_stats)
> 6cddcc7a Ben Greear 2014-09-29 1189
> 6cddcc7a Ben Greear 2014-09-29 1190 void ath10k_debug_get_et_strings(struct
> ieee80211_hw *hw,
> 6cddcc7a Ben Greear 2014-09-29 1191 struct
> ieee80211_vif *vif,
> 6cddcc7a Ben Greear 2014-09-29 1192 u32
> sset, u8 *data)
> 6cddcc7a Ben Greear 2014-09-29 1193 {
> 6cddcc7a Ben Greear 2014-09-29 1194 if (sset == ETH_SS_STATS)
> 6cddcc7a Ben Greear 2014-09-29 @1195 memcpy(data,
> *ath10k_gstrings_stats,
> 6cddcc7a Ben Greear 2014-09-29 1196
> sizeof(ath10k_gstrings_stats));
> 6cddcc7a Ben Greear 2014-09-29 1197 }
> 6cddcc7a Ben Greear 2014-09-29 1198
> 6cddcc7a Ben Greear 2014-09-29 1199 int
> ath10k_debug_get_et_sset_count(struct ieee80211_hw *hw,
> 6cddcc7a Ben Greear 2014-09-29 1200
> struct ieee80211_vif *vif, int sset)
> 6cddcc7a Ben Greear 2014-09-29 1201 {
> 6cddcc7a Ben Greear 2014-09-29 1202 if (sset == ETH_SS_STATS)
> 6cddcc7a Ben Greear 2014-09-29 1203 return
> ATH10K_SSTATS_LEN;
>
> ---
> 0-DAY kernel test infrastructure Open Source Technology Center
> http://lists.01.org/mailman/listinfo/kbuild Intel Corporation
_______________________________________________
kbuild mailing list
[email protected]
https://lists.01.org/mailman/listinfo/kbuild
