PSARC/2007/396: Removal of Duplicate BGE/NGE Kstats

Peter Memishian peter.memishian at sun.com
Mon Jul 2 13:18:40 PDT 2007


I'm filing this closed-approved-automatic fasttrack for Raymond Li.
The release binding is "minor".

  Removal of Duplicate BGE/NGE Kstats
  ===================================

    We propose to remove the duplicate "mii" and "parameters" kstats from
    the bge/nge drivers.

  Details
  -------

    The "mii" kstats duplicate the "mac" MII/GMII kstats that the GLDv3
    framework provides for all GLDv3-based Ethernet drivers.  When GLDv3
    was first integrated, it did not support all MII/GMII kstats, so the
    existing "mii" kstats in bge/nge were kept.  However, PSARC 2007/220
    completed the MII/GMII kstats, making it possible to remove the "mii"
    kstats from bge/nge.  Further, the bge/nge drivers also have the
    "parameter" kstats, which have identical names and semantics as the
    ndd(1M) parameters.  These are redundant and will also be removed.

    Attempts to retrieve MII/GMII kstats using "kstat -n mii" will fail
    after this cleanup; "kstat -n mac" should be used instead.  Likewise,
    attempts to retrieve ndd parameters using "kstat -n parameters" will
    fail after this cleanup; ndd should be used instead.  Since only
    bge/nge and export the two duplicate kstat blocks, while other drivers
    do not, applications and scripts are extremely unlikely to depend on
    their existence.  Thus, we believe this change is extremely low-risk.

    Note that the nxge driver has similar issues in that it has "Port
    Stats" which are mostly duplicated by MII/GMII kstats.  But GLDv3
    doesn't define 10GB MII kstats yet, so that cleanup will be deferred
    until GLDv3 is enhanced further.

  Interfaces
  ----------

    All of the following duplicate kstats will be removed:

     "mii" duplicates                     "parameters" duplicates
     -----------------------------        ------------------------------
    | adv_cap_1000fdx             |      | adv_1000fdx_cap              |
    | adv_cap_1000hdx             |      | adv_1000hdx_cap              |
    | adv_cap_100fdx              |      | adv_100fdx_cap               |
    | adv_cap_100hdx              |      | adv_100hdx_cap               |
    |                             |      | adv_100T4_cap                |
    | adv_cap_10fdx               |      | adv_10fdx_cap                |
    | adv_cap_10hdx               |      | adv_10hdx_cap                |
    | adv_cap_asmpause            |      | adv_asym_pause_cap           |
    | adv_cap_autoneg             |      | adv_autoneg_cap              |
    | adv_cap_pause               |      | adv_pause_cap                |
    | adv_rem_fault               |      |                              |
    | cap_1000fdx                 |      | 1000fdx_cap                  |
    | cap_1000hdx                 |      | 1000hdx_cap                  |
    | cap_100fdx                  |      | 100fdx_cap                   |
    | cap_100hdx                  |      | 100hdx_cap                   |
    |                             |      | 100T4_cap                    |
    | cap_10fdx                   |      | 10fdx_cap                    |
    | cap_10hdx                   |      | 10hdx_cap                    |
    | cap_asmpause                |      | asym_pause_cap               |
    | cap_autoneg                 |      | autoneg_cap                  |
    | cap_pause                   |      | pause_cap                    |
    | cap_rem_fault               |      |                              |
    | lp_cap_1000fdx              |      | lp_1000fdx_cap               |
    | lp_cap_1000hdx              |      | lp_1000hdx_cap               |
    | lp_cap_100fdx               |      | lp_100fdx_cap                |
    | lp_cap_100hdx               |      | lp_100hdx_cap                |
    |                             |      | lp_100T4_cap                 |
    | lp_cap_10fdx                |      | lp_10fdx_cap                 |
    | lp_cap_10hdx                |      | lp_10hdx_cap                 |
    | lp_cap_asmpause             |      | lp_asym_pause_cap            |
    | lp_cap_autoneg              |      | lp_autoneg_cap               |
    | lp_cap_pause                |      | lp_pause_cap                 |
    | lp_rem_fault                |      |                              |
    | link_duplex                 |      | link_duplex                  |
    | link_up                     |      | link_status                  |
    | link_asmpause               |      | link_rx_pause                |
    | link_pause                  |      | link_tx_pause                |
    |                             |      | link_autoneg                 |
    |                             |      | link_speed                   |
    | xcvr_addr                   |      |                              |
    | xcvr_id                     |      |                              |
    | xcvr_inuse                  |      |                              |
    |                             |      | drain_max                    |
    |                             |      | loop_mode                    |
    |                             |      | msi_cnt                      |
     -----------------------------        ------------------------------

-- 
meem



More information about the opensolaris-arc mailing list