From sacadmin Mon Jul  2 13:19:59 2007
Received: from sac.sfbay.sun.com (localhost [127.0.0.1])
	by sac.sfbay.sun.com (8.13.8+Sun/8.13.8) with ESMTP id l62KJxHQ014151;
	Mon, 2 Jul 2007 13:19:59 -0700 (PDT)
Received: (from meem@localhost)
	by sac.sfbay.sun.com (8.13.8+Sun/8.13.8/Submit) id l62KJxTP014148;
	Mon, 2 Jul 2007 13:19:59 -0700 (PDT)
Date: Mon, 2 Jul 2007 13:19:59 -0700 (PDT)
From: Peter Memishian <meem@sac.sfbay.sun.com>
Message-Id: <200707022019.l62KJxTP014148@sac.sfbay.sun.com>
To: PSARC-record@sac.sfbay.sun.com
Cc: raymond.li@sun.com, sowmini.varadhan@sun.com
Subject: Removal of Duplicate BGE/NGE Kstats [PSARC/2007/396 Self Review]
Status: RO
Content-Length: 566


Template Version: @(#)sac_nextcase 1.63 06/14/07 SMI
This information is Copyright 2007 Sun Microsystems
1. Introduction
    1.1. Project/Component Working Name:
	 Removal of Duplicate BGE/NGE Kstats
    1.2. Name of Document Author/Supplier:
	 Author:  Raymond Li
    1.3  Date of This Document:
	02 July, 2007
4. Technical Description
    See the case directory for more detail

6. Resources and Schedule
    6.4. Steering Committee requested information
   	6.4.1. Consolidation C-team Name:
		ON
    6.5. ARC review type: Automatic
    6.6. ARC Exposure: open


From meem@triplex.east.sun.com Mon Jul  2 13:21:17 2007
Received: from sunmail4.Singapore.Sun.COM (sunmail4.Singapore.Sun.COM [129.158.71.19])
	by sac.sfbay.sun.com (8.13.8+Sun/8.13.8) with ESMTP id l62KLGTS014172
	for <psarc-ext@sac.sfbay.Sun.COM>; Mon, 2 Jul 2007 13:21:17 -0700 (PDT)
Received: from nwk-avmta-2.sfbay.sun.com (nwk-avmta-2.SFBay.Sun.COM [129.145.155.6])
	by sunmail4.Singapore.Sun.COM (8.13.4+Sun/8.13.3/ENSMAIL,v2.2) with ESMTP id l62KJHDf024980;
	Tue, 3 Jul 2007 04:19:28 +0800 (SGT)
Received: from pmxchannel-daemon.nwk-avmta-2.sfbay.sun.com by
 nwk-avmta-2.sfbay.sun.com
 (Sun Java System Messaging Server 6.2-3.04 (built Jul 15 2005))
 id <0JKK00J0NJ4DWW00@nwk-avmta-2.sfbay.sun.com>; Mon,
 02 Jul 2007 13:19:25 -0700 (PDT)
Received: from triplex.East.Sun.COM ([129.148.174.104])
 by nwk-avmta-2.sfbay.sun.com
 (Sun Java System Messaging Server 6.2-3.04 (built Jul 15 2005))
 with ESMTP id <0JKK00HJBJ4BSK30@nwk-avmta-2.sfbay.sun.com>; Mon,
 02 Jul 2007 13:19:24 -0700 (PDT)
Received: from triplex.East.Sun.COM (localhost [127.0.0.1])
 by triplex.East.Sun.COM (8.14.1+Sun/8.14.1) with ESMTP id l62KIflI184212; Mon,
 02 Jul 2007 16:18:41 -0400 (EDT)
Received: (from meem@localhost)
 by triplex.East.Sun.COM (8.14.1+Sun/8.14.1/Submit) id l62KIfEP184209; Mon,
 02 Jul 2007 16:18:41 -0400 (EDT)
Date: Mon, 02 Jul 2007 16:18:40 -0400
From: Peter Memishian <peter.memishian@sun.com>
Subject: PSARC/2007/396: Removal of Duplicate BGE/NGE Kstats
To: psarc-ext@sun.com
Cc: raymond.li@sun.com, sowmini.vardhan@sun.com
Message-id: <18057.23968.946480.703160@gargle.gargle.HOWL>
MIME-version: 1.0
X-Mailer: VM 7.17 under 21.4 (patch 18) "Social Property" XEmacs Lucid
Content-type: text/plain; charset=us-ascii
Content-transfer-encoding: 7BIT
X-PMX-Version: 5.2.0.264296
Status: RO
Content-Length: 5279


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

From meem@triplex.east.sun.com Mon Jul  2 13:22:17 2007
Received: from sunmail5.uk.sun.com (sunmail5.UK.Sun.COM [129.156.85.165])
	by sac.sfbay.sun.com (8.13.8+Sun/8.13.8) with ESMTP id l62KMGk6014240
	for <psarc-ext@sac.sfbay.sun.com>; Mon, 2 Jul 2007 13:22:16 -0700 (PDT)
Received: from nwk-avmta-2.sfbay.sun.com (nwk-avmta-2.SFBay.Sun.COM [129.145.155.6])
	by sunmail5.uk.sun.com (8.13.7+Sun/8.13.7/ENSMAIL,v2.2) with ESMTP id l62KK7Ds008260;
	Mon, 2 Jul 2007 21:20:16 +0100 (BST)
Received: from pmxchannel-daemon.nwk-avmta-2.sfbay.sun.com by
 nwk-avmta-2.sfbay.sun.com
 (Sun Java System Messaging Server 6.2-3.04 (built Jul 15 2005))
 id <0JKK00J0ZJ5LY000@nwk-avmta-2.sfbay.sun.com>; Mon,
 02 Jul 2007 13:20:09 -0700 (PDT)
Received: from triplex.East.Sun.COM ([129.148.174.104])
 by nwk-avmta-2.sfbay.sun.com
 (Sun Java System Messaging Server 6.2-3.04 (built Jul 15 2005))
 with ESMTP id <0JKK00H9EJ5ISL30@nwk-avmta-2.sfbay.sun.com>; Mon,
 02 Jul 2007 13:20:07 -0700 (PDT)
Received: from triplex.East.Sun.COM (localhost [127.0.0.1])
	by triplex.East.Sun.COM (8.14.1+Sun/8.14.1) with ESMTP id l62KJNRg184222; Mon,
 02 Jul 2007 16:19:23 -0400 (EDT)
Received: (from meem@localhost)
	by triplex.East.Sun.COM (8.14.1+Sun/8.14.1/Submit) id l62KJNlg184219; Mon,
 02 Jul 2007 16:19:23 -0400 (EDT)
Date: Mon, 02 Jul 2007 16:19:23 -0400
From: Peter Memishian <peter.memishian@sun.com>
Subject: PSARC/2007/396: Removal of Duplicate BGE/NGE Kstats
To: psarc-ext@sun.com
Cc: raymond.li@sun.com, sowmini.varadhan@sun.com
Message-id: <18057.24011.777249.179160@gargle.gargle.HOWL>
MIME-version: 1.0
X-Mailer: VM 7.17 under 21.4 (patch 18) "Social Property" XEmacs Lucid
Content-type: text/plain; charset=us-ascii
Content-transfer-encoding: 7BIT
X-PMX-Version: 5.2.0.264296
X-PMX-Version: 5.2.0.264296
Status: RO
Content-Length: 5322


[ Resending since I botched the CC line ]

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

