From: <piergiorgio.beruto@gmail.com>
To: "'Vladimir Oltean'" <vladimir.oltean@nxp.com>, <netdev@vger.kernel.org>
Cc: "'Michal Kubecek'" <mkubecek@suse.cz>,
"'Pranavi Somisetty'" <pranavi.somisetty@amd.com>
Subject: RE: [PATCH v3 ethtool 0/4] MAC Merge layer support
Date: Sun, 12 Feb 2023 20:13:21 +0100 [thread overview]
Message-ID: <003c01d93f16$12d54f20$387fed60$@gmail.com> (raw)
In-Reply-To: <20230210213311.218456-1-vladimir.oltean@nxp.com>
Hi all,
From my perspective, it would be better to merge it on top of
"[v5,ethtool-next,1/1] add support for IEEE 802.3cg-2019 Clause 148".
Thanks,
Piergiorgio
-----Original Message-----
From: Vladimir Oltean <vladimir.oltean@nxp.com>
Sent: 10 February, 2023 22:33
To: netdev@vger.kernel.org
Cc: Michal Kubecek <mkubecek@suse.cz>; Pranavi Somisetty
<pranavi.somisetty@amd.com>; Piergiorgio Beruto
<piergiorgio.beruto@gmail.com>
Subject: [PATCH v3 ethtool 0/4] MAC Merge layer support
Add support for the following 2 new commands:
$ ethtool [ --include-statistics ] --show-mm <eth> $ ethtool --set-mm <eth>
[ ... ]
as well as for:
$ ethtool --include-statistics --show-pause <eth> --src pmac|emac|aggregate
$ ethtool --include-statistics --show-pause <eth> --src pmac|emac|aggregate
$ ethtool -S <eth> --groups eth-mac eth-phy eth-ctrl rmon -- --src pmac
and some modest amount of documentation (the bulk of it is already
distributed with the kernel's ethtool netlink rst).
This patch set applies on top of "[v5,ethtool-next,1/1] add support for IEEE
802.3cg-2019 Clause 148", because otherwise it would conflict with it:
https://patchwork.kernel.org/project/netdevbpf/patch/d51013e0bc617651c0e6d29
8f47cc6b82c0ffa88.1675327734.git.piergiorgio.beruto@gmail.com/
I believe Michal hasn't gotten to it yet, but I'm putting Piergiorgio on CC
too, in case there are other reasons why I shouldn't wait for his patch to
be merged first.
Vladimir Oltean (4):
netlink: add support for MAC Merge layer
netlink: pass the source of statistics for pause stats
netlink: pass the source of statistics for port stats
ethtool.8: update documentation with MAC Merge related bits
Makefile.am | 2 +-
ethtool.8.in | 107 +++++++++++++++++++
ethtool.c | 16 +++
netlink/extapi.h | 4 +
netlink/mm.c | 270 +++++++++++++++++++++++++++++++++++++++++++++++
netlink/pause.c | 33 +++++-
netlink/stats.c | 14 +++
7 files changed, 440 insertions(+), 6 deletions(-) create mode 100644
netlink/mm.c
--
2.34.1
next prev parent reply other threads:[~2023-02-12 19:14 UTC|newest]
Thread overview: 7+ messages / expand[flat|nested] mbox.gz Atom feed top
2023-02-10 21:33 [PATCH v3 ethtool 0/4] MAC Merge layer support Vladimir Oltean
2023-02-10 21:33 ` [PATCH v3 ethtool 1/4] netlink: add support for MAC Merge layer Vladimir Oltean
2023-02-10 21:33 ` [PATCH v3 ethtool 2/4] netlink: pass the source of statistics for pause stats Vladimir Oltean
2023-02-10 21:33 ` [PATCH v3 ethtool 3/4] netlink: pass the source of statistics for port stats Vladimir Oltean
2023-02-10 21:33 ` [PATCH v3 ethtool 4/4] ethtool.8: update documentation with MAC Merge related bits Vladimir Oltean
2023-02-12 19:13 ` piergiorgio.beruto [this message]
2023-02-13 18:10 ` [PATCH v3 ethtool 0/4] MAC Merge layer support patchwork-bot+netdevbpf
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='003c01d93f16$12d54f20$387fed60$@gmail.com' \
--to=piergiorgio.beruto@gmail.com \
--cc=mkubecek@suse.cz \
--cc=netdev@vger.kernel.org \
--cc=pranavi.somisetty@amd.com \
--cc=vladimir.oltean@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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).