LKML Archive on lore.kernel.org
help / color / mirror / Atom feed
From: Mark Lord <liml@rtr.ca>
To: Yinghai Lu <Yinghai.Lu@Sun.COM>
Cc: Jeff Garzik <jgarzik@redhat.com>,
	Linux Kernel Mailing List <linux-kernel@vger.kernel.org>,
	linux-ide@vger.kernel.org,
	Andrew Morton <akpm@linux-foundation.org>,
	Saeed Bishara <saeed@marvell.com>
Subject: Re: [PATCH] sata_mv: fix loop with last port
Date: Thu, 07 Feb 2008 23:30:42 -0500	[thread overview]
Message-ID: <47ABDAF2.7030209@rtr.ca> (raw)
In-Reply-To: <200802071506.17459.yinghai.lu@sun.com>

Yinghai Lu wrote:
> [PATCH] sata_mv: fix loop with last port
> 
> commit f351b2d638c3cb0b95adde3549b7bfaf3f991dfa
>         sata_mv: Support SoC controllers
> 
> cause panic:
> 
> scsi 4:0:0:0: Direct-Access     ATA      HITACHI HDS7225S V44O PQ: 0 ANSI: 5
> sd 4:0:0:0: [sde] 488390625 512-byte hardware sectors (250056 MB)
> sd 4:0:0:0: [sde] Write Protect is off
> sd 4:0:0:0: [sde] Mode Sense: 00 3a 00 00
> sd 4:0:0:0: [sde] Write cache: enabled, read cache: enabled, doesn't support DPO or FUA
> sd 4:0:0:0: [sde] 488390625 512-byte hardware sectors (250056 MB)
> sd 4:0:0:0: [sde] Write Protect is off
> sd 4:0:0:0: [sde] Mode Sense: 00 3a 00 00
> sd 4:0:0:0: [sde] Write cache: enabled, read cache: enabled, doesn't support DPO or FUA
>  sde:<1>BUG: unable to handle kernel NULL pointer dereference at 000000000000001a
> IP: [<ffffffff806262c7>] mv_interrupt+0x21c/0x4cc
> PGD 0
> Oops: 0000 [1] SMP
> CPU 3
> Modules linked in:
> Pid: 0, comm: swapper Not tainted 2.6.24-smp-08636-g0afc2ed-dirty #26
> RIP: 0010:[<ffffffff806262c7>]  [<ffffffff806262c7>] mv_interrupt+0x21c/0x4cc
> RSP: 0000:ffff8102050bbec8  EFLAGS: 00010297
> RAX: 0000000000000008 RBX: 0000000000000000 RCX: 0000000000000003
> RDX: 0000000000008000 RSI: 0000000000000286 RDI: ffff8102035180e0
> RBP: 0000000000000001 R08: 0000000000000003 R09: ffff8102036613e0
> R10: 0000000000000002 R11: ffffffff8061474c R12: ffff8102035bf828
> R13: 0000000000000008 R14: ffff81020348ece8 R15: ffffc20002cb2000
> FS:  0000000000000000(0000) GS:ffff810405025700(0000) knlGS:0000000000000000
> CS:  0010 DS: 0018 ES: 0018 CR0: 000000008005003b
> CR2: 000000000000001a CR3: 0000000000201000 CR4: 00000000000006e0
> DR0: 0000000000000000 DR1: 0000000000000000 DR2: 0000000000000000
> DR3: 0000000000000000 DR6: 00000000ffff0ff0 DR7: 0000000000000400
> Process swapper (pid: 0, threadinfo ffff810405094000, task ffff8102050b28c0)
> Stack:  000000010000000c 0002040000220400 0000001100000002 ffff81020348eda8
>  0000000000000001 ffff8102035f2cc0 0000000000000000 0000000000000000
>  0000000000000018 0000000000000000 0000000000000000 ffffffff80269ee8
> Call Trace:
>  <IRQ>  [<ffffffff80269ee8>] ? handle_IRQ_event+0x25/0x53
>  [<ffffffff8026b393>] ? handle_fasteoi_irq+0x90/0xc8
>  [<ffffffff802218e2>] ? do_IRQ+0xf1/0x15f
>  [<ffffffff8021df24>] ? default_idle+0x0/0x55
>  [<ffffffff8021f361>] ? ret_from_intr+0x0/0xa
>  <EOI>  [<ffffffff8023010c>] ? lapic_next_event+0x0/0xa
>  [<ffffffff8021df55>] ? default_idle+0x31/0x55
>  [<ffffffff8021df50>] ? default_idle+0x2c/0x55
>  [<ffffffff8021df24>] ? default_idle+0x0/0x55
>  [<ffffffff8021e00b>] ? cpu_idle+0x92/0xb8
> 
> 
> Code: 41 14 85 c0 89 44 24 14 0f 84 9d 02 00 00 f7 d0 01 d6 41 89 d5 89 41 14 8b 41 14 89 34 24 e9 7e 02 00 00 49 63 c5 49 8b 5c c6 48 <f6> 43 1a 80 4c 8b a3 20 37 00 00 0f 85 62 02 00 00 31 c9 41 83
> RIP  [<ffffffff806262c7>] mv_interrupt+0x21c/0x4cc
>  RSP <ffff8102050bbec8>
> CR2: 000000000000001a
> ---[ end trace 2583b5f7a5350584 ]---
> Kernel panic - not syncing: Aiee, killing interrupt handler!
> 
> last_port already include port0 base.
> this patch change use last_port directly, and move pp assignment later.
> 
> Signed-off-by: Yinghai Lu <yinghai.lu@sun.com>
..

Yup, obvious bug fixes, thanks.
Signed-off-by: Mark Lord <mlord@pobox.com>

> Index: linux-2.6/drivers/ata/sata_mv.c
> ===================================================================
> --- linux-2.6.orig/drivers/ata/sata_mv.c
> +++ linux-2.6/drivers/ata/sata_mv.c
> @@ -1716,14 +1716,16 @@ static void mv_host_intr(struct ata_host
>  	VPRINTK("ENTER, hc%u relevant=0x%08x HC IRQ cause=0x%08x\n",
>  		hc, relevant, hc_irq_cause);
>  
> -	for (port = port0; port < port0 + last_port; port++) {
> +	for (port = port0; port < last_port; port++) {
>  		struct ata_port *ap = host->ports[port];
> -		struct mv_port_priv *pp = ap->private_data;
> +		struct mv_port_priv *pp;
>  		int have_err_bits, hard_port, shift;
>  
>  		if ((!ap) || (ap->flags & ATA_FLAG_DISABLED))
>  			continue;
>  
> +		pp = ap->private_data;
> +
>  		shift = port << 1;		/* (port * 2) */
>  		if (port >= MV_PORTS_PER_HC) {
>  			shift++;	/* skip bit 8 in the HC Main IRQ reg */

  reply	other threads:[~2008-02-08  4:30 UTC|newest]

Thread overview: 4+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2008-02-07 23:06 Yinghai Lu
2008-02-08  4:30 ` Mark Lord [this message]
2008-02-08 21:45   ` Andrew Morton
2008-02-11 19:31 ` Jeff Garzik

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=47ABDAF2.7030209@rtr.ca \
    --to=liml@rtr.ca \
    --cc=Yinghai.Lu@Sun.COM \
    --cc=akpm@linux-foundation.org \
    --cc=jgarzik@redhat.com \
    --cc=linux-ide@vger.kernel.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=saeed@marvell.com \
    --subject='Re: [PATCH] sata_mv: fix loop with last port' \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).