From patchwork Wed Sep 9 21:08:06 2020 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Owen Hilyard X-Patchwork-Id: 77089 Return-Path: X-Original-To: patchwork@inbox.dpdk.org Delivered-To: patchwork@inbox.dpdk.org Received: from dpdk.org (dpdk.org [92.243.14.124]) by inbox.dpdk.org (Postfix) with ESMTP id C0D47A04B5; Wed, 9 Sep 2020 23:08:14 +0200 (CEST) Received: from [92.243.14.124] (localhost [127.0.0.1]) by dpdk.org (Postfix) with ESMTP id 7B1EA1BE0C; Wed, 9 Sep 2020 23:08:14 +0200 (CEST) Received: from mail-qk1-f171.google.com (mail-qk1-f171.google.com [209.85.222.171]) by dpdk.org (Postfix) with ESMTP id 9452E1DB8 for ; Wed, 9 Sep 2020 23:08:12 +0200 (CEST) Received: by mail-qk1-f171.google.com with SMTP id 16so3917144qkf.4 for ; Wed, 09 Sep 2020 14:08:12 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=iol.unh.edu; s=unh-iol; h=from:to:cc:subject:date:message-id:in-reply-to:references :mime-version:content-transfer-encoding; bh=48QuKTBfU7aM9Z5OtZ96LGB+MkBPqx5/Ouxj6n4HHXQ=; b=Jk1KGMDkPHVDHaHPQy9wWpqjYUG2jASUYEO+FO0A5NQrthC6r7sZHUWSH21bOPIj7Y lovcburdKTdSaB9SYRu+bTTQ6cz/cfszQVqx051NgXVpYf1osbmBvrJzZMyjkcMJlf4W k1zIkrVNUTqB+qIDz0l8MsL14MhUbjkLZCVIw= X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:from:to:cc:subject:date:message-id:in-reply-to :references:mime-version:content-transfer-encoding; bh=48QuKTBfU7aM9Z5OtZ96LGB+MkBPqx5/Ouxj6n4HHXQ=; b=K3bOLxy1xY4ceaEE4eY808bZ7Aqsrni1xRycxx1CUNK0mXhFeNczdFqRPuJc0YyDm7 errW0nhSv64lC+xlN0vIqeJEu9zRNNoKYYF2fygzpCNT8Hw1Gs+I7dJK6jkAlmelzM3w lDRdF159Z46BusFjfjUpGbM1pADe4Hajs6AGFELwKKWBBIsweu2qU2guI+32MEdL35gx QcFJcPTy/hhdfDnsvfcusdsVaAskLhV9AAE2qxp2fA/rctG7U2fjicistDwE+5OKuF+V e0RvavM2y1uIczvRnEOU1IJViN1CZ1ANZ3Lp591DleC3+YynxDWOie30NkuK+xmPCLon FHTw== X-Gm-Message-State: AOAM53079YwCp+y9mAG9Z6GYuayGkICAl8I+vYZn5Gm08P6UMWm7Ymbr /ESQWV9xDjJlruOwFctLuBKJOqdKWPJP0FBIWPTN6Qdgz9SX4WP1u2CJyVjCmDRQdS7yMafSMvI QQiva6fvSesepj2G5yHmJtV3yOy4jZpnMjAPXP2/1YRgb4FIS33Eql135Gw== X-Google-Smtp-Source: ABdhPJyMhyhqkTPGj1RpvpUGuhBZSIHBlH8yj2LcH3gW8ZXHEqIQL/WUcIoan5hwbzhFFHRcpkJv2A== X-Received: by 2002:a05:620a:13f6:: with SMTP id h22mr5305198qkl.9.1599685691722; Wed, 09 Sep 2020 14:08:11 -0700 (PDT) Received: from ohilyard-Alienware-m17.unh.edu (nt-238-68.w4.unh.edu. [132.177.238.68]) by smtp.googlemail.com with ESMTPSA id o2sm3973059qkk.42.2020.09.09.14.08.10 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Wed, 09 Sep 2020 14:08:11 -0700 (PDT) From: Owen Hilyard To: dts@dpdk.org, lihongx.ma@intel.com Cc: ohilyard@iol.unh.edu, lylavoie@iol.unh.edu Date: Wed, 9 Sep 2020 17:08:06 -0400 Message-Id: <20200909210807.55311-1-ohilyard@iol.unh.edu> X-Mailer: git-send-email 2.25.1 In-Reply-To: References: MIME-Version: 1.0 Subject: [dts] [PATCH V1] tests/checksum_offload: Bugfix for bug 529 X-BeenThere: dts@dpdk.org X-Mailman-Version: 2.1.15 Precedence: list List-Id: test suite reviews and discussions List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: dts-bounces@dpdk.org Sender: "dts" added missing quote in template for scapy packets. rearraged closing down scapy and testpmd to avoid issues with other test cases. Signed-off-by: Owen Hilyard Signed-off-by: Owen Hilyard > Signed-off-by: Owen Hilyard > --- tests/TestSuite_checksum_offload.py | 11 ++++++----- 1 file changed, 6 insertions(+), 5 deletions(-) diff --git a/tests/TestSuite_checksum_offload.py b/tests/TestSuite_checksum_offload.py index c4a877d..f2e9878 100644 --- a/tests/TestSuite_checksum_offload.py +++ b/tests/TestSuite_checksum_offload.py @@ -374,7 +374,7 @@ class TestChecksumOffload(TestCase): def replay_pcap_file_on_tester(self, iface, packet_file_path): self.tester.send_expect("scapy", ">>>") self.scapy_exec(f"packets = rdpcap('{packet_file_path}')") - self.scapy_exec(f"sendp(packets, iface={iface})") + self.scapy_exec(f"sendp(packets, iface='{iface}')") self.tester.send_expect("quit()", "# ") def validate_packet_list_checksums(self, packets): @@ -703,7 +703,7 @@ class TestChecksumOffload(TestCase): for l4 in l4_protos: for chksum in "", "chksum=0xf": vf = self.send_pkt_expect_good_bad_from_flag_catch_failure( - f"eth/IP({chksum})/{l4}()/(X'*50)", + f"eth/IP({chksum})/{l4}()/('X'*50)", "PKT_RX_IP_CKSUM_", f"{l4}", should_pass=(chksum == "")) if vf is not None: @@ -846,14 +846,13 @@ class TestChecksumOffload(TestCase): # if vf is not None: # verification_errors.append(vf) - # tunneled inner + self.tester.send_expect("quit", "#") + self.dut.send_expect("stop", "testpmd>") for err in verification_errors: self.logger.error(str(err)) self.verify(len(verification_errors) == 0, "See previous output") - self.tester.send_expect("quit", "#") - self.dut.send_expect("stop", "testpmd>") def test_hardware_checksum_check_l4_tx(self): self.checksum_enablehw(self.dut_ports[0]) @@ -887,7 +886,9 @@ class TestChecksumOffload(TestCase): error_messages = self.validate_packet_list_checksums(captured_packets) + self.tester.send_expect("quit", "#") self.dut.send_expect("stop", "testpmd>") + if len(error_messages) != 0: for error_msg in error_messages: self.logger.error(error_msg)