From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from mail-wr1-f50.google.com (mail-wr1-f50.google.com [209.85.221.50]) (using TLSv1.2 with cipher ECDHE-RSA-AES128-GCM-SHA256 (128/128 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id 03CF42F7EEB for ; Fri, 15 May 2026 17:14:40 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=209.85.221.50 ARC-Seal:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1778865282; cv=none; b=pdyxYl2/ADpNF7OU7Tz7+9/F2zq2kwSMT7oZW/eNm758x+WqQNQkoEFHZM/RuRuv8qiE5Nf5oPZIZDSzcLBRc0A2NVh/CV1BVdqKpmbAtjrWYklcnZVH2/G0T5Xv+qPi2tZHip27RXN/cPdHP+2SzEfp4sOgPCYm0R+EZYXlAfg= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1778865282; c=relaxed/simple; bh=LC9OHNbKrC9FZOQQTTEG4Hsi4cBNCQbZa9CGypc8kAU=; h=Message-ID:Date:MIME-Version:Subject:To:Cc:References:From: In-Reply-To:Content-Type; b=cl37riEz8jjgPsWy5q5mGf9nnLERqimp7hceHUw7o+SuRQVd6zGRMjWlG9jQmLVdhb6Q3bJdVd5QY41vuh77JNJBbZzb3JUiCspC90xtiHaVXXHsjLS35Co1LDYLHYZxk4qnyAvJNktqMnLAnN8kaiYV87wLZ+96Ej1YV0bekZw= ARC-Authentication-Results:i=1; smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=gmail.com; spf=pass smtp.mailfrom=gmail.com; dkim=pass (2048-bit key) header.d=gmail.com header.i=@gmail.com header.b=f5VFlTlw; arc=none smtp.client-ip=209.85.221.50 Authentication-Results: smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=gmail.com Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=gmail.com Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=gmail.com header.i=@gmail.com header.b="f5VFlTlw" Received: by mail-wr1-f50.google.com with SMTP id ffacd0b85a97d-44b330c5cc6so9346f8f.1 for ; Fri, 15 May 2026 10:14:40 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20251104; t=1778865279; x=1779470079; darn=vger.kernel.org; h=content-transfer-encoding:in-reply-to:from:content-language :references:cc:to:subject:user-agent:mime-version:date:message-id :from:to:cc:subject:date:message-id:reply-to; bh=DJZqzlC/MyiJvlvFMtDR9fBoSavw/3np+GiRJ/Jc3kc=; b=f5VFlTlwlYmSe8QBChqZRu+LWF1ATfuRN2mjf/AwvGNoh5XQat36fadiqP76vJdx7U xTvn0TnSmfimZJ2yFQAcdC9TfEQmkkR4Uwi5vd/zaVkbFc9AvfNIqhF2QYC8ULkCDfOD osUsSOvO9sMMtRvxJLHQ62H1yYXob9slmx/x/tgN35lB0VzMZEkSkaY69ENEe3As2SDb behhUt0p8zeE+Qa36HjuegqtsCTGYihudraO/f/AYlEXm0AhTUTKSYaTdVm/fNM2R3rf vW/S90iOpOKruwXLzVKOgT2fPMkcYbr0d/4hvGex+cwElBI3EjAcpcoTpo0lRmhGCVEb qRQA== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20251104; t=1778865279; x=1779470079; h=content-transfer-encoding:in-reply-to:from:content-language :references:cc:to:subject:user-agent:mime-version:date:message-id :x-gm-gg:x-gm-message-state:from:to:cc:subject:date:message-id :reply-to; bh=DJZqzlC/MyiJvlvFMtDR9fBoSavw/3np+GiRJ/Jc3kc=; b=T7CDRPR56oNUCQ6rTWGq9571TFWTCSSbZxLTmCwnBdlTLW06xuxPKr3vvYSnm5obtU G6FUA2RdWbhzYlXzUoyfhSrkXwXToPkykNpS4lko4xbiJktpYPZi+I8S7CkMXK0nEmf0 OxONpl37ROXyBxCiSKk8kFO8ip7qllBvFbGmZ07r442vOTbjE7FiyhDNNpWXpzgc+QAn C6rXapzSFYqs9sIg8vgMMNUwrFtxoVSglAZXUx80WpkjyA8k2QSUXmcwjtg4PM7LECN3 6LE9P3Il7o5PWLSiQMoSnTzIzOLtXQSKggnJZ6+noQ0sgWwgdIBdiHgughdAp2C4a1oY ww+w== X-Forwarded-Encrypted: i=1; AFNElJ+gaWC2k0JPRs00NeQ0yT0HLBuEe5EqqihAlEq+jpunnzyBmBhRzpooRG4ZNBKZbQXrZlD+DBTh7v8sn5M=@vger.kernel.org X-Gm-Message-State: AOJu0YwJzv6gzfLyCNZ0Rc1tmCyy+PXzVSva0pCbJgoB/j8XRUFCD03L +X9gbfu/YFAknbTuNZ45ti+STB75WtwgpVVzYubhB6d5soDZmOrUPzTI X-Gm-Gg: Acq92OEo13DeVF8V9b9kXRaw62ZO/qvPkQwAhDzQ8KBjI/HDCysGCT/Y2v7Z0wxkJaz GLeR72H7yRGRRx6AGq7j2LaUYp7WlzHmSPW7RJQ9MV/1rk3ZCpAbCyNYrSt7/jfwQrNkrtfO2Z6 iG0IxCMPoBXdbNPn3sPEcHbUZXGrdsDMQddJww8UMaQzB1VEOqEsZFyGb7c1PnAaPn5N2hJ+EZs 6Xa7VCYYiYMGyTHSa7mPnIJ+N2FZnHk/Pq9czKeWGnc0KOO7Y1CuPop1bUjxU45dCQaS6G2I27F fr4+QXqKMN1rU26kJpV8xl19JEdZYUt5c/FiBUvdi5oBAaMSrM1iUJ/ziDcP/iBUTNoatnuFRLL BdhWez0rWdVp0E5cNiKw1/Y1r+4SOOxJYUUJ4S6isC71E2ZHmkdwGRbBY1NcXBPJzODUHKBu6yx OUq2Ma3u1WryXrIUnZ5vTn6n1BbhdvSpHDFxinW9suFsIR5igS4Sh/9b3blXhvvQrRJJsbIeFz6 yx5FmU1qmZEXy6r75NAD84qNrRV2cKwenT7wQI4ggGE0HqrC493Tfdq4q2H9Y8Pnw== X-Received: by 2002:a05:6000:24c5:b0:45b:d2ff:4d4f with SMTP id ffacd0b85a97d-45e5c612fa3mr6819668f8f.40.1778865279184; Fri, 15 May 2026 10:14:39 -0700 (PDT) Received: from ?IPV6:2003:ea:8f17:5b00:3d43:d343:a252:9fd6? (p200300ea8f175b003d43d343a2529fd6.dip0.t-ipconnect.de. [2003:ea:8f17:5b00:3d43:d343:a252:9fd6]) by smtp.gmail.com with ESMTPSA id ffacd0b85a97d-45da0a19a0csm16005465f8f.20.2026.05.15.10.14.37 (version=TLS1_3 cipher=TLS_AES_128_GCM_SHA256 bits=128/128); Fri, 15 May 2026 10:14:38 -0700 (PDT) Message-ID: <18fbc6cc-2ec8-438e-afd0-bf25e576670a@gmail.com> Date: Fri, 15 May 2026 19:14:37 +0200 Precedence: bulk X-Mailing-List: linux-kernel@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 User-Agent: Mozilla Thunderbird Subject: Re: [PATCH] net: r8169: add RX allocation failure counters To: Kshitiz Bartariya , nic_swsd@realtek.com, andrew+netdev@lunn.ch, davem@davemloft.net, edumazet@google.com, kuba@kernel.org, pabeni@redhat.com Cc: netdev@vger.kernel.org, linux-kernel@vger.kernel.org References: <20260515143015.64018-1-kshitiz.bartariya@zohomail.in> Content-Language: en-US From: Heiner Kallweit In-Reply-To: <20260515143015.64018-1-kshitiz.bartariya@zohomail.in> Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 7bit On 15.05.2026 16:30, Kshitiz Bartariya wrote: > Add driver-private ethtool counters for RX allocation failures so packet > drops under memory pressure are easier to diagnose. > Could you please elaborate on the use case? What would you do based on this counter information? I'm not in favor of driver-private counters. If there's a valid use case, then it's driver-independent, and related code should be in net core. AFAIK the kernel complains anyway if allocations fail due to memory pressure. That's the reason why extra error messages for kmalloc() returning NULL are discouraged. > The new aggregate rx_alloc_fail counter is incremented when the driver > fails to allocate an RX page, fails to map an RX page for DMA, or fails > to allocate an skb in the RX receive path. Individual counters for finer > details are: > 1. rx_buf_alloc_fail : failed to allocate an RX page > 2. rx_dma_mapping_fail : failed to map an RX page for DMA > 3. rx_skb_alloc_fail : failed to allocate an skb > > These are kept in struct rtl8169_private instead of > struct rtl8169_counters, which contain the hardware tally counters. > > Signed-off-by: Kshitiz Bartariya > --- > drivers/net/ethernet/realtek/r8169_main.c | 21 ++++++++++++++++++++- > 1 file changed, 20 insertions(+), 1 deletion(-) > > diff --git a/drivers/net/ethernet/realtek/r8169_main.c b/drivers/net/ethernet/realtek/r8169_main.c > index 791277e750ba..c1a200e14f6f 100644 > --- a/drivers/net/ethernet/realtek/r8169_main.c > +++ b/drivers/net/ethernet/realtek/r8169_main.c > @@ -766,6 +766,10 @@ struct rtl8169_private { > dma_addr_t counters_phys_addr; > struct rtl8169_counters *counters; > struct rtl8169_tc_offsets tc_offset; > + atomic64_t rx_alloc_fail; > + atomic64_t rx_buf_alloc_fail; > + atomic64_t rx_dma_mapping_fail; > + atomic64_t rx_skb_alloc_fail; > u32 saved_wolopts; > > const char *fw_name; > @@ -1901,6 +1905,10 @@ static const char rtl8169_gstrings[][ETH_GSTRING_LEN] = { > "multicast", > "tx_aborted", > "tx_underrun", > + "rx_alloc_fail", > + "rx_buf_alloc_fail", > + "rx_dma_mapping_fail", > + "rx_skb_alloc_fail", > }; > > static int rtl8169_get_sset_count(struct net_device *dev, int sset) > @@ -1999,6 +2007,10 @@ static void rtl8169_get_ethtool_stats(struct net_device *dev, > data[10] = le32_to_cpu(counters->rx_multicast); > data[11] = le16_to_cpu(counters->tx_aborted); > data[12] = le16_to_cpu(counters->tx_underrun); > + data[13] = atomic64_read(&tp->rx_alloc_fail); > + data[14] = atomic64_read(&tp->rx_buf_alloc_fail); > + data[15] = atomic64_read(&tp->rx_dma_mapping_fail); > + data[16] = atomic64_read(&tp->rx_skb_alloc_fail); > } > > static void rtl8169_get_strings(struct net_device *dev, u32 stringset, u8 *data) > @@ -4167,12 +4179,17 @@ static struct page *rtl8169_alloc_rx_data(struct rtl8169_private *tp, > struct page *data; > > data = alloc_pages_node(node, GFP_KERNEL, get_order(R8169_RX_BUF_SIZE)); > - if (!data) > + if (!data) { > + atomic64_inc(&tp->rx_alloc_fail); > + atomic64_inc(&tp->rx_buf_alloc_fail); > return NULL; > + } > > mapping = dma_map_page(d, data, 0, R8169_RX_BUF_SIZE, DMA_FROM_DEVICE); > if (unlikely(dma_mapping_error(d, mapping))) { > netdev_err(tp->dev, "Failed to map RX DMA!\n"); > + atomic64_inc(&tp->rx_alloc_fail); > + atomic64_inc(&tp->rx_dma_mapping_fail); > __free_pages(data, get_order(R8169_RX_BUF_SIZE)); > return NULL; > } > @@ -4823,6 +4840,8 @@ static int rtl_rx(struct net_device *dev, struct rtl8169_private *tp, int budget > skb = napi_alloc_skb(&tp->napi, pkt_size); > if (unlikely(!skb)) { > dev->stats.rx_dropped++; > + atomic64_inc(&tp->rx_alloc_fail); > + atomic64_inc(&tp->rx_skb_alloc_fail); > goto release_descriptor; > } >