linux-mm.kvack.org archive mirror
 help / color / mirror / Atom feed
From: Ira Weiny <ira.weiny@intel.com>
To: dsterba@suse.cz, Linus Torvalds <torvalds@linux-foundation.org>,
	David Sterba <dsterba@suse.com>,
	Linux Kernel Mailing List <linux-kernel@vger.kernel.org>,
	Linux-MM <linux-mm@kvack.org>
Subject: Re: [GIT PULL] Kmap conversions for 5.12
Date: Thu, 25 Feb 2021 08:32:34 -0800	[thread overview]
Message-ID: <20210225163234.GD3014244@iweiny-DESK2.sc.intel.com> (raw)
In-Reply-To: <20210225131252.GA7604@suse.cz>

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 <dsterba@suse.com> 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.

Thank you and sorry for the extra work,
Ira


  reply	other threads:[~2021-02-25 16:32 UTC|newest]

Thread overview: 10+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2021-02-23 15:01 [GIT PULL] Kmap conversions for 5.12 David Sterba
2021-02-23 17:13 ` Linus Torvalds
2021-02-23 19:25   ` Ira Weiny
2021-02-24 12:30     ` David Sterba
2021-02-24 17:59       ` Ira Weiny
2021-02-24 18:10         ` Linus Torvalds
2021-02-25 13:12         ` David Sterba
2021-02-25 16:32           ` Ira Weiny [this message]
2021-02-26 14:23             ` David Sterba
2021-02-26 17:02               ` Ira Weiny

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20210225163234.GD3014244@iweiny-DESK2.sc.intel.com \
    --to=ira.weiny@intel.com \
    --cc=dsterba@suse.com \
    --cc=dsterba@suse.cz \
    --cc=linux-kernel@vger.kernel.org \
    --cc=linux-mm@kvack.org \
    --cc=torvalds@linux-foundation.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).