On Wed, 25 Nov 2020, Matthew Wilcox wrote: > On Tue, Nov 17, 2020 at 11:43:02PM +0000, Matthew Wilcox wrote: > > On Tue, Nov 17, 2020 at 07:15:13PM +0000, Matthew Wilcox wrote: > > > I find both of these functions exceptionally confusing. Does this > > > make it easier to understand? > > > > Never mind, this is buggy. I'll send something better tomorrow. > > That took a week, not a day. *sigh*. At least this is shorter. Thanks, I'll give it a try (along with the other 4, on top of the 12: maybe on -rc5, maybe on today's mmotm, I'll decide that later). Shorter you say, that's good: I was disheartened by the way it got more complicated, after your initial truncate_inode_partial_page() neatness. Any hints on what was wrong with my simple fixup to that? (But I didn't spend any more time trying to prove or disprove it.) Hugh