Received: by 2002:ac0:a5a7:0:0:0:0:0 with SMTP id m36-v6csp807422imm; Wed, 25 Jul 2018 06:36:07 -0700 (PDT) X-Google-Smtp-Source: AAOMgpeaE8vfCyDl0A2o+U7fYqMAijjNWW4RODIq5H/1YeYu17Sn+Fv67H2jnfjwP4dcSRaWLbGH X-Received: by 2002:a17:902:7c90:: with SMTP id y16-v6mr8524845pll.99.1532525767573; Wed, 25 Jul 2018 06:36:07 -0700 (PDT) ARC-Seal: i=1; a=rsa-sha256; t=1532525767; cv=none; d=google.com; s=arc-20160816; b=fETMV2yXDdEdXc3SCwGbTI6vkizJ6uc99++9rEONCH3eNp6awmtYA8LOaLun04OGiT RuW/Nb8M0x8FSZqxRcZWrhXiQy1HC62efHYn+cjdWf/00iczJ07EchRnpHAV1OQcCk7y Zl/nyJRUqtJjZ2O3nJ5M4h0BjGcfiMORIb55A27qDXrBEpKPOIGKr54khZN+6ToZjRXJ gRyz0ZtC/jzI81X+Ax3I4lrgrtAx0sNYa9Pk9s3IM/YNSCAUVSi5tSaTCUkHxqJScctw IjCsWhThUypU37B27RkLglVIIYyvYcKKlxW0yvNRjdWeFAUwBw4ivrwAaEK7p6XC2qau 3Org== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:sender:content-transfer-encoding :content-language:in-reply-to:mime-version:user-agent:date :message-id:from:references:cc:to:subject:dkim-signature :arc-authentication-results; bh=MzhSwLs6JDLcZZP9ZORoYcwAxFxuqOuAMh5qsQ586OY=; b=nIJItCNtNWTTkb14G9bPiMiR0Kjuv7sf5wmA/FeD0L36hv28o9fzi5/xxKBAV2MVUP 9MZffHiPwhVW114M6LjsE1D+smXrokyf0KvPCXc2dBHu++G1SX4STng2AngjZXPSivWk E1/W7pIOJez3ciF95eOIm/iudt9sCMKqbG9CVJePm5PBl+1rr4jVCVlk5L2qDXzZcrqy R0KTZ2oEQdoxin1/GzOMGYc+wj7FHEVAmJVI+OzuJmRnHOD3wSEIjIbIFcpJH2C2GKsS 2kRtAfVlbt57gjcgCq4P8Y9MHQKB6qB+voQZza6pwjCPZfM4if0hKVxKayHNzYkvEiky ACsg== ARC-Authentication-Results: i=1; mx.google.com; dkim=pass header.i=@gmail.com header.s=20161025 header.b=qlzSVD2A; spf=pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=pass (p=NONE sp=QUARANTINE dis=NONE) header.from=gmail.com Return-Path: Received: from vger.kernel.org (vger.kernel.org. [209.132.180.67]) by mx.google.com with ESMTP id n59-v6si12576390plb.198.2018.07.25.06.35.52; Wed, 25 Jul 2018 06:36:07 -0700 (PDT) Received-SPF: pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) client-ip=209.132.180.67; Authentication-Results: mx.google.com; dkim=pass header.i=@gmail.com header.s=20161025 header.b=qlzSVD2A; spf=pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=pass (p=NONE sp=QUARANTINE dis=NONE) header.from=gmail.com Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1729176AbeGYOpZ (ORCPT + 99 others); Wed, 25 Jul 2018 10:45:25 -0400 Received: from mail-pg1-f193.google.com ([209.85.215.193]:44044 "EHLO mail-pg1-f193.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1728529AbeGYOpZ (ORCPT ); Wed, 25 Jul 2018 10:45:25 -0400 Received: by mail-pg1-f193.google.com with SMTP id r1-v6so5339579pgp.11; Wed, 25 Jul 2018 06:33:43 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20161025; h=subject:to:cc:references:from:message-id:date:user-agent :mime-version:in-reply-to:content-language:content-transfer-encoding; bh=MzhSwLs6JDLcZZP9ZORoYcwAxFxuqOuAMh5qsQ586OY=; b=qlzSVD2A5ayTWCP0SzVyJCyBesSWLY+oDBr6ZMSiArl0TMFeqnYPqK1DflCqdfUiP2 pc6vbU9PwbBFnPuHYWBCXs1FBBYKIwS+AZb9sklxxDR+B2WytUP9iNmKRKUGi3IWRfPl S0aqmvcLEodNVol0inAMhhTb0c8G/5HuaCV6+d7orfYCyWw1uZEugBwDaIsULJg62diy GljuoyEiWyU8PRdDQgpTFJ709FHyCpugPjCL9w056QbjupmR1h2k7sP7YNZULNC5TSFn iXtwPiCXlj4foG74pcPwp9kUa57xpIBLGpi5zCSr8U/06VXulbBdGZ/96WtcyWgjmGMW HOBw== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:subject:to:cc:references:from:message-id:date :user-agent:mime-version:in-reply-to:content-language :content-transfer-encoding; bh=MzhSwLs6JDLcZZP9ZORoYcwAxFxuqOuAMh5qsQ586OY=; b=OMrYChjqQo/68fpsOt4GfV6kX/mdP25vdOa8FmnIHoX4iFthXDMPVw1HUdKfEktp/A GItBWaWPbtVXRB+0bQ+telZJpH4H4zREQvTtVsokZAO41cnPhxNOP5cG3Z7Mc1IQoUbv eN2X8Ow9gjf2u/kxYZuJMsoxOMTXmh3+KAeWNloj1Xu2MDAzOQ0J5i8Wr0WjDfStx9YL c4z5Xu3jRPyyArfSqR29413S35afJh6C8wQAZ5lSf89q0L7/stta3/73rzIqXk08tepD 1dmoX/Wil+nm2Zy480j0DBfLG5J9gRxKt15QhBOg9HCzVSXEguE0B+2JxZMJvAU/R2jV +hRw== X-Gm-Message-State: AOUpUlEZ3NgUmwqaLV7WzHD0xvE0Bx+BHrNIgx8vTsuxWzdhlAYDpwL1 PDxuosKKJGNLkfHTKn/J2j7y4t7U X-Received: by 2002:a62:9b57:: with SMTP id r84-v6mr22047014pfd.6.1532525623143; Wed, 25 Jul 2018 06:33:43 -0700 (PDT) Received: from [192.168.86.235] (c-67-180-167-114.hsd1.ca.comcast.net. [67.180.167.114]) by smtp.gmail.com with ESMTPSA id t14-v6sm15224855pgu.0.2018.07.25.06.33.42 (version=TLS1_2 cipher=ECDHE-RSA-AES128-GCM-SHA256 bits=128/128); Wed, 25 Jul 2018 06:33:42 -0700 (PDT) Subject: Re: [PATCH net-next] tcp: add SNMP counter for the number of packets pruned from ofo queue To: Yafang Shao , davem@davemloft.net, edumazet@google.com Cc: netdev@vger.kernel.org, linux-kernel@vger.kernel.org References: <1532524010-11855-1-git-send-email-laoar.shao@gmail.com> From: Eric Dumazet Message-ID: <047bc12f-9eb5-e512-10c5-6334c494c77e@gmail.com> Date: Wed, 25 Jul 2018 06:33:41 -0700 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:52.0) Gecko/20100101 Thunderbird/52.6.0 MIME-Version: 1.0 In-Reply-To: <1532524010-11855-1-git-send-email-laoar.shao@gmail.com> Content-Type: text/plain; charset=utf-8 Content-Language: en-US Content-Transfer-Encoding: 7bit Sender: linux-kernel-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On 07/25/2018 06:06 AM, Yafang Shao wrote: > LINUX_MIB_OFOPRUNED is used to count how many times ofo queue is pruned, > but sometimes we want to know how many packets are pruned from this queue, > that could help us to track the dropped packets. > > As LINUX_MIB_OFOPRUNED is a useful event for us, so I introduce a new > SNMP counter LINUX_MIB_OFOPRUNEDROP, which could be showed in netstat as > OfoPruneDrop. Okay, but why tracking number of skbs that are removed ? Skb can contain many segments (because of GRO and TCP coalescing) So your claim of tracking dropped packets is ill defined. Also I prefer having net tree being merged into net-next, since your patch would add a merge conflict.