linux-ext4.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
From: Theodore Ts'o <tytso@mit.edu>
To: Andreas Dilger <adilger@dilger.ca>
Cc: Li Xi <pkuelelixi@gmail.com>, Jan Kara <jack@suse.cz>,
	Dave Chinner <david@fromorbit.com>,
	linux-ext4 <linux-ext4@vger.kernel.org>
Subject: Re: Reporting project quota usage?
Date: Wed, 16 Mar 2016 20:48:47 -0400	[thread overview]
Message-ID: <20160317004847.GG23593@thunk.org> (raw)
In-Reply-To: <A21F30FD-8207-473D-860A-F1B2118D35C0@dilger.ca>

On Wed, Mar 16, 2016 at 03:49:31PM -0600, Andreas Dilger wrote:
> 
> At one time I thought Dave was going to patch xfs_quota to try the quota
> ioctls for "foreign" filesystems like ext4 that supported them, but I'm
> not sure if that patch was landed, nor whether it has been released.
> 
> On the opposite front, Jan was updating the generic quota tools to support
> the XFS ioctls that project quota is using?

It's quite possible I'm missing something, but it doesn't look like
either has landed.  I've hacked xfs_quota (not anything that I would
want to submit for merging) so I can report usage project quota stats,
and to be able to set hard and soft limits.  But I was trying to
figure out what we should be telling users who wanted to use Project
Quota, but I'm pretty sure "pull down xfsprogs sources, and apply this
hacky patch[1], and then when you use the tooll ignore a whole bunch
of spurious warnings" is the right way to go.  :-)

I am also curious how Li Xi was testing the patches he sent me.

     	     	       	   	       - Ted

[1]

diff --git a/libxcmd/paths.c b/libxcmd/paths.c
index 5c65737..6c69a35 100644
--- a/libxcmd/paths.c
+++ b/libxcmd/paths.c
@@ -301,8 +301,8 @@ fs_table_initialise_mounts(
 			return errno;
 
 	while ((mnt = getmntent(mtp)) != NULL) {
-		if (strcmp(mnt->mnt_type, "xfs") != 0)
-			continue;
+//		if (strcmp(mnt->mnt_type, "xfs") != 0)
+//			continue;
 		if (!realpath(mnt->mnt_dir, rmnt_dir))
 			continue;
 		if (!realpath(mnt->mnt_fsname, rmnt_fsname))

  reply	other threads:[~2016-03-17  0:48 UTC|newest]

Thread overview: 10+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2016-03-16  5:29 Reporting project quota usage? Theodore Ts'o
2016-03-16 21:49 ` Andreas Dilger
2016-03-17  0:48   ` Theodore Ts'o [this message]
2016-03-17  1:31     ` Dave Chinner
2016-03-18  9:40     ` Jan Kara
2016-03-17  1:47 ` Li Xi
2016-03-22 14:58   ` Jan Kara
2016-03-30  9:25     ` Jan Kara
2016-03-31  2:55       ` Li Xi
2016-03-31  8:47         ` Jan Kara

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20160317004847.GG23593@thunk.org \
    --to=tytso@mit.edu \
    --cc=adilger@dilger.ca \
    --cc=david@fromorbit.com \
    --cc=jack@suse.cz \
    --cc=linux-ext4@vger.kernel.org \
    --cc=pkuelelixi@gmail.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
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).