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 73F31C7EE29 for ; Sat, 10 Jun 2023 22:32:35 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S231325AbjFJWce (ORCPT ); Sat, 10 Jun 2023 18:32:34 -0400 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:36132 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S230432AbjFJWcd (ORCPT ); Sat, 10 Jun 2023 18:32:33 -0400 Received: from mail-pl1-x630.google.com (mail-pl1-x630.google.com [IPv6:2607:f8b0:4864:20::630]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id 6E72C35A9 for ; Sat, 10 Jun 2023 15:32:32 -0700 (PDT) Received: by mail-pl1-x630.google.com with SMTP id d9443c01a7336-1b3be6c0b4cso940755ad.3 for ; Sat, 10 Jun 2023 15:32:32 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=fromorbit-com.20221208.gappssmtp.com; s=20221208; t=1686436352; x=1689028352; h=in-reply-to:content-transfer-encoding:content-disposition :mime-version:references:message-id:subject:cc:to:from:date:from:to :cc:subject:date:message-id:reply-to; bh=yIMevQtafEsanIaRpoX6Xg8apLrTfiAQUxFguirgsS4=; b=ZSWqsuDCkm85kpo25ZJDyW9qqRe+Bhjz7weXfYfE4wwpRDqIECq9Dc0d6IWh4NK2zb bfMiGXgn6IxFDrngUtiI6bl4D6cLDKAjdiKbfpichvHe1Mw6IJSYTT5NRnT21JLqH34f 06tDZe1TOCnhJvsgMngvsHLQ7XCHkPXDS/0/+7Mt8BNXCyqae2LZs8zVp1aOrQmM7qpL JUc03DN44mfKkCudMfji15bkQYk26yGDLQ+l/cdei//8jfaDTIQ+Ax/myTJZ3rpeGMwv eUpwd6kqNs677UXDbSiwOUeTr703xjfsRQ6B2HqD5eHvKAh4HwB9PhlMZtXnBIxM3BTO ixqQ== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20221208; t=1686436352; x=1689028352; h=in-reply-to:content-transfer-encoding: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=yIMevQtafEsanIaRpoX6Xg8apLrTfiAQUxFguirgsS4=; b=XASq6S24hUFZUpJoACZ1keqciBuwkPmpitmFcwZsEF0Po7QLoKLfdL/3pmOD39Iw+Y JplMd1bZdjFaiXABVmTaU1d+14hhrvlU8TUfLvMUihOcSBr/ZXhXJp83hlZTmWseUfUL HcWekgflA6nV09JL40xULx+RiJEBkdNuy8qgiAUqr8QRTLSR+a+ko6dUWf1dBuwhOg++ ixqtcrqdGuXaitG/oLeN2j7v87dQiofvDxei1bKDqgmdrXjSXfUUUsC5IgyzK9dp8jEJ ozylzoDFpvtz2BaeJbj+nzANkvOf7Srwbm32VXu3RIrvW0rheFwmWmYjRX1MjXQXV74N Pbkg== X-Gm-Message-State: AC+VfDzi599/GabkdGJF6CUNRAnz+JigIO8HhqzXiPI0Cihlqx+tqlOi j2VIJPhl/c/xqDLbh54fpsOrgg== X-Google-Smtp-Source: ACHHUZ5mQ6sKDWv4O3omCBMHc2v4zVRCVzstNuuujtL90nnc2U9ek4CLvI8yW5NButAum0FJ0vyzbg== X-Received: by 2002:a17:902:7616:b0:1b0:7739:657a with SMTP id k22-20020a170902761600b001b07739657amr2674447pll.50.1686436351820; Sat, 10 Jun 2023 15:32:31 -0700 (PDT) Received: from dread.disaster.area (pa49-179-79-151.pa.nsw.optusnet.com.au. [49.179.79.151]) by smtp.gmail.com with ESMTPSA id k1-20020a170902694100b00199203a4fa3sm5417299plt.203.2023.06.10.15.32.30 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Sat, 10 Jun 2023 15:32:31 -0700 (PDT) Received: from dave by dread.disaster.area with local (Exim 4.96) (envelope-from ) id 1q878G-00AFdI-1H; Sun, 11 Jun 2023 08:32:28 +1000 Date: Sun, 11 Jun 2023 08:32:28 +1000 From: Dave Chinner To: Christian Brauner Cc: Amir Goldstein , Miklos Szeredi , linux-fsdevel@vger.kernel.org, linux-unionfs@vger.kernel.org Subject: Re: [PATCH] ovl: port to new mount api Message-ID: References: <20230605-fs-overlayfs-mount_api-v1-1-a8d78c3fbeaf@kernel.org> <20230609-hufen-zensor-490247280b6c@brauner> <20230609-tasten-raumfahrt-7b8a499ef787@brauner> MIME-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Disposition: inline Content-Transfer-Encoding: 8bit In-Reply-To: <20230609-tasten-raumfahrt-7b8a499ef787@brauner> Precedence: bulk List-ID: X-Mailing-List: linux-unionfs@vger.kernel.org On Fri, Jun 09, 2023 at 10:03:07AM +0200, Christian Brauner wrote: > On Fri, Jun 09, 2023 at 10:38:15AM +0300, Amir Goldstein wrote: > > On Fri, Jun 9, 2023 at 10:28 AM Christian Brauner wrote: > > > On Thu, Jun 08, 2023 at 09:29:39PM +0300, Amir Goldstein wrote: > > > > > fs/overlayfs/super.c | 896 ++++++++++++++++++++++++++++++++------------------- > > > > > 1 file changed, 568 insertions(+), 328 deletions(-) > > > > > > > > > > > > > Very big patch - Not so easy to review. > > > > It feels like a large refactoring mixed with the api change. > > > > Can it easily be split to just the refactoring patch > > > > and the api change patch? or any other split that will be > > > > easier to review. > > > > > > I don't really think so because you can't do a piecemeal conversion > > > unfortunately. But if you have concrete ideas I'm happy to hear them. > > > > > > > To me it looks like besides using new api you changed the order > > of config parsing to: > > - fill ovl_config and sanitize path arguments > > (replacing string with path in case of upper/workdir) > > Afaict this only makes sense if you cane have a sensible split between > option parsing and superblock creation. While the new mount api does > have that the old one doesn't. So ovl_fill_super() does option parsing, > verification, and superblock creation. This is the same problem we had with XFS and ext4 conversions, and to a lesser extent all the other simpler ones that have been done. There is no real intermediate step - the change from old parser to new parser as an atomic unit of change comes down to a single large patch. Yes, we whittled away at the edges to make it a bit smaller, but in the end the core API changeover was still a single large patch... > So I honestly thing this might end up being churn for churn. But I'll do > it if you insist. Yup, based on the experience of converting other filesytsems, I think it is largely a waste of time and effort to try to significantly rework this into smaller chunks. The bulk of the work has been done, reworking individual patches doesn't change the end result, so just review it as is, merge it and move on with more important stuff.... Cheers, Dave. -- Dave Chinner david@fromorbit.com