Re: [PATCH v2 3/4] introduce zero-filled page stat count

[Ric Mason <ric.masonn@gmail.com>]

Hi Konrad,
On 03/16/2013 09:06 PM, Konrad Rzeszutek Wilk wrote:
> On Thu, Mar 14, 2013 at 06:08:16PM +0800, Wanpeng Li wrote:
>> Introduce zero-filled page statistics to monitor the number of
>> zero-filled pages.
> Hm, you must be using an older version of the driver. Please
> rebase it against Greg KH's staging tree. This is where most if not
> all of the DebugFS counters got moved to a different file.

It seems that zcache debugfs in Greg's staging-next is buggy, Could you 
test it?

>
>> Acked-by: Dan Magenheimer <dan.magenheimer@oracle.com>
>> Signed-off-by: Wanpeng Li <liwanp@linux.vnet.ibm.com>
>> ---
>>   drivers/staging/zcache/zcache-main.c |    7 +++++++
>>   1 files changed, 7 insertions(+), 0 deletions(-)
>>
>> diff --git a/drivers/staging/zcache/zcache-main.c b/drivers/staging/zcache/zcache-main.c
>> index db200b4..2091a4d 100644
>> --- a/drivers/staging/zcache/zcache-main.c
>> +++ b/drivers/staging/zcache/zcache-main.c
>> @@ -196,6 +196,7 @@ static ssize_t zcache_eph_nonactive_puts_ignored;
>>   static ssize_t zcache_pers_nonactive_puts_ignored;
>>   static ssize_t zcache_writtenback_pages;
>>   static ssize_t zcache_outstanding_writeback_pages;
>> +static ssize_t zcache_pages_zero;
>>   
>>   #ifdef CONFIG_DEBUG_FS
>>   #include <linux/debugfs.h>
>> @@ -257,6 +258,7 @@ static int zcache_debugfs_init(void)
>>   	zdfs("outstanding_writeback_pages", S_IRUGO, root,
>>   				&zcache_outstanding_writeback_pages);
>>   	zdfs("writtenback_pages", S_IRUGO, root, &zcache_writtenback_pages);
>> +	zdfs("pages_zero", S_IRUGO, root, &zcache_pages_zero);
>>   	return 0;
>>   }
>>   #undef	zdebugfs
>> @@ -326,6 +328,7 @@ void zcache_dump(void)
>>   	pr_info("zcache: outstanding_writeback_pages=%zd\n",
>>   				zcache_outstanding_writeback_pages);
>>   	pr_info("zcache: writtenback_pages=%zd\n", zcache_writtenback_pages);
>> +	pr_info("zcache: pages_zero=%zd\n", zcache_pages_zero);
>>   }
>>   #endif
>>   
>> @@ -562,6 +565,7 @@ static void *zcache_pampd_eph_create(char *data, size_t size, bool raw,
>>   		kunmap_atomic(user_mem);
>>   		clen = 0;
>>   		zero_filled = true;
>> +		zcache_pages_zero++;
>>   		goto got_pampd;
>>   	}
>>   	kunmap_atomic(user_mem);
>> @@ -645,6 +649,7 @@ static void *zcache_pampd_pers_create(char *data, size_t size, bool raw,
>>   		kunmap_atomic(user_mem);
>>   		clen = 0;
>>   		zero_filled = true;
>> +		zcache_pages_zero++;
>>   		goto got_pampd;
>>   	}
>>   	kunmap_atomic(user_mem);
>> @@ -866,6 +871,7 @@ static int zcache_pampd_get_data_and_free(char *data, size_t *sizep, bool raw,
>>   		zpages = 0;
>>   		if (!raw)
>>   			*sizep = PAGE_SIZE;
>> +		zcache_pages_zero--;
>>   		goto zero_fill;
>>   	}
>>   
>> @@ -922,6 +928,7 @@ static void zcache_pampd_free(void *pampd, struct tmem_pool *pool,
>>   		zero_filled = true;
>>   		zsize = 0;
>>   		zpages = 0;
>> +		zcache_pages_zero--;
>>   	}
>>   
>>   	if (pampd_is_remote(pampd) && !zero_filled) {
>> -- 
>> 1.7.7.6
>>
> --
> To unsubscribe, send a message with 'unsubscribe linux-mm' in
> the body to majordomo@kvack.org.  For more info on Linux MM,
> see: http://www.linux-mm.org/ .
> Don't email: <a href=mailto:"dont@kvack.org"> email@kvack.org </a>

--
To unsubscribe, send a message with 'unsubscribe linux-mm' in
the body to majordomo@kvack.org.  For more info on Linux MM,
see: http://www.linux-mm.org/ .
Don't email: <a href=mailto:"dont@kvack.org"> email@kvack.org </a>