On Mon, 3 Feb 2014, David Rientjes wrote:
> On Tue, 4 Feb 2014, Joonsoo Kim wrote:
>
> > > > Okay. It can't fix your situation. Anyway, *normal* anon pages may be
> > > > mapped
> > > > and have positive page_count(), so your code such as
> > > > '!page_mapping(page) && page_count(page)' makes
On Tue, 4 Feb 2014, Joonsoo Kim wrote:
> > > Okay. It can't fix your situation. Anyway, *normal* anon pages may be
> > > mapped
> > > and have positive page_count(), so your code such as
> > > '!page_mapping(page) && page_count(page)' makes compaction skip these
> > > *normal*
> > > anon pages
On Mon, Feb 03, 2014 at 06:00:56PM -0800, David Rientjes wrote:
> On Tue, 4 Feb 2014, Joonsoo Kim wrote:
>
> > Okay. It can't fix your situation. Anyway, *normal* anon pages may be mapped
> > and have positive page_count(), so your code such as
> > '!page_mapping(page) && page_count(page)' makes
On Tue, 4 Feb 2014, Joonsoo Kim wrote:
> Okay. It can't fix your situation. Anyway, *normal* anon pages may be mapped
> and have positive page_count(), so your code such as
> '!page_mapping(page) && page_count(page)' makes compaction skip these *normal*
> anon pages and this is incorrect
On Mon, Feb 03, 2014 at 05:20:46PM -0800, David Rientjes wrote:
> On Tue, 4 Feb 2014, Joonsoo Kim wrote:
>
> > I think that you need more code to skip this type of page correctly.
> > Without page_mapped() check, this code makes migratable pages be skipped,
> > since if page_mapped() case,
On Tue, 4 Feb 2014, Joonsoo Kim wrote:
> I think that you need more code to skip this type of page correctly.
> Without page_mapped() check, this code makes migratable pages be skipped,
> since if page_mapped() case, page_count() may be more than zero.
>
> So I think that you need following
On Tue, 4 Feb 2014, Joonsoo Kim wrote:
I think that you need more code to skip this type of page correctly.
Without page_mapped() check, this code makes migratable pages be skipped,
since if page_mapped() case, page_count() may be more than zero.
So I think that you need following change.
On Mon, Feb 03, 2014 at 05:20:46PM -0800, David Rientjes wrote:
On Tue, 4 Feb 2014, Joonsoo Kim wrote:
I think that you need more code to skip this type of page correctly.
Without page_mapped() check, this code makes migratable pages be skipped,
since if page_mapped() case, page_count()
On Tue, 4 Feb 2014, Joonsoo Kim wrote:
Okay. It can't fix your situation. Anyway, *normal* anon pages may be mapped
and have positive page_count(), so your code such as
'!page_mapping(page) page_count(page)' makes compaction skip these *normal*
anon pages and this is incorrect behaviour.
On Mon, Feb 03, 2014 at 06:00:56PM -0800, David Rientjes wrote:
On Tue, 4 Feb 2014, Joonsoo Kim wrote:
Okay. It can't fix your situation. Anyway, *normal* anon pages may be mapped
and have positive page_count(), so your code such as
'!page_mapping(page) page_count(page)' makes compaction
On Tue, 4 Feb 2014, Joonsoo Kim wrote:
Okay. It can't fix your situation. Anyway, *normal* anon pages may be
mapped
and have positive page_count(), so your code such as
'!page_mapping(page) page_count(page)' makes compaction skip these
*normal*
anon pages and this is
On Mon, 3 Feb 2014, David Rientjes wrote:
On Tue, 4 Feb 2014, Joonsoo Kim wrote:
Okay. It can't fix your situation. Anyway, *normal* anon pages may be
mapped
and have positive page_count(), so your code such as
'!page_mapping(page) page_count(page)' makes compaction skip these
12 matches
Mail list logo