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 DB6E3432EA; Thu, 9 Nov 2023 20:41:41 +0100 (CET) Received: from mails.dpdk.org (localhost [127.0.0.1]) by mails.dpdk.org (Postfix) with ESMTP id 67F0F40DC9; Thu, 9 Nov 2023 20:41:41 +0100 (CET) Received: from mail-pl1-f181.google.com (mail-pl1-f181.google.com [209.85.214.181]) by mails.dpdk.org (Postfix) with ESMTP id 9080C4026B for ; Thu, 9 Nov 2023 20:41:40 +0100 (CET) Received: by mail-pl1-f181.google.com with SMTP id d9443c01a7336-1cc30bf9e22so10916195ad.1 for ; Thu, 09 Nov 2023 11:41:40 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=networkplumber-org.20230601.gappssmtp.com; s=20230601; t=1699558899; x=1700163699; darn=dpdk.org; h=content-transfer-encoding:mime-version:references:in-reply-to :message-id:subject:cc:to:from:date:from:to:cc:subject:date :message-id:reply-to; bh=IhfJoOZwy1lKGK7R/S4Rmik6iuBJ+tmB5RiPzAlOl38=; b=oyN9stNqownvxgW6lCwyQV5sDXFp+0Lf0Kbkvt4DN840cYl2jjSrCvCgICEjotCYjW aIWB/wtTGhyGFLMW+icV0NbfNjaqFthoq1sgKEn8asJ36INwtVNPVO8bSKX+ersQNwWl 21PYHuudZYF63GZAn6DPN8usRU5dWnJCJLo4oG8xoD/90WmuBE2KGVZ590Q8f0pi06WX 0zuV9MbqfyUOURnb+Mp92sf48b+eNpo3Py+clPca4L6xWb9lTdiiMGutRxP7Gb8O28KM NyJ0RkvNgiLLT9AkS/LbrLqY+I8kSvbXSuYblcULe+sJhp1VOZl7FQ///ohpuuJX+4/v C0gg== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20230601; t=1699558899; x=1700163699; h=content-transfer-encoding:mime-version:references:in-reply-to :message-id:subject:cc:to:from:date:x-gm-message-state:from:to:cc :subject:date:message-id:reply-to; bh=IhfJoOZwy1lKGK7R/S4Rmik6iuBJ+tmB5RiPzAlOl38=; b=mbb1F7QL0qMzWZrqAtBKnooBrZgoldHEBrNIYe44trU0ePyH2yglUDpwxGPpLbP2Di 6EV99muuFggRF/30M2cwsrgpzcWfi+NE8i2Q8YjM3CaIF5vnJRuFzlpy8Wyg9ic0yIah yIUhJ+AZJltFCiFI19F4S+X99yIH2KwtHOcqbjVZ2MslI4ObAe/b3FDaUq9Z3phIcwme LPTPSIzHcM2erlfCR3Y7sJhNU1tO7j33q574nmM/+CRgqLGMSs0dBvINZXcNeOfoh1Am k/4QWLsh52ndrz3xgBmTlXlDkvu7KVQkFPiFPCj/oGk6j9BmNbeEOYSIYfx8R+EWHzzY LonA== X-Gm-Message-State: AOJu0YxRZnshhdtTEUmArSqmDM9Zu1hHW+DmXzMw7EQK9vrbsh8yzkqM MUjM+RCbvCkY8QEac3+0XqjEBQ== X-Google-Smtp-Source: AGHT+IFKnLC+6fzrARuTXcsXfQKt6+fyj3rUQAjuS9gQt5VKm2DOZoz981RhCx9zTfPrgo+QXfO1Rg== X-Received: by 2002:a17:902:ce8e:b0:1cc:31c4:3426 with SMTP id f14-20020a170902ce8e00b001cc31c43426mr353657plg.11.1699558899504; Thu, 09 Nov 2023 11:41:39 -0800 (PST) Received: from hermes.local (204-195-123-141.wavecable.com. [204.195.123.141]) by smtp.gmail.com with ESMTPSA id jd18-20020a170903261200b001cc50146b43sm3885823plb.202.2023.11.09.11.41.39 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Thu, 09 Nov 2023 11:41:39 -0800 (PST) Date: Thu, 9 Nov 2023 11:41:37 -0800 From: Stephen Hemminger To: Gregory Etelson Cc: , , , Ori Kam , Aman Singh , Yuying Zhang Subject: Re: [PATCH v2] app/testpmd: fix indirect action list parameters parsing Message-ID: <20231109114137.7cf4292c@hermes.local> In-Reply-To: <20231109183637.498742-1-getelson@nvidia.com> References: <20231108163445.422772-1-getelson@nvidia.com> <20231109183637.498742-1-getelson@nvidia.com> MIME-Version: 1.0 Content-Type: text/plain; charset=US-ASCII Content-Transfer-Encoding: 7bit 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 On Thu, 9 Nov 2023 20:36:37 +0200 Gregory Etelson wrote: > diff --git a/app/test-pmd/cmdline_flow.c b/app/test-pmd/cmdline_flow.c > index 0d521159e9..397f9bc3eb 100644 > --- a/app/test-pmd/cmdline_flow.c > +++ b/app/test-pmd/cmdline_flow.c > @@ -11331,7 +11331,7 @@ parse_indlst_id2ptr(struct context *ctx, const struct token *token, > struct rte_flow_action *action = ctx->object; > struct rte_flow_action_indirect_list *action_conf; > const struct indlst_conf *indlst_conf; > - uint32_t id; > + uintptr_t id; > int ret; > > if (!action) > @@ -11350,7 +11350,8 @@ parse_indlst_id2ptr(struct context *ctx, const struct token *token, > action_conf->handle = (typeof(action_conf->handle)) > port_action_handle_get_by_id(ctx->port, id); > if (!action_conf->handle) { > - printf("no indirect list handle for id %u\n", id); > + printf("no indirect list handle for id %"PRIu64"\n", > + id); On 32 bit platforms uintptr_t is 32 bits. Uintptr_t is always a typedef for unsigned long so use %lu here instead.