On 9 May, Jeff Mahoney wrote: > Al Viro wrote: >>> On 9 May, Stefan Richter wrote: >>>> May 9 14:27:22 stein REISERFS warning (device sda1): jdm-20004 >>>> reiserfs_delete_xattrs: Couldn't delete all xattrs (-13) >>>> May 9 14:27:22 stein ... >> Could you try to bisect it? I bisected it with a series of git-format-patch output on top of 2.6.29.3. The last good one is "reiserfs: simplify xattr internal file lookups/opens", the first bad one is "reiserfs: eliminate per-super xattr lock". (Doh, what a surprise; grep'ing for the message in the diffs would have pointed to this one too... Well, the bisection homed in quickly though.) Of course that's only the patch which caused the symptoms to appear, not necessarily the one which introduced the bug. My config contains: CONFIG_REISERFS_FS=y # CONFIG_REISERFS_CHECK is not set # CONFIG_REISERFS_PROC_INFO is not set # CONFIG_REISERFS_FS_XATTR is not set (As mentioned, the filesystem was created and initially populated with a kernel which was differently configured; I don't have an immediate idea where to obtain the config.) The tested series: 0001-reiserfs-add-support-for-mount-count-incrementing.patch 0002-reiserfs-audit-transaction-ids-to-always-be-unsigne.patch 0003-reiserfs-use-buffer_info-for-leaf_paste_entries.patch 0004-reiserfs-use-more-consistent-printk-formatting.patch 0005-reiserfs-make-some-warnings-informational.patch 0006-reiserfs-rework-reiserfs_warning.patch 0007-reiserfs-prepare_error_buf-wrongly-consumes-va_arg.patch 0008-reiserfs-eliminate-reiserfs_warning-from-uniqueness.patch 0009-reiserfs-add-locking-around-error-buffer.patch 0010-reiserfs-rework-reiserfs_panic.patch 0011-reiserfs-rearrange-journal-abort.patch 0012-reiserfs-introduce-reiserfs_error.patch 0013-reiserfs-use-reiserfs_error.patch 0014-reiserfs-small-variable-cleanup.patch 0015-reiserfs-xattr-reiserfs_get_page-takes-offset-inste.patch 0016-reiserfs-remove-link-detection-code.patch 0017-reiserfs-remove-IS_PRIVATE-helpers.patch 0018-reiserfs-Clean-up-xattrs-when-REISERFS_FS_XATTR-is.patch 0019-reiserfs-simplify-xattr-internal-file-lookups-opens.patch 0020-reiserfs-eliminate-per-super-xattr-lock.patch 0021-reiserfs-make-per-inode-xattr-locking-more-fine-gra.patch # 0021 doesn't build, I skipped it. 0022-reiserfs-remove-i_has_xattr_dir.patch 0023-reiserfs-use-generic-xattr-handlers.patch 0024-reiserfs-journaled-xattrs.patch # 0024 and later build only with A. Bergalov's fix folded in. 0025-reiserfs-use-generic-readdir-for-operations-across.patch 0026-reiserfs-add-atomic-addition-of-selinux-attributes.patch 0027-reiserfs-factor-out-buffer_info-initialization.patch 0028-reiserfs-cleanup-path-functions.patch 0029-reiserfs-strip-trailing-whitespace.patch 0030-reiserfs-rename-p_s_sb-to-sb.patch 0031-reiserfs-rename-p_s_bh-to-bh.patch 0032-reiserfs-rename-p_s_inode-to-inode.patch 0033-reiserfs-rename-p_s_tb-to-tb.patch 0034-reiserfs-rename-p_._-variables.patch 0035-reiserfs-rename-cn-_-variables.patch > I'm looking into this right now. I have an idea, but I haven't been able > to reproduce it yet to test. Thanks, I can test something for you. -- Stefan Richter -=====-==--= -=-= -=--= http://arcgraph.de/sr/ -- To unsubscribe from this list: send the line "unsubscribe reiserfs-devel" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
