Date:   Wed, 15 Mar 2023 18:12:49 -0700
From:   Ira Weiny <ira.weiny@intel.com>
To:     Dave Hansen <dave.hansen@intel.com>,
        Ira Weiny <ira.weiny@intel.com>,
        Dave Hansen <dave.hansen@linux.intel.com>,
        Thomas Gleixner <tglx@linutronix.de>,
        Michael Ellerman <mpe@ellerman.id.au>,
        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>
Subject: Re: [PATCH 0/3] COVER: Remove memcpy_page_flushcache()
Message-ID: <64126d113d163_2595222942@iweiny-mobl.notmuch>
References: <20221230-kmap-x86-v1-0-15f1ecccab50@intel.com>
 <3523ddf9-03f5-3179-9f39-cec09f79aa97@intel.com>
Content-Type: text/plain; charset="us-ascii"
Content-Disposition: inline
In-Reply-To: <3523ddf9-03f5-3179-9f39-cec09f79aa97@intel.com>
MIME-Version: 1.0
X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1
X-MS-Exchange-AntiSpam-MessageData-0: =?us-ascii?Q?fwavKr2cHna/+qIPXz2NAFqBXVDcfkOl1nT6WQrgZCn/1GYmsK4rilLZhhXC?=
 =?us-ascii?Q?X6BMyd+1+TLXCERNODchmmBGKzyFF3L9rUKsriFTylM++XnybTWJqRqRKhvW?=
 =?us-ascii?Q?TRaoR1oEa94eg2haAQM82fHDQUKinT6u80lReRYGxzNZvlBzGxlInRpjnmRT?=
 =?us-ascii?Q?E17btHuSE0Cr9KA+OppsRK9GnmsG7SOmf6jGTi//jXNvNbXGM/dUnVJ6YBYr?=
 =?us-ascii?Q?JsdZ1xdnu1IMxoa8e4ql2AHCkDr5sr4lseo6BY1nLf8CJXg2qHYYkUoJ5lsP?=
 =?us-ascii?Q?d0eaxot33HkJXROY7tr4eKNQZhDulZttaa/2c7H3P/1kMaQNxq0PcgdBCGsl?=
 =?us-ascii?Q?N79sY6aiJLH9p1yivbONmJCWy0Pcu90VShdTycDTRm+5cZqE5YActA2szbfg?=
 =?us-ascii?Q?kZHtS+TgMJ1qEFtjPVHYtZcD6biN40Ht6w9yQ3mP/Ov0FbYFkgctwvCNU3gp?=
 =?us-ascii?Q?BJTMwZB5dXVJgke5CqadCGJGcjafdQowRDS32eoeOFFlG9dMVadz/VWbp1pf?=
 =?us-ascii?Q?+W3exSI9VmyHZuI4rugXav4SGkhHaHxIOwgkqzr9g5tXvYfVUGqrsriQgPnv?=
 =?us-ascii?Q?iPW/1WKnNzP1QoqrwzagtQ0PeapbGqOaHK+gdhZqsqvs3lmdu4R69edE5sQa?=
 =?us-ascii?Q?g06bpGZ2BFusKvvLddI9brYMm9MkMR0Yxf0NOAjI4WiBn22ZLJQWKxBpKSCQ?=
 =?us-ascii?Q?UFC7Q8Ddxf1w0+RZD+wFHzKqJvKu0mQubvgE5XTkLTCfeI9pTzWAeFDPdoMW?=
 =?us-ascii?Q?67lNQOdPTZG8Yv3Mz/N8xpe76V7Hs6dyA/sKmiI5+sfNah5VymjDqsE/2JEs?=
 =?us-ascii?Q?xIOPPSJAAcYy3TkGcElOUzgj6ihM115PfwIASpWxqhb+j36ZiGA8ISY8ysHg?=
 =?us-ascii?Q?YG3O4G3EtUTGhDbmJ8Ohro5CdVKgZXBYHZvuHXxykmXMlZiTHTuP1UTXzoQ/?=
 =?us-ascii?Q?RVV3CqWaxDX7JQ0KnqFMiQ56sQIJDLBN6GQf9s76xjSg31efdrnlm2gxBXIh?=
 =?us-ascii?Q?1wuf5NmDSy9dDHI0p89A3wqYfvorM1PtdJ3OmhYZo7wllFaFDIpMf/nss2/j?=
 =?us-ascii?Q?crL0yaFPXKsJ0Apncwb8EdRD1MgdX/n+vk2dX2FO4y3NVhKAUsThrnAQN5hg?=
 =?us-ascii?Q?41ICUUcDLbEu4vZ/NCM08oV9TsDjMdpxnoWlp3Og2L/J77Q107j+meCqvi/1?=
 =?us-ascii?Q?iDOOuki7kMNzJLTfLfco2RdEaPCHN1yc1mQX6vpQ4NgHDPLwgfAIkHaO32aJ?=
 =?us-ascii?Q?Ict7piVjhpmL20MjigpmMNAw5lf5Ey/QK0X2Dc5THcijYs3E8v/V2m+dpoJe?=
 =?us-ascii?Q?06BX+aojtZCWLytryG+av+uSYToSPAM1LcGxPfZTfFPzv6HsOnaCJtfYRB9j?=
 =?us-ascii?Q?kZGmNhTPhSu8/7DUK3n1Wm2HViEGjlrhKOoXCPuEw0xPSeqc3BL4umQc0DlK?=
 =?us-ascii?Q?to2346xBUfvGgRAFpAzMcjUcDDZlna6YErkcg9NGT9ng3BM0Y7P2IhS5Tcij?=
 =?us-ascii?Q?iju1QhTMek0xNoJ9OLIA8K02RFLw+A6j5vcdDus7UK5xO5k2l5h/AbajQXDi?=
 =?us-ascii?Q?+FbBTAKzIQDALTOikQUAL2gX9kW+jiSDhvGSwFtiti/xp/pBQAkUkZUTzorx?=
 =?us-ascii?Q?yw=3D=3D?=
X-MS-Exchange-CrossTenant-Network-Message-Id: 1f510235-82d0-45a1-85ac-08db25bb91bb
X-MS-Exchange-CrossTenant-AuthSource: SA1PR11MB6733.namprd11.prod.outlook.com
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-OriginalArrivalTime: 16 Mar 2023 01:12:53.8873
 (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: rAsFVUmb1GRwr1E0btoxQoLHxEaoHzOSswzLdAVR/ki8hJDs/927kmFaIZF8HhG33CsKwBkMo9S+BdgDB5LTXg==
X-MS-Exchange-Transport-CrossTenantHeadersStamped: PH0PR11MB7165
X-OriginatorOrg: intel.com
Precedence: bulk
List-ID: <linux-kernel.vger.kernel.org>
X-Mailing-List: linux-kernel@vger.kernel.org

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?

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.

Ira