On Mon 21-01-13 10:50:20, Ted Tso wrote: > On Mon, Jan 21, 2013 at 04:10:08PM +0100, Jan Kara wrote: > > Exactly. There shouldn't be any conflicts so just keep it in your tree. > > You can add to the patch: > > Acked-by: Jan Kara <jack@xxxxxxx> > > Great, thanks. > > BTW, do you have any plans to work on getting repquota support for > ocfs2 and ext4 with the internal quota support into quotactl(2) and > quotatools? It's missing functionality that I think users who want to > switch to the internal quota support will very much care about. As Carlos mentioned, it kind-of works. If repquota(8) doesn't have a way to iterate over quota structures directly, it iterates over entries in /etc/passwd and uses Q_GETQUOTA to find the information. It is not a perfect solution (e.g. when you have a huge LDAP database instead of /etc/passwd) but in lots of cases it works just fine. > 5) Add some kind of functional replacement for repquota. If > quotactl(2) support is not forthcoming in the near future, at the very > least debugfs should have a way of dumping out the repquota > information for an unmounted ext4 file system with internal quotas > enabled. I have it on my todo list for a long time. The problem is noone wants it enough to get to the top of the list... Also it won't be completely trivial interface because we'll have to return some handle with which we could associate scanning state. My plan would be to likely return an ordinary read-only file handle (mostly to avoid various problems with what happens when process exits without closing the handle or what happens on fork) where f_pos would mean [ug]id to report next and reading would fill the buffer with quota structures (if_dqblk + [ug]id). But I haven't tried coding this... Honza -- Jan Kara <jack@xxxxxxx> SUSE Labs, CR -- To unsubscribe from this list: send the line "unsubscribe linux-ext4" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html