All of lore.kernel.org
 help / color / mirror / Atom feed
From: Tao Ma <tao.ma@oracle.com>
To: ocfs2-devel@oss.oracle.com
Subject: [Ocfs2-devel] [PATCH 0/5 v2] Add readahead support in CoW for reflinked files.
Date: Thu, 12 Aug 2010 11:18:28 +0800	[thread overview]
Message-ID: <4C636804.209@oracle.com> (raw)
In-Reply-To: <20100812022531.GB7195@mail.oracle.com>



On 08/12/2010 10:25 AM, Joel Becker wrote:
> On Thu, Jul 15, 2010 at 11:04:37AM +0800, Tao Ma wrote:
>> On 07/15/2010 10:31 AM, Joel Becker wrote:
>>> On Wed, Jul 14, 2010 at 03:52:26PM +0800, Tao Ma wrote:
>>>> 	When we try to CoW some clusters for a reflinked file, we have to
>>>> read the contents first, allocate some clusters and then map these
>>>> pages to the new clusters. Currently, we use block_read_full_page to
>>>> read it, but it is a little bit slower.
>>>> 	So this patch set try to add readahead support for CoW. Before we
>>>> start, we call readahead first so that the pages can be read at the
>>>> very first. And during CoW, when we find a readahead page, we know
>>>> that we need to move the readahead window, so a new asyncreadahead
>>>> is called.
>>>
>>> 	I think this looks good.  Can you create a branch in your git
>>> tree for it?  I'll pull when I build merge-window.
>> done. please pull from
>> git://oss.oracle.com/git/tma/linux-2.6.git cow_readahead.
>
> 	I tried pulling this, and it conflicts with the zero-tail fix.
> Can you rebase it for me?  I want to make sure we get the readahead
> right.
done. Please pull the same branch. It should work now.

btw, I still have the patch "[PATCH v5] ocfs2: Let ocfs2_setattr use new 
truncate sequence." waiting for being included. So what's your option?

Regards,
Tao

      reply	other threads:[~2010-08-12  3:18 UTC|newest]

Thread overview: 10+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2010-07-14  7:52 [Ocfs2-devel] [PATCH 0/5 v2] Add readahead support in CoW for reflinked files Tao Ma
2010-07-14  7:53 ` [Ocfs2-devel] [PATCH 1/5] ocfs2: pass struct file* to ocfs2_write_begin_nolock Tao Ma
2010-07-14  7:53 ` [Ocfs2-devel] [PATCH 2/5] ocfs2: pass struct file* to ocfs2_prepare_inode_for_write Tao Ma
2010-07-14  7:53 ` [Ocfs2-devel] [PATCH 3/5] ocfs2: Add struct file to ocfs2_refcount_cow Tao Ma
2010-07-14  7:53 ` [Ocfs2-devel] [PATCH 4/5] ocfs2: Add readahead support for CoW Tao Ma
2010-07-14  7:53 ` [Ocfs2-devel] [PATCH 5/5] ocfs2: Add readhead during CoW Tao Ma
2010-07-15  2:31 ` [Ocfs2-devel] [PATCH 0/5 v2] Add readahead support in CoW for reflinked files Joel Becker
2010-07-15  3:04   ` Tao Ma
2010-08-12  2:25     ` Joel Becker
2010-08-12  3:18       ` Tao Ma [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=4C636804.209@oracle.com \
    --to=tao.ma@oracle.com \
    --cc=ocfs2-devel@oss.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.