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 05A7EA3160 for ; Wed, 9 Oct 2019 19:26:14 +0200 (CEST) Received: from [92.243.14.124] (localhost [127.0.0.1]) by dpdk.org (Postfix) with ESMTP id EC5571C1B8; Wed, 9 Oct 2019 19:26:13 +0200 (CEST) Received: from mail-pg1-f196.google.com (mail-pg1-f196.google.com [209.85.215.196]) by dpdk.org (Postfix) with ESMTP id 722E71C220 for ; Wed, 9 Oct 2019 19:26:12 +0200 (CEST) Received: by mail-pg1-f196.google.com with SMTP id e1so1839702pgj.6 for ; Wed, 09 Oct 2019 10:26:12 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=networkplumber-org.20150623.gappssmtp.com; s=20150623; h=date:from:to:cc:subject:message-id:in-reply-to:references :mime-version:content-transfer-encoding; bh=/t+me8OjrSWD+MJI3q0kX2x0yrnQn+NAvv8eZ5rZtsc=; b=mTavlYvQV6ei2swVFI5B0cqme02+oGAHWTuWxBWWrVMyXvezd8r3yfFB3hZrFdMy0h vsM5L6hUSThqXtClY8ZNycIWPhWmTQvyNYKYVNrobu9tfkZqkxGBFc0lzIxB2Wg+t00V cLljpfMuACJ/lJCao9agswmhkIrj4VEZmuGHgSeBeExuYsncmMh7gs+NxpRbfIzO7oyp oHIZOVG3WKugpJwcbq0eFYtgZKSM7IvKa7e2oPQf09zcpYvMquwcK5n2UIJlHhdhlGQ4 kgPpLF7fSSBWiUOjEM/QJlEHx5HhJi+tA1o/qAib9wa77D2Rq4UE9fc08LUuTdNPLWRm hLeA== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:date:from:to:cc:subject:message-id:in-reply-to :references:mime-version:content-transfer-encoding; bh=/t+me8OjrSWD+MJI3q0kX2x0yrnQn+NAvv8eZ5rZtsc=; b=DeEpwsWhQsJ0Zs9okJq28bGCOjLRhloigBKf8rPY6IDy0xMWaCoyz8gJdBdpPiCGS7 UlPSSk4IxV+rpz1Qc2sS2D3s2OefkRKFqSHriMCglTE5F4ewmiY/BLgs47ZTcHvyoVzW YqEGu2tNER6CZFryqptbfMvH1rnr4q5XYJ0yGOxi7T0xHrQ0HQJlV1E/80plYVt/am2F UX7yjUrDOT0e+QSIwAxCp74RtXlZlxHjruhJ/LB4hbxoRqXITUCNbUjs4Bqsg/w1k2+c BMKEuIi0FPEZKCqJlhKEW85w6sfmZFNE+ITt4MOQm/R42nqPwokMHce0UI8j3tGSZpXk euMQ== X-Gm-Message-State: APjAAAVUYz6s9bDSNWWjjZwtYs64lSvmSOOYj8NEHJyM8pyYqiq68wqQ pUSzasN/5/qkkYVGer3BC1BF4w== X-Google-Smtp-Source: APXvYqx3FNQmuisEPFQcd68dC2XCiyqA7/3BgzxS0tns18gC77qhTDDzZp2mnOa4kbQDdyt1TiD7CQ== X-Received: by 2002:a62:d408:: with SMTP id a8mr4923546pfh.15.1570641971639; Wed, 09 Oct 2019 10:26:11 -0700 (PDT) Received: from hermes.lan (204-195-22-127.wavecable.com. [204.195.22.127]) by smtp.gmail.com with ESMTPSA id x9sm3474268pje.27.2019.10.09.10.26.11 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Wed, 09 Oct 2019 10:26:11 -0700 (PDT) Date: Wed, 9 Oct 2019 10:26:09 -0700 From: Stephen Hemminger To: Flavia Musatescu Cc: dev@dpdk.org, "John W. Linville" , stable@dpdk.org, ciwillia@brocade.com Message-ID: <20191009102609.2961290a@hermes.lan> In-Reply-To: <1570634949-26819-1-git-send-email-flavia.musatescu@intel.com> References: <1570634949-26819-1-git-send-email-flavia.musatescu@intel.com> MIME-Version: 1.0 Content-Type: text/plain; charset=US-ASCII Content-Transfer-Encoding: 7bit Subject: Re: [dpdk-stable] [dpdk-dev] [PATCH] net/af_packet: improve Tx statistics accuracy 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 Wed, 9 Oct 2019 16:29:09 +0100 Flavia Musatescu wrote: > + if (sendto(pkt_q->sockfd, NULL, 0, MSG_DONTWAIT, NULL, 0) == -1 && > + errno != ENOBUFS) { > + /* Error sending. > + * When sendto call fails and ENOBUFS error is being set > + * some of the packets are actually successfully transmitted. > + * There is no available count of those packets, so in order > + * to make the statistics more accurate, all of the previously > + * enqueued packets will be considered successful, even though > + * this is not entirely correct. > + */ Agree with the change, but please keep only add a small code comment. Long comments are a personal pet peeve, it also tends to standout as "some different developer who had an issue left this comment"