Messages in this thread | | | Date | Tue, 24 Nov 2020 18:50:32 -0800 (PST) | From | Hugh Dickins <> | Subject | Re: [PATCH v4 00/16] Overhaul multi-page lookups for THP |
| |
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
| |