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

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Date:	Sun, 17 Mar 2013 20:58:46 +0800
From:	Ric Mason <ric.masonn@...il.com>
To:	Konrad Rzeszutek Wilk <konrad@...nok.org>
CC:	Wanpeng Li <liwanp@...ux.vnet.ibm.com>,
	Greg Kroah-Hartman <gregkh@...uxfoundation.org>,
	Andrew Morton <akpm@...ux-foundation.org>,
	Dan Magenheimer <dan.magenheimer@...cle.com>,
	Seth Jennings <sjenning@...ux.vnet.ibm.com>,
	Minchan Kim <minchan@...nel.org>, linux-mm@...ck.org,
	linux-kernel@...r.kernel.org
Subject: Re: [PATCH v2 3/4] introduce zero-filled page stat count

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@...cle.com>
>> Signed-off-by: Wanpeng Li <liwanp@...ux.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@...ck.org.  For more info on Linux MM,
> see: http://www.linux-mm.org/ .
> Don't email: <a href=mailto:"dont@...ck.org"> email@...ck.org </a>

--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/