All of lore.kernel.org
 help / color / mirror / Atom feed
From: Miao Xie <miaox@cn.fujitsu.com>
To: dsterba@suse.cz
Cc: Josef Bacik <jbacik@fusionio.com>,
	Linux Btrfs <linux-btrfs@vger.kernel.org>
Subject: Re: [PATCH] Btrfs: fix the deadlock between the transaction attach and commit
Date: Mon, 18 Feb 2013 18:36:39 +0800	[thread overview]
Message-ID: <51220437.8000209@cn.fujitsu.com> (raw)
In-Reply-To: <20130212125632.GV12339@twin.jikos.cz>

(Sorry for the late reply, I was on my vacation of the Spring Festival last week.)

On Tue, 12 Feb 2013 13:56:32 +0100, David Sterba wrote:
> On Mon, Feb 11, 2013 at 03:35:37PM -0500, Josef Bacik wrote:
>> or something like that.  Me and kdave reproduced by running 274 in a loop, it
>> happpened pretty quick.  I'd fix it myself but I have to leave my house for
>> people to come look at it.  If you haven't fixed this by tomorrow I'll fix it
>> up.  Thanks,
> 
> I found 224 stuck with this
> 
[SNIP]
> 
> mounted with noatime,space_cache

Thanks for your test. My test skipped the 274th case because it always fails,
and all the other cases passed, so I didn't hit this problem.

Anyways, very sorry for my stupid patch.
(I have reviewed Josef's fix patch, and commented on it, please see the reply
of that patch)

Thanks
Miao

  reply	other threads:[~2013-02-18 10:36 UTC|newest]

Thread overview: 5+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2013-02-08  6:55 [PATCH] Btrfs: fix the deadlock between the transaction attach and commit Miao Xie
2013-02-11 20:35 ` Josef Bacik
2013-02-12 12:56   ` David Sterba
2013-02-18 10:36     ` Miao Xie [this message]
2013-02-12 14:39 ` Josef Bacik

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=51220437.8000209@cn.fujitsu.com \
    --to=miaox@cn.fujitsu.com \
    --cc=dsterba@suse.cz \
    --cc=jbacik@fusionio.com \
    --cc=linux-btrfs@vger.kernel.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 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.