On Mon, Jan 25, 2021 at 9:28 AM Jason Gunthorpe jgg@ziepe.ca wrote:
On Wed, Jan 20, 2021 at 09:26:41AM -0500, Pavel Tatashin wrote:
I thought about this, and it would code a little cleaner. But, the reason I did not is because zero_page is perfectly pinnable, it is not pinnable only when it is in a movable zone (and it should not be in a movable zones for other reasons as well), but that is another bug that needs to be resolved, and once that bug is resolved this condition can be removed from gup migration.
My point is you've defined the zero page to be pinnable no matter what zone it is in, so is_pinnable(zero_page) == true
Sure, I will move it inside is_pinnable in the next update.
Thank you, Pasha
Jason