netdev - Re: [PATCH 14/17] octeontx2-pf: Add basic ethtool support

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-ID: <CA+sq2Ccr5jB1cBN62Y56C=19L-P7hgYPrD9o7EJN71Kroou9Ew@mail.gmail.com>
Date:   Sat, 11 Jan 2020 14:17:45 +0530
From:   Sunil Kovvuri <sunil.kovvuri@...il.com>
To:     Jakub Kicinski <jakub.kicinski@...ronome.com>
Cc:     Linux Netdev List <netdev@...r.kernel.org>,
        "David S. Miller" <davem@...emloft.net>,
        Christina Jacob <cjacob@...vell.com>,
        Sunil Goutham <sgoutham@...vell.com>
Subject: Re: [PATCH 14/17] octeontx2-pf: Add basic ethtool support

On Sat, Jan 11, 2020 at 12:58 AM Jakub Kicinski
<jakub.kicinski@...ronome.com> wrote:
>
> On Fri, 10 Jan 2020 17:11:58 +0530, sunil.kovvuri@...il.com wrote:
> > +static const struct otx2_stat otx2_dev_stats[] = {
> > +     OTX2_DEV_STAT(rx_bytes),
> > +     OTX2_DEV_STAT(rx_frames),
> > +     OTX2_DEV_STAT(rx_ucast_frames),
> > +     OTX2_DEV_STAT(rx_bcast_frames),
> > +     OTX2_DEV_STAT(rx_mcast_frames),
> > +     OTX2_DEV_STAT(rx_drops),
> > +
> > +     OTX2_DEV_STAT(tx_bytes),
> > +     OTX2_DEV_STAT(tx_frames),
> > +     OTX2_DEV_STAT(tx_ucast_frames),
> > +     OTX2_DEV_STAT(tx_bcast_frames),
> > +     OTX2_DEV_STAT(tx_mcast_frames),
> > +     OTX2_DEV_STAT(tx_drops),
> > +};
>
> Please don't duplicate the same exact stats which are exposed via
> ndo_get_stats64 via ethtool.

ndo_stats64 doesn't have separate stats for ucast, mcast and bcast on Rx and
Tx sides, they are combined ones. Hence added separate stats here.
The ones repeated here are bytes, frames and drops which are added to have
full set of stats at one place which could help anyone debugging pkt
drop etc issues.

Thanks,
Sunil.