All of lore.kernel.org
 help / color / mirror / Atom feed
From: Chris Dunlop <chris@onthe.net.au>
To: Joe Lawrence <joe.lawrence@stratus.com>
Cc: linux-raid@vger.kernel.org, linux-scsi@vger.kernel.org,
	NeilBrown <neilb@suse.de>
Subject: Re: mpt2sas + raid10 goes boom
Date: Tue, 9 Apr 2013 12:22:29 +1000	[thread overview]
Message-ID: <20130409022229.GA2872@onthe.net.au> (raw)
In-Reply-To: <20130408093242.34100af7@jlaw-desktop.mno.stratus.com>

On Mon, Apr 08, 2013 at 09:32:42AM -0400, Joe Lawrence wrote:
> On Mon, 8 Apr 2013 03:53:51 -0400
> "Chris Dunlop" <chris@onthe.net.au> wrote:
> 
>> Upon reflection, given this smells like a controller issue, this
>> may be better addressed to linux-scsi@vger.kernel.org...
>> 
>> On Mon, Apr 08, 2013 at 05:18:34PM +1000, Chris Dunlop wrote:
> [... snip ...]
>> Apr  8 15:08:53 b4 kernel: [  448.657142] md4: WRITE SAME failed.
>> Manually zeroing.
> 
> Hi Chris,
> 
> WRITE SAME was broken in MD RAID 1/10 up until: 
> 
>   c8dc9c6 md: raid1,10: Handle REQ_WRITE_SAME flag in write bios
> 
> and should be present in any kernel that includes the block layer WRITE
> SAME changes:
> 
>  4363ac7 block: Implement support for WRITE SAME
>  579e8f3 block: Make blkdev_issue_zeroout use WRITE SAME

Aha! That did it. My kernels, 3.7.3 and 3.7.10, include the
latter two changes but not the former. A cherry-pick of c8dc9c6
fixed the issue for me.

> Neil -- should c8dc9c6 go to stable?

I think it definitely should. Without it you can't create a
raid10 and it looks like you have a controller issue!


Thanks Joe, and to all who took an interest in my problem!

Cheers,

Chris.

  reply	other threads:[~2013-04-09  2:22 UTC|newest]

Thread overview: 13+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2013-04-08  7:18 mpt2sas + raid10 goes boom Chris Dunlop
2013-04-08  7:53 ` Chris Dunlop
2013-04-08  8:13   ` Baruch Even
2013-04-08 13:32   ` Joe Lawrence
2013-04-09  2:22     ` Chris Dunlop [this message]
2013-04-09  4:18       ` NeilBrown
2013-04-08  8:24 ` George Shuklin
2013-04-08 10:39 ` Brad Campbell
  -- strict thread matches above, loose matches on Subject: below --
2013-04-08 13:13 Matthias Prager
2013-04-08 17:02 ` Matthias Prager
2013-04-08 20:15   ` Baruch Even
2013-04-09 10:11     ` Matthias Prager
2013-04-09 12:45       ` Matthias Prager

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=20130409022229.GA2872@onthe.net.au \
    --to=chris@onthe.net.au \
    --cc=joe.lawrence@stratus.com \
    --cc=linux-raid@vger.kernel.org \
    --cc=linux-scsi@vger.kernel.org \
    --cc=neilb@suse.de \
    /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.