From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from aserp1040.oracle.com ([141.146.126.69]:23942 "EHLO aserp1040.oracle.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1750763AbdAXUFL (ORCPT ); Tue, 24 Jan 2017 15:05:11 -0500 Date: Tue, 24 Jan 2017 12:05:05 -0800 From: "Darrick J. Wong" Subject: Re: [PATCH v2] xfs: use per-AG reservations for the finobt Message-ID: <20170124200505.GL4780@birch.djwong.org> References: <1485194742-23185-1-git-send-email-hch@lst.de> <20170123192847.GM12985@birch.djwong.org> <20170124140247.GA26149@lst.de> <20170124164837.GI4780@birch.djwong.org> <20170124195050.GA1025@lst.de> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20170124195050.GA1025@lst.de> Sender: linux-xfs-owner@vger.kernel.org List-ID: List-Id: xfs To: Christoph Hellwig Cc: linux-xfs@vger.kernel.org On Tue, Jan 24, 2017 at 08:50:50PM +0100, Christoph Hellwig wrote: > On Tue, Jan 24, 2017 at 08:48:37AM -0800, Darrick J. Wong wrote: > > I'd just print out: > > > > "Per-AG reservation for AG ${agno} failed. Filesystem may run out of space." > > Sure, I can do that. > > > > > if any of the (now three) invocations of __xfs_ag_resv_init() returns > > ENOSPC, since the AG is more than 98% full. Some day, a sysadmin could > > then use the fsmap data to identify which inodes are using the most > > space in that AG and arrange to migrate/remove/etc the data to another > > AG. > > But for all but the reflink one this can't happen as the reservations > have been there since the beginning. And if they fail neverless we fail > the mount. Or am I missing something here? The code I wrote doesn't fail the mount (or remount) on ENOSPC so that it's still possible to delete files off the fs even when space is low. In theory it wasn't possible to run out, but I've been thinking that it's better to warn noisily just in case we /do/ run out of space later, since it's not entirely obvious when the fs goes offline because we tried to expand a btree and hit ENOSPC. (Also if people perform underhanded "upgrades" then they can hit this problem, but maybe that was just me. :P) --D