Re: [PATCH] CMA: add the amount of cma memory in meminfo

From: Xishi Qiu
Date: Sun Dec 14 2014 - 20:38:02 EST


On 2014/12/6 1:41, Michal Nazarewicz wrote:

> On Thu, Dec 04 2014, Xishi Qiu <qiuxishi@xxxxxxxxxx> wrote:
>> Add the amount of cma memory in the following meminfo.
>> /proc/meminfo
>> /sys/devices/system/node/nodeXX/meminfo
>>
>> Signed-off-by: Xishi Qiu <qiuxishi@xxxxxxxxxx>
>
> No second look:
>
> Acked-by: Michal Nazarewicz <mina86@xxxxxxxxxx>
>
>> ---
>> drivers/base/node.c | 16 ++++++++++------
>> fs/proc/meminfo.c | 12 +++++++++---
>> 2 files changed, 19 insertions(+), 9 deletions(-)
>>
>> diff --git a/drivers/base/node.c b/drivers/base/node.c
>> index 472168c..a27e4e0 100644
>> --- a/drivers/base/node.c
>> +++ b/drivers/base/node.c
>> @@ -120,6 +120,9 @@ static ssize_t node_read_meminfo(struct device *dev,
>> #ifdef CONFIG_TRANSPARENT_HUGEPAGE
>> "Node %d AnonHugePages: %8lu kB\n"
>> #endif
>> +#ifdef CONFIG_CMA
>> + "Node %d FreeCMAPages: %8lu kB\n"
>> +#endif
>> ,
>> nid, K(node_page_state(nid, NR_FILE_DIRTY)),
>> nid, K(node_page_state(nid, NR_WRITEBACK)),
>> @@ -136,14 +139,15 @@ static ssize_t node_read_meminfo(struct device *dev,
>> nid, K(node_page_state(nid, NR_SLAB_RECLAIMABLE) +
>> node_page_state(nid, NR_SLAB_UNRECLAIMABLE)),
>> nid, K(node_page_state(nid, NR_SLAB_RECLAIMABLE)),
>> -#ifdef CONFIG_TRANSPARENT_HUGEPAGE
>> nid, K(node_page_state(nid, NR_SLAB_UNRECLAIMABLE))
>> - , nid,
>> - K(node_page_state(nid, NR_ANON_TRANSPARENT_HUGEPAGES) *
>> - HPAGE_PMD_NR));
>> -#else
>> - nid, K(node_page_state(nid, NR_SLAB_UNRECLAIMABLE)));
>> +#ifdef CONFIG_TRANSPARENT_HUGEPAGE
>> + , nid, K(node_page_state(nid,
>> + NR_ANON_TRANSPARENT_HUGEPAGES) * HPAGE_PMD_NR)

Hi Michalï

The "mere white-space change" you said a few days agoïhow about change like this
", nid, K(...)" -> ",nid, K(xxx)"?

Thanks,
Xishi Qiu

>> +#endif
>> +#ifdef CONFIG_CMA
>> + , nid, K(node_page_state(nid, NR_FREE_CMA_PAGES))
>> #endif
>> + );
>> n += hugetlb_report_node_meminfo(nid, buf + n);
>> return n;
>> }
>> diff --git a/fs/proc/meminfo.c b/fs/proc/meminfo.c
>> index aa1eee0..d42e082 100644
>> --- a/fs/proc/meminfo.c
>> +++ b/fs/proc/meminfo.c
>> @@ -138,6 +138,9 @@ static int meminfo_proc_show(struct seq_file *m, void *v)
>> #ifdef CONFIG_TRANSPARENT_HUGEPAGE
>> "AnonHugePages: %8lu kB\n"
>> #endif
>> +#ifdef CONFIG_CMA
>> + "FreeCMAPages: %8lu kB\n"
>> +#endif
>> ,
>> K(i.totalram),
>> K(i.freeram),
>> @@ -187,11 +190,14 @@ static int meminfo_proc_show(struct seq_file *m, void *v)
>> vmi.used >> 10,
>> vmi.largest_chunk >> 10
>> #ifdef CONFIG_MEMORY_FAILURE
>> - ,atomic_long_read(&num_poisoned_pages) << (PAGE_SHIFT - 10)
>> + , atomic_long_read(&num_poisoned_pages) << (PAGE_SHIFT - 10)
>> #endif
>> #ifdef CONFIG_TRANSPARENT_HUGEPAGE
>> - ,K(global_page_state(NR_ANON_TRANSPARENT_HUGEPAGES) *
>> - HPAGE_PMD_NR)
>> + , K(global_page_state(NR_ANON_TRANSPARENT_HUGEPAGES) *
>> + HPAGE_PMD_NR)
>> +#endif
>> +#ifdef CONFIG_CMA
>> + , K(global_page_state(NR_FREE_CMA_PAGES))
>> #endif
>> );
>>
>> --
>> 2.0.0
>>
>>
>



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