Re: [patch 9/12] mm: page_state opt

[Andrew Morton]

Nick Piggin <nickpiggin@xxxxxxxxxxxx> wrote:
>
> Andrew Morton wrote:
> > Nick Piggin <nickpiggin@xxxxxxxxxxxx> wrote:
> > 
> >>-#define mod_page_state_zone(zone, member, delta)				\
> >> -	do {									\
> >> -		unsigned offset;						\
> >> -		if (is_highmem(zone))						\
> >> -			offset = offsetof(struct page_state, member##_high);	\
> >> -		else if (is_normal(zone))					\
> >> -			offset = offsetof(struct page_state, member##_normal);	\
> >> -		else								\
> >> -			offset = offsetof(struct page_state, member##_dma);	\
> >> -		__mod_page_state(offset, (delta));				\
> >> -	} while (0)
> > 
> > 
> > I suppose this needs updating to know about the dma32 zone.
> > 
> 
> Ah I didn't realise DMA32 is in the tree now. I think you're right.

It means adding a new field to /proc/vmstat of course.  Presumably someone
uses that file occasionally.

> I'll rebase this patchset when such an update is made. If you'd like
> I could look at doing said DMA32 update for you?

Sometime..
-
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/