On Thu, Jul 09, 2020 at 09:21:11PM +0100, Matthew Wilcox (Oracle) wrote:
> Thanks to everyone for your feedback on v1. I haven't added any acks
> from v1 because most of the patches have changed substantially from v1.
Aside from my aesthetic preference to have printk continuation line
below the opening '(' rather then full tab indented the patches look
good.
Acked-by: Mike Rapoport <rppt@xxxxxxxxxxxxx>
> Here's a sample dump of a pagecache tail page with all of the patches applied:
>
> page:000000006d1c49ca refcount:6 mapcount:0 mapping:00000000136b8d90 index:0x109 pfn:0x6c645
> head:000000008bd38076 order:2 compound_mapcount:0 compound_pincount:0
> aops:xfs_address_space_operations ino:800042 dentry name:"fd"
> flags: 0x4000000000012014(uptodate|lru|private|head)
> raw: 4000000000000000 ffffd46ac1b19101 ffffffff00000202 dead000000000004
> raw: 0000000000000001 0000000000000000 00000000ffffffff 0000000000000000
> head: 4000000000012014 ffffd46ac1b1bbc8 ffffd46ac1b1bc08 ffff91976f659560
> head: 0000000000000108 ffff919773220680 00000006ffffffff 0000000000000000
> page dumped because: testing
>
> Matthew Wilcox (Oracle) (6):
> mm: Handle page->mapping better in dump_page
> mm: Dump compound page information on a second line
> mm: Print head flags in dump_page
> mm: Switch dump_page to get_kernel_nofault
> mm: Print the inode number in dump_page
> mm: Print hashed address of struct page
>
> mm/debug.c | 75 +++++++++++++++++++++++++-----------------------------
> 1 file changed, 35 insertions(+), 40 deletions(-)
>
> --
> 2.27.0
>
--
Sincerely yours,
Mike.
