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 EC0FD42486; Wed, 25 Jan 2023 14:50:06 +0100 (CET) Received: from mails.dpdk.org (localhost [127.0.0.1]) by mails.dpdk.org (Postfix) with ESMTP id 890F142D3E; Wed, 25 Jan 2023 14:50:06 +0100 (CET) Received: from us-smtp-delivery-124.mimecast.com (us-smtp-delivery-124.mimecast.com [170.10.129.124]) by mails.dpdk.org (Postfix) with ESMTP id 9BCD542D31 for ; Wed, 25 Jan 2023 14:50:05 +0100 (CET) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=redhat.com; s=mimecast20190719; t=1674654605; h=from:from:reply-to:subject:subject:date:date:message-id:message-id: to:to:cc:cc:mime-version:mime-version:content-type:content-type: content-transfer-encoding:content-transfer-encoding: in-reply-to:in-reply-to:references:references; bh=xYjmY7jQUplUQ+jiZ+BvyIR56nb+MZj4ByE1hZG5lHM=; b=OLn+kwCtrtbGhKnpLgGShH1VaaWY2QF+o/uOUPSaXmrArPNOE0HBKHPC5iIoLlGeGPr5YK g066h/2w2ZEStDi8wra0moA5oOAbiCpDS4EHbnB2lWZLN2pheqPEeqZtKD98HTAUTgGU9e TjZgXPIKLxyhbVVUCujZe4Ha/u4sYaE= Received: from mail-wm1-f71.google.com (mail-wm1-f71.google.com [209.85.128.71]) by relay.mimecast.com with ESMTP with STARTTLS (version=TLSv1.3, cipher=TLS_AES_128_GCM_SHA256) id us-mta-221-jHPdm8wpO92GOGui03T_pw-1; Wed, 25 Jan 2023 08:50:03 -0500 X-MC-Unique: jHPdm8wpO92GOGui03T_pw-1 Received: by mail-wm1-f71.google.com with SMTP id p1-20020a05600c1d8100b003daff82f5edso10940029wms.8 for ; Wed, 25 Jan 2023 05:50:03 -0800 (PST) X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20210112; h=in-reply-to:references:to:from:subject:cc:message-id:date :content-transfer-encoding:mime-version:x-gm-message-state:from:to :cc:subject:date:message-id:reply-to; bh=xYjmY7jQUplUQ+jiZ+BvyIR56nb+MZj4ByE1hZG5lHM=; b=THit5V4Dary80h7uDzaEBiB6Fxpp4h2Ui5t0hVwitjMJmFqFQirDc/uf84KSk0sjy1 qU8mYHlfvyTOa8klSsaYwUu5Nji62UEolHGHu1p+Im9DyZ5V338l0TZP9z0kCTgOy8p8 VvGQ9UV5QzRbGd8HhdvEOVoZsOrCweb8Y+m9yrtQJ9xPOSluXJDS3QTg2azxkz65HHRV 8SVCg609aP1ZejQOaIuW6pCRDbEP3o/kUbwpgGJ8Rh7YXIhhw5mRKiYuYB1V8dbx5FHN qOvFhlxRhmF1sSNZoPaADrPeLXw+mTUJ7tuFi2A4LJJLuxdUf6FEUG0ck3he2uvkHila jPWg== X-Gm-Message-State: AFqh2kquchUi3UM+d2QaIFGFy/1/+/dsovfWWPtbA2LnPcZ8iG9l8GXv iRhakmjB9ULy55nasjk0P+q4O6UL8AwELi2283NY3e4L6bWZ1plvvxRdmxUM1yex7LYdkGZDyMx rf4c= X-Received: by 2002:a05:600c:1c23:b0:3db:b9f:f2e with SMTP id j35-20020a05600c1c2300b003db0b9f0f2emr29763363wms.14.1674654602843; Wed, 25 Jan 2023 05:50:02 -0800 (PST) X-Google-Smtp-Source: AMrXdXs4e6AmgDc0s6SyYU17MLg+Fotol061Kwfe82wNDkecSMKw2yx5LNRS2xH+IYZqwdljCSX5uQ== X-Received: by 2002:a05:600c:1c23:b0:3db:b9f:f2e with SMTP id j35-20020a05600c1c2300b003db0b9f0f2emr29763347wms.14.1674654602658; Wed, 25 Jan 2023 05:50:02 -0800 (PST) Received: from localhost (2a01cb000f483e0055ae3800781b5cbc.ipv6.abo.wanadoo.fr. [2a01:cb00:f48:3e00:55ae:3800:781b:5cbc]) by smtp.gmail.com with ESMTPSA id r20-20020a05600c35d400b003db16770bc5sm2169508wmq.6.2023.01.25.05.50.02 (version=TLS1_3 cipher=TLS_AES_128_GCM_SHA256 bits=128/128); Wed, 25 Jan 2023 05:50:02 -0800 (PST) Mime-Version: 1.0 Date: Wed, 25 Jan 2023 14:50:01 +0100 Message-Id: Cc: "Ferruh Yigit" , "Aman Singh" , "Yuying Zhang" Subject: Re: [PATCH 0/6] Testpmd code cleanup From: "Robin Jarry" To: "David Marchand" , X-Mailer: aerc/0.14.0-35-g8614fef5610d References: <20230124104742.1265439-1-david.marchand@redhat.com> In-Reply-To: <20230124104742.1265439-1-david.marchand@redhat.com> X-Mimecast-Spam-Score: 1 X-Mimecast-Originator: redhat.com Content-Transfer-Encoding: quoted-printable Content-Type: text/plain; charset=UTF-8; format=Flowed 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 David Marchand, Jan 24, 2023 at 11:47: > Here is a series to reduce code duplication in testpmd. > > This work started from looking at Robin series on reporting lcore busy=20 > cycles in telemetry, which is then added in testpmd [1]. While looking=20 > at the forward engines code, I saw way too much duplicated code. > > Warning: this is only compile tested. > > 1: https://patchwork.dpdk.org/project/dpdk/patch/20230119150656.418404-5-= rjarry@redhat.com/ Hi David, The code looks good to me. I have made some basic testing, it seems not=20 to break obvious things. Reviewed-by: Robin Jarry