From: Vladimir Oltean <vladimir.oltean@nxp.com>
To: Jakub Kicinski <kuba@kernel.org>
Cc: Eric Dumazet <edumazet@google.com>,
netdev@vger.kernel.org, linux-kernel@vger.kernel.org,
"David S. Miller" <davem@davemloft.net>,
Paolo Abeni <pabeni@redhat.com>,
Michal Kubecek <mkubecek@suse.cz>,
Claudiu Manoil <claudiu.manoil@nxp.com>,
Vinicius Costa Gomes <vinicius.gomes@intel.com>,
Xiaoliang Yang <xiaoliang.yang_1@nxp.com>,
Kurt Kanzenbach <kurt@linutronix.de>,
Rui Sousa <rui.sousa@nxp.com>,
Ferenc Fejes <ferenc.fejes@ericsson.com>,
Pranavi Somisetty <pranavi.somisetty@amd.com>,
Harini Katakam <harini.katakam@amd.com>,
Colin Foster <colin.foster@in-advantage.com>,
UNGLinuxDriver@microchip.com,
Alexandre Belloni <alexandre.belloni@bootlin.com>,
Andrew Lunn <andrew@lunn.ch>,
Florian Fainelli <f.fainelli@gmail.com>
Subject: Re: [PATCH v4 net-next 04/12] net: ethtool: netlink: retrieve stats from multiple sources (eMAC, pMAC)
Date: Tue, 24 Jan 2023 13:15:34 +0200 [thread overview]
Message-ID: <20230124111534.6eah24trofndera6@skbuf> (raw)
In-Reply-To: <20230123223033.3ad37ccc@kernel.org>
On Mon, Jan 23, 2023 at 10:30:33PM -0800, Jakub Kicinski wrote:
> On Tue, 24 Jan 2023 07:20:20 +0100 Eric Dumazet wrote:
> > > static int pause_prepare_data(const struct ethnl_req_info *req_base,
> > > struct ethnl_reply_data *reply_base,
> > > struct genl_info *info)
> > > {
> > > + const struct pause_req_info *req_info = PAUSE_REQINFO(req_base);
> > > struct pause_reply_data *data = PAUSE_REPDATA(reply_base);
> > > + enum ethtool_mac_stats_src src = req_info->src;
> > > + struct netlink_ext_ack *extack = info->extack;
> >
> > info can be NULL when called from ethnl_default_dump_one()
>
> Second time in a month, I think..
>
> Should we make a fake info to pass here? (until someone finds the time
> to combine the do/dump infos more thoroughly, at least)
Thanks for letting me know. I've sent 2 patches that fix this for
ETHTOOL_MSG_PAUSE_GET and for ETHTOOL_MSG_STATS_GET:
https://patchwork.kernel.org/project/netdevbpf/patch/20230124110801.3628545-1-vladimir.oltean@nxp.com/
https://patchwork.kernel.org/project/netdevbpf/patch/20230124111328.3630437-1-vladimir.oltean@nxp.com/
next prev parent reply other threads:[~2023-01-24 11:15 UTC|newest]
Thread overview: 20+ messages / expand[flat|nested] mbox.gz Atom feed top
2023-01-19 12:26 [PATCH v4 net-next 00/12] ethtool support for IEEE 802.3 MAC Merge layer Vladimir Oltean
2023-01-19 12:26 ` [PATCH v4 net-next 01/12] net: ethtool: netlink: introduce ethnl_update_bool() Vladimir Oltean
2023-01-19 12:26 ` [PATCH v4 net-next 02/12] net: ethtool: add support for MAC Merge layer Vladimir Oltean
2023-01-19 12:26 ` [PATCH v4 net-next 03/12] docs: ethtool-netlink: document interface " Vladimir Oltean
2023-01-19 12:26 ` [PATCH v4 net-next 04/12] net: ethtool: netlink: retrieve stats from multiple sources (eMAC, pMAC) Vladimir Oltean
2023-01-24 6:20 ` Eric Dumazet
2023-01-24 6:30 ` Jakub Kicinski
2023-01-24 11:15 ` Vladimir Oltean [this message]
2023-01-19 12:26 ` [PATCH v4 net-next 05/12] docs: ethtool: document ETHTOOL_A_STATS_SRC and ETHTOOL_A_PAUSE_STATS_SRC Vladimir Oltean
2023-01-19 12:26 ` [PATCH v4 net-next 06/12] net: ethtool: add helpers for aggregate statistics Vladimir Oltean
2023-01-19 12:26 ` [PATCH v4 net-next 07/12] net: ethtool: add helpers for MM fragment size translation Vladimir Oltean
2023-01-19 12:27 ` [PATCH v4 net-next 08/12] net: dsa: add plumbing for changing and getting MAC merge layer state Vladimir Oltean
2023-01-19 12:27 ` [PATCH v4 net-next 09/12] net: mscc: ocelot: allow ocelot_stat_layout elements with no name Vladimir Oltean
2023-01-19 12:27 ` [PATCH v4 net-next 10/12] net: mscc: ocelot: hide access to ocelot_stats_layout behind a helper Vladimir Oltean
2023-01-19 12:27 ` [PATCH v4 net-next 11/12] net: mscc: ocelot: export ethtool MAC Merge stats for Felix VSC9959 Vladimir Oltean
2023-01-19 12:27 ` [PATCH v4 net-next 12/12] net: mscc: ocelot: add MAC Merge layer support for VSC9959 Vladimir Oltean
2023-01-21 5:53 ` [PATCH v4 net-next 00/12] ethtool support for IEEE 802.3 MAC Merge layer Jakub Kicinski
2023-01-23 11:20 ` patchwork-bot+netdevbpf
2023-01-23 13:00 ` patchwork-bot+netdevbpf
2023-01-25 23:30 ` Vladimir Oltean
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20230124111534.6eah24trofndera6@skbuf \
--to=vladimir.oltean@nxp.com \
--cc=UNGLinuxDriver@microchip.com \
--cc=alexandre.belloni@bootlin.com \
--cc=andrew@lunn.ch \
--cc=claudiu.manoil@nxp.com \
--cc=colin.foster@in-advantage.com \
--cc=davem@davemloft.net \
--cc=edumazet@google.com \
--cc=f.fainelli@gmail.com \
--cc=ferenc.fejes@ericsson.com \
--cc=harini.katakam@amd.com \
--cc=kuba@kernel.org \
--cc=kurt@linutronix.de \
--cc=linux-kernel@vger.kernel.org \
--cc=mkubecek@suse.cz \
--cc=netdev@vger.kernel.org \
--cc=pabeni@redhat.com \
--cc=pranavi.somisetty@amd.com \
--cc=rui.sousa@nxp.com \
--cc=vinicius.gomes@intel.com \
--cc=xiaoliang.yang_1@nxp.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.