NetBSD Problem Report #32009

From www@netbsd.org  Mon Nov  7 17:59:20 2005
Return-Path: <www@netbsd.org>
Received: by narn.netbsd.org (Postfix, from userid 31301)
	id DD65863B938; Mon,  7 Nov 2005 17:59:20 +0000 (UTC)
Message-Id: <20051107175920.DD65863B938@narn.netbsd.org>
Date: Mon,  7 Nov 2005 17:59:20 +0000 (UTC)
From: tkuik@cisco.com
Reply-To: tkuik@cisco.com
To: gnats-bugs@netbsd.org
Subject: Use of pause frames are not properly auto-negotiated with wm driver
X-Send-Pr-Version: www-1.0

>Number:         32009
>Category:       kern
>Synopsis:       Use of pause frames are not properly auto-negotiated with wm driver
>Confidential:   no
>Severity:       non-critical
>Priority:       low
>Responsible:    kern-bug-people
>State:          closed
>Class:          sw-bug
>Submitter-Id:   net
>Arrival-Date:   Mon Nov 07 18:00:00 +0000 2005
>Closed-Date:    Mon May 03 03:45:22 +0000 2010
>Last-Modified:  Mon May 03 03:45:22 +0000 2010
>Originator:     Tim Kuik
>Release:        NetBSD-current
>Organization:
Cisco
>Environment:
NetBSD netbsd 3.99.8 NetBSD 3.99.8 ... i386

>Description:
The pause frame selection bits, being set in the transmit configuration word, are reserved (bits 10 & 11).  The proper bits are 7 & 8 for pause frame selection.
>How-To-Repeat:
Pause frame use is never autonegotiated properly.
>Fix:
I added two new defines for the TXCW register (0x178):
diff <NetBSD-current> if_wmreg.h
461a462,463
> #define TXCW_ANE_SYM_PAUSE (1U << 7)
> #define TXCW_ANE_ASYM_PAUSE (1U << 8)

Change the source to use the correct definitions:
diff <NetBSD-current> if_wm.c
3411c3411
<               sc->sc_txcw |= ANAR_X_PAUSE_SYM | ANAR_X_PAUSE_ASYM;
---
>               sc->sc_txcw |= TXCW_ANE_SYM_PAUSE | TXCW_ANE_ASYM_PAUSE;

>Release-Note:

>Audit-Trail:
From: SAITOH Masanobu <msaitoh@netbsd.org>
To: gnats-bugs@gnats.NetBSD.org
Cc: 
Subject: PR/32009 CVS commit: src/sys/dev/pci
Date: Tue, 7 Apr 2009 18:23:37 +0000

 Module Name:	src
 Committed By:	msaitoh
 Date:		Tue Apr  7 18:23:37 UTC 2009

 Modified Files:
 	src/sys/dev/pci: if_wm.c if_wmreg.h

 Log Message:
 Fix about TBI mode. This fix doesn't influence MII mode.
  - Fix SWDPIN(1)'s polarity on some chips.
  - Fix flow control stuff (includes PR#32009).
  - Stop RXCFG storm. It ocours easily.
  - And more fix about autonego.

 Tested on PRO/1000T, PRO/1000XF and PRO/1000 MF.


 To generate a diff of this commit:
 cvs rdiff -u -r1.172 -r1.173 src/sys/dev/pci/if_wm.c
 cvs rdiff -u -r1.26 -r1.27 src/sys/dev/pci/if_wmreg.h

 Please note that diffs are not public domain; they are subject to the
 copyright notices on the relevant files.

From: Soren Jacobsen <snj@netbsd.org>
To: gnats-bugs@gnats.NetBSD.org
Cc: 
Subject: PR/32009 CVS commit: [netbsd-5] src/sys/dev/pci
Date: Thu, 21 May 2009 01:13:49 +0000

 Module Name:	src
 Committed By:	snj
 Date:		Thu May 21 01:13:49 UTC 2009

 Modified Files:
 	src/sys/dev/pci [netbsd-5]: if_wm.c if_wmreg.h pcidevs

 Log Message:
 Pull up following revision(s) (requested by bouyer in ticket #711):
 	sys/dev/pci/pcidevs: revisions 1.975, 1.981, 1.982 via patch
 	sys/dev/pci/if_wm.c: revisions 1.164, 1.167, 1.173, 1.174 via patch
 	sys/dev/pci/if_wmreg.h: revisions 1.25, 1.27 via patch
 Add Intel 82567LM_3 ethernet
 --
 Add i82567LM-3
 --
 add i82567LF-3 LAN Controller
 --
 add an entry for 82567LF-3.
 fix the register access for ICH10DO.
 --
 Fix about TBI mode. This fix doesn't influence MII mode.
 --
  - Fix panic in mediachange.
  - Fix SWDPIN(1)'s polarity on some chips.
  - Fix flow control stuff (includes PR#32009).
  - Stop RXCFG storm. It ocours easily.
  - And more fix about autonego.
 --
 add 82801J_D_BM_LF (ICH10)
 --
 Reload sc_ctrl in wm_reset().
 Add an ICH10 entry.
 Remove some obsolete comments.


 To generate a diff of this commit:
 cvs rdiff -u -r1.162.4.8 -r1.162.4.9 src/sys/dev/pci/if_wm.c
 cvs rdiff -u -r1.24.20.1 -r1.24.20.2 src/sys/dev/pci/if_wmreg.h
 cvs rdiff -u -r1.962.4.3 -r1.962.4.4 src/sys/dev/pci/pcidevs

 Please note that diffs are not public domain; they are subject to the
 copyright notices on the relevant files.

From: Soren Jacobsen <snj@netbsd.org>
To: gnats-bugs@gnats.NetBSD.org
Cc: 
Subject: PR/32009 CVS commit: [netbsd-5-0] src/sys/dev/pci
Date: Sun, 19 Jul 2009 19:48:27 +0000

 Module Name:	src
 Committed By:	snj
 Date:		Sun Jul 19 19:48:26 UTC 2009

 Modified Files:
 	src/sys/dev/pci [netbsd-5-0]: if_wm.c if_wmreg.h

 Log Message:
 Pull up following revision(s) (requested by msaitoh in ticket #862):
 	sys/dev/pci/if_wm.c: revisions 1.168, 1.170, and 1.173 via patch
 	sys/dev/pci/if_wmreg.h: revision 1.27
 Fixes serious three bugs.
 1) On i82563, the em driver says that the ready bit in the MDIC
 register may be incorrectly set. Insert delay(200) like the em driver.
 Fixes PR#41014
 2) Add workaround for 82543GC. We need to force speed and duplex on
 the MAC equal to what the PHY speed and duplex configuration is.
 Fixes PR#36430.
 3) Fix many problems and panic on TBI's cards (includes PR#32009).


 To generate a diff of this commit:
 cvs rdiff -u -r1.162.4.3 -r1.162.4.3.2.1 src/sys/dev/pci/if_wm.c
 cvs rdiff -u -r1.24 -r1.24.26.1 src/sys/dev/pci/if_wmreg.h

 Please note that diffs are not public domain; they are subject to the
 copyright notices on the relevant files.

State-Changed-From-To: open->feedback
State-Changed-By: dholland@NetBSD.org
State-Changed-When: Sun, 19 Jul 2009 20:45:11 +0000
State-Changed-Why:
fixed?


State-Changed-From-To: feedback->closed
State-Changed-By: dholland@NetBSD.org
State-Changed-When: Mon, 03 May 2010 03:45:22 +0000
State-Changed-Why:
Feedback timeout.


>Unformatted:

Home	PR Database Search