On Sun, Feb 03, 2019 at 12:04:31AM +0100, Johannes Berg wrote: > Instead of relying on existing configuration (which may conflict > with other tests) and skipping otherwise, create a new radio with > two channels in this test and use it. Thanks, applied. > - set_no_group_iface(dev[0], 0) > - set_no_group_iface(dev[1], 0) > + set_no_group_iface(wpas, 0) > + set_no_group_iface(dev[1], 0) > - (grp_ifname0, grp_ifname1, ifnames) = p2ps_connect_p2ps_method(dev, keep_group=True) > + (grp_ifname0, grp_ifname1, ifnames) = p2ps_connect_p2ps_method(dev, keep_group=True) With that call fixed to use [ wpas, dev[1] ] as the first argument to maintain the expected testing behavior. PS. I did similar conversion for p2ps_channel_active_go_and_station_different_mcc and that remove the last one of those get_mcc() conditional test cases, so the CHANNELS=1 default configuration for mac80211_hwsim should now work with all test cases. -- Jouni Malinen PGP id EFC895FA _______________________________________________ Hostap mailing list Hostap@xxxxxxxxxxxxxxxxxxx http://lists.infradead.org/mailman/listinfo/hostap