All of lore.kernel.org
 help / color / mirror / Atom feed
From: Phil Turmel <philip@turmel.org>
To: Scott D'Vileskis <sdvileskis@gmail.com>
Cc: linux-raid@vger.kernel.org
Subject: Re: Raid stalls during --replace and other disk activity...
Date: Sat, 15 Mar 2014 16:59:45 -0400	[thread overview]
Message-ID: <5324BF41.3060401@turmel.org> (raw)
In-Reply-To: <CAK_KU4bZnVSPR+mF3AX_sjQjTdnMonP9x+v84g9SEAN5Necr5g@mail.gmail.com>

On 03/15/2014 03:53 PM, Scott D'Vileskis wrote:
>>
>> This is deliberate, and in my opinion, desirable.  Same thing happens
>> when doing a check or any kind of rebuild on multiple arrays that use
>> partitions on shared underlying devices.  If you don't limit the raid
>> background activity, the extra seek load on the devices can severely cut
>> performance, especially on spinning rust.
>>
> 
> If I was doing a 'normal' resync, I would also agree.
> 
> But my point was this...
> My raid was doing a --replace of /dev/sdf1 with /dev/sde1
> It was only reading/writing from/to those two devices, not the other
> disks in the raid. (i.e. /dev/sd[a-d] were not being used)
> The other job I started (dd if=/dev/zero of=/dev/sda2 bs=1M) was only
> writing to /dev/sda
> 
> These jobs should be are perfectly capable of running in parallel
> without slowing eachother down.

Good point.  If I were Neil, I'd say "Patch Welcome!".  :-)

> I think the logic that pauses a resync on 'related' disk activity
> should treat a --replace slightly different

But I would insist that the solution be generic -- make the existing
logic notice that the activity is on unrelated members, so all
background activity would benefit from the new algorithm.  IMHO.

Phil


      reply	other threads:[~2014-03-15 20:59 UTC|newest]

Thread overview: 4+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2014-03-15  4:28 Raid stalls during --replace and other disk activity Scott D'Vileskis
2014-03-15 19:19 ` Phil Turmel
2014-03-15 19:53   ` Scott D'Vileskis
2014-03-15 20:59     ` Phil Turmel [this message]

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=5324BF41.3060401@turmel.org \
    --to=philip@turmel.org \
    --cc=linux-raid@vger.kernel.org \
    --cc=sdvileskis@gmail.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.