Re: [PATCH] ethdev: sync ethtool link modes with Linux 6.15

[Stephen Hemminger <stephen@networkplumber.org>]

On Sun, 29 Jun 2025 18:37:31 +0200
Thomas Monjalon <thomas@monjalon.net> wrote:

> 26/06/2025 17:13, Thomas Monjalon:
> > 26/06/2025 16:26, Stephen Hemminger:  
> > > On Wed, 25 Jun 2025 15:42:02 +0200
> > > Thomas Monjalon <thomas@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.  
> 
> Unfortunately we cannot remove the backslashes,
> otherwise compilation of the comment is failing.
> 
> 

Put original patch into next-net