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  PHC 
Open Source and information security mailing list archives
 
Hash Suite: Windows password security audit tool. GUI, reports in PDF.
[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Date:	Mon, 15 Aug 2011 13:54:26 -0400
From:	Ted Ts'o <tytso@....edu>
To:	Eric Sandeen <sandeen@...hat.com>
Cc:	Lukas Czerner <lczerner@...hat.com>,
	Ext4 Developers List <linux-ext4@...r.kernel.org>
Subject: Re: [PATCH] ext4: don't give the "disabling delalloc" if not
 explicitly specified

On Mon, Aug 15, 2011 at 10:59:02AM -0500, Eric Sandeen wrote:
> 
> The giant behavior-options switch in ext4 is confusing enough; if enabling
> one option disables another default option, I think that explicitly stating
> it in the logs is useful.  Doing so silently just covers up the behavior.
> 
> If users are unhappy with the message, it's probably more because of 
> the fact of the matter, and not because of the presentation of the fact.  :)

Most users probably have no idea what "delalloc" actually means.  So
when they get a message that saying that data=journalled has disabled
delalloc, it could easily be seen as noise.  I was moved to do it
because I got tired of seeing the message over, and over, and over
again when running xfstests.

Maybe an improvement would be (1) to document what data=journal
implies in the Documentation/filesystems/ext4.txt, (2) change the
message to explicitly say "delayed allocation" instead of "delalloc"
(although many people won't have any idea what "delayed allocation"
means either), and (3) make it a printk_once thing.

I guess I don't agree with the fundamental presumption which is that
users should be looking at the dmesg output to understand what various
things mean, and if they didn't explicitly specify delalloc, why
should we complain about the fact that both delalloc and data=journal
were specified (when in fact it wasn't specified).

						- 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

Powered by blists - more mailing lists