Re: reiserfsprogs patches

[Hans Reiser <reiser@namesys.com>]

Thanks Jeff, Vitaly will review.

Hans

Jeff Mahoney wrote:

>
> Hey all -
>
> The SuSE copy of reiserfsprogs has a number of patches, mostly backports
> of fixes. The attached patches are the ones that are fixes, as of
> 3.6.18, we still apply to our tree and that I feel should be considered
> for inclusion in the official reiserfsprogs release.
>
> Here's the rundown:
> 03-cleanly-unmounted
>     - Just a cosmetic fix to change "cleanly umounted" to "clean"
>       in a few warnings
>
> 04-quota_sd_blocks-2
>     - Fixes fsck incorrectly warning about block counts for
>       symlinks when quotas are enabled
>
> 07-auto_quiet.diff
>     - Makes -a mode run quietly
>
> 10-blkgetsiz64.patch
>     - The BLKGETSIZE64 ioctl was, in earlier kernels, incorrectly
>       prototyped with sizeof (__u64), which expanded to
>       sizeof(sizeof (__u64)) - making it a size_t
>
> Please consider applying.
>
> -Jeff
>
> --
> Jeff Mahoney
> SuSE Labs


 >-------------------------

 >Index: reiserfsprogs-3.6.13/fsck/main.c
 >===================================================================
 >--- reiserfsprogs-3.6.13.orig/fsck/main.c    2004-02-17 
07:06:23.000000000 -0500
 >+++ reiserfsprogs-3.6.13/fsck/main.c    2004-03-08 10:40:14.382396213 
-0500
 >@@ -883,7 +883,7 @@ static void clean_attributes (reiserfs_f
    time (&t);

    if (get_sb_umount_state (fs->fs_ondisk_sb) != FS_CLEANLY_UMOUNTED) {
 >-        fsck_progress ("Filesystem does not look cleanly umounted\n"
 >+        fsck_progress ("Filesystem is not clean\n"
        "Check consistency of the partition first.\n");
        exit(EXIT_USER);
    }
 >Index: reiserfsprogs-3.6.13/reiserfscore/prints.c
 >===================================================================
 >--- reiserfsprogs-3.6.13.orig/reiserfscore/prints.c    2004-02-17 
07:06:23.000000000 -0500
 >+++ reiserfsprogs-3.6.13/reiserfscore/prints.c    2004-03-08 
10:41:23.648594772 -0500
 >@@ -646,7 +646,7 @@ int print_super_block (FILE * fp, reiser
              "bitmaps, data, reserved] blocks): %u\n", 
get_sb_free_blocks (sb));
    reiserfs_warning (fp, "Root block: %u\n", get_sb_root_block (sb));
    }
 >-    reiserfs_warning (fp, "Filesystem marked as %scleanly umounted\n",
 >+    reiserfs_warning (fp, "Filesystem is %sclean\n",
              (get_sb_umount_state (sb) == FS_CLEANLY_UMOUNTED) ? "" : 
"NOT ");

    if (short_print)


 >-------------------------

 >diff -ur reiserfsprogs-3.6.4/fsck/semantic_rebuild.c 
reiserfsprogs-3.6.4.suse/fsck/semantic_rebuild.c
 >--- reiserfsprogs-3.6.4/fsck/semantic_rebuild.c    2002-09-13 
05:41:41.000000000 -0400
 >+++ reiserfsprogs-3.6.4.suse/fsck/semantic_rebuild.c    2003-02-11 
10:36:49.000000000 -0500
 >@@ -169,7 +169,8 @@
    int ret = 0;

    if (S_ISREG (mode) || S_ISLNK (mode) || S_ISDIR (mode)) {
 >-    if (*blocks != sd_blocks) {
 >+    if ((!S_ISLNK(mode) && *blocks != sd_blocks) ||
 >+        (S_ISLNK(mode) && *blocks != sd_blocks && (ROUND_UP(*blocks) 
!= sd_blocks))) {
        fsck_log ("vpf-10680: The %s %K has the wrong block count in the 
StatData "
        "(%u)%s(%u)\n", S_ISDIR (mode) ? "directory" : "file",  key, 
sd_blocks,
        fsck_mode(fs) == FSCK_CHECK ? ", should be " : " - corrected to 
", *blocks);


 >-------------------------

 >--- progs/fsck/main.c.1    2003-09-01 12:44:41.000000000 -0400
 >+++ progs/fsck/main.c    2003-09-01 12:44:46.000000000 -0400
 >@@ -230,6 +230,7 @@

    case 'a':
    case 'p':
 >+        data->options |= OPT_QUIET;
        mode = FSCK_AUTO;
        break;
    


 >-------------------------

 >--- reiserfsprogs-3.6.13/lib/misc.c.blkgetsiz64    2004-05-18 
16:50:47.000000000 +0200
 >+++ reiserfsprogs-3.6.13/lib/misc.c    2004-05-18 16:53:19.000000000 +0200
 >@@ -463,7 +463,13 @@
}

#if defined(__linux__) && defined(_IOR) && !defined(BLKGETSIZE64)
 >-#   define BLKGETSIZE64 _IOR(0x12, 114, __u64)
 >+/* Note! Despite this call being called with *64, it must be encoded to
 >+ * return only sizeof(size_t), since in earlier kernel versions it was
 >+ * declared _IOR(0x12, 114, sizeof(u64)), making it use 
sizeof(sizeof(u64)).
 >+ *
 >+ * However, the call itself does always return 64bit!
 >+ */
 >+#   define BLKGETSIZE64 _IOR(0x12, 114, size_t)
#endif

/* To not have problem with last sectors on the block device when switching