From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from relay.sgi.com (relay1.corp.sgi.com [137.38.102.111]) by oss.sgi.com (Postfix) with ESMTP id CED057CA2 for ; Tue, 26 Jan 2016 09:00:31 -0600 (CST) Received: from cuda.sgi.com (cuda3.sgi.com [192.48.176.15]) by relay1.corp.sgi.com (Postfix) with ESMTP id BB69C8F8049 for ; Tue, 26 Jan 2016 07:00:28 -0800 (PST) Received: from mx1.redhat.com (mx1.redhat.com [209.132.183.28]) by cuda.sgi.com with ESMTP id dBSY3SQcRxzkD3nq (version=TLSv1.2 cipher=ECDHE-RSA-AES256-GCM-SHA384 bits=256 verify=NO) for ; Tue, 26 Jan 2016 07:00:27 -0800 (PST) Subject: Re: [PATCH 2/7] quota: add new quotactl Q_XGETNEXTQUOTA References: <1453487136-12681-1-git-send-email-sandeen@redhat.com> <1453487136-12681-3-git-send-email-sandeen@redhat.com> <20160126125710.GA23820@quack.suse.cz> From: Eric Sandeen Message-ID: <56A78A09.5080307@redhat.com> Date: Tue, 26 Jan 2016 09:00:25 -0600 MIME-Version: 1.0 In-Reply-To: <20160126125710.GA23820@quack.suse.cz> List-Id: XFS Filesystem from SGI List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Errors-To: xfs-bounces@oss.sgi.com Sender: xfs-bounces@oss.sgi.com To: Jan Kara Cc: linux-fsdevel@vger.kernel.org, xfs@oss.sgi.com On 1/26/16 6:57 AM, Jan Kara wrote: > On Fri 22-01-16 12:25:31, Eric Sandeen wrote: >> Q_XGETNEXTQUOTA is exactly like Q_XGETQUOTA, except that it >> will return quota information for the id equal to or greater >> than the id requested. In other words, if the requested id has >> no quota, the command will return quota information for the >> next higher id which does have a quota set. If no higher id >> has an active quota, -ESRCH is returned. > > Actually, is -ESRCH the right return value? It seems XFS traditionally > returns -ENOENT when id doesn't exist. So that would look more logical to > me. Hm, I was just going by the quotactl manpage, TBH, which says: ESRCH No disc quota is found for the indicated user. But yes, you are right, it is ENOENT for xfs... argh. I suppose the quotactl manpage could use an update as well, then. At this point do you want me to update the patches & resend or do you want to fix that up too? :( I reference -ESRCH in comments too, I think. -Eric _______________________________________________ xfs mailing list xfs@oss.sgi.com http://oss.sgi.com/mailman/listinfo/xfs