Re: [mac80211_hwsim] 148fe295b7: hwsim.ap_long_preamble.fail

[Philip Li]

On Fri, Oct 02, 2020 at 09:38:45AM -0700, Thomas Pedersen wrote:
> On 2020-10-02 01:29, kernel test robot wrote:
> > Greeting,
> > 
> > FYI, we noticed the following commit (built with gcc-9):
> > 
> > commit: 148fe295b7d9d892b2b0f47070233ccdc70c83cd ("mac80211_hwsim:
> > indicate support for S1G")
> > https://git.kernel.org/cgit/linux/kernel/git/next/linux-next.git master
> > 
> > 
> > in testcase: hwsim
> > version: hwsim-x86_64-6eb6cf0-1_20200619
> > with following parameters:
> > 
> > 	group: hwsim-04
> > 	ucode: 0x21
> > 
> > 
> > 
> > on test machine: 8 threads Intel(R) Core(TM) i7-3770K CPU @ 3.50GHz
> > with 16G memory
> > 
> > caused below changes (please refer to attached dmesg/kmsg for entire
> > log/backtrace):
> > 
> > 
> > If you fix the issue, kindly add following tag
> > Reported-by: kernel test robot <rong.a.chen@xxxxxxxxx>
> > 
> > 
> > 2020-10-06 00:22:09 	./run-tests.py ap_long_preamble
> > DEV: wlan0: 02:00:00:00:00:00
> > DEV: wlan1: 02:00:00:00:01:00
> > DEV: wlan2: 02:00:00:00:02:00
> > APDEV: wlan3
> > APDEV: wlan4
> > START ap_long_preamble 1/1
> > Test: AP with long preamble
> > Starting AP wlan3
> > Failed to enable hostapd interface wlan3
> > Traceback (most recent call last):
> >   File "./run-tests.py", line 533, in main
> >     t(dev, apdev)
> >   File "/lkp/benchmarks/hwsim/tests/hwsim/test_ap_params.py", line
> > 821, in test_ap_long_preamble
> >     hapd = hostapd.add_ap(apdev[0], params)
> >   File "/lkp/benchmarks/hwsim/tests/hwsim/hostapd.py", line 581, in
> > add_ap
> >     hapd.enable()
> >   File "/lkp/benchmarks/hwsim/tests/hwsim/hostapd.py", line 248, in
> > enable
> >     raise Exception("Failed to enable hostapd interface " + self.ifname)
> > Exception: Failed to enable hostapd interface wlan3
> > FAIL ap_long_preamble 0.064398 2020-10-06 00:22:09.608978
> > passed 0 test case(s)
> > skipped 0 test case(s)
> > failed tests: ap_long_preamble
> 
> This is actually a bug in hostap which was exposed by this commit (and fixed
> by https://patchwork.ozlabs.org/project/hostap/patch/20200827225940.18151-1-thomas@xxxxxxxxxxxx/).
Thanks for the info, we have upgraded hwsim test suite to contain this fix.
And apologize for the false positive.

> 
> -- 
> thomas
>