Re: [PATCH] mm: make sure isolate_lru_page() is never called for tail page

[Vlastimil Babka <vbabka@suse.cz>]

On 12/16/2015 03:47 PM, Michal Hocko wrote:
> On Wed 16-12-15 16:29:30, Kirill A. Shutemov wrote:
>> The VM_BUG_ON_PAGE() would catch such cases if any still exists.
>
> Thanks, this better than a silent breakage.
>
>> Signed-off-by: Kirill A. Shutemov <kirill.shutemov@linux.intel.com>
>
> Acked-by: Michal Hocko <mhocko@suse.com>

Acked-by: Vlastimil Babka <vbabka@suse.cz>

>
>> ---
>>   mm/vmscan.c | 1 +
>>   1 file changed, 1 insertion(+)
>>
>> diff --git a/mm/vmscan.c b/mm/vmscan.c
>> index 964390906167..05dd182f04fd 100644
>> --- a/mm/vmscan.c
>> +++ b/mm/vmscan.c
>> @@ -1436,6 +1436,7 @@ int isolate_lru_page(struct page *page)
>>   	int ret = -EBUSY;
>>
>>   	VM_BUG_ON_PAGE(!page_count(page), page);
>> +	VM_BUG_ON_PAGE(PageTail(page), page);
>>
>>   	if (PageLRU(page)) {
>>   		struct zone *zone = page_zone(page);
>> --
>> 2.6.2
>

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