From owner-freebsd-stable@FreeBSD.ORG Sun Mar 8 17:05:31 2009 Return-Path: Delivered-To: freebsd-stable@freebsd.org Received: from mx1.freebsd.org (mx1.freebsd.org [IPv6:2001:4f8:fff6::34]) by hub.freebsd.org (Postfix) with ESMTP id 9E0E61065677; Sun, 8 Mar 2009 17:05:31 +0000 (UTC) (envelope-from ianjhart@ntlworld.com) Received: from mtaout02-winn.ispmail.ntl.com (mtaout02-winn.ispmail.ntl.com [81.103.221.48]) by mx1.freebsd.org (Postfix) with ESMTP id E48A38FC0C; Sun, 8 Mar 2009 17:05:29 +0000 (UTC) (envelope-from ianjhart@ntlworld.com) Received: from aamtaout02-winn.ispmail.ntl.com ([81.103.221.35]) by mtaout02-winn.ispmail.ntl.com (InterMail vM.7.08.04.00 201-2186-134-20080326) with ESMTP id <20090308170528.IMMR4080.mtaout02-winn.ispmail.ntl.com@aamtaout02-winn.ispmail.ntl.com>; Sun, 8 Mar 2009 17:05:28 +0000 Received: from cpc1-cove3-0-0-cust909.sol2.cable.ntl.com ([86.20.31.142]) by aamtaout02-winn.ispmail.ntl.com (InterMail vG.2.02.00.01 201-2161-120-102-20060912) with ESMTP id <20090308170528.HVTO21638.aamtaout02-winn.ispmail.ntl.com@cpc1-cove3-0-0-cust909.sol2.cable.ntl.com>; Sun, 8 Mar 2009 17:05:28 +0000 X-Virus-Scanned: amavisd-new at cpc2-cove3-0-0-cust311.sol2.cable.ntl.com Received: from gamma.private.lan (gamma.private.lan [192.168.0.12]) by cpc1-cove3-0-0-cust909.sol2.cable.ntl.com (8.14.3/8.14.3) with ESMTP id n28H5CPV003346; Sun, 8 Mar 2009 17:05:13 GMT (envelope-from ianjhart@ntlworld.com) From: ian j hart To: pyunyh@gmail.com Date: Sun, 8 Mar 2009 17:05:12 +0000 User-Agent: KMail/1.9.10 References: <8dfae1c10901070639x67945324jeeecfcac647d7976@mail.gmail.com> <200903071717.57915.ianjhart@ntlworld.com> <20090308023642.GB1531@michelle.cdnetworks.co.kr> In-Reply-To: <20090308023642.GB1531@michelle.cdnetworks.co.kr> MIME-Version: 1.0 Content-Type: text/plain; charset="iso-8859-1" Content-Transfer-Encoding: 7bit Content-Disposition: inline Message-Id: <200903081705.12523.ianjhart@ntlworld.com> X-Spam-Status: No, score=-1.4 required=5.0 tests=ALL_TRUSTED autolearn=failed version=3.2.5 X-Spam-Checker-Version: SpamAssassin 3.2.5 (2008-06-10) on cpc1-cove3-0-0-cust909.sol2.cable.ntl.com X-Cloudmark-Analysis: v=1.0 c=1 a=2zH3nbq_TooA:10 a=S-5L88vOh18A:10 a=6I5d2MoRAAAA:8 a=OhSNbSWiAAAA:8 a=an4kULdV8BKMyoFNhZMA:9 a=cu224W3g2iVP7zmVgcEA:7 a=u8azVtT2Vny9Art_gjAuef5xLb0A:4 a=LY0hPdMaydYA:10 a=VcH4gEyNGmTbA7TE:21 a=Enx1yZu87UO49WiZ:21 Cc: Sascha Holzleiter , freebsd-stable@freebsd.org, Jung-uk Kim Subject: Re: FreeBSD 7.1 Breaks re and rl Network Interface Drivers X-BeenThere: freebsd-stable@freebsd.org X-Mailman-Version: 2.1.5 Precedence: list List-Id: Production branch of FreeBSD source code List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Sun, 08 Mar 2009 17:05:32 -0000 On Sunday 08 March 2009 02:36:42 Pyun YongHyeon wrote: > On Sat, Mar 07, 2009 at 05:17:57PM +0000, ian j hart wrote: > > On Tuesday 20 January 2009 02:45:19 Pyun YongHyeon wrote: > > > On Mon, Jan 19, 2009 at 06:33:46PM -0500, Jung-uk Kim wrote: > > > > On Monday 19 January 2009 04:33 pm, Jung-uk Kim wrote: > > > > > I found something interesting. I have another RTL8169SC that > > > > > works perfectly fine without the patch. The hardware revision is > > > > > 0x18000000. After reading Linux driver (drivers/net/r8169c), I > > > > > realised they use different masks for hardware revisions. With > > > > > their logic, non-working chip seems to be 0x98000000 (8110SCe) > > > > > while working chip seems to be 0x18000000 (8110SCd) with > > > > > 0xfc800000. FYI... > > > > > > > > Now armed with the information, I made it work without reverting > > > > memory mapped I/O. :-) > > > > > > > > http://people.freebsd.org/~jkim/re/re.current2.diff > > > > http://people.freebsd.org/~jkim/re/re.stable2.diff > > > > > > I like the patch. Since only RTL8169 family uses mask 0xfc800000 > > > it would be even better we can limit checking scope for RTL8169SC > > > by comparing PCI device id. I don't know what other side effect > > > would happen if the mask 0xfc800000 would be used on 8101/8168 > > > controllers. > > > If the patch works on RTL8169SC would you commit the patch? > > > I'd like to see multiple commits separated by each enhancements > > > as the patch contains several fixes which are not directly related > > > with the issue. > > > > Where are we on this? > > > > I have a headless firewall box which is not happy with 7.1-RELEASE. I've > > upgraded to 7.1-STABLE as of yesterday and now I'm getting 'PHY read > > failed' errors, although the network did come up, which was an > > improvement. > > > > Is there a patch I can try? > > > > http://www.jetway.com.tw/jw/ipcboard_view.asp?productid=174&proname=AD3RT > >LAN-G > > > > re0: port > > 0xf200-0xf2ff mem 0xfdfff000-0xfdfff0ff irq 18 at device 9.0 on pci0 re0: > > Chip rev. 0x18000000 > > re0: MAC rev. 0x00000000 > > re0: Ethernet address: 00:30:18:ae:1a:1b > > re0: [FILTER] > > re1: port > > 0xf000-0xf0ff mem 0xfdffd000-0xfdffd0ff irq 19 at device 11.0 on pci0 > > re1: Chip rev. 0x18000000 > > re1: MAC rev. 0x00000000 > > re1: Ethernet address: 00:30:18:ae:1a:1c > > re1: [FILTER] > > re2: port > > 0xec00-0xecff mem 0xfdffc000-0xfdffc0ff irq 16 at device 12.0 on pci0 > > re2: Chip rev. 0x18000000 > > re2: MAC rev. 0x00000000 > > re2: Ethernet address: 00:30:18:ae:1a:1d > > re2: [FILTER] > > > > re0@pci0:0:9:0: class=0x020000 card=0x10ec16f3 chip=0x816710ec rev=0x10 > > hdr=0x00 re1@pci0:0:11:0: class=0x020000 card=0x10ec16f3 > > chip=0x816710ec rev=0x10 hdr=0x00 re2@pci0:0:12:0: class=0x020000 > > card=0x10ec16f3 chip=0x816710ec rev=0x10 hdr=0x00 > > Have you tried re(4) in HEAD? > I had one report that re(4) in HEAD still does not fix the issue so > I posted a possible workaround for that. Unfortunately he didn't > report back so I don't know whether it was right workaround or not. > If re(4) in HEAD does not fix the issue, would you try attached > patch and let me know how it goes? Firstly IANAKH, my expertise in this area stops after "make kernel". I updated /usr/src/sys/dev/re/if_re.c /usr/src/sys/pci/if_rlreg.h to HEAD I still get "PHY read failed" with and without the patch. -- ian j hart