From patchwork Thu Jun 22 07:58:44 2017 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Andrei Otcheretianski X-Patchwork-Id: 779393 Return-Path: X-Original-To: incoming@patchwork.ozlabs.org Delivered-To: patchwork-incoming@bilbo.ozlabs.org Received: from bombadil.infradead.org (bombadil.infradead.org [65.50.211.133]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by ozlabs.org (Postfix) with ESMTPS id 3wtYsT20vCz9sBR for ; Thu, 22 Jun 2017 18:00:13 +1000 (AEST) Authentication-Results: ozlabs.org; dkim=pass (2048-bit key; unprotected) header.d=lists.infradead.org header.i=@lists.infradead.org header.b="AGKeFmUA"; dkim-atps=neutral DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=lists.infradead.org; s=bombadil.20170209; h=Sender: Content-Transfer-Encoding:Content-Type:MIME-Version:Cc:List-Subscribe: List-Help:List-Post:List-Archive:List-Unsubscribe:List-Id:Message-Id:Date: Subject:To:From:Reply-To:Content-ID:Content-Description:Resent-Date: Resent-From:Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID:In-Reply-To: References:List-Owner; bh=77JiNhGJAt27Y3+hd19cSEs3FU6GncI+AwrDI9TDf78=; b=AGK eFmUA9oO/26wx506lyQmVsV7Waz9Nk2DKbDttRXY2f9Vcf5qcLzdetkXrYYjSRdx8k/sAvHxxtvha FEVXgFbvArSr5LcDIHouw/N274DdIMVvKe4t6BtThMB/uYIZPwVwR/23kSWBJVyjik+bWIvQvbebT Hf0F3y+ZskAe/GQR9ZITCIWuYzIH4c7DBVWko6H+HsvFiVoTt4mQHusC5oOfGqDz1f3Q2oIqCiWqt FO1+Rz1/I1gckW3uO11ZBzp9Br3Ixh0ju/6z5XQoe4fB8q0mkdlyPeEfnDRkFvB7E2vypP4HesDdm FGbwSmkrgeOWUrLLCWOkwsTHmfxoWsQ==; Received: from localhost ([127.0.0.1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.87 #1 (Red Hat Linux)) id 1dNx1j-0002Ep-4j; Thu, 22 Jun 2017 07:59:43 +0000 Received: from mga11.intel.com ([192.55.52.93]) by bombadil.infradead.org with esmtps (Exim 4.87 #1 (Red Hat Linux)) id 1dNx1f-0002Ce-PC for hostap@lists.infradead.org; Thu, 22 Jun 2017 07:59:41 +0000 Received: from orsmga004.jf.intel.com ([10.7.209.38]) by fmsmga102.fm.intel.com with ESMTP/TLS/DHE-RSA-AES256-GCM-SHA384; 22 Jun 2017 00:59:09 -0700 X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="5.39,372,1493708400"; d="scan'208";a="100380689" Received: from jel00350.jer.intel.com (HELO TEMPHOSTNAME.ger.corp.intel.com) ([10.12.217.225]) by orsmga004.jf.intel.com with ESMTP; 22 Jun 2017 00:59:07 -0700 From: Andrei Otcheretianski To: hostap@lists.infradead.org Subject: [PATCH] tests: Use global control interface in test_p2p_channel.py Date: Thu, 22 Jun 2017 10:58:44 +0300 Message-Id: <1498118324-26495-1-git-send-email-andrei.otcheretianski@intel.com> X-Mailer: git-send-email 1.9.1 X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20170622_005939_898091_8B0E3CD1 X-CRM114-Status: UNSURE ( 6.33 ) X-CRM114-Notice: Please train this message. X-Spam-Score: -6.9 (------) X-Spam-Report: SpamAssassin version 3.4.1 on bombadil.infradead.org summary: Content analysis details: (-6.9 points) pts rule name description ---- ---------------------- -------------------------------------------------- -5.0 RCVD_IN_DNSWL_HI RBL: Sender listed at http://www.dnswl.org/, high trust [192.55.52.93 listed in list.dnswl.org] -0.0 SPF_PASS SPF: sender matches SPF record -0.0 T_RP_MATCHES_RCVD Envelope sender domain matches handover relay domain -1.9 BAYES_00 BODY: Bayes spam probability is 0 to 1% [score: 0.0000] X-BeenThere: hostap@lists.infradead.org X-Mailman-Version: 2.1.21 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Cc: Adiel Aloni MIME-Version: 1.0 Sender: "Hostap" Errors-To: hostap-bounces+incoming=patchwork.ozlabs.org@lists.infradead.org From: Adiel Aloni Previously p2p_channel_drv_pref_* tests would fail if dedicated P2P device is used, since the SET commands were sent to incorrect interface. Fix this by using a global control interface instead. Signed-off-by: Adiel Aloni --- tests/hwsim/test_p2p_channel.py | 20 ++++++++++---------- 1 file changed, 10 insertions(+), 10 deletions(-) diff --git a/tests/hwsim/test_p2p_channel.py b/tests/hwsim/test_p2p_channel.py index b5ff9ec..ffdea2e 100644 --- a/tests/hwsim/test_p2p_channel.py +++ b/tests/hwsim/test_p2p_channel.py @@ -1225,7 +1225,7 @@ def test_p2p_channel_vht80p80_persistent(dev): def test_p2p_channel_drv_pref_go_neg(dev): """P2P GO Negotiation with GO device channel preference""" - dev[0].set("get_pref_freq_list_override", "3:2417 4:2422") + dev[0].global_request("SET get_pref_freq_list_override 3:2417 4:2422") [i_res, r_res] = go_neg_pin_authorized(i_dev=dev[0], i_intent=15, r_dev=dev[1], r_intent=0, test_data=False) @@ -1237,8 +1237,8 @@ def test_p2p_channel_drv_pref_go_neg(dev): def test_p2p_channel_drv_pref_go_neg2(dev): """P2P GO Negotiation with P2P client device channel preference""" - dev[0].set("get_pref_freq_list_override", "3:2417,2422") - dev[1].set("get_pref_freq_list_override", "4:2422") + dev[0].global_request("SET get_pref_freq_list_override 3:2417,2422") + dev[1].global_request("SET get_pref_freq_list_override 4:2422") [i_res, r_res] = go_neg_pin_authorized(i_dev=dev[0], i_intent=15, r_dev=dev[1], r_intent=0, test_data=False) @@ -1250,7 +1250,7 @@ def test_p2p_channel_drv_pref_go_neg2(dev): def test_p2p_channel_drv_pref_go_neg3(dev): """P2P GO Negotiation with GO device channel preference""" - dev[1].set("get_pref_freq_list_override", "3:2417,2427 4:2422") + dev[1].global_request("SET get_pref_freq_list_override 3:2417,2427 4:2422") [i_res, r_res] = go_neg_pin_authorized(i_dev=dev[0], i_intent=0, r_dev=dev[1], r_intent=15, test_data=False) @@ -1262,8 +1262,8 @@ def test_p2p_channel_drv_pref_go_neg3(dev): def test_p2p_channel_drv_pref_go_neg4(dev): """P2P GO Negotiation with P2P client device channel preference""" - dev[0].set("get_pref_freq_list_override", "3:2417,2422,5180") - dev[1].request("P2P_SET override_pref_op_chan 115:36") + dev[0].global_request("SET get_pref_freq_list_override 3:2417,2422,5180") + dev[1].global_request("P2P_SET override_pref_op_chan 115:36") [i_res, r_res] = go_neg_pin_authorized(i_dev=dev[0], i_intent=15, r_dev=dev[1], r_intent=0, test_data=False) @@ -1275,9 +1275,9 @@ def test_p2p_channel_drv_pref_go_neg4(dev): def test_p2p_channel_drv_pref_go_neg5(dev): """P2P GO Negotiation with P2P client device channel preference""" - dev[0].set("get_pref_freq_list_override", "3:2417") - dev[1].set("get_pref_freq_list_override", "4:2422") - dev[1].request("P2P_SET override_pref_op_chan 115:36") + dev[0].global_request("SET get_pref_freq_list_override 3:2417") + dev[1].global_request("SET get_pref_freq_list_override 4:2422") + dev[1].global_request("P2P_SET override_pref_op_chan 115:36") [i_res, r_res] = go_neg_pin_authorized(i_dev=dev[0], i_intent=15, r_dev=dev[1], r_intent=0, test_data=False) @@ -1289,7 +1289,7 @@ def test_p2p_channel_drv_pref_go_neg5(dev): def test_p2p_channel_drv_pref_autogo(dev): """P2P autonomous GO with driver channel preference""" - dev[0].set("get_pref_freq_list_override", "3:2417,2422,5180") + dev[0].global_request("SET get_pref_freq_list_override 3:2417,2422,5180") res_go = autogo(dev[0]) if res_go['freq'] != "2417": raise Exception("Unexpected operating frequency: " + res_go['freq'])