Re: [PATCH 1/2] mm: Export vm_committed_as

[David Rientjes]

On Thu, 8 Nov 2012, Andrew Morton wrote:

> > > I don't think you should export the symbol itself to modules but rather a
> > > helper function that returns s64 that just wraps
> > > percpu_counter_read_positive() which your driver could use instead.
> > > 
> > > (And why percpu_counter_read_positive() returns a signed type is a
> > > mystery.)
> > 
> > Yes, this makes sense. I just want to access (read) this metric. Andrew, if you are willing to
> > take this patch, I could send one.
> 
> Sure.  I suppose that's better, although any module which modifies
> committed_as would never pass review (rofl).
> 

I was thinking of a function that all hypervisors can use (since xen also 
uses it) that can be well documented and maintain the semantics that they 
expect, whether that relines on vm_commited_as in the future or not.
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/