All of lore.kernel.org
 help / color / mirror / Atom feed
From: Chris Mason <chris.mason@oracle.com>
To: Badari Pulavarty <pbadari@gmail.com>
Cc: linux-fsdevel <linux-fsdevel@vger.kernel.org>,
	akpm@osdl.org, Zach Brown <zach.brown@oracle.com>
Subject: Re: [RFC PATCH 0/2] O_DIRECT locking rework
Date: Tue, 24 Oct 2006 18:22:07 -0400	[thread overview]
Message-ID: <20061024222207.GK12815@think.oraclecorp.com> (raw)
In-Reply-To: <1161726723.28727.0.camel@dyn9047017100.beaverton.ibm.com>

On Tue, Oct 24, 2006 at 02:52:03PM -0700, Badari Pulavarty wrote:
> > 
> > Thanks, I missed an i_mutex change.  Let me audit mutexes coming in and
> > out again and resend.
> > 
> > I think the new rules should be:
> > 
> > reads: don't need i_mutex at all
> > writes: don't need i_mutex for io inside i_size.
> > 
> > Oh, and don't mess with i_mutex under DIO_OWN_LOCKING.
> 
> No problem.
> 
> I ran into one more nasty looking bug ..

Neat.  Which fsx command line did you use?

-chris


  reply	other threads:[~2006-10-24 22:22 UTC|newest]

Thread overview: 17+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2006-10-20 18:32 [RFC PATCH 0/2] O_DIRECT locking rework Chris Mason
2006-10-20 18:38 ` [RFC PATCH 1/2] placeholder pages Chris Mason
2006-10-20 18:41 ` [RFC PATCH 1/2] page cache locking for O_DIRECT Chris Mason
2006-10-20 19:30 ` [RFC PATCH 0/2] O_DIRECT locking rework Andrew Morton
2006-10-20 20:03   ` Zach Brown
2006-10-20 20:12     ` Chris Mason
2006-10-20 20:05   ` Chris Mason
2006-10-20 20:23   ` Andi Kleen
2006-10-24 19:34 ` Chris Mason
2006-10-24 19:38   ` [RFC PATCH 1/2] placeholder pages Chris Mason
2006-10-24 19:40   ` [RFC PATCH 2/2] O_DIRECT locking via placeholders Chris Mason
2006-10-24 20:28   ` [RFC PATCH 0/2] O_DIRECT locking rework Badari Pulavarty
2006-10-24 20:50     ` Chris Mason
2006-10-24 21:52       ` Badari Pulavarty
2006-10-24 22:22         ` Chris Mason [this message]
2006-10-24 22:47           ` Badari Pulavarty
2006-10-24 22:37       ` Russell Cattelan

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=20061024222207.GK12815@think.oraclecorp.com \
    --to=chris.mason@oracle.com \
    --cc=akpm@osdl.org \
    --cc=linux-fsdevel@vger.kernel.org \
    --cc=pbadari@gmail.com \
    --cc=zach.brown@oracle.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.