From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from smtp.kernel.org (aws-us-west-2-korg-mail-1.web.codeaurora.org [10.30.226.201]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id E7148B673; Thu, 22 Aug 2024 02:00:25 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=10.30.226.201 ARC-Seal:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1724292027; cv=none; b=PZN6KvH21uqEAOy4xOFaPJS5SpJZAaiCcFEjreObIATChhnvbuLX86rOULfFiJzpNI52dlPi8bb8QJBcuzZFrFLpKtUzn1/9w8fj+wto2OWEIQHASMxFpNAF1OpQEvqy4ueiEIaB/uA83ZHN+oIygOQMJyfV7HwrVFbcHcoH1dg= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1724292027; c=relaxed/simple; bh=uqUsAf8+/12eNEYHCiPfQspPWgo8sfX5Z0Y3h0XmXQo=; h=Date:From:To:Cc:Subject:Message-ID:References:MIME-Version: Content-Type:Content-Disposition:In-Reply-To; b=sei9ImxNaPDuK311iseGnK5FEfkexr250GCT1HuPVpXTeBZEVHD8nTOMLTOqKtgL2WkFGZyk79teWExuSxyMLhoFY9C+ZD3XtY1sypXBi2XQVKr9zugW//Hb9CBRaUxuZwjvWwmYM2mHo2s80nzrLSUx6RzFfQLwLA3ERrPy0JI= ARC-Authentication-Results:i=1; smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=kernel.org header.i=@kernel.org header.b=OO2CMoNb; arc=none smtp.client-ip=10.30.226.201 Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=kernel.org header.i=@kernel.org header.b="OO2CMoNb" Received: by smtp.kernel.org (Postfix) with ESMTPSA id 20A1AC4AF0E; Thu, 22 Aug 2024 02:00:25 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=kernel.org; s=k20201202; t=1724292025; bh=uqUsAf8+/12eNEYHCiPfQspPWgo8sfX5Z0Y3h0XmXQo=; h=Date:From:To:Cc:Subject:References:In-Reply-To:From; b=OO2CMoNby9sNRrhyEh/LlMnHtHvkqLN7GQn6sMcVQMtWVJeMfn2Vnb/EEoI3TBcnQ CSjl9tT87onJZhacW7qkLFs6rZFpKTRMNdrpyiv2E97L6WOFvFoJIckA6+7j58w60h HmGvA0KyYXYRzrdCqm4yBaLALuYdNSmwAndcUMw3xNm1RIllVsEl/xGglIkNnZkXFw T4uMRSiaBDeFZ0qp1wtc5VdY2XFF+EK11/TUCm2KoTeL9guTJoekpeymYJvEtYYvcR 3r2+n49yhVOO30tA6VxdraR6FJ8Y82Z6TfhlVe31AkRjHMRZ8H2XBs+1etfwu9xfpN kVweVlQtAQQAg== Date: Wed, 21 Aug 2024 22:00:24 -0400 From: Mike Snitzer To: Jeff Layton Cc: linux-nfs@vger.kernel.org, Chuck Lever , Anna Schumaker , Trond Myklebust , NeilBrown , linux-fsdevel@vger.kernel.org Subject: Re: [PATCH v12 00/24] nfs/nfsd: add support for localio Message-ID: References: <20240819181750.70570-1-snitzer@kernel.org> <9dc7ec9b3d8a9a722046be2626b2d05fa714c8e6.camel@kernel.org> Precedence: bulk X-Mailing-List: linux-nfs@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: <9dc7ec9b3d8a9a722046be2626b2d05fa714c8e6.camel@kernel.org> Hey Jeff, On Wed, Aug 21, 2024 at 03:20:55PM -0400, Jeff Layton wrote: > > This looks much improved. I didn't see anything that stood out at me as > being problematic code-wise with the design or final product, aside > from a couple of minor things. BTW, thanks for this feedback, much appreciated! > But...this patchset is hard to review. My main gripe is that there is a > lot of "churn" -- places where you add code, just to rework it in a new > way in a later patch. > > For instance, the nfsd_file conversion should be integrated into the > new infrastructure much earlier instead of having a patch that later > does that conversion. Those kinds extraneous changes make this much > harder to review than it would be if this were done in a way that > avoided that churn. I think I've addressed all your v12 review comments from earlier today. I've pushed the new series out to my git repo here: https://git.kernel.org/pub/scm/linux/kernel/git/snitzer/linux.git/log/?h=nfs-localio-for-next No code changes, purely a bunch of rebasing to clean up like you suggested. Only outstanding thing is the nfsd tracepoints handling of NULL rqstp (would like to get Chuck's expert feedback on that point). Please feel free to have a look at my branch while I wait for any other v12 feedback from Chuck and/or others before I send out v13. I'd like to avoid spamming the list like I did in the past ;) Thanks, Mike