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 X-Spam-Level: X-Spam-Status: No, score=-11.3 required=3.0 tests=BAYES_00,DKIMWL_WL_HIGH, DKIM_SIGNED,DKIM_VALID,DKIM_VALID_AU,HEADER_FROM_DIFFERENT_DOMAINS, INCLUDES_CR_TRAILER,MAILING_LIST_MULTI,SPF_HELO_NONE,SPF_PASS,URIBL_BLOCKED autolearn=unavailable autolearn_force=no version=3.4.0 Received: from mail.kernel.org (mail.kernel.org [198.145.29.99]) by smtp.lore.kernel.org (Postfix) with ESMTP id 6B0AAC433E6 for ; Fri, 12 Feb 2021 16:44:16 +0000 (UTC) Received: from vger.kernel.org (vger.kernel.org [23.128.96.18]) by mail.kernel.org (Postfix) with ESMTP id 4316F64E8C for ; Fri, 12 Feb 2021 16:44:16 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S230196AbhBLQoA (ORCPT ); Fri, 12 Feb 2021 11:44:00 -0500 Received: from us-smtp-delivery-124.mimecast.com ([63.128.21.124]:53693 "EHLO us-smtp-delivery-124.mimecast.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S231245AbhBLQmq (ORCPT ); Fri, 12 Feb 2021 11:42:46 -0500 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=redhat.com; s=mimecast20190719; t=1613148078; h=from:from:reply-to:subject:subject:date:date:message-id:message-id: to:to:cc:cc:mime-version:mime-version:content-type:content-type: in-reply-to:in-reply-to:references:references; bh=ft1B0dmqfZyPUoyDvwnAfTze1mns3d3CFcFjBHQ5YEA=; b=aNRt9Ee3sV2xdj2/G38YJVresg4eVETnPdP9xDWtHOPJMqJe0efT11qDj6gc7WEQExSZGP N7/Y+Gsts8Ww7CUbqTDLJqyaxZ5vprO30+olfCn9kIjcjU4OnFYwHGeWme5ErQJOdiPEfT cJJ9GTyWMLNlBxzi8f30HU2JxnQJd9A= Received: from mail-ej1-f71.google.com (mail-ej1-f71.google.com [209.85.218.71]) (Using TLS) by relay.mimecast.com with ESMTP id us-mta-440-Jjju_bpYNU67422D9hKeuQ-1; Fri, 12 Feb 2021 11:41:16 -0500 X-MC-Unique: Jjju_bpYNU67422D9hKeuQ-1 Received: by mail-ej1-f71.google.com with SMTP id h24so156644ejl.16 for ; Fri, 12 Feb 2021 08:41:15 -0800 (PST) X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:mime-version:references:in-reply-to:from:date :message-id:subject:to:cc; bh=ft1B0dmqfZyPUoyDvwnAfTze1mns3d3CFcFjBHQ5YEA=; b=sZ4FRGWNDaF1RhNfA1iZ2l/uYyGBi3t60r7df3V0lW2Fw/W5xT4w+70zjpce657q19 7gTD0RRMNjNPJPHHc7Oo/VYhBoEXLYZB7DE1mTV4vsQXtaWVbJMx3jUfzzNgyXa5yURI q+vcSVuEVleYGXa8GRV1ZsdaMtIVzSKEEcSe9341DWM17+pPWhf5WGN6opUOIfYb8i7N 6chshoxML/gHxR1f2rQjGIP3BPz8Xr6dUv1rMjLzDNSr/j0O7S+OQnMhxk/GmQxBDYHY OSJUmkW8HdCneZyxz27UXp5fLKprFrUgU+phbaPpvXdQbZFsjax1PeokxqHjMr0vKIV4 Ap7Q== X-Gm-Message-State: AOAM533/wr9ReAWn5gFlwcwn42gFNzBibmTI7cA3WPnAUJS0h72kYxHR bv1v/iTZKjP/fwMNF5oksaxUB1siIC44aTqsnX1+dmcSDBOYr+vL5AIPGAkN+q+Zqw98LbTw3XP yY9QD6gtTmvMO+G600WuKTi5ZT8Wbd2FwZDWr/Ouj X-Received: by 2002:a17:906:1681:: with SMTP id s1mr3897284ejd.229.1613148074943; Fri, 12 Feb 2021 08:41:14 -0800 (PST) X-Google-Smtp-Source: ABdhPJwTIgnx7LJiTFOW+mhqlshoZww0YFi52MiJxBqcnsY+RGcoAoY9xA8AY8Yu0JSD5ZHRlga8wGOKDsNd1QerD6Y= X-Received: by 2002:a17:906:1681:: with SMTP id s1mr3897262ejd.229.1613148074745; Fri, 12 Feb 2021 08:41:14 -0800 (PST) MIME-Version: 1.0 References: <591237.1612886997@warthog.procyon.org.uk> <1330473.1612974547@warthog.procyon.org.uk> <1330751.1612974783@warthog.procyon.org.uk> <27816.1613085646@warthog.procyon.org.uk> In-Reply-To: <27816.1613085646@warthog.procyon.org.uk> From: David Wysochanski Date: Fri, 12 Feb 2021 11:40:38 -0500 Message-ID: Subject: Re: [GIT PULL] fscache: I/O API modernisation and netfs helper library To: David Howells Cc: Linus Torvalds , Matthew Wilcox , Jeff Layton , Anna Schumaker , Trond Myklebust , Steve French , Dominique Martinet , Alexander Viro , ceph-devel@vger.kernel.org, linux-afs@lists.infradead.org, linux-cachefs , CIFS , linux-fsdevel , "open list:NFS, SUNRPC, AND..." , v9fs-developer@lists.sourceforge.net, Linux Kernel Mailing List Content-Type: text/plain; charset="UTF-8" Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Thu, Feb 11, 2021 at 6:20 PM David Howells wrote: > > Linus Torvalds wrote: > > > Also, honestly, I really *REALLY* want your commit messages to talk > > about who has been cc'd, who has been part of development, and point > > to the PUBLIC MAILING LISTS WHERE THAT DISCUSSION WAS TAKING PLACE, so > > that I can actually see that "yes, other people were involved" > > Most of the development discussion took place on IRC and waving snippets of > code about in pastebin rather than email - the latency of email is just too > high. There's not a great deal I can do about that now as I haven't kept IRC > logs. I can do that in future if you want. > > > No, I don't require this in general, but exactly because of the > > history we have, I really really want to see that. I want to see a > > > > Link: https://lore.kernel.org/r/.... > > I can add links to where I've posted the stuff for review. Do you want this > on a per-patch basis or just in the cover for now? > > Also, do you want things like these: > > https://lore.kernel.org/linux-fsdevel/3326.1579019665@warthog.procyon.org.uk/ > https://lore.kernel.org/linux-fsdevel/4467.1579020509@warthog.procyon.org.uk/ > > which pertain to the overall fscache rewrite, but where the relevant changes > didn't end up included in this particular patchset? Or this: > > https://listman.redhat.com/archives/linux-cachefs/2020-December/msg00000.html > > where someone was testing the overall patchset of which this is a subset? > > > and the Cc's - or better yet, the Reviewed-by's etc - so that when I > > get a pull request, it really is very obvious to me when I look at it > > that others really have been involved. > > > > So if I continue to see just > > > > Signed-off-by: David Howells > > > > at the end of the commit messages, I will not pull. > > > > Yes, in this thread a couple of people have piped up and said that > > they were part of the discussion and that they are interested, but if > > I have to start asking around just to see that, then it's too little, > > too late. > > > > No more of this "it looks like David Howells did things in private". I > > want links I can follow to see the discussion, and I really want to > > see that others really have been involved. > > > > Ok? > > Sure. > > I can go and edit in link pointers into the existing patches if you want and > add Jeff's Review-and-tested-by into the appropriate ones. You would be able > to compare against the existing tag, so it wouldn't entirely invalidate the > testing. > You can add my Tested-by for your fscache-next branch series ending at commit 235299002012 netfs: Hold a ref on a page when PG_private_2 is set This series includes your commit c723f0232c9f8928b3b15786499637bda3121f41 discussed a little earlier in this email thread. I ran over 24 hours of NFS tests (unit, connectathon, xfstests, various servers and all NFS versions) on your latest series and it looks good. Note I did not run against pNFS servers due to known issue, and I did not do more advanced tests like error injections. I did get one OOM on xfstest generic/551 on one testbed, but that same' test passed on another testbed, so it's not clear what is happening there and it could very well be testbed or NFS related. In addition, I reviewed various patches in the series, especially the API portions of the netfs patches, so for those, Reviewed-by is appropriate as well. I have also reviewed some of the internals of the other infrastructure patches, but my review is more limited there. > Also, do you want links inserting into all the patches of the two keyrings > pull requests I've sent you? > > David >