Skip to content

reiserfs: patch queue (v2)

[email protected] reiserfs-devel 1 version In review

Get series with b4
b4 am [email protected]

Versions newest first

  1. 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 filesSizeRepliesApplied
1/40reiserfs: fix up lockdep warningsfs/reiserfs/xattr.c+3 -20 replies
2/40reiserfs: dont use BUG when panickingfs/reiserfs/prints.c+1 -60 replies
3/40reiserfs: use is_reusable to catch corruptionfs/reiserfs/bitmap.c+13 -81 reply
4/40reiserfs: make bitmap use cached first zero bitfs/reiserfs/bitmap.c+4 -10 replies
5/40reiserfs: use more consistent printk formattingfs/reiserfs/prints.c+14 -140 replies
6/40reiserfs: make some warnings informationalfs/reiserfs/bitmap.c fs/reiserfs/super.c fs/reiserfs/xattr.c+12 -160 replies
7/40reiserfs: rework reiserfs_warningfs/reiserfs/bitmap.c fs/reiserfs/do_balan.c fs/reiserfs/file.c +14 more+444 -4220 replies
8/40reiserfs: rework reiserfs_panicfs/reiserfs/do_balan.c fs/reiserfs/file.c fs/reiserfs/fix_node.c +11 more+212 -1970 replies
9/40reiserfs: rearrange journal abortfs/reiserfs/journal.c fs/reiserfs/prints.c include/linux/reiserfs_fs.h+6 -210 replies
10/40reiserfs: introduce reiserfs_error()fs/reiserfs/prints.c include/linux/reiserfs_fs.h+26 -00 replies
11/40reiserfs: use reiserfs_error()fs/reiserfs/bitmap.c fs/reiserfs/file.c fs/reiserfs/inode.c +7 more+134 -1380 replies
12/40reiserfs: simplify xattr internal file lookups/opensfs/reiserfs/xattr.c+66 -1340 replies
13/40reiserfs: eliminate per-super xattr lockfs/reiserfs/inode.c fs/reiserfs/namei.c fs/reiserfs/super.c +5 more+40 -1370 replies
14/40reiserfs: make per-inode xattr locking more fine grainedfs/reiserfs/inode.c fs/reiserfs/xattr.c fs/reiserfs/xattr_acl.c +2 more+28 -720 replies
15/40reiserfs: remove i_has_xattr_dirfs/reiserfs/xattr.c+0 -70 replies
16/40reiserfs: remove link detection codefs/reiserfs/xattr.c+0 -130 replies
17/40reiserfs: use generic xattr handlersfs/reiserfs/super.c fs/reiserfs/xattr.c fs/reiserfs/xattr_acl.c +6 more+206 -4210 replies
18/40reiserfs: use better open options for internal filesfs/reiserfs/xattr.c+3 -30 replies
19/40reiserfs: add per-file data=ordered mode and use it for xattrsfs/reiserfs/file.c fs/reiserfs/inode.c fs/reiserfs/xattr.c +2 more+17 -30 replies
20/40reiserfs: journaled xattrsfs/reiserfs/inode.c fs/reiserfs/namei.c fs/reiserfs/xattr.c +4 more+168 -400 replies
21/40reiserfs: use generic readdir for operations across all xattrsfs/reiserfs/xattr.c+129 -3100 replies
22/40reiserfs: add atomic addition of selinux attributes during inode creationfs/reiserfs/inode.c fs/reiserfs/namei.c fs/reiserfs/xattr_security.c +2 more+135 -80 replies
23/40reiserfs: cleanup path functionsfs/reiserfs/fix_node.c fs/reiserfs/stree.c+43 -740 replies
24/40reiserfs: strip trailing whitespacefs/reiserfs/README fs/reiserfs/do_balan.c fs/reiserfs/file.c +16 more+189 -1890 replies
26/40reiserfs: rename p_s_bh to bhfs/reiserfs/fix_node.c fs/reiserfs/stree.c include/linux/reiserfs_fs.h+93 -930 replies
27/40reiserfs: rename p_s_inode to inodefs/reiserfs/file.c fs/reiserfs/inode.c fs/reiserfs/stree.c +2 more+87 -870 replies
28/40reiserfs: rename p_s_tb to tbfs/reiserfs/fix_node.c fs/reiserfs/stree.c include/linux/reiserfs_fs.h+234 -2340 replies
29/40reiserfs: rename p_._ variablesfs/reiserfs/file.c fs/reiserfs/fix_node.c fs/reiserfs/stree.c +2 more+338 -3380 replies
30/40reiserfs: rename _* variablesfs/reiserfs/file.c fs/reiserfs/fix_node.c fs/reiserfs/stree.c +1 more+427 -4300 replies
31/40reiserfs: factor out buffer_info initializationfs/reiserfs/do_balan.c+60 -1150 replies
32/40reiserfs: Turn tb->snum and tb->sbytes into an arrayfs/reiserfs/do_balan.c fs/reiserfs/fix_node.c fs/reiserfs/prints.c +1 more+59 -760 replies
33/40reiserfs: split left balancing part of balance_leaf() offfs/reiserfs/do_balan.c+313 -4200 replies
34/40reiserfs: split right balancing part of balance_leaf() offfs/reiserfs/do_balan.c+290 -3810 replies
35/40reiserfs: split balance_leaf new node handling outfs/reiserfs/do_balan.c+338 -3550 replies
36/40reiserfs: split out current node handling from balance_leaffs/reiserfs/do_balan.c+142 -1310 replies
37/40reiserfs: clean up bl_when_deletefs/reiserfs/do_balan.c+33 -380 replies
38/40reiserfs: clean up balancing modesfs/reiserfs/prints.c include/linux/reiserfs_fs.h+33 -180 replies
39/40reiserfs: split bl_when_deletefs/reiserfs/do_balan.c+77 -700 replies
40/40reiserfs: reorganize do_balan.c commentsfs/reiserfs/do_balan.c+108 -1250 replies

Reviews trailers per patch

No Reviewed-by, Acked-by or Tested-by trailers for v1 yet.

Read the v1 thread

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