lists.openwall.net   lists  /  announce  owl-users  owl-dev  john-users  john-dev  passwdqc-users  yescrypt  popa3d-users  /  oss-security  kernel-hardening  musl  sabotage  tlsify  passwords  /  crypt-dev  xvendor  /  Bugtraq  Full-Disclosure  linux-kernel  linux-netdev  linux-ext4  linux-hardening  linux-cve-announce  PHC 
Open Source and information security mailing list archives
 
Hash Suite: Windows password security audit tool. GUI, reports in PDF.
[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <20070425085432.GD17130@htj.dyndns.org>
Date:	Wed, 25 Apr 2007 17:54:32 +0900
From:	Tejun Heo <htejun@...il.com>
To:	Andrew Morton <akpm@...ux-foundation.org>
Cc:	"Berck E. Nash" <flyboy@...il.com>,
	"linux-kernel@...r.kernel.org" <linux-kernel@...r.kernel.org>,
	Jeff Garzik <jeff@...zik.org>, linux-ide@...r.kernel.org
Subject: Re: 2.6.21-rc7-mm1 BUG at kernel/sched-clock.c:175 init_sched_clock()

On Wed, Apr 25, 2007 at 05:42:41PM +0900, Tejun Heo wrote:
> Andrew Morton wrote:
> >> [    2.953862] RIP: 0010:[<ffffffff803ae98b>]  [<ffffffff803ae98b>] scsi_schedule_eh+0xa/0x57
> >> [    3.058672]  [<ffffffff803e1e01>] ata_port_schedule_eh+0x4c/0x50
> >> [    3.064725]  [<ffffffff803e1ea7>] ata_port_abort+0xa2/0xae
> >> [    3.070248]  [<ffffffff803e1ef9>] ata_port_freeze+0x46/0x57
> >> [    3.075853]  [<ffffffff803e4c39>] ahci_interrupt+0x300/0x47a
> >> [    3.081552]  [<ffffffff8020eb83>] handle_IRQ_event+0x27/0x57
> >> [    3.087253]  [<ffffffff8029a021>] handle_edge_irq+0xee/0x133
> >> [    3.092960]  [<ffffffff8025f4e1>] do_IRQ+0x6d/0xd5
> >> [    3.097793]  [<ffffffff80255071>] ret_from_intr+0x0/0xa
> >> [    3.103059]  [<ffffffff8024e88b>] mwait_idle+0x46/0x4b
> >> [    3.108231]  [<ffffffff802422f4>] cpu_idle+0x87/0xaa
> >> [    3.113227]  [<ffffffff8025c988>] rest_init+0x49/0x4b
> >> [    3.118322]  [<ffffffff805dca5d>] start_kernel+0x291/0x29c
> >> [    3.123837]  [<ffffffff805dc13a>] _sinittext+0x13a/0x141
> >>
> > 
> > So we took an AHCI interrupt when ata_port.scsi_host was still NULL.
> > 
> > It appears that ATA is presently requesting its IRQ before allocating all
> > the resources which are needed to handle an interrupt.  Does this
> > (resource-leaky) patch fix things?
> 
> No, that would break host probing.  The port is in frozen (controller
> initialized and IRQs masked off) state, so it's not allowed to take
> interrupt.  If interrupt triggers at this point, it's low level driver
> bug.  Berck, how reliably can you reproduce this problem?  Can you post
> the result of 'lspci -nn'?

Berck, does this patch fix your problem?

diff --git a/drivers/ata/ahci.c b/drivers/ata/ahci.c
index 72c286e..a128717 100644
--- a/drivers/ata/ahci.c
+++ b/drivers/ata/ahci.c
@@ -1369,10 +1369,14 @@ static unsigned int ahci_qc_issue(struct ata_queued_cmd *qc)
 
 static void ahci_freeze(struct ata_port *ap)
 {
+	void __iomem *mmio = ap->host->iomap[AHCI_PCI_BAR];
 	void __iomem *port_mmio = ahci_port_base(ap);
 
 	/* turn IRQ off */
 	writel(0, port_mmio + PORT_IRQ_MASK);
+
+	/* clear IRQ pending bit */
+	writel(1 << ap->port_no, mmio + HOST_IRQ_STAT);
 }
 
 static void ahci_thaw(struct ata_port *ap)
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

Powered by blists - more mailing lists

Powered by Openwall GNU/*/Linux Powered by OpenVZ