ocfs2/xattr: transaction merge patch set. v3
Get series with b4
b4 am [email protected]Versions newest first
- v1 Latest5 patches 0 repliesNew
Patches in v1 No applies recorded yet
| # | Subject and files | Size | Replies | Applied |
|---|---|---|---|---|
| 1/5 | ocfs2/xattr: Only extend xattr bucket in need.fs/ | +3 | 0 replies | |
| 2/5 | ocfs2: Add clusters free in dealloc_ctxt.fs/ fs/ | +103 | 0 replies | |
| 3/5 | ocfs2/xattr: Move clusters free into dealloc.fs/ | +1 | 0 replies | |
| 4/5 | ocfs2/xattr: Reserve meta/data at the beginning of ocfs2_xattr_set.fs/ fs/ | +361 | 0 replies | |
| 5/5 | ocfs2/xattr: Merge xattr set transaction.fs/ | +325 | 0 replies |
Reviews trailers per patch
No Reviewed-by, Acked-by or Tested-by trailers for v1 yet.
Cover letter [Ocfs2-devel] [PATCH 0/5] ocfs2/xattr: transaction merge patch set. v3
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