Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1754449AbZJCDDc (ORCPT ); Fri, 2 Oct 2009 23:03:32 -0400 Received: (majordomo@vger.kernel.org) by vger.kernel.org id S1752967AbZJCDDc (ORCPT ); Fri, 2 Oct 2009 23:03:32 -0400 Received: from SpacedOut.fries.net ([67.64.210.234]:33090 "EHLO SpacedOut.fries.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1752428AbZJCDDb (ORCPT ); Fri, 2 Oct 2009 23:03:31 -0400 X-Greylist: delayed 510 seconds by postgrey-1.27 at vger.kernel.org; Fri, 02 Oct 2009 23:03:31 EDT Date: Fri, 2 Oct 2009 21:54:16 -0500 From: David Fries To: linux-kernel@vger.kernel.org Cc: d.stussy@yahoo.com, Joao Ramos , Linus Torvalds , "Rafael J. Wysocki" Subject: [bisected] 2.6.31 regression sis5513 PIO Mode 0 hang Message-ID: <20091003025415.GA28295@spacedout.fries.net> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline User-Agent: Mutt/1.5.18 (2008-05-17) X-Greylist: Sender is SPF-compliant, not delayed by milter-greylist-3.0 (SpacedOut.fries.net [127.0.0.1]); Fri, 02 Oct 2009 21:54:22 -0500 (CDT) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 4400 Lines: 105 I just did a git bisection from 2.6.30 to 2.6.31 because 2.6.31 will not boot on this system. d.stussy@yahoo.com's post September 12th looks the same, different CPU but both have sis5513 IDE chips. His would normally init the ethernet chip next, mine would do PS/2 next, both hang right after the Uniform CD-ROM message. This was bisected down to chaging PIO mode 0 for probing. What problem was that patch trying to solve? Reverting just that patch at the top of 2.6.31 tree works. I can test patches. Assigned bugzilla.kernel.org bug 14310 Uniform Multi-Platform E-IDE driver sis5513 0000:00:02.5: SiS635 ATA 100 (2nd gen) controller sis5513 0000:00:02.5: IDE controller (0x1039:0x5513 rev 0xd0) sis5513 0000:00:02.5: not 100% native mode: will probe irqs later ide0: BM-DMA at 0xff00-0xff07 ide1: BM-DMA at 0xff08-0xff0f Probing IDE interface ide0... Switched to high resolution mode on CPU 0 hda: ST3250823A, ATA DISK drive hda: host max PIO4 wanted PIO255(auto-tune) selected PIO4 hda: UDMA/100 mode selected Probing IDE interface ide1... hdc: _NEC DVD_RW ND-2500A, ATAPI CD/DVD-ROM drive hdc: host max PIO4 wanted PIO255(auto-tune) selected PIO4 hdc: UDMA/33 mode selected ide0 at 0x1f0-0x1f7,0x3f6 on irq 14 ide1 at 0x170-0x177,0x376 on irq 15 ide_generic: please use "probe_mask=0x3f" module parameter for probing all legacy ISA IDE ports ide-gd driver 1.18 hda: max request size: 512KiB hda: 488397168 sectors (250059 MB) w/8192KiB Cache, CHS=30401/255/63 hda: cache flushes supported hda: hda1 hda2 hda3 ide-cd driver 5.00 ide-cd: hdc: ATAPI 40X DVD-ROM DVD-R CD-R/RW drive, 2048kB Cache Uniform CD-ROM driver Revision: 3.20 commit 6029336426a2b43e4bc6f4a84be8789a047d139e Author: Joao Ramos Date: Sun May 17 17:22:54 2009 +0200 ide: try to use PIO Mode 0 during probe if possible Initially set PIO Mode 0 for all host drivers that have a 'set_pio_mode' method before the IDE core figures out the most suited PIO mode for the attached device. Signed-off-by: Joao Ramos Cc: Sergei Shtylyov Signed-off-by: Bartlomiej Zolnierkiewicz diff --git a/drivers/ide/ide-probe.c b/drivers/ide/ide-probe.c index 7f264ed..b609a58 100644 --- a/drivers/ide/ide-probe.c +++ b/drivers/ide/ide-probe.c @@ -1032,6 +1032,15 @@ static void ide_port_init_devices(ide_hwif_t *hwif) if (port_ops && port_ops->init_dev) port_ops->init_dev(drive); } + + ide_port_for_each_dev(i, drive, hwif) { + /* + * default to PIO Mode 0 before we figure out + * the most suited mode for the attached device + */ + if (port_ops && port_ops->set_pio_mode) + port_ops->set_pio_mode(drive, 0); + } } static void ide_init_port(ide_hwif_t *hwif, unsigned int port, vendor_id : CentaurHauls cpu family : 6 model : 7 model name : VIA Samuel 2 lspci 00:00.0 Host bridge: Silicon Integrated Systems [SiS] 635 Host (rev 11) 00:01.0 PCI bridge: Silicon Integrated Systems [SiS] Virtual PCI-to-PCI bridge (AGP) 00:02.0 ISA bridge: Silicon Integrated Systems [SiS] SiS85C503/5513 (LPC Bridge) 00:02.1 SMBus: Silicon Integrated Systems [SiS] SiS961/2 SMBus Controller 00:02.2 USB Controller: Silicon Integrated Systems [SiS] USB 1.1 Controller (rev 07) 00:02.3 USB Controller: Silicon Integrated Systems [SiS] USB 1.1 Controller (rev 07) 00:02.5 IDE interface: Silicon Integrated Systems [SiS] 5513 [IDE] (rev d0) 00:02.6 Modem: Silicon Integrated Systems [SiS] AC'97 Modem Controller (rev a0) 00:02.7 Multimedia audio controller: Silicon Integrated Systems [SiS] AC'97 Sound Controller (rev a0) 00:03.0 Ethernet controller: Silicon Integrated Systems [SiS] SiS900 PCI Fast Ethernet (rev 90) 00:09.0 Multimedia video controller: Conexant CX23880/1/2/3 PCI Video and Audio Decoder (rev 05) 00:09.2 Multimedia controller: Conexant CX23880/1/2/3 PCI Video and Audio Decoder [MPEG Port] (rev 05) 01:00.0 VGA compatible controller: Silicon Integrated Systems [SiS] 315PRO PCI/AGP VGA Display Adapter -- David Fries http://fries.net/~david/ (PGP encryption key available) -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/