Re: [PATCH] btrfs-progs: fix one bracket issue in mkfs.btrfs manpage

2013-03-21 Thread Eric Sandeen
n mkfs.btrfs", you will see the difference. > > Signed-off-by: Zhi Yong Wu Whoops, yes. Thanks for spotting that. Reviewed-by: Eric Sandeen > --- > man/mkfs.btrfs.8.in | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/man/mkfs.btrfs.8.in b/man/mk

Re: Apparent serious progressive ext4 data corruption bug in 3.6.3 (and other stable branches?)

2012-10-23 Thread Eric Sandeen
On 10/23/12 5:19 PM, Theodore Ts'o wrote: > On Tue, Oct 23, 2012 at 09:57:08PM +0100, Nix wrote: >> >> It is now quite clear that this is a bug introduced by one or more of >> the post-3.6.1 ext4 patches (which have all been backported at least to >> 3.5, so the problem is probably there too). >> >

Re: Apparent serious progressive ext4 data corruption bug in 3.6.3 (and other stable branches?)

2012-10-23 Thread Eric Sandeen
On 10/23/12 3:57 PM, Nix wrote: > (I'd provide more sample errors, but this bug has been eating > newly-written logs in /var all day, so not much has survived.) > > I rebooted into 3.6.1 rescue mode and fscked everything: lots of > orphans, block group corruption and cross-linked files. The pro

Re: Apparent serious progressive ext4 data corruption bug in 3.6.3 (and other stable branches?)

2012-10-23 Thread Eric Sandeen
On 10/23/12 11:15 PM, Nix wrote: > On 24 Oct 2012, Eric Sandeen uttered the following: > >> On 10/23/12 3:57 PM, Nix wrote: >>> The only unusual thing about the filesystems on this machine are that >>> they have hardware RAID-5 (using the Areca driver), so I&

Re: Apparent serious progressive ext4 data corruption bug in 3.6.3 (and other stable branches?)

2012-10-24 Thread Eric Sandeen
On 10/24/2012 12:23 AM, Theodore Ts'o wrote: > On Tue, Oct 23, 2012 at 11:27:09PM -0500, Eric Sandeen wrote: >> >> Ok, fair enough. If the BBU is working, nobarrier is ok; I don't trust >> journal_async_commit, but that doesn't mean this isn't a regre

Re: Apparent serious progressive ext4 data corruption bug in 3.6.3 (and other stable branches?)

2012-10-24 Thread Eric Sandeen
On 10/24/2012 02:49 PM, Nix wrote: > On 24 Oct 2012, Theodore Ts'o spake thusly: >> Toralf, Nix, if you could try applying this patch (at the end of this >> message), and let me know how and when the WARN_ON triggers, and if it >> does, please send the empty_bug_workaround plus the WARN_ON(1) repor

Re: EXT4-fs error w/ external USB drive

2012-10-25 Thread Eric Sandeen
On 10/25/12 1:20 PM, Theodore Ts'o wrote: > On Thu, Oct 25, 2012 at 06:39:30PM +0200, Toralf Förster wrote: >> After a lot of file operations (Gentoo emerging, kernel build, git >> pulls, ...) I s2disk the system (that with the external USB drive) >> yesterday, wake it up today, rebooted it - >> an

Re: xfs [_fsr] probs in 2.6.24.0

2008-02-12 Thread Eric Sandeen
Linda Walsh wrote: > > David Chinner wrote: >> Filesystem bugs rarely hang systems hard like that - more likely is >> a hardware or driver problem. And neither of the lockdep reports >> below are likely to be responsible for a system wide, no-response >> hang. > --- > "Ish", the 32-bitter, h

Re: xfs [_fsr] probs in 2.6.24.0

2008-02-12 Thread Eric Sandeen
Linda Walsh wrote: > > Eric Sandeen wrote: >> Linda Walsh wrote: >>> David Chinner wrote: >>>> Filesystem bugs rarely hang systems hard like that - more likely is >>>> a hardware or driver problem. And neither of the lockdep reports >>>>

Re: [RFC] ext3 freeze feature

2008-02-15 Thread Eric Sandeen
Takashi Sato wrote: > Hi, > > Christoph Hellwig wrote: >> On Fri, Feb 08, 2008 at 08:26:57AM -0500, Andreas Dilger wrote: >>> You may as well make the common ioctl the same as the XFS version, >>> both by number and parameters, so that applications which already >>> understand the XFS ioctl will w

Re: [PATCH V2] perf: Fix parallel build

2012-09-20 Thread Eric Sandeen
On 9/20/12 9:07 PM, Namhyung Kim wrote: > Hi again, > > On Thu, 20 Sep 2012 20:12:50 -0500, Eric Sandeen wrote: >> -$(OUTPUT)util/parse-events-flex.c: util/parse-events.l >> +$(OUTPUT)util/parse-events-flex.c: util/parse-events.l >> util/parse-events-bison.c > &

[PATCH V3] perf: Fix parallel build

2012-09-20 Thread Eric Sandeen
front of the bison calls in tools/perf/Makefile and running make -j4 on a smaller box i.e.: sleep 10; $(QUIET_BISON)$(BISON) -v util/pmu.y -d -o $(OUTPUT)util/pmu-bison.c Adding the following dependencies fixes it for me. Signed-off-by: Eric Sandeen --- V2: Fix other bison dependency

Re: Apparent serious progressive ext4 data corruption bug in 3.6.3 (and other stable branches?)

2012-10-26 Thread Eric Sandeen
On 10/24/12 3:17 PM, Jan Kara wrote: > On Tue 23-10-12 19:57:09, Eric Sandeen wrote: >> On 10/23/12 5:19 PM, Theodore Ts'o wrote: >>> On Tue, Oct 23, 2012 at 09:57:08PM +0100, Nix wrote: >>>> >>>> It is now quite clear that this is a bug introd

Re: Apparent serious progressive ext4 data corruption bug in 3.6.3 (and other stable branches?)

2012-10-26 Thread Eric Sandeen
On 10/23/12 3:57 PM, Nix wrote: > [Bruce, Trond, I fear it may be hard for me to continue chasing this NFS > lockd crash as long as ext4 on 3.6.3 is hosing my filesystems like > this. Apologies.] > The only unusual thing about the filesystems on this machine are that > they have hardware RAID-

Re: Apparent serious progressive ext4 data corruption bug in 3.6.3 (and other stable branches?)

2012-10-27 Thread Eric Sandeen
On 10/27/12 7:45 AM, Nix wrote: > [nfs people purged from Cc] > > On 27 Oct 2012, Theodore Ts'o verbalised: > >> Huh? It's not turned on by default. If you mount with no mount >> options, journal checksums are *not* turned on. > > ?! it's turned on for me, and though I use weird mount options

Re: Apparent serious progressive ext4 data corruption bug in 3.6.3 (and other stable branches?)

2012-10-27 Thread Eric Sandeen
On 10/27/12 1:47 PM, Nix wrote: > On 27 Oct 2012, Theodore Ts'o said: > >> On Sat, Oct 27, 2012 at 01:45:25PM +0100, Nix wrote: >>> Ah! it's turned on by journal_async_commit. OK, that alone argues >>> against use of journal_async_commit, tested or not, and I'd not have >>> turned it on if I'd not

Re: Apparent serious progressive ext4 data corruption bug in 3.6.3 (and other stable branches?)

2012-10-27 Thread Eric Sandeen
On 10/27/12 4:19 PM, Eric Sandeen wrote: > On 10/27/12 1:47 PM, Nix wrote: >> On 27 Oct 2012, Theodore Ts'o said: >> >>> On Sat, Oct 27, 2012 at 01:45:25PM +0100, Nix wrote: >>>> Ah! it's turned on by journal_async_commit. OK, that alone argues >&

[PATCH] ext4: fix unjournaled inode bitmap modification

2012-10-27 Thread Eric Sandeen
fix it survives many iterations. Signed-off-by: Eric Sandeen cc: Nix --- A little more going on here to try to properly handle error cases & moving to the next group; despite ext4_handle_release_buffer being a no-op, I've tried to sprinkle it in at the right places. Double checking on r

Re: Apparent serious progressive ext4 data corruption bug in 3.6.3 (and other stable branches?)

2012-10-28 Thread Eric Sandeen
On 10/28/12 8:00 PM, Theodore Ts'o wrote: > On Sat, Oct 27, 2012 at 05:42:07PM -0500, Eric Sandeen wrote: >> >> It looks like the inode_bitmap_bh is being modified outside a transaction: >> >> ret2 = ext4_test_and_set_bit(ino, inode_bitm

Re: Apparent serious progressive ext4 data corruption bug in 3.6.3 (and other stable branches?)

2012-10-28 Thread Eric Sandeen
On 10/28/12 9:34 PM, Theodore Ts'o wrote: > On Sun, Oct 28, 2012 at 09:24:19PM -0500, Eric Sandeen wrote: >> Yeah, I knew it wasn't ;) I did resend >> [PATCH] ext4: fix unjournaled inode bitmap modification >> which is a bit more involved. > > Yeah, sorr

