linux-ext4 - Re: [PATCH] resize2fs: fix overly-pessimistic calculation of minimum size required

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [day] [month] [year] [list]

Message-ID: <alpine.LFD.2.00.1404291326020.2183@localhost.localdomain>
Date:	Tue, 29 Apr 2014 13:30:18 +0200 (CEST)
From:	Lukáš Czerner <lczerner@...hat.com>
To:	"Theodore Ts'o" <tytso@....edu>
cc:	Dmitry Monakhov <dmonakhov@...nvz.org>,
	Ext4 Developers List <linux-ext4@...r.kernel.org>
Subject: Re: [PATCH] resize2fs: fix overly-pessimistic calculation of minimum
 size required

On Mon, 28 Apr 2014, Theodore Ts'o wrote:

> Date: Mon, 28 Apr 2014 07:47:55 -0400
> From: Theodore Ts'o <tytso@....edu>
> To: Dmitry Monakhov <dmonakhov@...nvz.org>
> Cc: Ext4 Developers List <linux-ext4@...r.kernel.org>
> Subject: Re: [PATCH] resize2fs: fix overly-pessimistic calculation of minimum
>     size required
> 
> On Mon, Apr 28, 2014 at 11:34:54AM +0400, Dmitry Monakhov wrote:
> > > I'm going to have to self-NACK this.  This patch causes the resize2fs
> > > regression tests to fail.  (In fact, Dmitry's original patch also
> > > causes the resize2fs regression tests to fail.)
> >
> > Agree, regressions are not acceptable. Can you please spacify
> > which tests are failed. As far as i know xfstetsts has no tests
> > for resize2fs.
> 
> It was 2 of the tests from e2fsprogs's "make check", where off-one
> resize2fs regression tests belong.  The xfstests suite is a good place
> to test on-line resize functionality, where the bulk of the code that
> has to function correctly is in the kernel,, but for testing
> functionality which is specific to e2fsprogs, it's better to test it
> within the context of e2fsprogs's regression tests.

While xfstests is definitely a good place for online resize test, I
do not see any reason not to put some offline resize tests into
xfstests as well. If nothing it is run more often and by much more
people than e2fsprogs tests.

Currently AFAICT we only have one resize2fs test in xfstests.

Thanks!
-Lukas

> 
> Anyway, the patch set I sent out fixes up the problem.
> 
> 	    	      	     	       	  - Ted
> --
> To unsubscribe from this list: send the line "unsubscribe linux-ext4" in
> the body of a message to majordomo@...r.kernel.org
> More majordomo info at  http://vger.kernel.org/majordomo-info.html
> 
--
To unsubscribe from this list: send the line "unsubscribe linux-ext4" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html