[HMM 08/15] mm/ZONE_DEVICE: special case put_page() for device private pages v2

2017-05-24 Thread Jérôme Glisse
A ZONE_DEVICE page that reach a refcount of 1 is free ie no longer have any user. For device private pages this is important to catch and thus we need to special case put_page() for this. Changed since v1: - use static key to disable special code path in put_page() by default - uninline pu

Re: [HMM 08/15] mm/ZONE_DEVICE: special case put_page() for device private pages

2017-05-23 Thread Kirill A. Shutemov
On Mon, May 22, 2017 at 12:51:59PM -0400, Jérôme Glisse wrote: > A ZONE_DEVICE page that reach a refcount of 1 is free ie no longer > have any user. For device private pages this is important to catch > and thus we need to special case put_page() for this. > > Signed-off-by: Jérôme Glisse > Cc: K

Re: [HMM 08/15] mm/ZONE_DEVICE: special case put_page() for device private pages

2017-05-23 Thread kbuild test robot
Hi Jérôme, [auto build test ERROR on mmotm/master] [also build test ERROR on next-20170523] [cannot apply to v4.12-rc2] [if your patch is applied to the wrong git tree, please drop us a note to help improve the system] url: https://github.com/0day-ci/linux/commits/J-r-me-Glisse/HMM-Heterogen

Re: [HMM 08/15] mm/ZONE_DEVICE: special case put_page() for device private pages

2017-05-22 Thread Jerome Glisse
On Mon, May 22, 2017 at 01:22:22PM -0700, Hugh Dickins wrote: > On Mon, 22 May 2017, Jerome Glisse wrote: > > On Mon, May 22, 2017 at 12:29:53PM -0700, Dan Williams wrote: > > > On Mon, May 22, 2017 at 9:51 AM, Jerome Glisse wrote: > > > > A ZONE_DEVICE page that reach a refcount of 1 is free ie n

Re: [HMM 08/15] mm/ZONE_DEVICE: special case put_page() for device private pages

2017-05-22 Thread Jerome Glisse
On Mon, May 22, 2017 at 01:19:29PM -0700, Dan Williams wrote: > On Mon, May 22, 2017 at 1:14 PM, Jerome Glisse wrote: > > On Mon, May 22, 2017 at 12:29:53PM -0700, Dan Williams wrote: > >> On Mon, May 22, 2017 at 9:51 AM, Jérôme Glisse wrote: > >> > A ZONE_DEVICE page that reach a refcount of 1 i

Re: [HMM 08/15] mm/ZONE_DEVICE: special case put_page() for device private pages

2017-05-22 Thread Hugh Dickins
On Mon, 22 May 2017, Jerome Glisse wrote: > On Mon, May 22, 2017 at 12:29:53PM -0700, Dan Williams wrote: > > On Mon, May 22, 2017 at 9:51 AM, Jerome Glisse wrote: > > > A ZONE_DEVICE page that reach a refcount of 1 is free ie no longer > > > have any user. For device private pages this is importa

Re: [HMM 08/15] mm/ZONE_DEVICE: special case put_page() for device private pages

2017-05-22 Thread Dan Williams
On Mon, May 22, 2017 at 1:14 PM, Jerome Glisse wrote: > On Mon, May 22, 2017 at 12:29:53PM -0700, Dan Williams wrote: >> On Mon, May 22, 2017 at 9:51 AM, Jérôme Glisse wrote: >> > A ZONE_DEVICE page that reach a refcount of 1 is free ie no longer >> > have any user. For device private pages this

Re: [HMM 08/15] mm/ZONE_DEVICE: special case put_page() for device private pages

2017-05-22 Thread Jerome Glisse
On Mon, May 22, 2017 at 12:29:53PM -0700, Dan Williams wrote: > On Mon, May 22, 2017 at 9:51 AM, Jérôme Glisse wrote: > > A ZONE_DEVICE page that reach a refcount of 1 is free ie no longer > > have any user. For device private pages this is important to catch > > and thus we need to special case p

Re: [HMM 08/15] mm/ZONE_DEVICE: special case put_page() for device private pages

2017-05-22 Thread Dan Williams
On Mon, May 22, 2017 at 9:51 AM, Jérôme Glisse wrote: > A ZONE_DEVICE page that reach a refcount of 1 is free ie no longer > have any user. For device private pages this is important to catch > and thus we need to special case put_page() for this. > > Signed-off-by: Jérôme Glisse > Cc: Kirill A.

[HMM 08/15] mm/ZONE_DEVICE: special case put_page() for device private pages

2017-05-22 Thread Jérôme Glisse
A ZONE_DEVICE page that reach a refcount of 1 is free ie no longer have any user. For device private pages this is important to catch and thus we need to special case put_page() for this. Signed-off-by: Jérôme Glisse Cc: Kirill A. Shutemov Cc: Dan Williams Cc: Ross Zwisler --- include/linux/m