Date:   Thu, 16 Mar 2023 09:16:34 -0700
From:   Ira Weiny <ira.weiny@intel.com>
To:     Michael Ellerman <mpe@ellerman.id.au>,
        Ira Weiny <ira.weiny@intel.com>,
        Dave Hansen <dave.hansen@intel.com>,
        Dave Hansen <dave.hansen@linux.intel.com>,
        Thomas Gleixner <tglx@linutronix.de>,
        Nicholas Piggin <npiggin@gmail.com>,
        Christophe Leroy <christophe.leroy@csgroup.eu>,
        Catalin Marinas <catalin.marinas@arm.com>,
        Will Deacon <will@kernel.org>, Ingo Molnar <mingo@redhat.com>,
        "Borislav Petkov" <bp@alien8.de>, <x86@kernel.org>,
        "H. Peter Anvin" <hpa@zytor.com>
CC:     Dan Williams <dan.j.williams@intel.com>,
        Al Viro <viro@zeniv.linux.org.uk>,
        <linux-kernel@vger.kernel.org>, <linuxppc-dev@lists.ozlabs.org>,
        <linux-arm-kernel@lists.infradead.org>,
        Konstantin Ryabitsev <konstantin@linuxfoundation.org>
Subject: Re: [PATCH 0/3] COVER: Remove memcpy_page_flushcache()
Message-ID: <641340e2998b4_2695182944f@iweiny-mobl.notmuch>
References: <20221230-kmap-x86-v1-0-15f1ecccab50@intel.com>
 <3523ddf9-03f5-3179-9f39-cec09f79aa97@intel.com>
 <64126d113d163_2595222942@iweiny-mobl.notmuch>
 <87lejxmax8.fsf@mpe.ellerman.id.au>
Content-Type: text/plain; charset="us-ascii"
Content-Disposition: inline
In-Reply-To: <87lejxmax8.fsf@mpe.ellerman.id.au>
MIME-Version: 1.0
X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1
X-MS-Exchange-AntiSpam-MessageData-0: =?us-ascii?Q?aV7NwIzpRQN2a3aUiZhR2O3HiK/i93JjbHlhPCbAiaIux7nmXErxNn1GwYIz?=
 =?us-ascii?Q?fPuF3iCHZzeTD7GrD0/jrYU/XO+dhK14Ujy6nd4HI3WD/gO3anwueUGwcjQl?=
 =?us-ascii?Q?mbkxquvoY3gmrrJK9tINg3eySwgjzaTVX2AiKmP6fH39Mk2Sx9FkFBx6UpO4?=
 =?us-ascii?Q?EKuMDLrfYQeu1y7X6x4eb62yTDLg/lVRoTHRaKAd9BvGsA7M+rYCmFZ3UGDK?=
 =?us-ascii?Q?kZRRmyyyQwHTHrdS1WgrOgkamQBdKdEQd2M2+TFfT0qHzxkHcNeaQHPg4Qag?=
 =?us-ascii?Q?8gO0G1YgjUTfcRlQzJB+/uaUWhBcGYcoVztH2eiLhymw8M+4nag2fB17i+4t?=
 =?us-ascii?Q?P4X4lsk0UQpPafku/G5s28HBQ+GaHCw24kBRtJFPFBlWeRUm1mLwP+3s8jZ3?=
 =?us-ascii?Q?E5CUajirkroBsabGRuPvacO8Ucp1Ff69weA/5ZLU6D+gJXZYdKEivZ8CPcyx?=
 =?us-ascii?Q?aY00egU35/b1nHKDxnW5LXSMIhR3gdtVWDsCRYj0W5W8zT+W5kUheq0FoUxi?=
 =?us-ascii?Q?FQyu1vVLuCj3sAeThxTsujDPnH+kdzM1a76DrUOa+HwsiTvs7jla5+F+hfIx?=
 =?us-ascii?Q?g0iopQPh84AQlC6tBA08B90oa3o0ADWPfvJvr+FU8glb/4ETG9a1HrTySfwC?=
 =?us-ascii?Q?npN6/s9aUsKOE7YcUcPad8GJ1I6ZQE/4/dEaZf03um8BksX/b7Lvif+xE8wl?=
 =?us-ascii?Q?/LFBbJqvLeTIfODzpqU3Liy/nsqccBoZc0FQPZKaby+XJycmGJ4KATkSGDOQ?=
 =?us-ascii?Q?7kc2kXzNPDUwRfP5EAtdkJRUiQrFX+Zw+bf+BqponCN6v8/IdEt6CSq+uXyr?=
 =?us-ascii?Q?klDG2Iib55p96f4od7DkpRvC/pSCbP/7hFlDMOwTaX4wc8hGWdMJNZeL51Jb?=
 =?us-ascii?Q?bVcvZCCdG+5iNv5dHwkP0phbzcdHGV04UtAk+1Q/Vk2a3GYoZQQyOA1W+xJ1?=
 =?us-ascii?Q?NHGVQrm9RZSSx2K/RPpo3OjgLZb07DXynEBH0DB8FBsxiqjwFKDEv8qX8l/+?=
 =?us-ascii?Q?/ZBFr7TI/9fgOczKZxGItXII9yjTUaHE4hDV226GtZ9BYbe38Mg0yj2v2m4x?=
 =?us-ascii?Q?VOOoTFdAs+lZ248lPl9gcY+BWT5CO1+6wv4bUw/igqHYTpJHor6UjwoC2cOa?=
 =?us-ascii?Q?Lmr1sXNwJ7EZ3HtB6S4Pc/Q5bPVjGH1SKQZSFmxbQBIDYV/t9tOdZEgFc9kQ?=
 =?us-ascii?Q?XksSOMo/Ujqjh4gz13riN5ZilJFUI+QtRohoHycvFe3zQd+USvLu1eh3Hrmr?=
 =?us-ascii?Q?WuXN9GCl85qPybmcZ2CkTJDkG3SF5AJMG0aAItmDsNPFioGy07KATMVXWcl5?=
 =?us-ascii?Q?mDUXjxOiVoeq8heB70aFSCOxgCgjCRecH5oO29OxL4c63jVZZGmXgk6yxR/6?=
 =?us-ascii?Q?oVgp7YY+H9eIRtBhaRaDBdjzEwbWxWkHYDuKRDSdwPfupeFq0bqhDCxrVMTp?=
 =?us-ascii?Q?L0OZHcYK0X70n/iUT/gJ84gs8GDEF9WlaUjOdfzUh1k/5SoRD4mUvnpr8SLW?=
 =?us-ascii?Q?I/uzQqJNCEd+wZUtaqlsUI07py25Hu8KdvYAltzZWSXCA4YqvChQShhUr3St?=
 =?us-ascii?Q?/vHXwO16FMzBv8fyObe28KSJ2T1+5mdYML2AxQf6edJZ93ny7UJtX80GQ7ng?=
 =?us-ascii?Q?dA=3D=3D?=
