Re: 2.5.37 oopses at boot in ide_toggle_bounce

[Jens Axboe <axboe@suse.de>]

On Mon, Sep 23 2002, Jens Axboe wrote:
> On Mon, Sep 23 2002, Andries Brouwer wrote:
> > On Mon, Sep 23, 2002 at 09:41:42AM +0200, Jens Axboe wrote:
> > 
> > > Patch is fine, thanks Andries.
> > 
> > Yes, that patch allows the kernel to boot.
> > The booted system has two main problems that 2.5.33 does not have:
> > (i) It no longer sees my disks on an HPT366,
> > (ii) pgrp handling changed, so that some programs hang.
> 
> I'm assuming we are talking about 2.5.38? Can you send me the kernel
> boot log, thanks.

Ah hang on, please boot with this patch from Ivan. That should make it
work again.

--- linux/drivers/ide/setup-pci.c.bk	Sat Sep 21 12:58:45 2002
+++ linux/drivers/ide/setup-pci.c	Sat Sep 21 12:59:59 2002
@@ -421,20 +421,17 @@ static ide_hwif_t *ide_hwif_configure(st
 {
 	unsigned long ctl = 0, base = 0;
 	ide_hwif_t *hwif;
-	
-	if ((dev->class >> 8) != PCI_CLASS_STORAGE_IDE)
-	{
-	    	/*  Possibly we should fail if these checks report true */
-	    	ide_pci_check_iomem(dev, d, 2*port);
-	    	ide_pci_check_iomem(dev, d, 2*port+1);
-	 
-		ctl  = pci_resource_start(dev, 2*port+1);
-		base = pci_resource_start(dev, 2*port);
-		if ((ctl && !base) || (base && !ctl)) {
-			printk(KERN_ERR "%s: inconsistent baseregs (BIOS) "
-				"for port %d, skipping\n", d->name, port);
-			return NULL;
-		}
+
+	/*  Possibly we should fail if these checks report true */
+	ide_pci_check_iomem(dev, d, 2*port);
+	ide_pci_check_iomem(dev, d, 2*port+1);
+ 
+	ctl  = pci_resource_start(dev, 2*port+1);
+	base = pci_resource_start(dev, 2*port);
+	if ((ctl && !base) || (base && !ctl)) {
+		printk(KERN_ERR "%s: inconsistent baseregs (BIOS) "
+			"for port %d, skipping\n", d->name, port);
+		return NULL;
 	}
 	if (!ctl)
 	{


-- 
Jens Axboe