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] ocfs2/xattr: transaction merge patch set. v3
Date: Wed, 12 Nov 2008 16:20:51 +0800	[thread overview]
Message-ID: <491A91E3.7010408@oracle.com> (raw)

Hi all,
	In current ocfs2/xattr, the whole xattr set is divided into many steps 
are many transaction are used, this make the xattr set process isn't 
like a real transaction, so this patch try to merge all the transaction 
into one. Another benefit is that acl can use it easily now.

Modification from v2 to v3:
patch 2: set c_global_allocator to NULL after we free the cluster.

patch 4:
   1) initialize ocfs2_xattr_set_ctxt in a more efficient way.
   2) add one inline function ocfs2_dealloc_has_cluster so that we only 
flush truncate log when dealloc has clusters to free(Mark, you are free 
to change the name if it looks ugly. ;) ).

Mark,
	Since Joel and you have already reviewed this patch set, I'd like you 
to push these patches to merge_window now. So I can ask tristan to pull 
from your branch and do some test on this ASAP(actually I have already 
done some, but I'd like to hear the voice of the really testers :)). 
Another side effect is that Joel may need this for his bucket work on 
the new _set() path. And what's more, tiger's acl can base on your 
merge_window directly now.

Regards,
Tao

             reply	other threads:[~2008-11-12  8:20 UTC|newest]

Thread overview: 7+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2008-11-12  8:20 Tao Ma [this message]
2008-11-12  0:26 ` [Ocfs2-devel] [PATCH 1/5] ocfs2/xattr: Only extend xattr bucket in need Tao Ma
2008-11-12  0:26 ` [Ocfs2-devel] [PATCH 2/5] ocfs2: Add clusters free in dealloc_ctxt Tao Ma
2008-11-12  0:26 ` [Ocfs2-devel] [PATCH 3/5] ocfs2/xattr: Move clusters free into dealloc Tao Ma
2008-11-12  0:27 ` [Ocfs2-devel] [PATCH 4/5] ocfs2/xattr: Reserve meta/data at the beginning of ocfs2_xattr_set Tao Ma
2008-11-12  0:27 ` [Ocfs2-devel] [PATCH 5/5] ocfs2/xattr: Merge xattr set transaction Tao Ma
2008-11-12 21:23 ` [Ocfs2-devel] [PATCH 0/5] ocfs2/xattr: transaction merge patch set. v3 Mark Fasheh

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=491A91E3.7010408@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.