[dpdk-dev] [PATCH 0/3] drivers/net: add support for IF-MIB and EtherLike-MIB

Stephen Hemminger stephen at networkplumber.org
Mon May 22 18:11:00 CEST 2017


On Mon, 22 May 2017 16:31:59 +0200
Michal Jastrzebski <michalx.k.jastrzebski at intel.com> wrote:

> Extend Intel NICs (i40e, ixgbe, e1000) xstats for IF-MIB and
> EtherLike-MIB.
> 
> If-MIB xstats:
> ifNumber
> ifIndex
> ifType
> ifMtu
> ifSpeed
> ifPhysAddress
> ifOperStatus
> ifLastChange
> ifHighSpeed
> ifConnectorPresent
> ifCounterDiscontinuityTime
> 
> EtherLike-MIB xstats:
> dot3PauseOperMode
> dot3StatsDuplexStatus
> dot3StatsRateControlAbility
> dot3StatsRateControlStatus
> dot3ControlFunctionsSupported
> 
> Michal Jastrzebski (3):
>   drivers/net: add support for IF-MIB and EtherLike-MIB for e1000
>   drivers/net: add support for IF-MIB and EtherLike-MIB for i40e
>   drivers/net: add support for IF-MIB and EtherLike-MIB for ixgbe
> 
>  drivers/net/e1000/e1000_ethdev.h |   59 ++++++++
>  drivers/net/e1000/igb_ethdev.c   |  296 ++++++++++++++++++++++++++++++++++----
>  drivers/net/i40e/i40e_ethdev.c   |  171 +++++++++++++++++++++-
>  drivers/net/i40e/i40e_ethdev.h   |   60 ++++++++
>  drivers/net/ixgbe/ixgbe_ethdev.c |  275 ++++++++++++++++++++++++++++++++---
>  drivers/net/ixgbe/ixgbe_ethdev.h |   59 ++++++++
>  6 files changed, 878 insertions(+), 42 deletions(-)
> 

Having all the SNMP MIB information is a worth goal. Thank you for starting
the effort.

But you need to rethink how you are implementing this.
Doing it in a driver specific manner is not helpful to application writers.
All API's for features like this should be at ethdev level, and be supported
by generic code.

If you are going to add something like this it has to work for all devices
virtual, physical, layered and not just Intel hardware. It is a much
bigger task.


More information about the dev mailing list