On Wed, Feb 10, 2021 at 12:55:02PM +0000, Christoph Hellwig wrote: > On Tue, Feb 09, 2021 at 10:22:17PM -0800, [email protected] wrote: > > From: Ira Weiny <[email protected]> > > > > 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] And I thought it was a good idea. Any file system development should have tests with DEBUG_VM which should cover Matthew's concern while not having the overhead in production. Seemed like a decent compromise? Ira [1] https://lore.kernel.org/lkml/[email protected]/ > > Otherwise looks good: > > Reviewed-by: Christoph Hellwig <[email protected]>
