Re: [PATCH v5 04/14] mm/gup: check for isolation errors

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

To: Jason Gunthorpe <jgg@xxxxxxxx>
Subject: Re: [PATCH v5 04/14] mm/gup: check for isolation errors
From: Pavel Tatashin <pasha.tatashin@xxxxxxxxxx>
Date: Tue, 19 Jan 2021 13:27:45 -0500
Cc: LKML <linux-kernel@xxxxxxxxxxxxxxx>, linux-mm <linux-mm@xxxxxxxxx>, Andrew Morton <akpm@xxxxxxxxxxxxxxxxxxxx>, Vlastimil Babka <vbabka@xxxxxxx>, Michal Hocko <mhocko@xxxxxxxx>, David Hildenbrand <david@xxxxxxxxxx>, Oscar Salvador <osalvador@xxxxxxx>, Dan Williams <dan.j.williams@xxxxxxxxx>, Sasha Levin <sashal@xxxxxxxxxx>, Tyler Hicks <tyhicks@xxxxxxxxxxxxxxxxxxx>, Joonsoo Kim <iamjoonsoo.kim@xxxxxxx>, mike.kravetz@xxxxxxxxxx, Steven Rostedt <rostedt@xxxxxxxxxxx>, Ingo Molnar <mingo@xxxxxxxxxx>, Peter Zijlstra <peterz@xxxxxxxxxxxxx>, Mel Gorman <mgorman@xxxxxxx>, Matthew Wilcox <willy@xxxxxxxxxxxxx>, David Rientjes <rientjes@xxxxxxxxxx>, John Hubbard <jhubbard@xxxxxxxxxx>, Linux Doc Mailing List <linux-doc@xxxxxxxxxxxxxxx>, Ira Weiny <ira.weiny@xxxxxxxxx>, linux-kselftest@xxxxxxxxxxxxxxx
In-reply-to: <20210119182200.GZ4605@ziepe.ca>

> This should have a fixme too, this is a bug.

OK

>
> The patch looks OK, but I keep feeling this logic is all really
> overcomplicated...

I agree, I have a simplification patch for this logic, check out patch 12/14.

>
> Reviewed-by: Jason Gunthorpe <jgg@xxxxxxxxxx>

Thank you for your review.

Pasha

References:
- [PATCH v5 00/14] prohibit pinning pages in ZONE_MOVABLE
  - From: Pavel Tatashin
- [PATCH v5 04/14] mm/gup: check for isolation errors
  - From: Pavel Tatashin
- Re: [PATCH v5 04/14] mm/gup: check for isolation errors
  - From: Jason Gunthorpe

Prev by Date: Re: [PATCH v5 08/14] mm/gup: do not allow zero page for pinned pages
Next by Date: Re: [PATCH v5 02/14] mm/gup: check every subpage of a compound page during isolation
Previous by thread: Re: [PATCH v5 04/14] mm/gup: check for isolation errors
Next by thread: [PATCH v5 08/14] mm/gup: do not allow zero page for pinned pages
Index(es):
- Date
- Thread

[Index of Archives] [Linux Wireless] [Linux Kernel] [ATH6KL] [Linux Bluetooth] [Linux Netdev] [Kernel Newbies] [Share Photos] [IDE] [Security] [Git] [Netfilter] [Bugtraq] [Yosemite News] [MIPS Linux] [ARM Linux] [Linux Security] [Linux RAID] [Linux ATA RAID] [Samba] [Device Mapper]