From mboxrd@z Thu Jan 1 00:00:00 1970 From: Eric Sandeen Subject: Re: [PATCH] xfs_quota: allow operation on foreign filesystem types Date: Fri, 15 Jan 2016 22:41:15 -0600 Message-ID: <5699C9EB.10307@sandeen.net> References: <1452729788-5396-1-git-send-email-david@fromorbit.com> Mime-Version: 1.0 Content-Type: text/plain; charset=windows-1252 Content-Transfer-Encoding: 8bit Cc: "linux-ext4@vger.kernel.org" , fstests To: xfs@oss.sgi.com Return-path: In-Reply-To: <1452729788-5396-1-git-send-email-david@fromorbit.com> Sender: fstests-owner@vger.kernel.org List-Id: linux-ext4.vger.kernel.org On 1/13/16 6:03 PM, Dave Chinner wrote: > From: Dave Chinner > > This allows xfs_quota to be used on ext4 for project quota testing > in xfstests. > > Signed-off-by: Dave Chinner > --- ... > diff --git a/quota/report.c b/quota/report.c > index c77b24f..eaf3a67 100644 > --- a/quota/report.c > +++ b/quota/report.c > @@ -658,6 +658,7 @@ report_init(void) > dump_cmd.args = _("[-gpu] [-f file]"); > dump_cmd.oneline = _("dump quota information for backup utilities"); > dump_cmd.help = dump_help; > + dump_cmd.flags = CMD_FLAG_FOREIGN_OK; > > report_cmd.name = "report"; > report_cmd.altname = "repquota"; > @@ -668,6 +669,7 @@ report_init(void) > report_cmd.args = _("[-bir] [-gpu] [-ahnt] [-f file]"); > report_cmd.oneline = _("report filesystem quota information"); > report_cmd.help = report_help; > + report_cmd.flags = CMD_FLAG_FOREIGN_OK; There's another report_cmd.flags = CMD_FLAG_GLOBAL; above this which gets overwritten here; this command needs to be: report_cmd.flags = CMD_FLAG_GLOBAL | CMD_FLAG_FOREIGN_OK; or else xfs_quota -x -c "report -a" goes bonkers, iterating in nested loops and printing filesystems multiple times. It also seems that "report -a" should not be changing defaults this way such that non-xfs filesystems will get printed; this will be an unexpected change to default behavior. Perhaps a top-level "-f" option to explicitly enable foreign filesystem detection would be best? -Eric