linux-kernel - RE: [RFC PATCH] zcache2 cleanups (s/int/bool + debugfs movement).

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>] [day] [month] [year] [list]

Message-ID: <0e2b1932-628b-4110-8c80-7cfbe3323452@default>
Date:	Tue, 6 Nov 2012 16:02:51 -0800 (PST)
From:	Dan Magenheimer <dan.magenheimer@...cle.com>
To:	Konrad Rzeszutek Wilk <konrad.wilk@...cle.com>,
	linux-kernel@...r.kernel.org, sjenning@...ux.vnet.ibm.com,
	dan.magenheimer@...cle.com, ngupta@...are.org, minchan@...nel.org,
	rcj@...ux.vnet.ibm.com, linux-mm@...ck.org,
	gregkh@...uxfoundation.org, devel@...verdev.osuosl.org
Cc:	akpm@...ux-foundation.org
Subject: RE: [RFC PATCH] zcache2 cleanups (s/int/bool + debugfs movement).

> From: Konrad Rzeszutek Wilk [mailto:konrad.wilk@...cle.com]
> Sent: Monday, November 05, 2012 7:37 AM
> To: linux-kernel@...r.kernel.org; sjenning@...ux.vnet.ibm.com; dan.magenheimer@...cle.com;
> ngupta@...are.org; minchan@...nel.org; rcj@...ux.vnet.ibm.com; linux-mm@...ck.org;
> gregkh@...uxfoundation.org; devel@...verdev.osuosl.org
> Cc: akpm@...ux-foundation.org
> Subject: [RFC PATCH] zcache2 cleanups (s/int/bool + debugfs movement).
> 
> Looking at the zcache2 code there were a couple of things that I thought
> would make sense to move out of the code. For one thing it makes it easier
> to read, and for anoter - it can be cleanly compiled out. It also allows
> to have a clean seperation of counters that we _need_ vs the optional ones.
> Which means that in the future we could get rid of the optional ones.
> 
> This patchset is based on the patchset that Dan sent out
> (https://lkml.org/lkml/2012/10/31/790). I've stuck
> them (and addressed some review comments) and put them in my branch:
> 
>  git://git.kernel.org/pub/scm/linux/kernel/git/konrad/mm.git devel/zcache.v3
> 
> I am going to repost the module loading some time later this week - Bob Liu had
> some comments that I want to address.
> 
> So back to this patchset - it fixes some outstanding compile warnings, cleans
> up some of the code, and rips out the debug counters out of zcache-main.c
> and sticks them in a debug.c file.
> 
> I was hoping it would end up with less code, but sadly it ended up with
> a bit more due to the empty non-debug functions.
> 
>  drivers/staging/ramster/Kconfig       |    8 +
>  drivers/staging/ramster/Makefile      |    1 +
>  drivers/staging/ramster/debug.c       |   66 ++++++
>  drivers/staging/ramster/debug.h       |  225 +++++++++++++++++++
>  drivers/staging/ramster/zcache-main.c |  384 ++++++++-------------------------
>  5 files changed, 389 insertions(+), 295 deletions(-)
> 
> Konrad Rzeszutek Wilk (11):
>       zcache2: s/int/bool/ on the various options.
>       zcache: Module license is defined twice.
>       zcache: Provide accessory functions for counter increase
>       zcache: Provide accessory functions for counter decrease.
>       zcache: The last of the atomic reads has now an accessory function.
>       zcache: Fix compile warnings due to usage of debugfs_create_size_t
>       zcache: Make the debug code use pr_debug
>       zcache: Move debugfs code out of zcache-main.c file.
>       zcache: Use an array to initialize/use debugfs attributes.
>       zcache: Move the last of the debugfs counters out
>       zcache: Coalesce all debug under CONFIG_ZCACHE2_DEBUG

FWIW, for all these patches, please add my:

Reviewed-by: Dan Magenheimer <dan.magenheimer@...cle.com>
--
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/