Re: [Cryptodev-linux-devel] [PATCH 1/4] whitespace and indenting cleanup

2012-07-16 Thread Nikos Mavrogiannopoulos
Applied! On 07/16/2012 03:10 PM, Phil Sutter wrote: > > Signed-off-by: Phil Sutter > --- > authenc.c | 28 ++-- > cryptlib.c |2 +- > cryptlib.h |2 +- > cryptodev_int.h |2 +- > ioctl.c |4 ++-- > lib/benchmark.c |8 ---

[Cryptodev-linux-devel] [PATCH 2/4] zc.c: simplify code a bit by exiting early

2012-07-16 Thread Phil Sutter
Also, hiding expressions in one-liners is not worth the lines saved. Signed-off-by: Phil Sutter --- zc.c | 98 - 1 files changed, 48 insertions(+), 50 deletions(-) diff --git a/zc.c b/zc.c index 773dfb4..db28b58 100644 --- a/zc.c

[Cryptodev-linux-devel] [PATCH 4/4] cryptlib: simplify cryptodev_cipher_auth()

2012-07-16 Thread Phil Sutter
Signed-off-by: Phil Sutter --- cryptlib.h |6 ++ 1 files changed, 2 insertions(+), 4 deletions(-) diff --git a/cryptlib.h b/cryptlib.h index 729679c..1745d0f 100644 --- a/cryptlib.h +++ b/cryptlib.h @@ -36,10 +36,8 @@ ssize_t cryptodev_cipher_encrypt(struct cipher_data *cdata, inline s

[Cryptodev-linux-devel] [PATCH 1/4] whitespace and indenting cleanup

2012-07-16 Thread Phil Sutter
Signed-off-by: Phil Sutter --- authenc.c | 28 ++-- cryptlib.c |2 +- cryptlib.h |2 +- cryptodev_int.h |2 +- ioctl.c |4 ++-- lib/benchmark.c |8 lib/combo.c | 22 +++--- lib/hash.c | 16

[Cryptodev-linux-devel] [PATCH 3/4] no need to hold mm->mmap_sem in write lock

2012-07-16 Thread Phil Sutter
As other examples show, read lock suffices although pages are being written later on. Signed-off-by: Phil Sutter --- zc.c |4 ++-- 1 files changed, 2 insertions(+), 2 deletions(-) diff --git a/zc.c b/zc.c index db28b58..884dbab 100644 --- a/zc.c +++ b/zc.c @@ -58,10 +58,10 @@ int __get_user