From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mga07.intel.com (mga07.intel.com [134.134.136.100]) by dpdk.org (Postfix) with ESMTP id BA7E7282 for ; Thu, 30 Mar 2017 21:31:05 +0200 (CEST) Received: from fmsmga002.fm.intel.com ([10.253.24.26]) by orsmga105.jf.intel.com with ESMTP; 30 Mar 2017 12:31:05 -0700 X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="5.36,248,1486454400"; d="scan'208";a="1148967604" Received: from silpixa00398672.ir.intel.com ([10.237.223.128]) by fmsmga002.fm.intel.com with ESMTP; 30 Mar 2017 12:31:04 -0700 From: Harry van Haaren To: dev@dpdk.org Cc: jerin.jacob@caviumnetworks.com, Harry van Haaren Date: Thu, 30 Mar 2017 20:30:30 +0100 Message-Id: <1490902250-32164-3-git-send-email-harry.van.haaren@intel.com> X-Mailer: git-send-email 2.7.4 In-Reply-To: <1490902250-32164-1-git-send-email-harry.van.haaren@intel.com> References: <1490829963-106807-1-git-send-email-harry.van.haaren@intel.com> <1490902250-32164-1-git-send-email-harry.van.haaren@intel.com> Subject: [dpdk-dev] [PATCH v7 02/22] test/eventdev: pass timeout ticks unsupported X-BeenThere: dev@dpdk.org X-Mailman-Version: 2.1.15 Precedence: list List-Id: DPDK patches and discussions List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Thu, 30 Mar 2017 19:31:06 -0000 This commit reworks the return value handling of the timeout ticks test. This feature is not mandatory for a pmd, the eventdev layer returns -ENOTSUP if the PMD doesn't implement the function. The test is modified to check if the return value is -ENOTSUP, and return -ENOTSUP to the test framework, which can handle "unsupported" tests since patch[1]. As such, this test will function correctly if the patchset linked below is applied, it fails if the patch is not applied and the PMD doesn't the timeout ticks function. Note it does not depend (as a compile time dependency) on the patchset linked below. Signed-off-by: Harry van Haaren Acked-by: Jerin Jacob [1] http://dpdk.org/dev/patchwork/patch/21979/ --- v6: - Fix return value to "return ret" instead of -ENOTSUP (Jerin) --- test/test/test_eventdev.c | 5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-) diff --git a/test/test/test_eventdev.c b/test/test/test_eventdev.c index 0f1deb6..b568470 100644 --- a/test/test/test_eventdev.c +++ b/test/test/test_eventdev.c @@ -519,9 +519,10 @@ test_eventdev_timeout_ticks(void) uint64_t timeout_ticks; ret = rte_event_dequeue_timeout_ticks(TEST_DEV_ID, 100, &timeout_ticks); - TEST_ASSERT_SUCCESS(ret, "Fail to get timeout_ticks"); + if (ret != -ENOTSUP) + TEST_ASSERT_SUCCESS(ret, "Fail to get timeout_ticks"); - return TEST_SUCCESS; + return ret; } -- 2.7.4