Re: [PATCH V2 4/8] mm/highmem: Add VM_BUG_ON() to mem*_page() calls

From: Ira Weiny
Date: Wed Feb 10 2021 - 12:05:28 EST


On Wed, Feb 10, 2021 at 04:41:34PM +0000, Christoph Hellwig wrote:
> On Wed, Feb 10, 2021 at 08:29:01AM -0800, Ira Weiny wrote:
> > On Wed, Feb 10, 2021 at 12:55:02PM +0000, Christoph Hellwig wrote:
> > > On Tue, Feb 09, 2021 at 10:22:17PM -0800, ira.weiny@xxxxxxxxx wrote:
> > > > From: Ira Weiny <ira.weiny@xxxxxxxxx>
> > > >
> > > > Add VM_BUG_ON bounds checks to ensure the newly lifted and created page
> > > > memory operations do not result in corrupted data in neighbor pages and
> > > > to make them consistent with zero_user().[1][2]
> > >
> > > s/VM_BUG_ON/BUG_ON/g ?
> >
> > Andrew wanted VM_BUG_ON.[1]
>
> I don't care either way, but the patch uses BUG_ON, so the description
> should match.

Oh man... I changed the commit message after spliting the patch and forgot to
change the code... <doh>

Thanks,
Ira