From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org Received: from vger.kernel.org (vger.kernel.org [23.128.96.18]) by smtp.lore.kernel.org (Postfix) with ESMTP id F007CC4332F for ; Tue, 12 Dec 2023 19:21:49 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1377128AbjLLTVm (ORCPT ); Tue, 12 Dec 2023 14:21:42 -0500 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:52032 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S230370AbjLLTVk (ORCPT ); Tue, 12 Dec 2023 14:21:40 -0500 Received: from smtp.kernel.org (relay.kernel.org [52.25.139.140]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id 6AC239F for ; Tue, 12 Dec 2023 11:21:47 -0800 (PST) Received: by smtp.kernel.org (Postfix) with ESMTPSA id C50E8C433C9; Tue, 12 Dec 2023 19:21:46 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=kernel.org; s=k20201202; t=1702408907; bh=mj3SAwKcmqSqhMBqORrj4sHEFg6g5cS+PgxB5jSbueY=; h=Date:From:To:Cc:Subject:In-Reply-To:References:From; b=KEQHZ1B5w9+KQEUURp8DI6uCXRF4SjDEK2K0PH8YTdG4Q7EUrSRK+V6rfabwGbsQV UFPCt5lBYyOGcrgt/KV18WrYpGDH39q6CTQmi2Fg1MJ5mYXJBSjWfsegyeIPjQrjBb lUzWYVkaqwQe2KFCL32uqg1nrUVFn16KSIxhXk3+T5Ive6WDYeS2KkznrBGz/XpFnO N/ekEg3TGKl5HjMu37kyXAaILa7GZyZGBkp3RToBqCq2K61E7CKMSW4sIGUDXcT7Wc 5kJpD5I+OCycKUCY6IrSsCzwoOGMLFNxVqQeiobdYQOqq8dbhbPEYiFaJP9qyi3QCx 8ETOjngt76Vdg== Date: Tue, 12 Dec 2023 11:21:45 -0800 From: Jakub Kicinski To: Justin Lai Cc: , , , , , , , Subject: Re: [PATCH net-next v14 10/13] rtase: Implement ethtool function Message-ID: <20231212112145.25953351@kernel.org> In-Reply-To: <20231208094733.1671296-11-justinlai0215@realtek.com> References: <20231208094733.1671296-1-justinlai0215@realtek.com> <20231208094733.1671296-11-justinlai0215@realtek.com> MIME-Version: 1.0 Content-Type: text/plain; charset=US-ASCII Content-Transfer-Encoding: 7bit Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Fri, 8 Dec 2023 17:47:30 +0800 Justin Lai wrote: > +static const char rtase_gstrings[][ETH_GSTRING_LEN] = { > + "tx_packets", > + "rx_packets", > + "tx_errors", > + "rx_errors", > + "rx_missed", > + "align_errors", > + "tx_single_collisions", > + "tx_multi_collisions", > + "unicast", > + "broadcast", > + "multicast", > + "tx_aborted", > + "tx_underrun", Please take a look at struct rtnl_link_stats, anything that's already present there should be reported via ndo_get_stats64, not ethtool