X-MS-Exchange-CrossTenant-Network-Message-Id: 38d322cf-15cb-45bb-88ed-08db2639d27e
X-MS-Exchange-CrossTenant-AuthSource: SA1PR11MB6733.namprd11.prod.outlook.com
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-OriginalArrivalTime: 16 Mar 2023 16:16:39.1587
 (UTC)
X-MS-Exchange-CrossTenant-FromEntityHeader: Hosted
X-MS-Exchange-CrossTenant-Id: 46c98d88-e344-4ed4-8496-4ed7712e255d
X-MS-Exchange-CrossTenant-MailboxType: HOSTED
X-MS-Exchange-CrossTenant-UserPrincipalName: NccUQ7rVBktIjcahhsasBZjwq/fEBYbWljjl4Z7/7Sp/QfTxsW4qufYorlLKnXm75ucz9gY2UC4NRyQlec0cLQ==
X-MS-Exchange-Transport-CrossTenantHeadersStamped: DS0PR11MB7532
X-OriginatorOrg: intel.com
Precedence: bulk
List-ID: <linux-kernel.vger.kernel.org>
X-Mailing-List: linux-kernel@vger.kernel.org

+ Konstantin

Michael Ellerman wrote:
> Ira Weiny <ira.weiny@intel.com> writes:
> > Dave Hansen wrote:
> >> On 3/15/23 16:20, Ira Weiny wrote:
> >> > Commit 21b56c847753 ("iov_iter: get rid of separate bvec and xarray 
> >> > callbacks") removed the calls to memcpy_page_flushcache().
> >> > 
> >> > kmap_atomic() is deprecated and used in the x86 version of
> >> > memcpy_page_flushcache().
> >> > 
> >> > Remove the unnecessary memcpy_page_flushcache() call from all arch's.
> >> 
> >> Hi Ira,
> >> 
> >> Since the common code user is already gone these three patches seem
> >> quite independent.  It seems like the right thing to do is have
> >> individual arch maintainers cherry pick their arch patch and carry it
> >> independently.
> >
> > Yes.
> >
> >> 
> >> Is there a compelling reason to have someone pick up and carry these all
> >> together that I'm missing?
> >
> > No reason.  Would you like me to submit them individually?
> 
> I'll just grab the powerpc one from the thread, no need to resend.

Thanks.

> 
> > Sorry, submitting them separately crossed my mind when I wrote them but I
> > kind of forgot as they were all on the same branch and I was waiting for
> > after the merge window to submit them.
> 
> It's also much easier to run git-send-email HEAD^^^, rather than running
> it three separate times, let alone if it's a 20 patch series.

Exactly.  And I'm using b4 which would have forced me to create a separate
branch for each of the patches to track.  So I was keeping them around in
a single branch to let 0day run after the merge window.  Then I forgot
about the idea of splitting them because b4 had it all packaged up nice!

> 
> I wonder if we could come up with some convention to indicate that a
> series is made up of independent patches, and maintainers are free to
> pick them individually - but still sent as a single series.

Maybe.  But perhaps b4 could have a send option which would split them
out?  I'll see about adding an option to b4 but I've Cc'ed Konstantin as
well for the idea.

Thanks for picking this up!
Ira

> 
> cheers