All of lore.kernel.org
 help / color / mirror / Atom feed
From: James Bottomley <James.Bottomley@HansenPartnership.com>
To: Dan Williams <dan.j.williams@intel.com>
Cc: Tom Rini <trini@ti.com>,
	linux-scsi@vger.kernel.org, Xiangliang Yu <yuxiangl@marvell.com>
Subject: Re: Commit a692b0e broke my mvsas card
Date: Tue, 17 Apr 2012 19:47:17 +0400	[thread overview]
Message-ID: <1334677637.3766.66.camel@dabdike> (raw)
In-Reply-To: <1334633159.13899.4.camel@dwillia2-mobl>

On Mon, 2012-04-16 at 20:25 -0700, Dan Williams wrote:
> On Mon, 2012-04-16 at 18:53 -0700, Tom Rini wrote:
> > > Thanks, yes it looks like a new regression.  Can you tell me if the
> > > following fixes it?
> > 
> > No joy, oopses.  http://alnk.org/43youngkirby for the dmesg and oops.
> > 
> 
> Agh, sorry, I rushed that one.  The phy array is initialized later, here
> is another run at it:

Are we sure it's initialised correctly in all the other SAS drivers that
use (well, one other: aic94xx)?

Given the oops issue, perhaps revert this for now and get a working
patch in for the next merge window?

James



  parent reply	other threads:[~2012-04-17 15:47 UTC|newest]

Thread overview: 15+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2012-04-16 22:42 Commit a692b0e broke my mvsas card Tom Rini
2012-04-16 23:12 ` Dan Williams
2012-04-16 23:34   ` Tom Rini
2012-04-16 23:59     ` Dan Williams
2012-04-17  1:53       ` Tom Rini
2012-04-17  3:25         ` Dan Williams
2012-04-17 14:44           ` Tom Rini
2012-04-17 16:50             ` Dan Williams
2012-04-17 17:21               ` Tom Rini
2012-04-17 15:47           ` James Bottomley [this message]
2012-04-17 17:17             ` Dan Williams
2012-04-18  5:11               ` Dan Williams
2012-04-18  5:37                 ` James Bottomley
2012-04-18 15:23                 ` Tom Rini
2012-04-18 15:46                   ` Dan Williams

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=1334677637.3766.66.camel@dabdike \
    --to=james.bottomley@hansenpartnership.com \
    --cc=dan.j.williams@intel.com \
    --cc=linux-scsi@vger.kernel.org \
    --cc=trini@ti.com \
    --cc=yuxiangl@marvell.com \
    /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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.