Re: [RFC 18/22] sh: Use generic show_mem()

From: Johannes Weiner
Date: Thu Apr 03 2008 - 09:09:09 EST


Hi Paul,

Paul Mundt <lethal@xxxxxxxxxxxx> writes:

> On Wed, Apr 02, 2008 at 10:40:24PM +0200, Johannes Weiner wrote:
>> - printk(KERN_INFO "Total of %ld pages in page table cache\n",
>> - quicklist_total_size());
>> -}
>> -
>
> NACK. The quicklists are also absent from the generic implementation.
> Doing things generically is nice and all, but please do not go around
> removing all of the different implementations and consolidating on the
> simplest point of commonality you could come up with. Either combine
> everything in to a generic show_mem() that doesn't sacrifice
> functionality, or only convert the platforms that are identical.

Sorry, I made it an RFC because I was not quite sure if the generic pfn
walker is valid on all affected architectures. If it is okay, I will
send a patch-set meant for application that does not miss any output.

Thanks,

Hannes
--
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/