All of lore.kernel.org
 help / color / mirror / Atom feed
From: Dmitry Monakhov <dmonakhov@openvz.org>
To: Justin Maggard <jmaggard10@gmail.com>
Cc: ext4 development <linux-ext4@vger.kernel.org>
Subject: Re: System hang with EXT4
Date: Fri, 11 Dec 2009 05:37:17 +0300	[thread overview]
Message-ID: <873a3ick2q.fsf@openvz.org> (raw)
In-Reply-To: <150c16850912101817g497588b2g3aa4dcaf7c5aed7a@mail.gmail.com>

Justin Maggard <jmaggard10@gmail.com> writes:

> On Thu, Dec 10, 2009 at 5:29 PM, Dmitry Monakhov <dmonakhov@openvz.org> wrote:
>> Justin Maggard <jmaggard10@gmail.com> writes:
>>> I have a little more information to add.  After noticing the recent
>>> "Fix potential quota deadlock" patch on the mailing list, I figured it
>>> would be worth a shot to try it without quotas enabled.  This also
>>> avoids the system hang.  I tried applying that patch, but still had
>>> the same symptoms using that kernel.  So I'm seeing a consistent
>>> system hang with ext4 when delalloc and quotas are enabled on an SMP
>>> system.  With either quotas or delalloc disabled, it doesn't hang.
>>> Both enabled on a single processor system also doesn't hang.
>> You also may try another patch
>> "[PATCH] ext4: fix sleep inside spinlock issue aka #14739 V2"
>
> Yes, that patch looks like it did the trick.  Thanks!
Ohhh.. in fact i've to apologies my patch is wrong a little bit
after i_block_reservation_lock was reacquired second time
i_reserved_meta_blocks block's may be changed so we have to
use add instead of assign
 
-       EXT4_I(inode)->i_reserved_meta_blocks = mdblocks;
+       EXT4_I(inode)->i_reserved_meta_blocks += md_needed;
As result quota leak is possible on heavy SMP stress test.
I've already send correct version.
--
To unsubscribe from this list: send the line "unsubscribe linux-ext4" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

      reply	other threads:[~2009-12-11  2:37 UTC|newest]

Thread overview: 6+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2009-11-24  2:16 System hang with EXT4 Justin Maggard
2009-12-08 15:18 ` Jan Kara
     [not found]   ` <150c16850912091723x7659bf4ke42f209595ceb147@mail.gmail.com>
     [not found]     ` <150c16850912101541k5293b3d7l947b5850fed707b1@mail.gmail.com>
2009-12-10 23:43       ` Justin Maggard
2009-12-11  1:29         ` Dmitry Monakhov
2009-12-11  2:17           ` Justin Maggard
2009-12-11  2:37             ` Dmitry Monakhov [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=873a3ick2q.fsf@openvz.org \
    --to=dmonakhov@openvz.org \
    --cc=jmaggard10@gmail.com \
    --cc=linux-ext4@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.