reiserfs: patch queue (v2)
Get series with b4
b4 am [email protected]Versions newest first
- v1 Latest39 of 40 patches 3 repliesIn review
Changes from the cover letter
- Reworked journal abort code to provide cleaner syslog output
- Always abort journal on error, unless panicking
- Always use bl_ prefix in do_balan.c
- Fixed unchecked deref in reiserfs_free_block
39 of 40 patches are in this archive. The cover letter announces 40.
Patches in v1 No applies recorded yet
| # | Subject and files | Size | Replies | Applied |
|---|---|---|---|---|
| 1/40 | reiserfs: fix up lockdep warningsfs/ | +3 | 0 replies | |
| 2/40 | reiserfs: dont use BUG when panickingfs/ | +1 | 0 replies | |
| 3/40 | reiserfs: use is_reusable to catch corruptionfs/ | +13 | 1 reply | |
| 4/40 | reiserfs: make bitmap use cached first zero bitfs/ | +4 | 0 replies | |
| 5/40 | reiserfs: use more consistent printk formattingfs/ | +14 | 0 replies | |
| 6/40 | reiserfs: make some warnings informationalfs/ fs/ fs/ | +12 | 0 replies | |
| 7/40 | reiserfs: rework reiserfs_warningfs/ fs/ fs/ +14 more | +444 | 0 replies | |
| 8/40 | reiserfs: rework reiserfs_panicfs/ fs/ fs/ +11 more | +212 | 0 replies | |
| 9/40 | reiserfs: rearrange journal abortfs/ fs/ include/ | +6 | 0 replies | |
| 10/40 | reiserfs: introduce reiserfs_error()fs/ include/ | +26 | 0 replies | |
| 11/40 | reiserfs: use reiserfs_error()fs/ fs/ fs/ +7 more | +134 | 0 replies | |
| 12/40 | reiserfs: simplify xattr internal file lookups/opensfs/ | +66 | 0 replies | |
| 13/40 | reiserfs: eliminate per-super xattr lockfs/ fs/ fs/ +5 more | +40 | 0 replies | |
| 14/40 | reiserfs: make per-inode xattr locking more fine grainedfs/ fs/ fs/ +2 more | +28 | 0 replies | |
| 15/40 | reiserfs: remove i_has_xattr_dirfs/ | +0 | 0 replies | |
| 16/40 | reiserfs: remove link detection codefs/ | +0 | 0 replies | |
| 17/40 | reiserfs: use generic xattr handlersfs/ fs/ fs/ +6 more | +206 | 0 replies | |
| 18/40 | reiserfs: use better open options for internal filesfs/ | +3 | 0 replies | |
| 19/40 | reiserfs: add per-file data=ordered mode and use it for xattrsfs/ fs/ fs/ +2 more | +17 | 0 replies | |
| 20/40 | reiserfs: journaled xattrsfs/ fs/ fs/ +4 more | +168 | 0 replies | |
| 21/40 | reiserfs: use generic readdir for operations across all xattrsfs/ | +129 | 0 replies | |
| 22/40 | reiserfs: add atomic addition of selinux attributes during inode creationfs/ fs/ fs/ +2 more | +135 | 0 replies | |
| 23/40 | reiserfs: cleanup path functionsfs/ fs/ | +43 | 0 replies | |
| 24/40 | reiserfs: strip trailing whitespacefs/ fs/ fs/ +16 more | +189 | 0 replies | |
| 26/40 | reiserfs: rename p_s_bh to bhfs/ fs/ include/ | +93 | 0 replies | |
| 27/40 | reiserfs: rename p_s_inode to inodefs/ fs/ fs/ +2 more | +87 | 0 replies | |
| 28/40 | reiserfs: rename p_s_tb to tbfs/ fs/ include/ | +234 | 0 replies | |
| 29/40 | reiserfs: rename p_._ variablesfs/ fs/ fs/ +2 more | +338 | 0 replies | |
| 30/40 | reiserfs: rename _* variablesfs/ fs/ fs/ +1 more | +427 | 0 replies | |
| 31/40 | reiserfs: factor out buffer_info initializationfs/ | +60 | 0 replies | |
| 32/40 | reiserfs: Turn tb->snum and tb->sbytes into an arrayfs/ fs/ fs/ +1 more | +59 | 0 replies | |
| 33/40 | reiserfs: split left balancing part of balance_leaf() offfs/ | +313 | 0 replies | |
| 34/40 | reiserfs: split right balancing part of balance_leaf() offfs/ | +290 | 0 replies | |
| 35/40 | reiserfs: split balance_leaf new node handling outfs/ | +338 | 0 replies | |
| 36/40 | reiserfs: split out current node handling from balance_leaffs/ | +142 | 0 replies | |
| 37/40 | reiserfs: clean up bl_when_deletefs/ | +33 | 0 replies | |
| 38/40 | reiserfs: clean up balancing modesfs/ include/ | +33 | 0 replies | |
| 39/40 | reiserfs: split bl_when_deletefs/ | +77 | 0 replies | |
| 40/40 | reiserfs: reorganize do_balan.c commentsfs/ | +108 | 0 replies |
Reviews trailers per patch
No Reviewed-by, Acked-by or Tested-by trailers for v1 yet.
Cover letter [patch 00/40] reiserfs: patch queue (v2)
Hi all -
The following are a series of patches that have been kicking around for a while. I thought I'd just put them out there to gauge interest and to get some testing before submitting to -mm.
I've re-ordered a few of the patches. This is actually four distinct, unrelated, though context-dependent, patch series. I've moved the trivial changes to the front of the list to better reflect that.
There are a few groups:
* Little changes (01-04)
- Fixes lockdep warnings in xattr code.
- Changes reiserfs_panic() to panic() instead of BUG.
- Includes bitmap is_reusable unconditionally and uses to to catch corruption. This is light-weight now that the stupid loop is gone.
- Make bitmap used cached first zero bit.
* Error handling (05-11)
- Reformat error/warning/panic/info/etc messages to be consistent with one another.
- Makes some warnings informational. Rather than printing the reiserfs_warning() full preamble to declare that a mount option was invalid, we use reiserfs_info instead.
- Rework reiserfs_warning to take a unique ID, print the function name, print the device name.
- Rework reiserfs_panic similarly.
- Add reiserfs_error() to handle metadata errors.
- Use reiserfs_error().
* Extended attributes (12-22)
- Simplify internal file lookups/opens.
- Eliminate per-super xattr lock.
- Make per-inode xattr lock more fine-grained.
- Remove i_has_xattr_dir flag.
- Remove shared xattr detection (unused).
- Use generic xattr handler infrastructure.
- Use O_NOATIME for internal file opens.
- Add per-file data=ordered mode and use it for xattrs.
- Journal xattrs.
- Kill xattr_readdir in favor of a for_each_xattr.
- Add atomic addition of selinux attributes during inode creation.
* Cleanups, code readability (23-30)
- Cleanup path functions to reduce redundancy.
- Strip trailing whitespace from all reiserfs code.
- Rename p_s_sb to sb.
- Rename p_s_bh to bh.
- Rename p_s_inode to inode.
- Rename p_s_tb to tb.
- Rename misc p_._ variables.
- Rename [cn]_* variables.
* Clean up do_balan.c (31-40)
- Factor out buffer_info initializations.
- Turn tb->s[12]num and tb[12]bytes into an array.
- Split balance_leaf from the 2500 line beast to more manageable chunks.
- Clean up balance_leaf_when_delete.
- Clean up comments.
I look forward to comments. The journaled xattr changes in particular need testing as deadlocks may still lurk in there.
Changes:
- Reworked journal abort code to provide cleaner syslog output
- Always abort journal on error, unless panicking
- Always use bl_ prefix in do_balan.c
- Fixed unchecked deref in reiserfs_free_block
I've also posted the patch set here:
ftp://ftp.suse.com/pub/people/jeffm/reiserfs/kernel-v2.6/2.6.22-rc4-git3/
-Jeff
Signature · 2 lines
Jeff Mahoney SUSE Labs