Re: [PATCH -v3] ext4: fix unjournaled inode bitmap modification

2012-10-28 Thread Eric Sandeen
On 10/28/12 9:30 PM, Theodore Ts'o wrote: > On Sat, Oct 27, 2012 at 11:23:57PM -0500, Eric Sandeen wrote: >> A little more going on here to try to properly handle error >> cases & moving to the next group; despite >> ext4_handle_release_buffer being a no-op, I'v

Re: ext4: fix memory leak in xattr code.

2013-09-20 Thread Eric Sandeen
e leak either way. > > Spotted with Coverity. > > Signed-off-by: Dave Jones Looks right to me. Reviewed-by: Eric Sandeen Thanks! -Eric > diff --git a/fs/ext4/xattr.c b/fs/ext4/xattr.c > index c081e34..f3a6220 100644 > --- a/fs/ext4/xattr.c > +++ b/fs/e

Re: [PATCH v3 0/2] ext4: increase mbcache scalability

2013-09-10 Thread Eric Sandeen
On 9/10/13 4:02 PM, Theodore Ts'o wrote: > On Tue, Sep 10, 2013 at 02:47:33PM -0600, Andreas Dilger wrote: >> I agree that SELinux is enabled on enterprise distributions by default, >> but I'm also interested to know how much overhead this imposes. I would >> expect that writing large external xat

Re: [PATCH v3 0/2] ext4: increase mbcache scalability

2013-09-11 Thread Eric Sandeen
On 9/11/13 6:30 AM, Theodore Ts'o wrote: > On Tue, Sep 10, 2013 at 10:13:16PM -0500, Eric Sandeen wrote: >> >> Above doesn't tell us the prevalence of various contexts on the actual >> system, >> but they are all under 100 bytes in any case. > > OK, s

Re: [PATCH v3 0/2] ext4: increase mbcache scalability

