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
next 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.