linux-ext4 - Re: [v8 3/5] ext4: adds project quota support

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] [thread-next>] [day] [month] [year] [list]

Message-ID: <20150106215251.GA30687@quack.suse.cz>
Date:	Tue, 6 Jan 2015 22:52:51 +0100
From:	Jan Kara <jack@...e.cz>
To:	Andreas Dilger <adilger@...ger.ca>
Cc:	Li Xi <pkuelelixi@...il.com>,
	Linux Filesystem Development List 
	<linux-fsdevel@...r.kernel.org>,
	ext4 development <linux-ext4@...r.kernel.org>,
	linux-api@...r.kernel.org, tytso@....edu, jack@...e.cz,
	viro@...iv.linux.org.uk, hch@...radead.org, dmonakhov@...nvz.org
Subject: Re: [v8 3/5] ext4: adds project quota support

On Tue 06-01-15 13:01:21, Andreas Dilger wrote:
> On Dec 8, 2014, at 10:22 PM, Li Xi <pkuelelixi@...il.com> wrote:
> > 
> > This patch adds mount options for enabling/disabling project quota
> > accounting and enforcement. A new specific inode is also used for
> > project quota accounting.
> 
> Judy looking through these patches again and saw a minor issue:
> 
> > @@ -1128,10 +1128,11 @@ enum {
> > 	Opt_journal_path, Opt_journal_checksum, Opt_journal_async_commit,
> > 	Opt_abort, Opt_data_journal, Opt_data_ordered, Opt_data_writeback,
> > 	Opt_data_err_abort, Opt_data_err_ignore,
> > -	Opt_usrjquota, Opt_grpjquota, Opt_offusrjquota, Opt_offgrpjquota,
> > +	Opt_usrjquota, Opt_grpjquota, Opt_prjjquota,
> > +	Opt_offusrjquota, Opt_offgrpjquota, Opt_offprjjquota,
> > 	Opt_jqfmt_vfsold, Opt_jqfmt_vfsv0, Opt_jqfmt_vfsv1, Opt_quota,
> > 	Opt_noquota, Opt_barrier, Opt_nobarrier, Opt_err,
> > -	Opt_usrquota, Opt_grpquota, Opt_i_version,
> > +	Opt_usrquota, Opt_grpquota, Opt_prjquota, Opt_i_version,
> > 	Opt_stripe, Opt_delalloc, Opt_nodelalloc, Opt_mblk_io_submit,
> > 	Opt_nomblk_io_submit, Opt_block_validity, Opt_noblock_validity,
> > 	Opt_inode_readahead_blks, Opt_journal_ioprio,
> > @@ -1183,6 +1184,8 @@ static const match_table_t tokens = {
> > 	{Opt_usrjquota, "usrjquota=%s"},
> > 	{Opt_offgrpjquota, "grpjquota="},
> > 	{Opt_grpjquota, "grpjquota=%s"},
> > +	{Opt_offprjjquota, "prjjquota="},
> > +	{Opt_prjjquota, "prjjquota=%s"},
> 
> We don't need to be able to specify the project quota inodes on the
> mount command line.  This was only needed for supporting old (non-
> journaled) user and group quotas, but there is no chance of having
> old project quotas.
> 
> > 	{Opt_jqfmt_vfsold, "jqfmt=vfsold"},
> > 	{Opt_jqfmt_vfsv0, "jqfmt=vfsv0"},
> > 	{Opt_jqfmt_vfsv1, "jqfmt=vfsv1"},
> > @@ -1190,6 +1193,7 @@ static const match_table_t tokens = {
> > 	{Opt_noquota, "noquota"},
> > 	{Opt_quota, "quota"},
> > 	{Opt_usrquota, "usrquota"},
> > +	{Opt_prjquota, "prjquota"},
> 
> Do we want to allow non-journaled project quota at all?  I don't
> see that as a benefit, and it was only allowed for user/group
> quota for compatibility reasons.  Since the project quota support
> will need an updated e2fsprogs anyway, it may as well be journaled
> from the start.
  Yeah, probably you are right that it doesn't make a whole lot of sense to
support project quotas in user visible files. So these could be removed.

								Honza
-- 
Jan Kara <jack@...e.cz>
SUSE Labs, CR
--
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