2013-09-11 Thread Eric Sandeen
On 9/11/13 11:49 AM, Eric Sandeen wrote: > On 9/11/13 6:30 AM, Theodore Ts'o wrote: >> On Tue, Sep 10, 2013 at 10:13:16PM -0500, Eric Sandeen wrote: >>> >>> Above doesn't tell us the prevalence of various contexts on the actual >>> system, &g

Re: [PATCH v3 0/2] ext4: increase mbcache scalability

2013-09-11 Thread Eric Sandeen
On 9/11/13 3:32 PM, David Lang wrote: > On Wed, 11 Sep 2013, Eric Sandeen wrote: > >>> The reason why I'm pushing here is that mbcache shouldn't be showing >>> up in the profiles at all if there is no external xattr block. And so >>> if newer versions

Re: [PATCH v3 0/2] ext4: increase mbcache scalability

2013-09-11 Thread Eric Sandeen
On 9/11/13 3:36 PM, Thavatchai Makphaibulchoke wrote: > I seem to be seeing the same thing as Eric is seeing. ... > For both filesystems, the security xattr are about 32.17 and 34.87 bytes > respectively. ... Can you triple-check the inode size on your fs, for good measure? dumpe2fs -h /dev/wh

Re: [PATCH, 3.7-rc7, RESEND] fs: revert commit bbdd6808 to fallocate UAPI

2012-12-07 Thread Eric Sandeen
On 12/7/12 3:14 PM, Theodore Ts'o wrote: > On Fri, Dec 07, 2012 at 02:30:19PM -0500, Steven Rostedt wrote: >> How is this similar? By adding this bit, we removed incentive from a >> group of developers that have the means to fix the real issue at hand >> (the performance problem with ext4). Thus, i

Re: [PATCH, 3.7-rc7, RESEND] fs: revert commit bbdd6808 to fallocate UAPI

2012-12-07 Thread Eric Sandeen
On 12/7/12 3:57 PM, Chris Mason wrote: > On Fri, Dec 07, 2012 at 02:49:04PM -0700, Ric Wheeler wrote: >> On 12/07/2012 04:43 PM, Chris Mason wrote: >>> On Fri, Dec 07, 2012 at 02:27:43PM -0700, Theodore Ts'o wrote: On Fri, Dec 07, 2012 at 04:09:32PM -0500, Chris Mason wrote: > Persistent t

Re: [PATCH, 3.7-rc7, RESEND] fs: revert commit bbdd6808 to fallocate UAPI

2012-12-07 Thread Eric Sandeen
On 12/7/12 3:57 PM, Chris Mason wrote: > On Fri, Dec 07, 2012 at 02:49:04PM -0700, Ric Wheeler wrote: >> On 12/07/2012 04:43 PM, Chris Mason wrote: >>> On Fri, Dec 07, 2012 at 02:27:43PM -0700, Theodore Ts'o wrote: On Fri, Dec 07, 2012 at 04:09:32PM -0500, Chris Mason wrote: > Persistent t

Re: [PATCH] xfs: return -EINVAL instead of -EUCLEAN when mounting non-xfs

2012-12-29 Thread Eric Sandeen
On 12/29/12 5:16 PM, Sergei Trofimovich wrote: > It fixes boot panic when trying to boot from btrfs filesystem. > kernel tries to mount as xfs and gets fatal -EUCLEAN: > > [0.17] VFS: Cannot open root device "ubda" or unknown-block(98,0): > error -117 > [0.17] Please append a corr

Re: 3.9.4 Oops running xfstests (WAS Re: 3.9.3: Oops running xfstests)

2013-05-30 Thread Eric Sandeen
On 5/30/13 10:03 PM, CAI Qian wrote: > OK, so the minimal workload to trigger this I found so far was to > run trinity, ltp and then xfstests. I have been able to easily > reproduced on 3 servers so far, and I'll post full logs here for > LKML and linux-mm as this may unrelated to XFS only. As far

Re: [PATCH 0/3] ext4: introduce two new ioctls

