linux-mm.kvack.org archive mirror
 help / color / mirror / Atom feed
* Re: Strange memory usage reporting
       [not found] ` <Pine.LNX.4.44.0308261756570.1632-100000@localhost.localdomain>
@ 2003-08-27  7:52   ` Ingo Oeser
  2003-08-27 13:45     ` Hugh Dickins
  0 siblings, 1 reply; 2+ messages in thread
From: Ingo Oeser @ 2003-08-27  7:52 UTC (permalink / raw)
  To: Hugh Dickins; +Cc: Jaroslav Kysela, linux-kernel, linux-mm

Hi,

On Tue, Aug 26, 2003 at 06:03:14PM +0100, Hugh Dickins wrote:
> Which is the driver involved?  Though it's not wrong to give do_no_page
> a Reserved page, beware of the the page->count accounting: while it's
> Reserved, get_page or page_cache_get raises the count, but put_page
> or page_cache_release does not decrement it - very easy to end up
> with the page never freed.

Why is this so asymetric? I would understand ignoring these pages
in the freeing logic, but why exclude them also from refcounting?

Regards

Ingo Oeser
--
To unsubscribe, send a message with 'unsubscribe linux-mm' in
the body to majordomo@kvack.org.  For more info on Linux MM,
see: http://www.linux-mm.org/ .
Don't email: <a href=mailto:"aart@kvack.org"> aart@kvack.org </a>

^ permalink raw reply	[flat|nested] 2+ messages in thread

* Re: Strange memory usage reporting
  2003-08-27  7:52   ` Strange memory usage reporting Ingo Oeser
@ 2003-08-27 13:45     ` Hugh Dickins
  0 siblings, 0 replies; 2+ messages in thread
From: Hugh Dickins @ 2003-08-27 13:45 UTC (permalink / raw)
  To: Ingo Oeser; +Cc: Jaroslav Kysela, linux-kernel, linux-mm

On Wed, 27 Aug 2003, Ingo Oeser wrote:
> On Tue, Aug 26, 2003 at 06:03:14PM +0100, Hugh Dickins wrote:
> > Which is the driver involved?  Though it's not wrong to give do_no_page
> > a Reserved page, beware of the the page->count accounting: while it's
> > Reserved, get_page or page_cache_get raises the count, but put_page
> > or page_cache_release does not decrement it - very easy to end up
> > with the page never freed.
> 
> Why is this so asymetric? I would understand ignoring these pages
> in the freeing logic, but why exclude them also from refcounting?

I don't think there's a _good_ reason, it just evolved that way.

The real answer is to get rid of PageReserved completely, which
I'll embark on again in 2.7 (I did start a couple of times in 2.5,
but each time it was too late).

There was a halfway-house suggestion in 2.5 about three months ago,
inspired (as usual) by Reserved page problems in AIO's get_user_pages,
to do as you suggest: submit them to normal refcounting.  I don't
know what became of that, I didn't have much time to get involved.

Hugh

--
To unsubscribe, send a message with 'unsubscribe linux-mm' in
the body to majordomo@kvack.org.  For more info on Linux MM,
see: http://www.linux-mm.org/ .
Don't email: <a href=mailto:"aart@kvack.org"> aart@kvack.org </a>

^ permalink raw reply	[flat|nested] 2+ messages in thread

end of thread, other threads:[~2003-08-27 13:45 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
     [not found] <Pine.LNX.4.44.0308261550240.1958-100000@pnote.perex-int.cz>
     [not found] ` <Pine.LNX.4.44.0308261756570.1632-100000@localhost.localdomain>
2003-08-27  7:52   ` Strange memory usage reporting Ingo Oeser
2003-08-27 13:45     ` Hugh Dickins

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox