Re: [PATCHv3 04/24] rmap: add argument to charge compound page

2015-03-04 Thread Kirill A. Shutemov
On Wed, Mar 04, 2015 at 01:09:51PM +0100, Vlastimil Babka wrote: > On 03/04/2015 12:52 PM, Kirill A. Shutemov wrote: > >On Mon, Feb 23, 2015 at 05:21:31PM +0100, Vlastimil Babka wrote: > >>On 02/12/2015 05:18 PM, Kirill A. Shutemov wrote: > >>>@@ -1052,21 +1052,24 @@ void page_add_anon_rmap(struct

Re: [PATCHv3 04/24] rmap: add argument to charge compound page

2015-03-04 Thread Vlastimil Babka
On 03/04/2015 12:52 PM, Kirill A. Shutemov wrote: On Mon, Feb 23, 2015 at 05:21:31PM +0100, Vlastimil Babka wrote: On 02/12/2015 05:18 PM, Kirill A. Shutemov wrote: @@ -1052,21 +1052,24 @@ void page_add_anon_rmap(struct page *page, * Everybody else should continue to use page_add_anon_rmap

Re: [PATCHv3 04/24] rmap: add argument to charge compound page

2015-03-04 Thread Kirill A. Shutemov
On Mon, Feb 23, 2015 at 05:21:31PM +0100, Vlastimil Babka wrote: > On 02/12/2015 05:18 PM, Kirill A. Shutemov wrote: > > We're going to allow mapping of individual 4k pages of THP compound > > page. It means we cannot rely on PageTransHuge() check to decide if map > > small page or THP. > > > >

Re: [PATCHv3 04/24] rmap: add argument to charge compound page

2015-03-04 Thread Kirill A. Shutemov
On Mon, Feb 23, 2015 at 05:21:31PM +0100, Vlastimil Babka wrote: On 02/12/2015 05:18 PM, Kirill A. Shutemov wrote: We're going to allow mapping of individual 4k pages of THP compound page. It means we cannot rely on PageTransHuge() check to decide if map small page or THP. The patch

Re: [PATCHv3 04/24] rmap: add argument to charge compound page

2015-03-04 Thread Vlastimil Babka
On 03/04/2015 12:52 PM, Kirill A. Shutemov wrote: On Mon, Feb 23, 2015 at 05:21:31PM +0100, Vlastimil Babka wrote: On 02/12/2015 05:18 PM, Kirill A. Shutemov wrote: @@ -1052,21 +1052,24 @@ void page_add_anon_rmap(struct page *page, * Everybody else should continue to use page_add_anon_rmap

Re: [PATCHv3 04/24] rmap: add argument to charge compound page

2015-03-04 Thread Kirill A. Shutemov
On Wed, Mar 04, 2015 at 01:09:51PM +0100, Vlastimil Babka wrote: On 03/04/2015 12:52 PM, Kirill A. Shutemov wrote: On Mon, Feb 23, 2015 at 05:21:31PM +0100, Vlastimil Babka wrote: On 02/12/2015 05:18 PM, Kirill A. Shutemov wrote: @@ -1052,21 +1052,24 @@ void page_add_anon_rmap(struct page

Re: [PATCHv3 04/24] rmap: add argument to charge compound page

2015-02-23 Thread Vlastimil Babka
On 02/12/2015 05:18 PM, Kirill A. Shutemov wrote: > We're going to allow mapping of individual 4k pages of THP compound > page. It means we cannot rely on PageTransHuge() check to decide if map > small page or THP. > > The patch adds new argument to rmap function to indicate whethe we want > to

Re: [PATCHv3 04/24] rmap: add argument to charge compound page

2015-02-23 Thread Vlastimil Babka
On 02/12/2015 05:18 PM, Kirill A. Shutemov wrote: We're going to allow mapping of individual 4k pages of THP compound page. It means we cannot rely on PageTransHuge() check to decide if map small page or THP. The patch adds new argument to rmap function to indicate whethe we want to map

Re: [PATCHv3 04/24] rmap: add argument to charge compound page

2015-02-20 Thread Jerome Marchand
On 02/16/2015 04:20 PM, Kirill A. Shutemov wrote: > On Thu, Feb 12, 2015 at 04:10:21PM -0500, Rik van Riel wrote: >> -BEGIN PGP SIGNED MESSAGE- >> Hash: SHA1 >> >> On 02/12/2015 11:18 AM, Kirill A. Shutemov wrote: >> >>> +++ b/include/linux/rmap.h @@ -168,16 +168,24 @@ static inline void

Re: [PATCHv3 04/24] rmap: add argument to charge compound page

2015-02-20 Thread Jerome Marchand
On 02/16/2015 04:20 PM, Kirill A. Shutemov wrote: On Thu, Feb 12, 2015 at 04:10:21PM -0500, Rik van Riel wrote: -BEGIN PGP SIGNED MESSAGE- Hash: SHA1 On 02/12/2015 11:18 AM, Kirill A. Shutemov wrote: +++ b/include/linux/rmap.h @@ -168,16 +168,24 @@ static inline void

Re: [PATCHv3 04/24] rmap: add argument to charge compound page

2015-02-16 Thread Kirill A. Shutemov
On Thu, Feb 12, 2015 at 04:10:21PM -0500, Rik van Riel wrote: > -BEGIN PGP SIGNED MESSAGE- > Hash: SHA1 > > On 02/12/2015 11:18 AM, Kirill A. Shutemov wrote: > > > +++ b/include/linux/rmap.h @@ -168,16 +168,24 @@ static inline void > > anon_vma_merge(struct vm_area_struct *vma, > > > >

Re: [PATCHv3 04/24] rmap: add argument to charge compound page

2015-02-16 Thread Kirill A. Shutemov
On Thu, Feb 12, 2015 at 04:10:21PM -0500, Rik van Riel wrote: -BEGIN PGP SIGNED MESSAGE- Hash: SHA1 On 02/12/2015 11:18 AM, Kirill A. Shutemov wrote: +++ b/include/linux/rmap.h @@ -168,16 +168,24 @@ static inline void anon_vma_merge(struct vm_area_struct *vma, struct

Re: [PATCHv3 04/24] rmap: add argument to charge compound page

2015-02-12 Thread Rik van Riel
-BEGIN PGP SIGNED MESSAGE- Hash: SHA1 On 02/12/2015 11:18 AM, Kirill A. Shutemov wrote: > +++ b/include/linux/rmap.h @@ -168,16 +168,24 @@ static inline void > anon_vma_merge(struct vm_area_struct *vma, > > struct anon_vma *page_get_anon_vma(struct page *page); > > +/* flags for

[PATCHv3 04/24] rmap: add argument to charge compound page

2015-02-12 Thread Kirill A. Shutemov
We're going to allow mapping of individual 4k pages of THP compound page. It means we cannot rely on PageTransHuge() check to decide if map small page or THP. The patch adds new argument to rmap function to indicate whethe we want to map whole compound page or only the small page. Signed-off-by:

Re: [PATCHv3 04/24] rmap: add argument to charge compound page

2015-02-12 Thread Rik van Riel
-BEGIN PGP SIGNED MESSAGE- Hash: SHA1 On 02/12/2015 11:18 AM, Kirill A. Shutemov wrote: +++ b/include/linux/rmap.h @@ -168,16 +168,24 @@ static inline void anon_vma_merge(struct vm_area_struct *vma, struct anon_vma *page_get_anon_vma(struct page *page); +/* flags for

[PATCHv3 04/24] rmap: add argument to charge compound page

2015-02-12 Thread Kirill A. Shutemov
We're going to allow mapping of individual 4k pages of THP compound page. It means we cannot rely on PageTransHuge() check to decide if map small page or THP. The patch adds new argument to rmap function to indicate whethe we want to map whole compound page or only the small page. Signed-off-by: