>>>> Andy Whitcroft wrote on Mon, March 29, 2004 4:30 AM > Indeed. The previous patches I submitted only address #1. Attached is > another patch which should address #2, it supplies hugetlb commit > accounting. This is checked and applied when the segment is created. It > also supplements the meminfo information to display this new commitment. > The patch only implments strict commitment, but as has been stated here > often, it is not clear that overcommit of unswappable memory makes any > sense in the absence of demand allocation. When that is implemented then > this will likely need a policy. > > Patch applies on top of my previous patch and has been tested on i386. +int hugetlbfs_report_meminfo(char *buf) +{ + long htlb = atomic_read(&hugetlb_committed_space); + return sprintf(buf, "HugeCommited_AS: %5lu\n", htlb); +} "HugeCommited_AS", typo?? Should that be double "t"? Also can we print in terms of kB instead of num pages to match all other entries? Something Like: htlb<<(PAGE_SHIFT-10)? overcomit is not checked for hugetlb mmap, is it intentional here? - Ken - To unsubscribe from this list: send the line "unsubscribe linux-ia64" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.htmlReceived on Mon Mar 29 15:53:49 2004
This archive was generated by hypermail 2.1.8 : 2005-08-02 09:20:25 EST