From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from mail-yb1-f175.google.com (mail-yb1-f175.google.com [209.85.219.175]) (using TLSv1.2 with cipher ECDHE-RSA-AES128-GCM-SHA256 (128/128 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id 210EE14E2DC for ; Thu, 22 Feb 2024 15:48:04 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=209.85.219.175 ARC-Seal:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1708616886; cv=none; b=m30oM/xnMMorBgyK1pkIkieg589V+2Flu2HDHOePnRp7An97V9h2ArQnprLaiNEXn2ugoG896U1JYpulLrLSp4mYXOzYrXnpB3GRRPCYRfXUqEiAnYrkuwEWRBLGbIDTgGRxN59TDRu8UVE6JIatVzYSRm0kklGAby1rLYVKyqk= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1708616886; c=relaxed/simple; bh=/URg4PQnQQuQavo2tWzU6iVHhXxnRjfAkpj2MvvU244=; h=Date:From:To:Cc:Subject:Message-ID:References:MIME-Version: Content-Type:Content-Disposition:In-Reply-To; b=XIvISMy3yojNr8HlmF1rwCtVcNnwIkSTtf5m0/XLdiRsf44UoT2oPOf5FFekSqggvvJ+qycLhW/jLDbheT2ulk7fUvSJEaNXfVq61Gvp2X8bm6TS1A3aNfYEW/qYrWQ3mIHhnWa6YkKPG7EDqYLAhlb1rMEfMBthLJIOhj47khU= ARC-Authentication-Results:i=1; smtp.subspace.kernel.org; dmarc=none (p=none dis=none) header.from=toxicpanda.com; spf=none smtp.mailfrom=toxicpanda.com; dkim=pass (2048-bit key) header.d=toxicpanda-com.20230601.gappssmtp.com header.i=@toxicpanda-com.20230601.gappssmtp.com header.b=RIEywIvv; arc=none smtp.client-ip=209.85.219.175 Authentication-Results: smtp.subspace.kernel.org; dmarc=none (p=none dis=none) header.from=toxicpanda.com Authentication-Results: smtp.subspace.kernel.org; spf=none smtp.mailfrom=toxicpanda.com Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=toxicpanda-com.20230601.gappssmtp.com header.i=@toxicpanda-com.20230601.gappssmtp.com header.b="RIEywIvv" Received: by mail-yb1-f175.google.com with SMTP id 3f1490d57ef6-dc6dcd9124bso1998907276.1 for ; Thu, 22 Feb 2024 07:48:04 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=toxicpanda-com.20230601.gappssmtp.com; s=20230601; t=1708616884; x=1709221684; darn=vger.kernel.org; h=in-reply-to:content-disposition:mime-version:references:message-id :subject:cc:to:from:date:from:to:cc:subject:date:message-id:reply-to; bh=QIa8oVIZuT7lP1k8X8l2WJQdq609coU/Nz2aGnBWwe4=; b=RIEywIvvscu1E8xViuV4ZuAJOikoT2p1+xmOQaCMPZpoQ5BUx/TnMjs2uOiFXVRR5t Phh57Z4wdTLREqHqs61NjoO3Iknt5R4e49VX887QUDXR/WQfNIjffCEOLEp+hzX2JfSQ 7mYiEamFKSWVW4qKy756x+ZdsL850EDSFBQ6aT2vDN0O6LWiu1D1IsEHpKgwq/EUuk+w fy1d5afa6gPYlpjt/rxtXtUmLRtfYb4zRb0KJR/wAf/ZwfTJ3VurlPCdyt1ySZZwf7Ey MW3R+XNomF88wZDCrC9WcSAzOoWZSHsPQfwAC2w8ZgWdrfCdK4VZeLpiBusKI1ZhDuIb BdSQ== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20230601; t=1708616884; x=1709221684; h=in-reply-to:content-disposition:mime-version:references:message-id :subject:cc:to:from:date:x-gm-message-state:from:to:cc:subject:date :message-id:reply-to; bh=QIa8oVIZuT7lP1k8X8l2WJQdq609coU/Nz2aGnBWwe4=; b=HdqvS1uuh0iuuYgmDWmyyPMuwkGb6U7P/T4NN4+6Kbwf/j1ftOmCvFpqlaO7D5sd9D +8hvoJx6gNjQjLvQzDnXiWvPU2b94s1yWdfNPfHcETi9oHb3KIjyfpCERHIaiUrQUTtV rdhUJdAckFWIOmEs7OiS9q6ZVKQISJ4e9jQLzW0y6sFMKcPcjGtOgC4/HcCT7YOg6N13 6OTrnFcdPIKsqc2TD+3t/AdPMZUaoACxPFJiAmNSvq38tCYzXzawoK0s364KGrQFBsYu MXKG5+fbXH7t57PhoYbjbKEvdYNUq8bKKVND81fPk3guZvCN+6TvegSFbVLOmsdZg7SN CZPQ== X-Forwarded-Encrypted: i=1; AJvYcCXWRh14LynwCSzEOxMxhLVoYvDDJvp8D5Hd09jxNHJsw2FhSeh4Y7msOeD22h2bDtmzZpji/RiCw2qEQuKTkCBKW38VLnWfB/8Nz0n6rw== X-Gm-Message-State: AOJu0YwOcOFmpHnC611BrlhdWtr+hYXBq6z5NIqx0GPdEIlAnFUBlMNu +pQvpNNRz865oR73CNiwbnliihNQYi/O8JCzUwbWqwMF6XLG8PhSoWiHAnho2cA= X-Google-Smtp-Source: AGHT+IHeRNYTHAQHlTw6FyZDjFXRERNo4oRgHEr44ZmqBtCgNvJrfmcbtG60abBs4SP3t2oerm+vSw== X-Received: by 2002:a25:8b85:0:b0:dcd:2f2d:7a05 with SMTP id j5-20020a258b85000000b00dcd2f2d7a05mr2616558ybl.35.1708616884094; Thu, 22 Feb 2024 07:48:04 -0800 (PST) Received: from localhost (076-182-020-124.res.spectrum.com. [76.182.20.124]) by smtp.gmail.com with ESMTPSA id w131-20020a253089000000b00dc25d5f4c75sm2942698ybw.10.2024.02.22.07.48.03 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Thu, 22 Feb 2024 07:48:03 -0800 (PST) Date: Thu, 22 Feb 2024 10:48:02 -0500 From: Josef Bacik To: Miklos Szeredi Cc: Kent Overstreet , linux-bcachefs@vger.kernel.org, linux-btrfs@vger.kernel.org, linux-fsdevel@vger.kernel.org, linux-kernel@vger.kernel.org, lsf-pc@lists.linux-foundation.org Subject: Re: [LSF TOPIC] statx extensions for subvol/snapshot filesystems & more Message-ID: <20240222154802.GA1219527@perftesting> References: <2uvhm6gweyl7iyyp2xpfryvcu2g3padagaeqcbiavjyiis6prl@yjm725bizncq> <20240221210811.GA1161565@perftesting> Precedence: bulk X-Mailing-List: linux-fsdevel@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: On Thu, Feb 22, 2024 at 10:14:20AM +0100, Miklos Szeredi wrote: > On Wed, 21 Feb 2024 at 22:08, Josef Bacik wrote: > > > > On Wed, Feb 21, 2024 at 04:06:34PM +0100, Miklos Szeredi wrote: > > > On Wed, 21 Feb 2024 at 01:51, Kent Overstreet wrote: > > > > > > > > Recently we had a pretty long discussion on statx extensions, which > > > > eventually got a bit offtopic but nevertheless hashed out all the major > > > > issues. > > > > > > > > To summarize: > > > > - guaranteeing inode number uniqueness is becoming increasingly > > > > infeasible, we need a bit to tell userspace "inode number is not > > > > unique, use filehandle instead" > > > > > > This is a tough one. POSIX says "The st_ino and st_dev fields taken > > > together uniquely identify the file within the system." > > > > > > > Which is what btrfs has done forever, and we've gotten yelled at forever for > > doing it. We have a compromise and a way forward, but it's not a widely held > > view that changing st_dev to give uniqueness is an acceptable solution. It may > > have been for overlayfs because you guys are already doing something special, > > but it's not an option that is afforded the rest of us. > > Overlayfs tries hard not to use st_dev to give uniqueness and instead > partitions the 64bit st_ino space within the same st_dev. There are > various fallback cases, some involve switching st_dev and some using > non-persistent st_ino. > > What overlayfs does may or may not be applicable to btrfs/bcachefs, > but that's not my point. My point is that adding a flag to statx does > not solve anything. You can't just say that from now on btrfs > doesn't have use unique st_ino/st_dev because we've just indicated > that in statx and everything is fine. That will trigger the > no-regressions rule and then it's game over. At least I would expect > that to happen. Right, nobody is arguing that. Our plan is to 1) Introduce some sort of statx mechanism to expose this information. 2) Introduce an incompat fs feature flag to give unique inode numbers for people that want them, and there stop doing the st_dev thing we currently do. 3) Continue doing the st_dev thing we currently do for the non-feature flag case. 4) Wait 50 years and then maybe stop doing the st_dev thing once people have adopted whatever statx flag has been introduced. God willing I will have been hit by a bus well before then. Nobody is arguing about reverting existing behavior, we're stuck with it. We're trying to define how we'd like it to look going forward, and start working towards that. Thanks, Josef