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=-10.2 required=3.0 tests=BAYES_00, HEADER_FROM_DIFFERENT_DOMAINS,MAILING_LIST_MULTI,MENTIONS_GIT_HOSTING, SPF_HELO_NONE,SPF_PASS,URIBL_BLOCKED,USER_AGENT_SANE_1 autolearn=ham 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 DBBA5C433DB for ; Fri, 26 Feb 2021 17:03:00 +0000 (UTC) Received: from kanga.kvack.org (kanga.kvack.org [205.233.56.17]) by mail.kernel.org (Postfix) with ESMTP id 5D3C164EC4 for ; Fri, 26 Feb 2021 17:03:00 +0000 (UTC) DMARC-Filter: OpenDMARC Filter v1.3.2 mail.kernel.org 5D3C164EC4 Authentication-Results: mail.kernel.org; dmarc=fail (p=none dis=none) header.from=intel.com Authentication-Results: mail.kernel.org; spf=pass smtp.mailfrom=owner-linux-mm@kvack.org Received: by kanga.kvack.org (Postfix) id 940736B0005; Fri, 26 Feb 2021 12:02:59 -0500 (EST) Received: by kanga.kvack.org (Postfix, from userid 40) id 8F2756B0006; Fri, 26 Feb 2021 12:02:59 -0500 (EST) X-Delivered-To: int-list-linux-mm@kvack.org Received: by kanga.kvack.org (Postfix, from userid 63042) id 8087A6B006C; Fri, 26 Feb 2021 12:02:59 -0500 (EST) X-Delivered-To: linux-mm@kvack.org Received: from forelay.hostedemail.com (smtprelay0231.hostedemail.com [216.40.44.231]) by kanga.kvack.org (Postfix) with ESMTP id 69BB96B0005 for ; Fri, 26 Feb 2021 12:02:59 -0500 (EST) Received: from smtpin12.hostedemail.com (10.5.19.251.rfc1918.com [10.5.19.251]) by forelay02.hostedemail.com (Postfix) with ESMTP id 2D436D21F for ; Fri, 26 Feb 2021 17:02:59 +0000 (UTC) X-FDA: 77861038878.12.F43BC74 Received: from mga17.intel.com (mga17.intel.com [192.55.52.151]) by imf13.hostedemail.com (Postfix) with ESMTP id 999F8E00013D for ; Fri, 26 Feb 2021 17:02:53 +0000 (UTC) IronPort-SDR: KNtsege4Ml5nSpIKjQ3K3hN9hQSeRlSRuEJ7tIT0WP2Rl9ZrIWR+7lJ4gdNh+dLlhFKApIb+rN U/FiayFGcZmw== X-IronPort-AV: E=McAfee;i="6000,8403,9907"; a="165795021" X-IronPort-AV: E=Sophos;i="5.81,209,1610438400"; d="scan'208";a="165795021" Received: from orsmga001.jf.intel.com ([10.7.209.18]) by fmsmga107.fm.intel.com with ESMTP/TLS/ECDHE-RSA-AES256-GCM-SHA384; 26 Feb 2021 09:02:49 -0800 IronPort-SDR: ReI+MBbVQEBTQiTBgijphNyVsN9PMQvLjXWg9g0Fli0A+nknkgedIMoa7QP8sH8Exv9XWbxY9u JwuOWuWq3Zgw== X-IronPort-AV: E=Sophos;i="5.81,209,1610438400"; d="scan'208";a="443137334" Received: from iweiny-desk2.sc.intel.com (HELO localhost) ([10.3.52.147]) by orsmga001-auth.jf.intel.com with ESMTP/TLS/ECDHE-RSA-AES256-GCM-SHA384; 26 Feb 2021 09:02:48 -0800 Date: Fri, 26 Feb 2021 09:02:48 -0800 From: Ira Weiny To: dsterba@suse.cz, Linus Torvalds , David Sterba , Linux Kernel Mailing List , Linux-MM Subject: Re: [GIT PULL] Kmap conversions for 5.12 Message-ID: <20210226170248.GH3014244@iweiny-DESK2.sc.intel.com> References: <20210223192506.GY3014244@iweiny-DESK2.sc.intel.com> <20210224123049.GX1993@twin.jikos.cz> <20210224175912.GA3014244@iweiny-DESK2.sc.intel.com> <20210225131252.GA7604@suse.cz> <20210225163234.GD3014244@iweiny-DESK2.sc.intel.com> <20210226142339.GK7604@suse.cz> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20210226142339.GK7604@suse.cz> User-Agent: Mutt/1.11.1 (2018-12-01) X-Stat-Signature: nzb4g13sicxwsoagm3kfu7epag5si4yg X-Rspamd-Server: rspam05 X-Rspamd-Queue-Id: 999F8E00013D Received-SPF: none (intel.com>: No applicable sender policy available) receiver=imf13; identity=mailfrom; envelope-from=""; helo=mga17.intel.com; client-ip=192.55.52.151 X-HE-DKIM-Result: none/none X-HE-Tag: 1614358973-201936 X-Bogosity: Ham, tests=bogofilter, spamicity=0.000000, version=1.2.4 Sender: owner-linux-mm@kvack.org Precedence: bulk X-Loop: owner-majordomo@kvack.org List-ID: On Fri, Feb 26, 2021 at 03:23:40PM +0100, David Sterba wrote: > On Thu, Feb 25, 2021 at 08:32:34AM -0800, Ira Weiny wrote: > > On Thu, Feb 25, 2021 at 02:12:52PM +0100, David Sterba wrote: > > > On Wed, Feb 24, 2021 at 09:59:12AM -0800, Ira Weiny wrote: > > > > On Wed, Feb 24, 2021 at 01:30:49PM +0100, David Sterba wrote: > > > > > On Tue, Feb 23, 2021 at 11:25:06AM -0800, Ira Weiny wrote: > > > > > > On Tue, Feb 23, 2021 at 09:13:42AM -0800, Linus Torvalds wrote: > > > > > > > On Tue, Feb 23, 2021 at 7:03 AM David Sterba wrote: > > > > > [...] > > > > > > > > > > > Sorry. I will change it. > > > > > > > > > > Let me know how you want to proceed with the patchset/pull request. > > > > > > > > To be clear I'd like to just drop the 2 patches which use zero_user() for this > > > > merge window. > > > > > > > > I've already submitted some additional btrfs changes for 5.13[1]. I can rework > > > > these zero_user() patches and submit them through Andrew for 5.13 as separate > > > > set. That is what I meant by 'I will change it'. > > > > > > > > > I > > > > > can play the messenger again but now it seems a round of review is > > > > > needed and with some testing it'll be possible in some -rc. At that > > > > > point you may take the patches via the mm tree, unless Linus is ok with > > > > > a late pull. > > > > > > > > I'm ok with delaying the memzero_page() change to 5.13. There are a lot of > > > > kmap changes to come. But I'm trying to do them as smaller series just for > > > > this reason. I don't want valid changes to be denied due to my messing up just > > > > a few patches... :-( Hopefully you and Linus can forgive me on this one. > > > > > > > > Is ok to just drop them and merge the rest of this series in 5.12? > > > > > > Ok, no problem. Please let me know exactly which patches to drop, I'll > > > respin the branch. Thanks. > > > > Drop These 2: > > > > [PATCH V2 5/8] iov_iter: Remove memzero_page() in favor of zero_user() > > https://lore.kernel.org/lkml/20210210062221.3023586-6-ira.weiny@intel.com/ > > > > [PATCH V2 8/8] btrfs: convert to zero_user() > > https://lore.kernel.org/lkml/20210210062221.3023586-9-ira.weiny@intel.com/ > > > > > > Keep: > > > > [PATCH V2 1/8] mm/highmem: Lift memcpy_[to|from]_page to core > > [PATCH V2 2/8] mm/highmem: Convert memcpy_[to|from]_page() to kmap_local_page() > > [PATCH V2 3/8] mm/highmem: Introduce memcpy_page(), memmove_page(), and memset_page() > > [PATCH V2 4/8] mm/highmem: Add VM_BUG_ON() to mem*_page() calls > > ... > > [PATCH V2 6/8] btrfs: use memcpy_[to|from]_page() and kmap_local_page() > > [PATCH V2 7/8] btrfs: use copy_highpage() instead of 2 kmaps() > > ... > > > > I would resend but I'd rather keep the exact commits you had in your testing > > rather than potentially messing up the rebase this late. > > Got it, thanks. It's easier for me to delete the patches once I have > them in the branch, that's been updated and now pushed to kernel org > again (https://git.kernel.org/pub/scm/linux/kernel/git/kdave/linux.git/log/?h=kmap-conversion-for-5.12) Looks good. thank you. > > I'll add it to testing branches and let it test over the weekend, > sending the pull request next week. > Sounds like a good plan. Once again thank you for dealing with this. Sorry for the mix up, Ira