2013-06-23 Thread Eric Sandeen
On Jun 23, 2013, at 12:01 PM, Andreas Dilger wrote: > On 2013-06-23, at 0:07, Namjae Jeon wrote: > >> From: Namjae Jeon >> >> This patch series introduces 2 new ioctls for ext4. >> >> Truncate_block_range ioctl truncates blocks from source file. > > How is this different from fallocate(FALL

Re: [PATCH 01/12] Btrfs: Remove superfluous casts from u64 to unsigned long long

2013-08-20 Thread Eric Sandeen
On 8/20/13 6:20 AM, Geert Uytterhoeven wrote: > u64 is "unsigned long long" on all architectures now, so there's no need to > cast it when formatting it using the "ll" length modifier. > > Signed-off-by: Geert Uytterhoeven > --- > fs/btrfs/backref.c | 24 ++-- > fs/btrfs/check-integri

Re: [PATCH 01/12] Btrfs: Remove superfluous casts from u64 to unsigned long long

2013-08-20 Thread Eric Sandeen
On 8/20/13 9:16 AM, Eric Sandeen wrote: > On 8/20/13 6:20 AM, Geert Uytterhoeven wrote: >> u64 is "unsigned long long" on all architectures now, so there's no need to >> cast it when formatting it using the "ll" length modifier. >> >> Si

Re: [PATCH 0/7] uselex.rb as a tiny tool to find dead code

2013-08-07 Thread Eric Sandeen
On 8/7/13 4:43 PM, Sergei Trofimovich wrote: ... > Meet uselex.rb: one-file script to parse 'nm' output: > > https://github.com/trofi/uselex/blob/master/uselex.rb Nice to meet you! I think I've met your close relative, ftp://ftp.samba.org/pub/unpacked/junkcode/findstatic.pl :) #!/usr/bin

Re: [PATCH 1/1] ext4: Fix Opts: (null)

2013-07-23 Thread Eric Sandeen
On 7/22/13 5:24 PM, Jóhann B. Guðmundsson wrote: > null null null no more Opts: (null) but something that actually makes sense to > human beings... It's not clear to me how this changes the (null) output... Have you tested it? What's the difference in output? -Eric > Signed-off-by: Jóhann B. Gu

Re: linux-next: Tree for Aug 13 (xfs)

2013-08-13 Thread Eric Sandeen
On 8/13/13 11:59 AM, Randy Dunlap wrote: > On 08/13/13 01:28, Stephen Rothwell wrote: >> Hi all, >> >> Changes since 20130812: >> > > on i386: > > fs/built-in.o: In function `xfs_log_calc_minimum_size': > (.text+0x1797a9): undefined reference to `__udivdi3' > > See: [PATCH] xfs: call roundup_

Re: [GIT PULL] ext4 updates for 3.9

2013-03-01 Thread Eric Sandeen
On 2/27/13 2:58 PM, Dmitry Monakhov wrote: > On Wed, 27 Feb 2013 14:29:07 -0500, Theodore Ts'o wrote: >> On Wed, Feb 27, 2013 at 02:19:23PM -0500, Dave Jones wrote: >>> >>> Looks like it's fixed here too. >>> >>> How did this make it through -next without anyone hitting it ? >>> >>> I can't rememb

Re: EXT4 regression caused 4eec7

2013-05-13 Thread Eric Sandeen
On 5/12/13 4:01 AM, Dmitry Monakhov wrote: > In fact '4eec70' are vexing because I have reviewed and tested this patch > before > it was marked as Review-by, but missed the bug. This is because xfstests > was executed manually logs was full of warnings but tainted flag was not > checked at the end

Re: EXT4 regression caused 4eec7

2013-05-13 Thread Eric Sandeen
On 5/13/13 12:01 PM, Jan Kara wrote: > On Mon 13-05-13 11:34:12, Eric Sandeen wrote: >> On 5/12/13 4:01 AM, Dmitry Monakhov wrote: >>> In fact '4eec70' are vexing because I have reviewed and tested this patch >>> before >>> it was marked as Review-by

Re: EXT4 regression caused 4eec7

2013-05-14 Thread Eric Sandeen
On 5/14/13 2:11 AM, Dmitry Monakhov wrote: > On Mon, 13 May 2013 12:09:22 -0500, Eric Sandeen wrote: >> On 5/13/13 12:01 PM, Jan Kara wrote: >>> On Mon 13-05-13 11:34:12, Eric Sandeen wrote: >>>> On 5/12/13 4:01 AM, Dmitry Monakhov wrote: >>>>> In fac

Re: v3.10: unmount won't work

2013-05-14 Thread Eric Sandeen
On 5/14/13 2:12 PM, Toralf Förster wrote: > At a stable 32 bit stable Gentoo with kernel v3.10-rc1-113-ga2c7a54 I cannot > umount an (EXT4) fs > which was created in a file located in a tmpfs partition and loop mounted : > > That file system was used to hold victims files shared via NFSv4 to an

Re: XFS assertion from truncate. (3.10-rc2)

2013-05-24 Thread Eric Sandeen
On 5/24/13 3:03 AM, Dave Chinner wrote: > Right, patch below should fix the problem. > > What a frustrating bug. Now, where's my bottle of scotch? In your pantry, Dave. Next to the others! ;) -Eric > Cheers, > > Dave. > -- To unsubscribe from this list: send the line "unsubscribe linux-ker

[PATCH] perf: Fix parallel build

2012-09-20 Thread Eric Sandeen
front of the bison call in tools/perf/Makefile and running make -j4 on a smaller box: sleep 10; $(QUIET_BISON)$(BISON) -v util/pmu.y -d -o $(OUTPUT)util/pmu-bison.c Adding the following dependency fixes it for me: Signed-off-by: Eric Sandeen --- diff --git a/tools/perf/Makefile b/tools

Re: [PATCH] perf: Fix parallel build

2012-09-20 Thread Eric Sandeen
On 9/20/12 7:24 PM, Namhyung Kim wrote: > Hi Eric, > > On Thu, 20 Sep 2012 18:53:01 -0500, Eric Sandeen wrote: >> Parallel builds of perf were failing for me on a 32p box, with: >> >> * new build flags or prefix >> util/pmu.l:7:23: error: pmu-b

[PATCH V2] perf: Fix parallel build

2012-09-20 Thread Eric Sandeen
front of the bison calls in tools/perf/Makefile and running make -j4 on a smaller box i.e.: sleep 10; $(QUIET_BISON)$(BISON) -v util/pmu.y -d -o $(OUTPUT)util/pmu-bison.c Adding the following dependencies fixes it for me. Signed-off-by: Eric Sandeen --- V2: Fix other bison dependency

Re: Upgraded from 3.4 to 3.5.1 kernel: machine does not boot

2012-08-12 Thread Eric Sandeen
On 8/10/12 11:14 PM, Justin Piszcz wrote: > On Fri, Aug 10, 2012 at 7:07 PM, Justin Piszcz >> >> Hi, >> >> Found the root cause, the 3.5.1 kernel cannot mount my ext4 filesystem >> (60TB). You are a brave man running ext4 at 60T, but thank you for testing :) Backing out 8aeb00ff85ad25453765dd339b

[PATCH] reduce large do_mount stack usage with noinlines

