From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mails.dpdk.org (mails.dpdk.org [217.70.189.124]) by inbox.dpdk.org (Postfix) with ESMTP id B43D84626E; Thu, 20 Feb 2025 03:02:12 +0100 (CET) Received: from mails.dpdk.org (localhost [127.0.0.1]) by mails.dpdk.org (Postfix) with ESMTP id EE8DD402C6; Thu, 20 Feb 2025 03:01:54 +0100 (CET) Received: from linux.microsoft.com (linux.microsoft.com [13.77.154.182]) by mails.dpdk.org (Postfix) with ESMTP id 4719A402C4 for ; Thu, 20 Feb 2025 03:01:48 +0100 (CET) Received: by linux.microsoft.com (Postfix, from userid 1213) id 48EA9204E5A3; Wed, 19 Feb 2025 18:01:47 -0800 (PST) DKIM-Filter: OpenDKIM Filter v2.11.0 linux.microsoft.com 48EA9204E5A3 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=linux.microsoft.com; s=default; t=1740016907; bh=jaf8IHbxA1OeStjJ5GJLM5EXMesSgLeWDrGHlc9auNk=; h=From:To:Cc:Subject:Date:In-Reply-To:References:From; b=X1kp6udS5lT9jQMS73edXC/xVpANyeCER1jNfXaQbIoygKkrsx3EMHwXM1BFWeBSl EPw2LYAgkMfUkRj6aUn8o8bgeh1NsMGLP4MjrS7r9tjXoic90Ed/wv4chHw3wAcxi7 x2O/n2WVwZ2l7QU/gaX1Sa39abHHhB6b/8fgg//4= From: Andre Muezerie To: andremue@linux.microsoft.com Cc: dev@dpdk.org, Chengwen Feng Subject: [PATCH v3 02/10] test_alarm: avoid warning about different qualifiers Date: Wed, 19 Feb 2025 18:01:31 -0800 Message-Id: <1740016899-2817-3-git-send-email-andremue@linux.microsoft.com> X-Mailer: git-send-email 1.8.3.1 In-Reply-To: <1740016899-2817-1-git-send-email-andremue@linux.microsoft.com> References: <1739311325-14425-1-git-send-email-andremue@linux.microsoft.com> <1740016899-2817-1-git-send-email-andremue@linux.microsoft.com> X-BeenThere: dev@dpdk.org X-Mailman-Version: 2.1.29 Precedence: list List-Id: DPDK patches and discussions List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: dev-bounces@dpdk.org Compiling with MSVC results in the warning below: app/test/test_alarm.c(54): warning C4090: 'function': different '_Atomic' qualifiers The fix is to use a macro to explicitly drop the qualifier. Signed-off-by: Andre Muezerie Signed-off-by: Chengwen Feng --- app/test/test_alarm.c | 12 ++++++------ 1 file changed, 6 insertions(+), 6 deletions(-) diff --git a/app/test/test_alarm.c b/app/test/test_alarm.c index 9ed8c6f72c..6445f713fe 100644 --- a/app/test/test_alarm.c +++ b/app/test/test_alarm.c @@ -51,12 +51,12 @@ test_alarm(void) "Expected rte_eal_alarm_cancel to fail with null callback parameter"); /* check if can set a alarm for one second */ - TEST_ASSERT_SUCCESS(rte_eal_alarm_set(US_PER_SEC, test_alarm_callback, &triggered), - "Setting one second alarm failed"); + TEST_ASSERT_SUCCESS(rte_eal_alarm_set(US_PER_SEC, test_alarm_callback, + RTE_PTR_UNQUAL(&triggered)), "Setting one second alarm failed"); /* set a longer alarm that will be canceled. */ - TEST_ASSERT_SUCCESS(rte_eal_alarm_set(10 * US_PER_SEC, test_alarm_callback, &later), - "Setting ten second alarm failed"); + TEST_ASSERT_SUCCESS(rte_eal_alarm_set(10 * US_PER_SEC, test_alarm_callback, + RTE_PTR_UNQUAL(&later)), "Setting ten second alarm failed"); /* wait for alarm to happen */ while (rte_atomic_load_explicit(&triggered, rte_memory_order_acquire) == false) @@ -65,11 +65,11 @@ test_alarm(void) TEST_ASSERT(!rte_atomic_load_explicit(&later, rte_memory_order_acquire), "Only one alarm should have fired."); - ret = rte_eal_alarm_cancel(test_alarm_callback, &triggered); + ret = rte_eal_alarm_cancel(test_alarm_callback, RTE_PTR_UNQUAL(&triggered)); TEST_ASSERT(ret == 0 && rte_errno == ENOENT, "Canceling alarm after run ret %d: %s", ret, rte_strerror(rte_errno)); - ret = rte_eal_alarm_cancel(test_alarm_callback, &later); + ret = rte_eal_alarm_cancel(test_alarm_callback, RTE_PTR_UNQUAL(&later)); TEST_ASSERT(ret == 1, "Canceling ten second alarm failed %d: %s", ret, rte_strerror(rte_errno)); -- 2.48.1.vfs.0.0