linux-ext4.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
* [PATCH-e2fsprogs] Allow tune2fs -I if only one group per flex
@ 2009-03-02  8:29 Peng Tao
  2009-03-02 12:40 ` Theodore Tso
  0 siblings, 1 reply; 2+ messages in thread
From: Peng Tao @ 2009-03-02  8:29 UTC (permalink / raw)
  To: linux-ext4; +Cc: Peng Tao

This can be caused by tune2fs -O flex_bg.
And tune2fs -I should work okay for such partitions.

Signed-off-by: Peng Tao <bergwolf@gmail.com>
---
 misc/tune2fs.c |    3 ++-
 1 files changed, 2 insertions(+), 1 deletions(-)

diff --git a/misc/tune2fs.c b/misc/tune2fs.c
index 887a702..ee982e5 100644
--- a/misc/tune2fs.c
+++ b/misc/tune2fs.c
@@ -1715,7 +1715,8 @@ retry_open:
 			exit(1);
 		}
 		if (fs->super->s_feature_incompat &
-		    EXT4_FEATURE_INCOMPAT_FLEX_BG) {
+		    EXT4_FEATURE_INCOMPAT_FLEX_BG &&
+		    fs->super->s_log_groups_per_flex) {
 			fputs(_("Changing the inode size not supported for "
 				"filesystems with the flex_bg\n"
 				"feature enabled.\n"),
-- 
1.6.1.3


^ permalink raw reply related	[flat|nested] 2+ messages in thread

* Re: [PATCH-e2fsprogs] Allow tune2fs -I if only one group per flex
  2009-03-02  8:29 [PATCH-e2fsprogs] Allow tune2fs -I if only one group per flex Peng Tao
@ 2009-03-02 12:40 ` Theodore Tso
  0 siblings, 0 replies; 2+ messages in thread
From: Theodore Tso @ 2009-03-02 12:40 UTC (permalink / raw)
  To: Peng Tao; +Cc: linux-ext4

On Mon, Mar 02, 2009 at 04:29:20PM +0800, Peng Tao wrote:
> This can be caused by tune2fs -O flex_bg.
> And tune2fs -I should work okay for such partitions.

It will *probably* work, but again, it would be better to check to
make sure it really was caused by an accidental "tune2fs -O flex_bg".

     	     	    	       	     - Ted

^ permalink raw reply	[flat|nested] 2+ messages in thread

end of thread, other threads:[~2009-03-02 12:40 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2009-03-02  8:29 [PATCH-e2fsprogs] Allow tune2fs -I if only one group per flex Peng Tao
2009-03-02 12:40 ` Theodore Tso

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).