26/06/2025 16:26, Stephen Hemminger:
> On Wed, 25 Jun 2025 15:42:02 +0200
> Thomas Monjalon <tho...@monjalon.net> wrote:
>
> > diff --git a/lib/ethdev/ethdev_linux_ethtool.c
> > b/lib/ethdev/ethdev_linux_ethtool.c
> > index ec42d3054a..f508cdba6c 100644
> > --- a/lib/ethdev/ethdev_linux_ethtool.c
> > +++ b/lib/ethdev/ethdev_linux_ethtool.c
> > @@ -17,8 +17,9 @@
> > *
> > * The array below is built from bit definitions with this shell command:
> > * sed -rn 's;.*(ETHTOOL_LINK_MODE_)([0-9]+)([0-9a-zA-Z_]*).*=
> > *([0-9]*).*;'\
> > - * '[\4] = \2, /\* \1\2\3 *\/;p' /usr/include/linux/ethtool.h |
> > - * awk '/_Half_/{$3=$3+1","}1'
> > + * '[\4] \2 \1\2\3;p' /usr/include/linux/ethtool.h |
> > + * awk '/_Half_/{$2=$2+1}1' |
> > + * awk '{printf "\t%5s = %7s, /\* %s *\/\n", $1, $2, $3}'
> > */
>
> The commands in the comment never worked verbatim.
It works on my machine.
> $ sed -rn 's;.*(ETHTOOL_LINK_MODE_)([0-9]+)([0-9a-zA-Z_]*).*= *([0-9]*).*;'\
> '[\4] \2 \1\2\3;p' /usr/include/linux/ethtool.h |
> awk '/_Half_/{$2=$2+1}1' |
> awk '{printf "\t%5s = %7s, /\* %s *\/\n", $1, $2, $3}'
>
> > > > sed: -e expression #1, char 63: unterminated `s' command
> awk: cmd. line:1: warning: escape sequence `\*' treated as plain `*'
> awk: cmd. line:1: warning: escape sequence `\/' treated as plain `/'
The backslashes were added to help the syntax highlighting.
But I can remove them.
> Fixing the comment would be:
> $ git diff
> diff --git a/lib/ethdev/ethdev_linux_ethtool.c
> b/lib/ethdev/ethdev_linux_ethtool.c
> index f508cdba6c..8a148ae0cd 100644
> --- a/lib/ethdev/ethdev_linux_ethtool.c
> +++ b/lib/ethdev/ethdev_linux_ethtool.c
> @@ -16,10 +16,11 @@
> * and allows to compile with new bits included even on an old kernel.
> *
> * The array below is built from bit definitions with this shell command:
> - * sed -rn 's;.*(ETHTOOL_LINK_MODE_)([0-9]+)([0-9a-zA-Z_]*).*=
> *([0-9]*).*;'\
> - * '[\4] \2 \1\2\3;p' /usr/include/linux/ethtool.h |
> - * awk '/_Half_/{$2=$2+1}1' |
> - * awk '{printf "\t%5s = %7s, /\* %s *\/\n", $1, $2, $3}'
> + * sed -rn \
> + * 's;.*(ETHTOOL_LINK_MODE_)([0-9]+)([0-9a-zA-Z_]*).*= *([0-9]*).*;[\4] \2
> \1\2\3;p' \
> + * /usr/include/linux/ethtool.h | \
> + * awk '/_Half_/{$2=$2+1}1' | \
> + * awk '{printf "\t%5s = %7s, /* %s */\n", $1, $2, $3}'
