From: Jack Wang <jinpu.wang@profitbricks.com>
To: NeilBrown <neilb@suse.de>
Cc: Jack Wang <xjtuwjp@gmail.com>,
linux-raid@vger.kernel.org,
Sebastian Riemer <sebastian.riemer@profitbricks.com>,
stable@vger.kernel.org
Subject: Re: [BUG] md hang at schedule in md_write_start
Date: Thu, 12 Sep 2013 09:55:01 +0200 [thread overview]
Message-ID: <52317355.6020201@profitbricks.com> (raw)
In-Reply-To: <20130912085933.54b5021b@notabene.brown>
On 09/12/2013 12:59 AM, NeilBrown wrote:
> On Wed, 11 Sep 2013 09:40:08 +0200 Jack Wang <xjtuwjp@gmail.com> wrote:
>
>> On 09/11/2013 01:54 AM, NeilBrown wrote:
>>> On Tue, 10 Sep 2013 13:09:05 +0200 Jack Wang <jinpu.wang@profitbricks.com>
>>> wrote:
>>>
>>>> snip
>>>>
>>>> Hi Neil,
>>>>
>>>> I notice you send out pull request for md update, which include fix for
>>>> this bug.
>>>>
>>>> I think we'd better include the fix to stable tree at least from 3.4
>>>> above, what do you think?
>>>
>>> I don't think it is a situation that is at all like to occur in normal usage,
>>> so it doesn't seem justified for -stable.
>>>
>>> Do you disagree? Did you ever experience the deadlock in normal usage or
>>> only in artificial situations?
>>
>> Yes, we do see this BUG in our production environment, so I think it's
>> good to include it in stable tree.
>>
>
> I was hoping you would explain how....
>
> Maybe I'm misunderstanding, but as I see it the deadlock can only occur if
> you run "mdadm --stop" while some other process has the block device open
> and is writing to it. That seems like a dumb thing to do and my suggest
> would be to not do it.
> Is there a good reason why you try to stop the array while it is being
> written to.
> Would it make sense for the process to open the block device with O_EXCL.
> This would encourage exclusive access, and would also prevent the deadlock
> from happening.
>
> NeilBrown
>
Thanks Neil for suggestion.
I will look into the code which is developed by other colleagues, and we
will fix that if it is.
Regards,
Jack
prev parent reply other threads:[~2013-09-12 7:55 UTC|newest]
Thread overview: 10+ messages / expand[flat|nested] mbox.gz Atom feed top
2013-08-12 16:33 [BUG] md hang at schedule in md_write_start Jack Wang
2013-08-13 4:31 ` NeilBrown
2013-08-13 7:42 ` Jack Wang
2013-08-14 0:44 ` NeilBrown
2013-08-14 8:09 ` Jack Wang
2013-09-10 11:09 ` Jack Wang
2013-09-10 23:54 ` NeilBrown
2013-09-11 7:40 ` Jack Wang
2013-09-11 22:59 ` NeilBrown
2013-09-12 7:55 ` Jack Wang [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=52317355.6020201@profitbricks.com \
--to=jinpu.wang@profitbricks.com \
--cc=linux-raid@vger.kernel.org \
--cc=neilb@suse.de \
--cc=sebastian.riemer@profitbricks.com \
--cc=stable@vger.kernel.org \
--cc=xjtuwjp@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.