Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S964891AbVIMRJw (ORCPT ); Tue, 13 Sep 2005 13:09:52 -0400 Received: (majordomo@vger.kernel.org) by vger.kernel.org id S964905AbVIMRJv (ORCPT ); Tue, 13 Sep 2005 13:09:51 -0400 Received: from [204.13.84.100] ([204.13.84.100]:38733 "EHLO stargazer.tbdnetworks.com") by vger.kernel.org with ESMTP id S964891AbVIMRJv (ORCPT ); Tue, 13 Sep 2005 13:09:51 -0400 Subject: Re: 2.6.13.1 locks machine after some time, 2.6.12.5 work fine From: Norbert Kiesel To: Linus Torvalds Cc: linux-kernel@vger.kernel.org In-Reply-To: References: <1126569577.25875.25.camel@defiant> <20050913033814.GA879@tbdnetworks.com> Content-Type: multipart/signed; micalg=pgp-sha1; protocol="application/pgp-signature"; boundary="=-iq5fGUOaCRgtZQnhHVO1" Organization: TBD Networks Date: Tue, 13 Sep 2005 10:09:45 -0700 Message-Id: <1126631386.4555.13.camel@defiant> Mime-Version: 1.0 X-Mailer: Evolution 2.2.3 Sender: linux-kernel-owner@vger.kernel.org X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 14756 Lines: 345 --=-iq5fGUOaCRgtZQnhHVO1 Content-Type: text/plain Content-Transfer-Encoding: quoted-printable On Tue, 2005-09-13 at 07:25 -0700, Linus Torvalds wrote: >=20 > On Mon, 12 Sep 2005, Norbert Kiesel wrote: > >=20 > > diff is appended. Regarding the -rc3 and friends, currently I can't as > > I jumped directly from 12.5 to 13. This is my desktop at work, so I > > try to keep it somewhat stable. However, if you have a guess which > > versions to try, I can give it a spin. It takes some time though to > > test, as the lockup normally only happens after 1 hour or so (although > > I could propably speed this up by doing lots of disk IO). >=20 > No need. The numbers made it clear: this is the same bug that hit the=20 > hpt366 driver: >=20 > 0000:00:10.0 RAID bus controller: Silicon Image, Inc. SiI 0649 > Ultra ATA/100 PCI to ATA Host Controller (rev 01) > ... > 00: 95 10 49 06 07 00 90 02 01 00 04 01 00 40 00 00 > 10: 01 b8 00 00 01 bc 00 00 01 c0 00 00 01 c4 00 00 > 20: 01 c8 00 00 00 00 00 00 00 00 00 00 95 10 49 06 > -30: 00 00 00 00 60 00 00 00 00 00 00 00 0c 01 02 04 > +30: 01 00 00 00 60 00 00 00 00 00 00 00 0c 01 02 04 >=20 > and the exact same cause too.=20 >=20 > I wonder who the _hell_ has been sprinkling these _byte_ writes to the RO= M > enable logic around? >=20 > I bet this will fix it.. >=20 > Linus > --- > diff --git a/drivers/ide/pci/cmd64x.c b/drivers/ide/pci/cmd64x.c > --- a/drivers/ide/pci/cmd64x.c > +++ b/drivers/ide/pci/cmd64x.c > @@ -608,7 +608,7 @@ static unsigned int __devinit init_chips > =20 > #ifdef __i386__ > if (dev->resource[PCI_ROM_RESOURCE].start) { > - pci_write_config_byte(dev, PCI_ROM_ADDRESS, dev->resource[PCI_ROM_RESO= URCE].start | PCI_ROM_ADDRESS_ENABLE); > + pci_write_config_dword(dev, PCI_ROM_ADDRESS, dev->resource[PCI_ROM_RES= OURCE].start | PCI_ROM_ADDRESS_ENABLE); > printk(KERN_INFO "%s: ROM enabled at 0x%08lx\n", name, dev->resource[P= CI_ROM_RESOURCE].start); > } > #endif >=20 Ok, I applied the patch and I'm running it right now, so far so good. Here is the the output of lspci from the patched 2.6.13.1 (not sure if a diff to the unpatched 2.6.13.1 or the 2.6.12.5 would be more useful, so I settled for no diff :-). 0000:00:00.0 Host bridge: VIA Technologies, Inc. VT8363/8365 [KT133/KM133] (rev 02) Control: I/O- Mem+ BusMaster+ SpecCycle- MemWINV- VGASnoop- ParErr- Stepping- SERR- FastB2B- Status: Cap+ 66MHz- UDF- FastB2B- ParErr- DEVSEL=3Dmedium >TAbort- SERR- TAbort- SERR- Reset- FastB2B- Capabilities: [80] Power Management version 2 Flags: PMEClk- DSI- D1+ D2- AuxCurrent=3D0mA PME(D0-,D1-,D2-,D3hot-,D3cold-) Status: D0 PME-Enable- DSel=3D0 DScale=3D0 PME- 00: 06 11 05 83 07 00 30 22 00 00 04 06 00 00 01 00 10: 00 00 00 00 00 00 00 00 00 01 01 00 90 90 00 00 20: 00 e4 f0 e5 00 d0 f0 df 00 00 00 00 00 00 00 00 30: 00 00 00 00 80 00 00 00 00 00 00 00 00 00 0c 00 0000:00:07.0 ISA bridge: VIA Technologies, Inc. VT82C686 [Apollo Super South] (rev 22) Subsystem: VIA Technologies, Inc. VT82C686/A PCI to ISA Bridge Control: I/O+ Mem+ BusMaster+ SpecCycle- MemWINV- VGASnoop- ParErr- Stepping+ SERR- FastB2B- Status: Cap+ 66MHz- UDF- FastB2B- ParErr- DEVSEL=3Dmedium >TAbort- SERR- TAbort- SERR- TAbort- SERR- TAbort- SERR- TAbort- SERR- TAbort- SERR- TAbort- SERR- TAbort- SERR- TAbort- SERR- TAbort- SERR- TAbort- SERR-