<div dir="ltr">Oops. Just realized I still hadn't responded. Sorry.<br><div class="gmail_extra"><br><br><div class="gmail_quote">On 20 March 2013 22:09, Dave Chinner <span dir="ltr"><<a href="mailto:david@fromorbit.com" target="_blank">david@fromorbit.com</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex"><div class="im">> > # xfs_quota  -x -c "quota -N -u brian"<br>
</div><div class="im">
><br>
> This prints the quota information 20 times. Huh?<br>
<br>
</div>How many mount points do you have? 20?<br></blockquote><div><br></div><div style>No. More actually. Today I get</div><div style><br></div><div>hq ~ # cat /proc/mounts  | wc --lines</div><div>34</div><div>hq ~ # xfs_quota  -x -c "quota -N -u brian" | wc --lines</div>
<div>44</div><div><div>hq ~ # xfs_quota  -x -c "quota -N -u brian" /home /share | wc --lines</div><div>4</div></div><div style> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex">
<div class="im"><span style="color:rgb(34,34,34)">The command has once for each of the mount points, and the command</span><br></div>
itself iterates mount points because you haven't given it a mount<br>
point to look at. So, double iteration.  This same problem was fixed<br>
for the report command by setting it to be a global command i.e. the<br>
fix I mentioned previously that you tested should prevent this<br>
second case, and maybe even the first.<br></blockquote><div><br></div><div style>As far as I can tell this happens with the patched xfs_quota.</div><div style><br></div><div style><div>hq /home/brian/xfsprogs # git diff</div>
<div>diff --git a/quota/quot.c b/quota/quot.c</div><div>index a7782b4..6cb665d 100644</div><div>--- a/quota/quot.c</div><div>+++ b/quota/quot.c</div><div>@@ -418,6 +418,7 @@ quot_init(void)</div><div>        quot_cmd.args = _("[-bir] [-gpu] [-acv] [-f file]");</div>
<div>        quot_cmd.oneline = _("summarize filesystem ownership");</div><div>        quot_cmd.help = quot_help;</div><div>+       quot_cmd.flags = CMD_FLAG_GLOBAL;</div><div> </div><div>        if (expert)</div>
<div>                add_command(&quot_cmd);</div><div><br></div><div style>As this version is based on git, I just did a git pull:</div><div style><br></div><div style><div>hq /home/brian/xfsprogs # git pull</div><div>
Updating b326e06..64ae7fc</div><div>Fast-forward</div><div> mkfs/xfs_mkfs.c |   18 ++++++++++++++++++</div><div> 1 files changed, 18 insertions(+), 0 deletions(-)</div><div><br></div><div style>Doesn't look related to this, don't think I need to retest.</div>
</div></div></div>-- <br>Brian May <<a href="mailto:brian@microcomaustralia.com.au" target="_blank">brian@microcomaustralia.com.au</a>>
</div></div>