[Ocfs2-devel] [PATCH] ocfs2: Prevent mount with "-o acl" if acl isn't supported.

[Joel Becker]

On Thu, Jun 11, 2009 at 05:03:55AM +0800, Tao Ma wrote:
> Currently, if we don't enable CONFIG_OCFS2_FS_POSIX_ACL, ocfs2
> will not support acl. That is OK.
> 
> But in the following case:
> 1. kernel isn't built to support acl.
> 2. mount -t ocfs2 -o acl /dev/sdx /mnt/ocfs2.
> It sucesses. And what's more, if I do "mount|grep sdx" will get
> /dev/sdx on /mnt/ocfs2 type ocfs2 (rw,acl,heartbeat=none)
> 
> So a normal user will just think acl now is ok for the volume and
> he will surely be puzzled when he tries setfacl and fails with
> "Operation not supported".
> 
> The good thing is that the kernel already printk some useful
> information, so this patch just tries to prevent the user from
> mounting the volume.

	Hmm, everyone else seems to do it the way we have been.  Trying
to think about the surprise you describe vs the surprise of ocfs2 being
different than other linux filesystms.

Joel

-- 

"Here's something to think about:  How come you never see a headline
 like ``Psychic Wins Lottery''?"
	- Jay Leno

Joel Becker
Principal Software Developer
Oracle
E-mail: joel.becker at oracle.com
Phone: (650) 506-8127