2014-06-14 22:09:49

by Fabian Frédérick

Subject: [PATCH 1/1] drivers/crypto/nx/nx-842.c: use PAGE_ALIGNED instead of IS_ALIGNED(PAGE_SIZE

use mm.h definition

Cc: Nathan Fontenot <[email protected]>
Cc: Marcelo Henrique Cerri <[email protected]>
Signed-off-by: Fabian Frederick <[email protected]>
---
drivers/crypto/nx/nx-842.c | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/drivers/crypto/nx/nx-842.c b/drivers/crypto/nx/nx-842.c
index 502edf0..923c4b6 100644
--- a/drivers/crypto/nx/nx-842.c
+++ b/drivers/crypto/nx/nx-842.c
@@ -350,7 +350,7 @@ int nx842_compress(const unsigned char *in, unsigned int inlen,
* the alignment is guaranteed.
*/
inbuf = (unsigned long)in;
- if (!IS_ALIGNED(inbuf, PAGE_SIZE) || inlen != PAGE_SIZE)
+ if (!PAGE_ALIGNED(inbuf) || inlen != PAGE_SIZE)
return -EINVAL;

rcu_read_lock();
@@ -545,7 +545,7 @@ int nx842_decompress(const unsigned char *in, unsigned int inlen,

/* Ensure page alignment and size */
outbuf = (unsigned long)out;
- if (!IS_ALIGNED(outbuf, PAGE_SIZE) || *outlen != PAGE_SIZE)
+ if (!PAGE_ALIGNED(outbuf) || *outlen != PAGE_SIZE)
return -EINVAL;

rcu_read_lock();
--
1.8.4.5

2014-06-16 14:55:23

by Nathan Fontenot

Subject: Re: [PATCH 1/1] drivers/crypto/nx/nx-842.c: use PAGE_ALIGNED instead of IS_ALIGNED(PAGE_SIZE

On 06/14/2014 05:09 PM, Fabian Frederick wrote:
> use mm.h definition
>
> Cc: Nathan Fontenot <[email protected]>
> Cc: Marcelo Henrique Cerri <[email protected]>
> Signed-off-by: Fabian Frederick <[email protected]>

Acked-by: Nathan Fontenot <[email protected]>

> ---
> drivers/crypto/nx/nx-842.c | 4 ++--
> 1 file changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/drivers/crypto/nx/nx-842.c b/drivers/crypto/nx/nx-842.c
> index 502edf0..923c4b6 100644
> --- a/drivers/crypto/nx/nx-842.c
> +++ b/drivers/crypto/nx/nx-842.c
> @@ -350,7 +350,7 @@ int nx842_compress(const unsigned char *in, unsigned int inlen,
> * the alignment is guaranteed.
> */
> inbuf = (unsigned long)in;
> - if (!IS_ALIGNED(inbuf, PAGE_SIZE) || inlen != PAGE_SIZE)
> + if (!PAGE_ALIGNED(inbuf) || inlen != PAGE_SIZE)
> return -EINVAL;
>
> rcu_read_lock();
> @@ -545,7 +545,7 @@ int nx842_decompress(const unsigned char *in, unsigned int inlen,
>
> /* Ensure page alignment and size */
> outbuf = (unsigned long)out;
> - if (!IS_ALIGNED(outbuf, PAGE_SIZE) || *outlen != PAGE_SIZE)
> + if (!PAGE_ALIGNED(outbuf) || *outlen != PAGE_SIZE)
> return -EINVAL;
>
> rcu_read_lock();
>