xfs
[Top] [All Lists]

Re: [PATCH 0/4] quota: add new quotactl Q_XGETQUOTA2

To: Jan Kara <jack@xxxxxxx>
Subject: Re: [PATCH 0/4] quota: add new quotactl Q_XGETQUOTA2
From: Christoph Hellwig <hch@xxxxxxxxxxxxx>
Date: Mon, 18 Jan 2016 03:00:46 -0800
Cc: Eric Sandeen <sandeen@xxxxxxxxxxx>, Christoph Hellwig <hch@xxxxxxxxxxxxx>, fsdevel <linux-fsdevel@xxxxxxxxxxxxxxx>, Eric Sandeen <sandeen@xxxxxxxxxx>, xfs@xxxxxxxxxxx
Delivered-to: xfs@xxxxxxxxxxx
In-reply-to: <20160118103302.GD6850@xxxxxxxxxxxxx>
References: <568FEA2C.6080708@xxxxxxxxxx> <20160109072600.GA21636@xxxxxxxxxxxxx> <20160111132617.GD6262@xxxxxxxxxxxxx> <5693D33A.5090307@xxxxxxxxxxx> <20160111162807.GK6262@xxxxxxxxxxxxx> <5696D27A.9070700@xxxxxxxxxxx> <20160115093507.GA15950@xxxxxxxxxxxxx> <56992CD4.6030408@xxxxxxxxxxx> <20160118103302.GD6850@xxxxxxxxxxxxx>
User-agent: Mutt/1.5.24 (2015-08-30)
On Mon, Jan 18, 2016 at 11:33:02AM +0100, Jan Kara wrote:
> OK, ok. I like Dave's proposal for quotactl2(). So let's leave the unification
> for later and implent Q_GETNEXTQUOTA and Q_XGETNEXTQUOTA with the
> functionality of your original Q_XGETQUOTA2. Having separate call to get
> next ID would save us one new quotactl but OTOH we would need two syscalls
> (and quota structure lookups) to report one structure and there are
> potentially *lots* of them.

Oh well.  Guess I should give in and just accept the original patch..

<Prev in Thread] Current Thread [Next in Thread>