Re: [PATCH -mm] mm, pagemap: Fix soft dirty marking for PMD migration entry

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



"Kirill A. Shutemov" <kirill@xxxxxxxxxxxxx> writes:

> On Tue, Oct 17, 2017 at 04:18:18PM +0800, Huang, Ying wrote:
>> From: Huang Ying <ying.huang@xxxxxxxxx>
>> 
>> Now, when the page table is walked in the implementation of
>> /proc/<pid>/pagemap, pmd_soft_dirty() is used for both the PMD huge
>> page map and the PMD migration entries.  That is wrong,
>> pmd_swp_soft_dirty() should be used for the PMD migration entries
>> instead because the different page table entry flag is used.
>> 
>> Cc: Michal Hocko <mhocko@xxxxxxxx>
>> Cc: "Kirill A. Shutemov" <kirill.shutemov@xxxxxxxxxxxxxxx>
>> Cc: David Rientjes <rientjes@xxxxxxxxxx>
>> Cc: Arnd Bergmann <arnd@xxxxxxxx>
>> Cc: Hugh Dickins <hughd@xxxxxxxxxx>
>> Cc: "J.r.me Glisse" <jglisse@xxxxxxxxxx>
>> Cc: Daniel Colascione <dancol@xxxxxxxxxx>
>> Cc: Zi Yan <zi.yan@xxxxxxxxxxxxxx>
>> Cc: Naoya Horiguchi <n-horiguchi@xxxxxxxxxxxxx>
>> Signed-off-by: "Huang, Ying" <ying.huang@xxxxxxxxx>
>
> What is effect of the misbehaviour? pagemap reports garbage?

Yes.  pagemap may report incorrect soft dirty information for PMD
migration entries.

> Shoudn't it be in stable@? And maybe add Fixes: <sha1>.

Yes.  Will do that in the next version.

> Otherwise, looks good to me.
>
> Acked-by: Kirill A. Shutemov <kirill.shutemov@xxxxxxxxxxxxxxx>

Thanks!

Best Regards,
Huang, Ying

--
To unsubscribe, send a message with 'unsubscribe linux-mm' in
the body to majordomo@xxxxxxxxx.  For more info on Linux MM,
see: http://www.linux-mm.org/ .
Don't email: <a href=mailto:"dont@xxxxxxxxx";> email@xxxxxxxxx </a>



[Index of Archives]     [Linux ARM Kernel]     [Linux ARM]     [Linux Omap]     [Fedora ARM]     [IETF Annouce]     [Bugtraq]     [Linux OMAP]     [Linux MIPS]     [eCos]     [Asterisk Internet PBX]     [Linux API]
  Powered by Linux