On Tue, May 15, 2018 at 11:27:25AM +0200, Vlastimil Babka wrote: > On 05/04/2018 08:33 PM, Matthew Wilcox wrote: > > From: Matthew Wilcox <mawilcox@xxxxxxxxxxxxx> > > > > x86 overloads the page->index field to store a pointer to the mm_struct. > > Maybe start the sentence with "For page table pages, ..." or "For pgd > page table pages, ..." ? Thanks, done. > > - unsigned long _pt_pad_3; > > + struct mm_struct *pt_mm; > > Add comment that it's x86-only so somebody doesn't try to write a > generic code expecting it? Done. My plan is to actually make this true for all page table pages so that we can always track a page table back to its owner, but that's not part of this patch set, and we can remove the comment when that changes. That's part of the generic infrastructure we need to be able to survive an uncorrectable error in a process's page table.