2008-02-06 Thread Eric Sandeen
called only as do_mount() helpers; none of them should be in any path that would see a performance benefit from inlining... Signed-off-by: Eric Sandeen <[EMAIL PROTECTED]> --- Index: linux-2.6.24-mm1/fs/namespace.c === ---

Re: [PATCH] reduce large do_mount stack usage with noinlines

2008-02-06 Thread Eric Sandeen
Andrew Morton wrote: > Does the patch actually help? I mean, if a() calls b() and both use N > bytes of locals, our worst-case stack usage remains ~2N whether or not b() > was inlined in a()? In fact, uninlining makes things a little worse due to > callframe stuff. I think it does. [linux-2.6.

Re: [PATCH] reduce large do_mount stack usage with noinlines

2008-02-06 Thread Eric Sandeen
Arjan van de Ven wrote: > On Wed, 6 Feb 2008 14:34:57 -0800 > Andrew Morton <[EMAIL PROTECTED]> wrote: > >> Does the patch actually help? I mean, if a() calls b() and both use N >> bytes of locals, our worst-case stack usage remains ~2N whether or >> not b() was inlined in a()? In fact, uninlini

Re: [PATCH] reduce large do_mount stack usage with noinlines

2008-02-06 Thread Eric Sandeen
168, which is mostly the struct nameidata nd; left on the stack. These functions are called only as do_mount() helpers; none of them should be in any path that would see a performance benefit from inlining... Signed-off-by: Eric Sandeen <[EMAIL PROTECTED]> --- Index: linux-2.6.24-mm1/fs/na

Re: [PATCH] reduce large do_mount stack usage with noinlines

2008-02-06 Thread Eric Sandeen
Andrew Morton wrote: > On Wed, 06 Feb 2008 17:11:38 -0600 > Eric Sandeen <[EMAIL PROTECTED]> wrote: > >> /* >> * recursively change the type of the mountpoint. >> + * noinline this do_mount helper to save do_mount stack space. >> */ >> -static

Re: [PATCH] reduce large do_mount stack usage with noinlines

2008-02-07 Thread Eric Sandeen
Andrew Morton wrote: > On Wed, 06 Feb 2008 17:11:38 -0600 > Eric Sandeen <[EMAIL PROTECTED]> wrote: > >> /* >> * recursively change the type of the mountpoint. >> + * noinline this do_mount helper to save do_mount stack space. >> */ >> -static

Re: [PATCH] reduce large do_mount stack usage with noinlines

2008-02-08 Thread Eric Sandeen
Andi Kleen wrote: > Andrew Morton <[EMAIL PROTECTED]> writes: >>> */ >>> -static int do_change_type(struct nameidata *nd, int flag) >>> +static noinline int do_change_type(struct nameidata *nd, int flag) >> What we could do here is defined a new noinline_because_of_stack_suckiness >> and use that

Re: WARNING: at drivers/tty/tty_buffer.c:476 flush_to_ldisc()

2013-01-19 Thread Eric Sandeen
ad in [1] for more details. > > If you are on Linux-Next (next-20130118) you need the following three patches. > > Ilya Zykov (2): > tty: Correct tty buffer flush. > tty: Add driver unthrottle in ioctl(...,TCFLSH,..). > > Eric Sandeen (1): > jbd2: don

Re: jbd2: don't wake kjournald unnecessarily

2013-01-22 Thread Eric Sandeen
On 1/22/13 5:50 PM, Jan Kara wrote: > On Mon 21-01-13 18:11:30, Ted Tso wrote: >> On Tue, Jan 22, 2013 at 12:04:32AM +0100, Sedat Dilek wrote: >>> >>> Beyond the FUSE/LOOP fun, will you apply this patch to your linux-next GIT >>> tree? >>> >>> Feel free to add... >>> >>> Tested-by: Sedat Dile

Re: jbd2: don't wake kjournald unnecessarily

2013-01-23 Thread Eric Sandeen
On 1/23/13 3:44 AM, Jan Kara wrote: > On Tue 22-01-13 19:37:46, Eric Sandeen wrote: >> On 1/22/13 5:50 PM, Jan Kara wrote: >>> On Mon 21-01-13 18:11:30, Ted Tso wrote: >>>> On Tue, Jan 22, 2013 at 12:04:32AM +0100, Sedat Dilek wrote: >>>>> >>>

[PATCH] include falloc.h in header-y

2008-02-22 Thread Eric Sandeen
Include falloc.h in header-y; it defines a flag for the fallocate sysctl. Signed-off-by: Eric Sandeen <[EMAIL PROTECTED]> --- --- linux.orig/include/linux/Kbuild +++ linux/include/linux/Kbuild @@ -61,6 +61,7 @@ header-y += efs_fs_sb.h header-y += elf-fdpic.h header-y += elf-em.h he

Re: [xfs-masters] filesystem corruption on xfs after 2.6.25-rc1 (bisected, powerpc related?)

2008-02-25 Thread Eric Sandeen
Gaudenz Steinlin wrote: > Hi > > Since upgrading to 2.6.25-rc1 I see filesystem corruption on my XFS > filesystem. I can reproduce this by doing "git reset --hard v2.6.25-rc1" > on a git checkout which is on some other revision. Git outputs strange > error messages (like file xxx is a directory wh

Re: [xfs-masters] filesystem corruption on xfs after 2.6.25-rc1 (bisected, powerpc related?)

2008-02-25 Thread Eric Sandeen
Rafael J. Wysocki wrote: > On Monday, 25 of February 2008, Eric Sandeen wrote: >> If you're feeling motivated, maybe you can narrow it down to which of >> the changes - xfs_highbit32, xfs_highbit64, xfs_lowbit32, or >> xfs_lowbit64 - is causing the problem? (or maybe

Re: [RFC] ext3 freeze feature ver 0.2

2008-02-26 Thread Eric Sandeen
Takashi Sato wrote: > o Elevate XFS ioctl numbers (XFS_IOC_FREEZE and XFS_IOC_THAW) to the VFS > As Andreas Dilger and Christoph Hellwig advised me, I have elevated > them to include/linux/fs.h as below. > #define FIFREEZE_IOWR('X', 119, int) >   #define FITHAW _IOWR('X',

Re: [xfs-masters] Re: filesystem corruption on xfs after 2.6.25-rc1 (bisected, powerpc related?)

2008-02-26 Thread Eric Sandeen
Gaudenz Steinlin wrote: > On Tue, Feb 26, 2008 at 01:13:56AM +0100, Rafael J. Wysocki wrote: >> On Tuesday, 26 of February 2008, Christoph Hellwig wrote: >>> On Tue, Feb 26, 2008 at 12:52:56AM +0100, Rafael J. Wysocki wrote: > I'm not suggesting a partial revert; I just wonder which part of the

"kaweth" usb ethernet driver in 2.4?

2001-02-03 Thread Eric Sandeen
I'm wondering about the status of the "kaweth" Kawasaki LSI KL5KUSB100 USB to Ethernet Controller driver for 2.4. According to http://www.hiru.aoba.yokohama.jp/%7eura/USB/usbether.html, this chipset is used in the 3Com USB Network Adapter, Linksys USB10T, D-Link DSB-650, SMC 2102USB, Netgear EA10

Re: "kaweth" usb ethernet driver in 2.4?

2001-02-04 Thread Eric Sandeen
Michael Rothwell wrote: > > It also doesn't seem to work in 2.2. :) The original development of > > this driver was going on at http://drivers.rd.ilan.net/kaweth/ but there > > have been no updates for quite some time. > > Well, it doesn't work you _you_ on 2.2, obviously. But it works for us

Re: "kaweth" usb ethernet driver in 2.4?

2001-02-05 Thread Eric Sandeen
Ok, so the problem with the current driver is that it will attempt to load the firmware even if firmware is already loaded. This will hang the device. The trick is to look at the device release number (dev->descriptor.bcdDevice) - if no firmware is present, it returns 0x0002, if firmware is pres

[PATCH] updates for KLSI usb->ethernet

2001-02-06 Thread Eric Sandeen
This patch, against 2.4.1-ac4, does the following for the KLSI USB->ethernet adapter: (patch at http://lager.dyndns.org/kaweth/KLSI-2.4.1-ac4.patch.bz2) o Fixes firmware downloading. If firmware is already loaded and an attempt is made to download it again, the device will hang. This will

Re: [NEW][PATCH] updates for KLSI

2001-02-08 Thread Eric Sandeen
Greg KH wrote: > Silently changing descriptor ids while connected is just asking for > trouble :) Ok, fair enough - here's a patch against 2.4.1-ac6 which causes the device to disconnect/reconnect after firmware load (and a couple other minor kernel logging & formatting cleanups, plus one more d

Re: [xfs-masters] Re: [PATCH] pull XFS support out of Kconfig submenu

2005-08-18 Thread Eric Sandeen
Chris Wedgwood wrote: On Wed, Aug 17, 2005 at 10:45:48PM +0200, Jesper Juhl wrote: It seems slightly odd to me that XFS support should be in a separate submenu, when all the other filesystems are not using submenus but are directly selectable from the Filesystems menu. XFS also has an out-o

Re: [xfs-masters] Re: [PATCH] pull XFS support out of Kconfig submenu

2005-08-18 Thread Eric Sandeen
Eric Sandeen wrote: I have no problem with removing the submenu. Hm, however, if this is a purely cosmetic thing, let's go all the way and format the xfs options like the others, with indentation etc: config XFS_RT - bool "Realtime support (EXPERIMENTAL)" + bool

[PATCH] (UPDATED3) ecryptfs: check for existing key_tfm at mount time

2008-01-07 Thread Eric Sandeen
e cipher on the cached key_tfm_list, and sets a pointer to it if it exists. This can then be called from ecryptfs_parse_options(), and new key_tfm's can be added only when a cached one is not found. With list locking changes suggested by akpm. Signed-off-by: Eric Sandeen <[EMAIL PROTE

[PATCH] ecryptfs: make show_options reflect actual mount options

2007-12-07 Thread Eric Sandeen
t indeterminate for a given fs, but in any case the reported mount options can be used in a new mount command to get the same behavior. Signed-off-by: Eric Sandeen <[EMAIL PROTECTED]> Acked-by: Michael Halcrow <[EMAIL PROTECTED]&

[PATCH] ecryptfs: initialize new auth_tokens before teardown

2007-12-11 Thread Eric Sandeen
his, and it's happy with the following change. Signed-off-by: Eric Sandeen <[EMAIL PROTECTED]> --- Index: linux/fs/ecryptfs/keystore.c === --- linux.orig/fs/ecryptfs/keystore.c +++ linux/fs/ecryptfs/keystore.c @@ -

Re: 2.6.24-rc8 oops ext3_clear_inode+0x25/0xa0

2008-01-19 Thread Eric Sandeen
Soeren Sonnenburg wrote: > Dear all, > > I've just got this oops (causing the machine to hang finally)... > > Any ideas? > Soeren I've seen an awful lot of oopses out there on this path, kswapd->shrink_icache_memory; some get a little further and oops in ext3_discard_reservation. A few were cha

Re: [PATCH] 2.4: fix memory corruption from misinterpreted bad_inode_ops return values

2008-01-24 Thread Eric Sandeen
Willy Tarreau wrote: > Hi Dann, > > On Wed, Jan 23, 2008 at 11:12:12PM -0700, dann frazier wrote: >> This is a 2.4 backport of a linux-2.6 change by Eric Sandeen >> (commit be6aab0e9fa6d3c6d75aa1e38ac972d8b4ee82b8) >> >> CVE-2006-5753 was assigned for this issue.

Re: [PATCH] 2.4: fix memory corruption from misinterpreted bad_inode_ops return values

2008-01-24 Thread Eric Sandeen
dann frazier wrote: > Thanks Eric. Sounds like my comment about exercising these code paths > wasn't too clear - the comments with your patch do make the issue > clear, and this program demonstrates the void cast promotion issue > well. I'm just not sure of a good way to demonstrate that my backpo

[PATCH] ecryptfs: check for existing key_tfm at mount time

2007-12-20 Thread Eric Sandeen
lled from ecryptfs_parse_options(), and new key_tfm's can be added only when a cached one is not found. Signed-off-by: Eric Sandeen <[EMAIL PROTECTED]> --- Index: linux-2.6.24-rc3/fs/ecryptfs/crypto.c === --- linux-2.6

[PATCH] (UPDATED) ecryptfs: check for existing key_tfm at mount time

2007-12-22 Thread Eric Sandeen
hen be called from ecryptfs_parse_options(), and new key_tfm's can be added only when a cached one is not found. With list locking changes suggested by akpm. Signed-off-by: Eric Sandeen <[EMAIL PROTECTED]> --- Index: linux-2.

Re: [PATCH] (UPDATED) ecryptfs: check for existing key_tfm at mount time

2007-12-22 Thread Eric Sandeen
Andrew Morton wrote: > On Sat, 22 Dec 2007 11:42:37 -0600 Eric Sandeen <[EMAIL PROTECTED]> wrote: > >> Andrew Morton wrote: >> >>> It would all look a lot more solid if this locking was retained and both >>> ecryptfs_tfm_exists() and ecryptfs_add_new_key_

[PATCH] (UPDATED2) ecryptfs: check for existing key_tfm at mount time

2007-12-23 Thread Eric Sandeen
lled from ecryptfs_parse_options(), and new key_tfm's can be added only when a cached one is not found. With list locking changes suggested by akpm. Signed-off-by: Eric Sandeen <[EMAIL PROTECTED]> --- Index: linux-2.6.24-rc3/fs/ec

Re: [PATCH] UPDATED: hfs: handle more on-disk corruptions without oopsing

2007-12-23 Thread Eric Sandeen
Roman Zippel wrote: > Hi, > > On Thursday 20 December 2007, Eric Sandeen wrote: > >> Index: linux-2.6.24-rc3/fs/hfs/brec.c >> === >> --- linux-2.6.24-rc3.orig/fs/hfs/brec.c >> +++ linux-2.6.24-r

Re: [xfs-masters] [Patch 7/8] FS: Remove 'TOPDIR' from Makefiles

2008-01-01 Thread Eric Sandeen
WANG Cong wrote: > TOPDIR is obsolete, use objtree instead. > This patch removes TOPDIR from all fs/ Makefiles. > diff --git a/fs/xfs/Makefile b/fs/xfs/Makefile > index 49e3e7e..d1d3d49 100644 > --- a/fs/xfs/Makefile > +++ b/fs/xfs/Makefile > @@ -1 +1 @@ > -include $(TOPDIR)/fs/xfs/Makefile-linux-

Re: [PATCH] [KBUILD] fix external module install path

2008-01-01 Thread Eric Sandeen
Sam Ravnborg wrote: > Hi Eric. > > Took a look at this now and fixed it like this. > Let me know if you see other issues. > > [I know this is more than 6 months ago you reported it - sorry] > > Sam Wow, blast from the past; if you'd done it a bit earlier it would have been in the same ye

[PATCH] UPDATED2: hfs: handle more on-disk corruptions without oopsing

2008-01-02 Thread Eric Sandeen
Roman, with this on top does it look better to you? I'll get hfsplus done in a bit. Thanks, -Eric - Fix up previous hfs fsfuzzer patch to address Roman's comments. Signed-off-by: Eric Sandeen <[EMAIL PROTECTED]> --- diff -u linux-2.6.24-rc3/fs/hfs/brec.c lin

Re: [2.6.24 patch] let EXT4DEV_FS depend on BROKEN

2008-01-02 Thread Eric Sandeen
Adrian Bunk wrote: > Most people and all distributions use CONFIG_EXPERIMENTAL=y simply > because too many options (including options required for hardware > support) depend on it. > > Compare e.g.: > - "Marvell SATA support (HIGHLY EXPERIMENTAL)" > - "Provide NFSv4 client support (EXPERIMENTAL)

Re: [EXT4 set 6][PATCH 1/1]Export jbd stats through procfs

2007-11-30 Thread Eric Sandeen
Mingming Cao wrote: > [PATCH] jbd2 stats through procfs > > The patch below updates the jbd stats patch to 2.6.20/jbd2. > The initial patch was posted by Alex Tomas in December 2005 > (http://marc.info/?l=linux-ext4&m=113538565128617&w=2). > It provides statistics via procfs such as transaction li

[PATCH] fix up ext2_fs.h for userspace after reservations backport

2007-11-26 Thread Eric Sandeen
def __KERNEL__ around the function seems to help, patch below. BR, Tobias Signed-off-by: Eric Sandeen <[EMAIL PROTECTED]> --- Index: linux-2.6.24-rc1/include/linux/ext2_fs.h === --- linux-2.6.24-rc1.orig/include/linux/ext

Re: [PATCH] fix up ext2_fs.h for userspace after reservations backport

2007-11-27 Thread Eric Sandeen
Andrew Morton wrote: > I did this instead: (moved offending function from ext2_fs.h to ext2.h) Look like a better plan, thanks. -Eric - To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to [EMAIL PROTECTED] More majordomo info at http://vger.kern

[PATCH] address hfs on-disk corruption robustness review comments

2008-01-14 Thread Eric Sandeen
Address Roman's review comments for the previously sent on-disk corruption hfs robustness patch. I still owe a patch for hfsplus. Signed-off-by: Eric Sandeen <[EMAIL PROTECTED]> --- Index: linux-2.6.24-rc6-mm1/fs/

Re: fs/hfs/btree.c: new NULL dereference

2008-01-14 Thread Eric Sandeen
Adrian Bunk wrote: > The Coverity checker spotted the following NULL dereference introduced > by commit cf0594625083111ae522496dc1c256f7476939c2: Oops. Thanks, Adrian. Patch sent on a different thread, following another fixup patch for Roman - you're cc'd. Thanks, -Eric -- To unsubscribe from

[PATCH] hfs: fix coverity-found null deref

2008-01-14 Thread Eric Sandeen
Fix potential null deref introduced by commit cf0594625083111ae522496dc1c256f7476939c2 http://bugzilla.kernel.org/show_bug.cgi?id=9748 Signed-off-by: Eric Sandeen <[EMAIL PROTECTED]> --- Index: linux-2.6.24-rc6-mm1/fs/hfs/b

Re: [PATCH] address hfs on-disk corruption robustness review comments

2008-01-14 Thread Eric Sandeen
Andrew Morton wrote: > On Mon, 14 Jan 2008 15:15:04 -0600 > Eric Sandeen <[EMAIL PROTECTED]> wrote: > >> Address Roman's review comments for the previously sent on-disk >> corruption hfs robustness patch. >> >> I still owe a patch for hfsplus. >>

Re: [Patch] document ext3 requirements (was Re: [RFD] Incremental fsck)

2008-01-16 Thread Eric Sandeen
Alan Cox wrote: >> Writeback cache on disk in iteself is not bad, it only gets bad if the >> disk is not engineered to save all its dirty cache on power loss, >> using the disk motor as a generator or alternatively a small battery. >> It would be awfully nice to know which brands fail here, if any,

[PATCH] ecryptfs: set s_blocksize from lower fs in sb

2007-12-14 Thread Eric Sandeen
eCryptfs wasn't setting s_blocksize in it's superblock; just pick it up from the lower FS. Having an s_blocksize of 0 made things like "filefrag" which call FIGETBSZ unhappy. Signed-off-by: Eric Sandeen <[EMAIL PROTECTED]> --- Index: linux-2.6

[PATCH] ext3: issue warning when bad inode found via ext3_lookup

2007-12-14 Thread Eric Sandeen
te, as it indicates filesystem corruption. Signed-off-by: Eric Sandeen <[EMAIL PROTECTED] --- Index: linux-2.6.24-rc3/fs/ext3/namei.c === --- linux-2.6.24-rc3.orig/fs/ext3/namei.c +++ linux-2.6.24-rc3/fs/ext3/namei.c @@ -1049,6 +1049,9 @@

[PATCH] ext4: issue warning when bad inode found via ext4_lookup

2007-12-14 Thread Eric Sandeen
te, as it indicates filesystem corruption. Signed-off-by: Eric Sandeen <[EMAIL PROTECTED] --- Index: linux-2.6.24-rc3/fs/ext4/namei.c === --- linux-2.6.24-rc3.orig/fs/ext4/namei.c +++ linux-2.6.24-rc3/fs/ext4/namei.c @@ -1050,6 +1050,9 @@

[PATCH] export iov_shorten for ext4's use

2007-12-14 Thread Eric Sandeen
files in ext4_file_write(). This patch is currently living in the ext4 patch queue. Signed-off-by: Eric Sandeen <[EMAIL PROTECTED]> --- Index: linux-2.6.24-rc3/fs/read_write.c === --- linux-2.6.24-rc3.orig/fs/read_write.c +++ linu

[PATCH] ecryptfs: fix fsx data corruption problems

2007-12-16 Thread Eric Sandeen
mmap ops disabled - mmap still needs work. (A version of this patch on a RHEL5 kernel ran for over 110 million fsx ops) I added a few comments as well, to the best of my understanding as I read through the code. Signed-off-by: Eric Sandeen <[EMAIL PROTECTED]> --- Index: linux-2.6.24-rc3/

Re: [PATCH] export iov_shorten for ext4's use

2007-12-17 Thread Eric Sandeen
tes - pos); } to fix up too-large writes to these files in ext4_file_write(). This patch is currently living in the ext4 patch queue. Signed-off-by: Eric Sandeen <[EMAIL PROTECTED]> --- Index: linux-2.6.24-rc3/

[PATCH] ecryptfs: fix string overflow on long cipher names

2007-12-18 Thread Eric Sandeen
Passing a cipher name > 32 chars on mount results in an overflow when the cipher name is printed, because the last character in the struct ecryptfs_key_tfm's cipher_name string was never zeroed. Signed-off-by: Eric Sandeen <[EMAIL PROTECTED]> --- Index: linux-2.6.24-rc3/fs/ecr

  1   2   3   4   >