From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org Received: from vger.kernel.org (vger.kernel.org [23.128.96.18]) by smtp.lore.kernel.org (Postfix) with ESMTP id 3D8BBC4332F for ; Mon, 6 Nov 2023 22:42:17 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S233195AbjKFWmQ (ORCPT ); Mon, 6 Nov 2023 17:42:16 -0500 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:57704 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S232478AbjKFWmP (ORCPT ); Mon, 6 Nov 2023 17:42:15 -0500 Received: from mail-qv1-xf2f.google.com (mail-qv1-xf2f.google.com [IPv6:2607:f8b0:4864:20::f2f]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id A2E901BF for ; Mon, 6 Nov 2023 14:42:12 -0800 (PST) Received: by mail-qv1-xf2f.google.com with SMTP id 6a1803df08f44-66d122e0c85so32121596d6.3 for ; Mon, 06 Nov 2023 14:42:12 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=toxicpanda-com.20230601.gappssmtp.com; s=20230601; t=1699310532; x=1699915332; 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=Jh4u9lQwcTBmzRwQYOONZw1QExGbYz2ed9uS0nODmN0=; b=Gj4cpOIFNZRTzGJK6DSxt+fqBPLlSG99yiCcosFJftu9dGP0IADLA4be+hl9vVsGYQ d8dsoDUrgkL6iMciv0Kld5W5dnDMYTXcPFQItoZmrzEXD0lV/1u5k+tBsgfh4Th9TMGx Mhv99MCS2NOw74TW+AfZO6TW4nFtI0hpjKmeNThpxsgsO67T2lAIELlm7EvFA7qniocp 5FaKL+fjdElOC67EStIpjzAOV8GIqAdvh2WblLQDEYIFdQjfvcxskg53QOYfItQeRRTL LO09dShSAQ/Ck9403sEFVNqyYR33kwBMyBqBmGa0MlqRz3r51SaeX5X5ovyq4b+5GwAJ iPXQ== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20230601; t=1699310532; x=1699915332; 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=Jh4u9lQwcTBmzRwQYOONZw1QExGbYz2ed9uS0nODmN0=; b=bVnEmDBc9xin74k9h2FLWSnEvpqIkNXdzjTT/2p8cVGD4PpQb4SUdT3tbDMl5Qr8XP ZMUtjRORjdfr25lrObJT0GzIbnoCZqB+XRNW/JOScsgwLCZzAPf4Ls3QG8hdk5t7zydA ViH6OCowZdSqGEvdTrT/ThSbAK+c+YmT4VesrDjKuMin9+v/OQwuWEvljutB001m5tjD 5ADoiMTbehBFSvxxvN2ytmVrnEueDm67jYYsujNEoqT+Oh2oQd/wwypydTNOpwLCKAn5 3ZTqY1zFD1DrzXRS1Jh4MQm1nFjlo+wqbpj4xMcCBnDFJINUxz82dMVt9s2RB9MpIa4U Njxw== X-Gm-Message-State: AOJu0YzEKb6yn3tEdyPsc1si6wF6YFv9JXZetTylsxe/dANCBnZdtbEq H6A26DlyBGMJbJjzFuOMxJ3A1DyeqaQ7tmzApvsUbA== X-Google-Smtp-Source: AGHT+IE1BxGrUrgmYdZ4xKqxUp8ea8KUUcrhOF2gcRjFuJLv0yBe7E6hyFa7RnF2hITxbs50SL/EXw== X-Received: by 2002:a05:6214:501c:b0:66d:20f7:8d8d with SMTP id jo28-20020a056214501c00b0066d20f78d8dmr42120778qvb.28.1699310531808; Mon, 06 Nov 2023 14:42:11 -0800 (PST) Received: from localhost (cpe-76-182-20-124.nc.res.rr.com. [76.182.20.124]) by smtp.gmail.com with ESMTPSA id e16-20020a056214111000b00670a8921170sm3838537qvs.112.2023.11.06.14.42.11 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Mon, 06 Nov 2023 14:42:11 -0800 (PST) Date: Mon, 6 Nov 2023 17:42:10 -0500 From: Josef Bacik To: Christoph Hellwig Cc: Christian Brauner , Qu Wenruo , Amir Goldstein , Jan Kara , Chris Mason , David Sterba , linux-btrfs@vger.kernel.org, linux-fsdevel@vger.kernel.org Subject: Re: [PATCH 0/3] fanotify support for btrfs sub-volumes Message-ID: <20231106224210.GA3812457@perftesting> References: <20231101-neigen-storch-cde3b0671902@brauner> <20231102051349.GA3292886@perftesting> <20231102-schafsfell-denkzettel-08da41113e24@brauner> <20231103-kursleiter-proklamieren-aae0a02aa1a4@brauner> <20231106-fragment-geweigert-1d80138523e5@brauner> <20231106-unser-fiskus-9d1eba9fc64c@brauner> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: Precedence: bulk List-ID: X-Mailing-List: linux-btrfs@vger.kernel.org On Mon, Nov 06, 2023 at 09:13:19AM -0800, Christoph Hellwig wrote: > On Mon, Nov 06, 2023 at 02:47:16PM +0100, Christian Brauner wrote: > > Granted, an over-generalization but non in any way different from > > claiming that currently on one needs to know about btrfs subvolumes or > > that the proposed vfsmount solution will make it magically so that no > > one needs to care anymore. > > I don't think any one has claimed "no one" needs to care any more. What > the vfsmounts buy us that is that software that doesn't know and > should't know about btrfs subvolumes isn't silently broken. Software > that actually wants to do something fancy with them always need special > casing. Again, this is where I'm confused, because this doesn't change anything, we're still going to report st_dev as being different, which is what you hate. You pointed out above that user space thinks that different st_dev means different inode space. That is why Chris put the st_dev hack in, because rsync got confused, and this made it not confused. So we're actually doing exactly what user space wants, letting them know that they've wandered into a different inode ino space when they cross into a subvolume. But that's the crux of the problem, new subvolume == new inode number space. I'm not changing this. I don't think you want us to change this. We want to let user space who don't care to know about btrfs be able to operate cleanly. We have that with the st_dev hack. Changing to vfsmounts doesn't fix this. Userspace doesn't know it's wandered into a new directory, and in fact if you look at the rsync code they have a special python script you have to use if you want to exclude bind mounts, which is all the automount thing would accomplish. At this point I don't care, tell me what you want me to do so you'll stop complaining anytime we try to expose more btrfs specific information to user space and I'll do it. I'm tired of having this argument. You would have had auto mount patches in your inbox last week but it took me longer to get the new mount api conversion done than anticipated. But it doesn't appear to me there's agreement on the way forward. vfsmounts aren't going to do anything from what I can tell, but I could very well be missing some detail. And Christian doesn't seem to want that as a solution. Thanks, Josef