From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from dpdk.org (dpdk.org [92.243.14.124]) by inbox.dpdk.org (Postfix) with ESMTP id AFD95A046B for ; Fri, 28 Jun 2019 11:09:19 +0200 (CEST) Received: from [92.243.14.124] (localhost [127.0.0.1]) by dpdk.org (Postfix) with ESMTP id 6D59A1B96B; Fri, 28 Jun 2019 11:09:19 +0200 (CEST) Received: from mail-vs1-f68.google.com (mail-vs1-f68.google.com [209.85.217.68]) by dpdk.org (Postfix) with ESMTP id 2F9A54F98 for ; Fri, 28 Jun 2019 11:09:16 +0200 (CEST) Received: by mail-vs1-f68.google.com with SMTP id q64so3587891vsd.1 for ; Fri, 28 Jun 2019 02:09:16 -0700 (PDT) X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:mime-version:references:in-reply-to:from:date :message-id:subject:to:cc; bh=tEgvrxv62MeIdfXSvnUp4zv1N2XX7tRjdgCkBB/YTRs=; b=TP3T+1y2Yhr249/CV9F/CQFmM06VJVtwTLplAyJDXjKzcjT7JVkHXp29gr53m8KtiO 6OtkGqqLEaP4lcy92wWPVnRVfJa5xi8XqqYTuXyjomSHnk4VszuRfli5vy+9lKopmtcl MUFS4nZ5MqUD+cqePlOF69L046DAY8SI+FLv95W2T66V48btZNieKulzW5jFLfy7d2Id qYzdqHa+bxEgcSrckus9XoytrJh5K4DQNBZsiPAv/9KiYmnuZjxeGJkBcceTBw2w9oyD kYRNLLKB83M8petN5Zq3wTh06jQUXnXa5knnU4JEZM7C1v9pRagOqp/lRJU8c7HbKMht 995A== X-Gm-Message-State: APjAAAX9PkuvGu6QU2wcaZolvaRhdiaKIxCnaY4WtPrszn9yxCiW0bTq zbuQwlo2A5c/MvnA3C3rFifWMkEgSkVtDpp+6pRvYrq8V9E= X-Google-Smtp-Source: APXvYqx5cMhveOCr3D995cZ8eTjwVCMZFzoYvUY3St8aDAqwCeag2ZUHq1SoXjxpKZksXb+Rjpc3h8fudsthyVB+RPg= X-Received: by 2002:a67:2ec8:: with SMTP id u191mr5945384vsu.39.1561712955460; Fri, 28 Jun 2019 02:09:15 -0700 (PDT) MIME-Version: 1.0 References: <20190628034406.5399-1-honnappa.nagarahalli@arm.com> In-Reply-To: <20190628034406.5399-1-honnappa.nagarahalli@arm.com> From: David Marchand Date: Fri, 28 Jun 2019 11:09:04 +0200 Message-ID: To: Honnappa Nagarahalli Cc: dev , ruifeng.wang@arm.com, nd , dpdk stable Content-Type: text/plain; charset="UTF-8" X-Content-Filtered-By: Mailman/MimeDel 2.1.15 Subject: Re: [dpdk-stable] [dpdk-dev] [PATCH 1/2] test/rcu: increase the size of num cores variable X-BeenThere: stable@dpdk.org X-Mailman-Version: 2.1.15 Precedence: list List-Id: patches for DPDK stable branches List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: stable-bounces@dpdk.org Sender: "stable" On Fri, Jun 28, 2019 at 5:44 AM Honnappa Nagarahalli < honnappa.nagarahalli@arm.com> wrote: > num_cores is of type uint8_t. This results in the following > compilation error. > > test_rcu_qsbr_perf.c:649:16: error: comparison is always false > due to limited range of data type [-Werror=type-limits] > if (num_cores >= RTE_MAX_LCORE) { > ^~ > > RTE_MAX_LCORE is set to 256 for armv8 config. > Weird I did not see this error in travis. Just tried again: https://travis-ci.com/david-marchand/dpdk/jobs/211768426 > Fixes: e6a14121f4ae ("test/rcu: remove arbitrary limit on max core count") > Cc: stable@dpdk.org > > Signed-off-by: Honnappa Nagarahalli > Reviewed-by: Ruifeng Wang > --- > app/test/test_rcu_qsbr.c | 4 ++-- > app/test/test_rcu_qsbr_perf.c | 2 +- > 2 files changed, 3 insertions(+), 3 deletions(-) > > diff --git a/app/test/test_rcu_qsbr.c b/app/test/test_rcu_qsbr.c > index 943a1e370..0c6267ee9 100644 > --- a/app/test/test_rcu_qsbr.c > +++ b/app/test/test_rcu_qsbr.c > @@ -27,7 +27,7 @@ > #define TEST_RCU_QSBR_CNT_INIT 1 > > uint16_t enabled_core_ids[RTE_MAX_LCORE]; > -uint8_t num_cores; > +uint16_t num_cores; > If we want to be closer to the eal API, those should be unsigned int. > static uint32_t *keys; > #define TOTAL_ENTRY (1024 * 8) > @@ -891,7 +891,7 @@ static int > test_rcu_qsbr_mw_mv_mqs(void) > { > int i, j; > - uint8_t test_cores; > + uint16_t test_cores; > > writer_done = 0; > test_cores = num_cores / 4; > diff --git a/app/test/test_rcu_qsbr_perf.c b/app/test/test_rcu_qsbr_perf.c > index 363365f46..b1a910423 100644 > --- a/app/test/test_rcu_qsbr_perf.c > +++ b/app/test/test_rcu_qsbr_perf.c > @@ -17,7 +17,7 @@ > > /* Check condition and return an error if true. */ > static uint16_t enabled_core_ids[RTE_MAX_LCORE]; > -static uint8_t num_cores; > +static uint16_t num_cores; > > static uint32_t *keys; > #define TOTAL_ENTRY (1024 * 8) > -- > 2.17.1 > -- David Marchand