All of lore.kernel.org
 help / color / mirror / Atom feed
From: "Geir A. Myrestrand" <geir.myrestrand@falconstor.com>
To: xfs@oss.sgi.com
Subject: Re: New CentOS4/RHEL4-compatible xfs module rpms
Date: Thu, 07 Dec 2006 21:58:38 -0500	[thread overview]
Message-ID: <4578D4DE.1090400@falconstor.com> (raw)
In-Reply-To: <20061207232641.GP33919298@melbourne.sgi.com>

[-- Attachment #1: Type: text/plain, Size: 1906 bytes --]

David Chinner wrote:
> On Thu, Dec 07, 2006 at 01:17:32PM -0500, Geir A. Myrestrand wrote:
>>> Geir A. Myrestrand wrote:
>>>
>>>> However, I run into issues with xfs_freeze as it often locks up when I 
>>>> try to freeze a file system where there is I/O activity. Sometimes it 
>>>> happen on the first xfs_freeze invocation to freeze the file system, 
>>>> other times I have to unfreeze and then it happens on the second time I 
>>>> freeze. xfs_freeze never returns when this happens.
>>>>
>>>> Looks like xfs_io get stuck --see partial output from `ps auxf`:
>>>>
>>>> strace -ff -o freeze.txt xfs_freeze -f /mnt/xfs
>>>>  \_ /bin/sh -f /usr/sbin/xfs_freeze -f /mnt/xfs
>>>>      \_ /usr/sbin/xfs_io -r -p xfs_freeze -x -c freeze /mnt/xfs
>>>>
>>>> Anyone else encountering this issue?
> 
> Yes, and I fixed it about a 2 weeks ago. It's an ABBA deadlock between
> lookup of multiple, already dirty, metadata buffers and synchronous buftarg
> flushing (that occurs when trying to freeze a filesystem)
> 
> <sigh>

That is awesome news, mate!  :-)

> The problem is that during a freeze, the filesystem may
> still be doing stuff - like flushing delalloc data buffers -
> in the background and hence we can be trying to lock buffers
> that were on the delwri list at the same time.  Hence we can
> get ABBA deadlocks between threads doing allocation and the
> buftarg flush (freeze) thread.

That sounds like an accurate description of my test environment. I bet 
this is the issue...

> Fix it by skipping locked (and pinned) buffers as we traverse the
> delwri buffer list.

Good to know that you're one step ahead!

> And the diff was:
> 
> http://oss.sgi.com/cgi-bin/cvsweb.cgi/linux-2.6-xfs/fs/xfs/linux-2.6/xfs_buf.c.diff?r1=1.229;r2=1.230

Excellent. I will try this tomorrow (it's late in the evening here in 
New York now). I'll let you know how it works out.

Thanks!


-- 

Geir A. Myrestrand

[-- Attachment #2: geir.myrestrand.vcf --]
[-- Type: text/x-vcard, Size: 446 bytes --]

begin:vcard
fn:Geir A. Myrestrand
n:Myrestrand;Geir A.
org:FalconStor Software, Inc.
adr:Suite 2S01;;2 Huntington Quadrangle;Melville;NY;11747;U.S.A.
email;internet:geir.myrestrand@falconstor.com
title:Senior Software Engineer
tel;work:(631) 773-5842
tel;cell:(631) 747-5049
note;quoted-printable:Skype: gmyrestrand=0D=0A=
	MSN Messenger: gmyrestrand@hotmail.com=0D=0A=
	
x-mozilla-html:FALSE
url:http://www.falconstor.com
version:2.1
end:vcard


  reply	other threads:[~2006-12-08  3:00 UTC|newest]

Thread overview: 27+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2006-11-19 19:07 New CentOS4/RHEL4-compatible xfs module rpms Eric Sandeen
2006-12-07 17:25 ` Geir A. Myrestrand
2006-12-07 17:52   ` Eric Sandeen
2006-12-07 18:17     ` Geir A. Myrestrand
2006-12-07 23:26       ` David Chinner
2006-12-08  2:58         ` Geir A. Myrestrand [this message]
2006-12-07 18:19     ` Geir A. Myrestrand
2006-12-07 20:51     ` Geir A. Myrestrand
2006-12-07 21:11       ` Nathan Scott
2006-12-07 21:35         ` Geir A. Myrestrand
2006-12-07 21:40           ` Nathan Scott
2006-12-07 21:51             ` Geir A. Myrestrand
2006-12-07 22:06               ` Nathan Scott
2006-12-07 22:16                 ` Geir A. Myrestrand
2006-12-07 22:18                   ` Eric Sandeen
2006-12-07 22:25                     ` Geir A. Myrestrand
2006-12-07 22:30                       ` Eric Sandeen
2006-12-07 22:52                         ` Geir A. Myrestrand
2006-12-07 22:53                           ` Eric Sandeen
2006-12-07 23:08                             ` Geir A. Myrestrand
2007-07-19 14:37 ` James Braid
2007-07-19 15:15   ` Eric Sandeen
2007-07-21  3:34     ` James Braid
2007-07-21 14:58       ` Eric Sandeen
2007-07-21 15:36         ` James Braid
  -- strict thread matches above, loose matches on Subject: below --
2007-08-08 16:04 James Pearson
2007-08-08 16:09 ` Eric Sandeen

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=4578D4DE.1090400@falconstor.com \
    --to=geir.myrestrand@falconstor.com \
    --cc=xfs@oss.sgi.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.