Messages in this thread | | | Date | Sun, 5 Jul 2009 22:19:47 +0900 | Subject | Re: [PATCH 5/5] add NR_ANON_PAGES to OOM log | From | KOSAKI Motohiro <> |
| |
>> > > + printk("%ld total anon pages\n", global_page_state(NR_ANON_PAGES)); >> > > printk("%ld total pagecache pages\n", global_page_state(NR_FILE_PAGES)); >> > >> > Can we put related items together, ie. this looks more friendly: >> > >> > Anon:XXX active_anon:XXX inactive_anon:XXX >> > File:XXX active_file:XXX inactive_file:XXX >> >> hmmm. Actually NR_ACTIVE_ANON + NR_INACTIVE_ANON != NR_ANON_PAGES. >> tmpfs pages are accounted as FILE, but it is stay in anon lru. > > Right, that's exactly the reason I propose to put them together: to > make the number of tmpfs pages obvious. > >> I think your proposed format easily makes confusion. this format cause to >> imazine Anon = active_anon + inactive_anon. > > Yes it may confuse normal users :( > >> At least, we need to use another name, I think. > > Hmm I find it hard to work out a good name. > > But instead, it may be a good idea to explicitly compute the tmpfs > pages, because the excessive use of tmpfs pages could be a common > reason of OOM.
Yeah, explicite tmpfs/shmem accounting is also useful for /proc/meminfo. -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
| |