Re: [PATCH] iommu/amd: Take mmap_sem when calling get_user_pages

From: Suravee Suthikulanit
Date: Tue May 06 2014 - 15:27:31 EST


On 5/6/2014 2:11 PM, Suravee Suthikulanit wrote:
On 5/6/2014 1:14 PM, Joerg Roedel wrote:
On Mon, Apr 28, 2014 at 05:27:46PM -0500, Suthikulpanit, Suravee wrote:
From: Jay Cornwall <jay.cornwall@xxxxxxx>

get_user_pages requires caller to hold a read lock on mmap_sem.

Right, but can't we just switch to get_user_pages_fast instead?


Joerg


You're right. I think it can. Let me spin out another patch.

Suravee


Actually, we would have to provide the pointer to task_struct for the corresponding PASID which is not necessary "current". So I take it back. I don't think we can use _fast.

Suravee
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/