Messages in this thread | | | Date | Mon, 07 Jan 2002 11:14:10 +0100 | From | Peter Wächtler <> | Subject | Re: e2compress on Linux 2.4.16 |
| |
Peter Wächtler schrieb: > > > Bonjour, > > > > I tried your e2compress patches and it didn't work for me at the > > first run :-( After compression of some larger files (>400KiB) > > the files couldn't be stat()ed -> the i_blocks field was "negative". > > > > in fs/ext2/cpmpress.c::ext2_compress_cluster:2886 > > inode->i_blocks -= ((u_nblk - c_nblk) > > << (inode->i_sb->s_blocksize_bits - 9)); > > > > has to be deleted. Otherwise the i_blocks field gets decremented > > twice. Depending on the glibc version you use, a stat() fails > > due to some conversion code between lstat64() and userspace > > returning EOVERFLOW and "negative i_blocks" (viewable with debugfs) > > > > ext2_free_blocks() calls DQUOT_FREE_BLOCK() which calls > > include/linux/fs.h::inode_sub_bytes() regardless if CONFIG_QUOTA > > is enabled or not. > > In 2.2 the quota code was different - so there it was correct > > to substract the free'ed blocks. > > > > With this "fix" all seems to work fine. So thanx for your work! > >
Since I didn't get any reply so far (perhaps you have longer vacation than me:)
I resend my mail and also cc:lkml - take it as a positive feedback! - To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
| |