Re: [PATCH 5/8] tests: setup the wlantest only once in check_qos_map()

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Thu, Jun 23, 2016 at 08:16:33PM +0300, Jonathan Afek wrote:
> Some tests call the check_qos_map() func more than once.
> Make sure it setups the wlantest only once on the first
> time it is called.

> diff --git a/tests/hwsim/test_ap_qosmap.py b/tests/hwsim/test_ap_qosmap.py
> @@ -17,7 +17,8 @@ def check_qos_map(ap, hapd, dev, sta, dscp, tid, ap_tid=None):

> -    Wlantest.setup(hapd)
> +    if not Wlantest.setup_done:
> +        Wlantest.setup(hapd)

Is this really the correct thing to do? Isn't that Wlantest.setup_done
shared through all test cases? What if the QoS mapping test cases get
run after a test case that used wlantest on another channel? If this
commit is trying to optimize test execution (the commit message should
really provide the reason for doing this change..), that would likely
need to be done by maintaining a local variable for the test case(s) in
question rather than relying on Wlantest.setup_done.

-- 
Jouni Malinen                                            PGP id EFC895FA

_______________________________________________
Hostap mailing list
Hostap@xxxxxxxxxxxxxxxxxxx
http://lists.infradead.org/mailman/listinfo/hostap



[Index of Archives]     [Linux Wireless]     [Linux Kernel]     [ATH6KL]     [Linux Bluetooth]     [Linux Netdev]     [Kernel Newbies]     [IDE]     [Security]     [Git]     [Netfilter]     [Bugtraq]     [Yosemite News]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux RAID]     [Linux ATA RAID]     [Samba]     [Device Mapper]

  Powered by Linux