On Sat, 26 Jan 2008, Andrew Morton wrote:
> >
> > - if (!page->mapping)
> > + if (!page->mapping) {
> > + if (!PageAnon(page) && PagePrivate(page))
> > + try_to_release_page(page, GFP_KERNEL);
> > goto rcu_unlock;
> > + }
>
> We call
On Sat, 26 Jan 2008, Andrew Morton wrote:
> We call something(GFP_KERNEL) under rcu_read_lock()? I've lost track of
> the myriad flavours of rcu which we purport to support, but I don't think
> they'll all like us blocking under rcu_read_lock().
>
> We _won't_ block, because
On Sat, 26 Jan 2008, Andrew Morton wrote:
We call something(GFP_KERNEL) under rcu_read_lock()? I've lost track of
the myriad flavours of rcu which we purport to support, but I don't think
they'll all like us blocking under rcu_read_lock().
We _won't_ block, because try_to_release_page()
On Sat, 26 Jan 2008, Andrew Morton wrote:
- if (!page-mapping)
+ if (!page-mapping) {
+ if (!PageAnon(page) PagePrivate(page))
+ try_to_release_page(page, GFP_KERNEL);
goto rcu_unlock;
+ }
We call something(GFP_KERNEL) under
On Mon, 2008-01-28 at 12:43 +1100, Nick Piggin wrote:
> On Sunday 27 January 2008 17:03, Andrew Morton wrote:
> > > On Fri, 25 Jan 2008 14:03:25 +0800 Shaohua Li <[EMAIL PROTECTED]>
> > > wrote:
> > >
> > > - if (!page->mapping)
> > > + if (!page->mapping) {
> > > + if (!PageAnon(page) &&
On Sunday 27 January 2008 17:03, Andrew Morton wrote:
> > On Fri, 25 Jan 2008 14:03:25 +0800 Shaohua Li <[EMAIL PROTECTED]>
> > wrote:
> >
> > - if (!page->mapping)
> > + if (!page->mapping) {
> > + if (!PageAnon(page) && PagePrivate(page))
> > +
On Mon, 2008-01-28 at 12:43 +1100, Nick Piggin wrote:
On Sunday 27 January 2008 17:03, Andrew Morton wrote:
On Fri, 25 Jan 2008 14:03:25 +0800 Shaohua Li [EMAIL PROTECTED]
wrote:
- if (!page-mapping)
+ if (!page-mapping) {
+ if (!PageAnon(page) PagePrivate(page))
+
> On Fri, 25 Jan 2008 14:03:25 +0800 Shaohua Li <[EMAIL PROTECTED]> wrote:
>
> - if (!page->mapping)
> + if (!page->mapping) {
> + if (!PageAnon(page) && PagePrivate(page))
> + try_to_release_page(page, GFP_KERNEL);
> goto rcu_unlock;
> +
On Fri, 2008-01-25 at 15:17 +1100, Nick Piggin wrote:
> On Friday 25 January 2008 15:01, Christoph Lameter wrote:
> > Acked-by: Christoph Lameter <[EMAIL PROTECTED]>
> >
> > Nick? Ok with you too?
>
> Yeah, for memory hot remove that makes sense. A comment
> might be in order, at least a
On Friday 25 January 2008 14:09, Shaohua Li wrote:
> On Fri, 2008-01-25 at 14:03 +1100, Nick Piggin wrote:
> > On Wednesday 23 January 2008 17:22, Shaohua Li wrote:
> > > Anonymous page might have fs-private metadata, the page is truncated.
> > > As the page hasn't mapping, page migration refuse
On Fri, 25 Jan 2008, Nick Piggin wrote:
> Yeah, for memory hot remove that makes sense. A comment
> might be in order, at least a reference to the orphaned
> page code in vmscan.c.
Right. The surrounding comments in mm/migrate.c also need to be made
consistent. The comment before is now
On Friday 25 January 2008 15:01, Christoph Lameter wrote:
> Acked-by: Christoph Lameter <[EMAIL PROTECTED]>
>
> Nick? Ok with you too?
Yeah, for memory hot remove that makes sense. A comment
might be in order, at least a reference to the orphaned
page code in vmscan.c.
Otherwise, it is OK by me.
Acked-by: Christoph Lameter <[EMAIL PROTECTED]>
Nick? Ok with you too?
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to [EMAIL PROTECTED]
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at
On Thu, 2008-01-24 at 19:37 -0800, Christoph Lameter wrote:
> On Wed, 23 Jan 2008, Shaohua Li wrote:
>
> > +
> > + /*
> > +* See truncate_complete_page(). Anonymous page might have
> > +* fs-private metadata, the page is truncated. Such page can't be
> > +* migrated. Try to free
On Wed, 23 Jan 2008, Shaohua Li wrote:
> +
> + /*
> + * See truncate_complete_page(). Anonymous page might have
> + * fs-private metadata, the page is truncated. Such page can't be
> + * migrated. Try to free metadata, so the page can be freed.
> + */
Well maybe you
On Thu, 2008-01-24 at 19:12 -0800, Christoph Lameter wrote:
> On Fri, 25 Jan 2008, Shaohua Li wrote:
>
> > the page is still in lru list. Memory hot remove will try to migrate the
> > page.
>
> So this is an abandoned page with a refcount that only exists because of
> fs private data?
Yes,
On Fri, 25 Jan 2008, Shaohua Li wrote:
> the page is still in lru list. Memory hot remove will try to migrate the
> page.
So this is an abandoned page with a refcount that only exists because of
fs private data? Truncate race?
--
To unsubscribe from this list: send the line "unsubscribe
On Fri, 25 Jan 2008, Nick Piggin wrote:
> On Wednesday 23 January 2008 17:22, Shaohua Li wrote:
> > Anonymous page might have fs-private metadata, the page is truncated. As
> > the page hasn't mapping, page migration refuse to migrate the page. It
> > appears the page is only freed in page
On Fri, 2008-01-25 at 14:03 +1100, Nick Piggin wrote:
> On Wednesday 23 January 2008 17:22, Shaohua Li wrote:
> > Anonymous page might have fs-private metadata, the page is truncated. As
> > the page hasn't mapping, page migration refuse to migrate the page. It
> > appears the page is only freed
On Wednesday 23 January 2008 17:22, Shaohua Li wrote:
> Anonymous page might have fs-private metadata, the page is truncated. As
> the page hasn't mapping, page migration refuse to migrate the page. It
> appears the page is only freed in page reclaim and if zone watermark is
> low, the page is
On Fri, 25 Jan 2008, Shaohua Li wrote:
the page is still in lru list. Memory hot remove will try to migrate the
page.
So this is an abandoned page with a refcount that only exists because of
fs private data? Truncate race?
--
To unsubscribe from this list: send the line unsubscribe
On Fri, 25 Jan 2008, Nick Piggin wrote:
On Wednesday 23 January 2008 17:22, Shaohua Li wrote:
Anonymous page might have fs-private metadata, the page is truncated. As
the page hasn't mapping, page migration refuse to migrate the page. It
appears the page is only freed in page reclaim and
On Fri, 2008-01-25 at 14:03 +1100, Nick Piggin wrote:
On Wednesday 23 January 2008 17:22, Shaohua Li wrote:
Anonymous page might have fs-private metadata, the page is truncated. As
the page hasn't mapping, page migration refuse to migrate the page. It
appears the page is only freed in page
On Wed, 23 Jan 2008, Shaohua Li wrote:
+
+ /*
+ * See truncate_complete_page(). Anonymous page might have
+ * fs-private metadata, the page is truncated. Such page can't be
+ * migrated. Try to free metadata, so the page can be freed.
+ */
Well maybe you should
On Wednesday 23 January 2008 17:22, Shaohua Li wrote:
Anonymous page might have fs-private metadata, the page is truncated. As
the page hasn't mapping, page migration refuse to migrate the page. It
appears the page is only freed in page reclaim and if zone watermark is
low, the page is never
On Thu, 2008-01-24 at 19:12 -0800, Christoph Lameter wrote:
On Fri, 25 Jan 2008, Shaohua Li wrote:
the page is still in lru list. Memory hot remove will try to migrate the
page.
So this is an abandoned page with a refcount that only exists because of
fs private data?
Yes, this is what
On Thu, 2008-01-24 at 19:37 -0800, Christoph Lameter wrote:
On Wed, 23 Jan 2008, Shaohua Li wrote:
+
+ /*
+* See truncate_complete_page(). Anonymous page might have
+* fs-private metadata, the page is truncated. Such page can't be
+* migrated. Try to free metadata, so
Acked-by: Christoph Lameter [EMAIL PROTECTED]
Nick? Ok with you too?
--
To unsubscribe from this list: send the line unsubscribe linux-kernel in
the body of a message to [EMAIL PROTECTED]
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at
On Friday 25 January 2008 15:01, Christoph Lameter wrote:
Acked-by: Christoph Lameter [EMAIL PROTECTED]
Nick? Ok with you too?
Yeah, for memory hot remove that makes sense. A comment
might be in order, at least a reference to the orphaned
page code in vmscan.c.
Otherwise, it is OK by me.
On Fri, 25 Jan 2008, Nick Piggin wrote:
Yeah, for memory hot remove that makes sense. A comment
might be in order, at least a reference to the orphaned
page code in vmscan.c.
Right. The surrounding comments in mm/migrate.c also need to be made
consistent. The comment before is now slightly
On Friday 25 January 2008 14:09, Shaohua Li wrote:
On Fri, 2008-01-25 at 14:03 +1100, Nick Piggin wrote:
On Wednesday 23 January 2008 17:22, Shaohua Li wrote:
Anonymous page might have fs-private metadata, the page is truncated.
As the page hasn't mapping, page migration refuse to migrate
On Fri, 2008-01-25 at 15:17 +1100, Nick Piggin wrote:
On Friday 25 January 2008 15:01, Christoph Lameter wrote:
Acked-by: Christoph Lameter [EMAIL PROTECTED]
Nick? Ok with you too?
Yeah, for memory hot remove that makes sense. A comment
might be in order, at least a reference to the
Anonymous page might have fs-private metadata, the page is truncated. As
the page hasn't mapping, page migration refuse to migrate the page. It
appears the page is only freed in page reclaim and if zone watermark is
low, the page is never freed, as a result migration always fail. I
thought we
Anonymous page might have fs-private metadata, the page is truncated. As
the page hasn't mapping, page migration refuse to migrate the page. It
appears the page is only freed in page reclaim and if zone watermark is
low, the page is never freed, as a result migration always fail. I
thought we
34 matches
Mail list logo