linuxppc-dev.lists.ozlabs.org archive mirror
 help / color / mirror / Atom feed
From: Brian King <brking@us.ibm.com>
To: Josh Boyer <jwboyer@linux.vnet.ibm.com>
Cc: akpm@osdl.org, James Bottomley <jejb@mulgrave.il.steeleye.com>,
	John H Rose <johnrose@us.ibm.com>,
	linuxppc-dev@ozlabs.org, torvalds@osdl.org,
	Paul Mackerras <paulus@samba.org>,
	Jeff Garzik <jgarzik@pobox.com>
Subject: Re: ipr SATA support is causing problems
Date: Fri, 10 Nov 2006 09:07:35 -0600	[thread overview]
Message-ID: <455495B7.4050701@us.ibm.com> (raw)
In-Reply-To: <1163164499.4168.3.camel@zod.rchland.ibm.com>

Josh Boyer wrote:
> On Fri, 2006-11-10 at 23:07 +1100, Paul Mackerras wrote:
>> Josh Boyer writes:
>>
>>> John Rose reported this a few days ago.  Brian responded with this:
>>>
>>> http://marc.theaimsgroup.com/?l=linux-ide&m=116169938407596&w=2
>> Didn't John say he still saw the crash with that change?
> 
> He did on my suggestion, which technically wasn't correct.  I haven't
> seen anything from him after Brian replied with a real patch.
> 
> I know I helped debug a crash that looked very much like that on a P5
> box with a 2.6.19-rcX kernel and Brian's patch fixed it.

Paul,

The backtrace you posted is identical to the problem fixed by the patch
that Josh pointed you to. The patch has been submitted to Jeff. I asked
Jeff two days ago if this could be pulled in for 2.6.19. I verified the
bug and the fix on my machine. As far as I know, John still has not tried
the actual patch pointed to above. Just to clarify, without this patch,
any monolithic kernel will not work on pseries, but modular kernels will
still work. If someone can reproduce any problem with the libata patch
above applied, please let me know.

Brian

-- 
Brian King
eServer Storage I/O
IBM Linux Technology Center

  reply	other threads:[~2006-11-10 15:08 UTC|newest]

Thread overview: 10+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2006-11-10 11:37 ipr SATA support is causing problems Paul Mackerras
2006-11-10 11:48 ` Josh Boyer
2006-11-10 12:07   ` Paul Mackerras
2006-11-10 13:14     ` Josh Boyer
2006-11-10 15:07       ` Brian King [this message]
2006-11-10 15:23 ` Brian King
2006-11-10 23:06   ` Paul Mackerras
2006-11-11 23:56     ` Brian King
2006-11-12  0:50       ` Benjamin Herrenschmidt
2006-11-12  3:55         ` Brian King

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=455495B7.4050701@us.ibm.com \
    --to=brking@us.ibm.com \
    --cc=akpm@osdl.org \
    --cc=jejb@mulgrave.il.steeleye.com \
    --cc=jgarzik@pobox.com \
    --cc=johnrose@us.ibm.com \
    --cc=jwboyer@linux.vnet.ibm.com \
    --cc=linuxppc-dev@ozlabs.org \
    --cc=paulus@samba.org \
    --cc=torvalds@osdl.org \
    /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
Be sure your reply has a Subject: header at the top and a blank line before the message body.
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).