All of lore.kernel.org
 help / color / mirror / Atom feed
* [PATCH] fscrypt: properly declare on-stack completion
@ 2017-02-01 20:32 Richard Weinberger
  2017-02-01 20:34 ` Richard Weinberger
  2017-02-07  4:21 ` Theodore Ts'o
  0 siblings, 2 replies; 3+ messages in thread
From: Richard Weinberger @ 2017-02-01 20:32 UTC (permalink / raw)
  To: linux-fsdevel
  Cc: linux-kernel, jaegeuk, tytso, ebiggers, david, Richard Weinberger

When a completion is declared on-stack we have to use
COMPLETION_INITIALIZER_ONSTACK().

Fixes: 0b81d07790726 ("fs crypto: move per-file encryption from f2fs
tree to fs/crypto")
Signed-off-by: Richard Weinberger <richard@nod.at>

---
 fs/crypto/fscrypt_private.h | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/fs/crypto/fscrypt_private.h b/fs/crypto/fscrypt_private.h
index aeab032d7d35..a0494a164f97 100644
--- a/fs/crypto/fscrypt_private.h
+++ b/fs/crypto/fscrypt_private.h
@@ -81,7 +81,7 @@ struct fscrypt_completion_result {
 
 #define DECLARE_FS_COMPLETION_RESULT(ecr) \
 	struct fscrypt_completion_result ecr = { \
-		COMPLETION_INITIALIZER((ecr).completion), 0 }
+		COMPLETION_INITIALIZER_ONSTACK((ecr).completion), 0 }
 
 
 /* crypto.c */
-- 
2.10.2

^ permalink raw reply related	[flat|nested] 3+ messages in thread

* Re: [PATCH] fscrypt: properly declare on-stack completion
  2017-02-01 20:32 [PATCH] fscrypt: properly declare on-stack completion Richard Weinberger
@ 2017-02-01 20:34 ` Richard Weinberger
  2017-02-07  4:21 ` Theodore Ts'o
  1 sibling, 0 replies; 3+ messages in thread
From: Richard Weinberger @ 2017-02-01 20:34 UTC (permalink / raw)
  To: linux-fsdevel; +Cc: linux-kernel, jaegeuk, Theodore Ts'o, ebiggers, david

Am 01.02.2017 um 21:32 schrieb Richard Weinberger:
> When a completion is declared on-stack we have to use
> COMPLETION_INITIALIZER_ONSTACK().
> 
> Fixes: 0b81d07790726 ("fs crypto: move per-file encryption from f2fs
> tree to fs/crypto")
> Signed-off-by: Richard Weinberger <richard@nod.at>
> 
> ---
>  fs/crypto/fscrypt_private.h | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/fs/crypto/fscrypt_private.h b/fs/crypto/fscrypt_private.h
> index aeab032d7d35..a0494a164f97 100644
> --- a/fs/crypto/fscrypt_private.h
> +++ b/fs/crypto/fscrypt_private.h
> @@ -81,7 +81,7 @@ struct fscrypt_completion_result {
>  
>  #define DECLARE_FS_COMPLETION_RESULT(ecr) \
>  	struct fscrypt_completion_result ecr = { \
> -		COMPLETION_INITIALIZER((ecr).completion), 0 }
> +		COMPLETION_INITIALIZER_ONSTACK((ecr).completion), 0 }
>  
>  
>  /* crypto.c */
> 

Replying with Ted's corrected mail address... ;-\

Thanks,
//richard

^ permalink raw reply	[flat|nested] 3+ messages in thread

* Re: [PATCH] fscrypt: properly declare on-stack completion
  2017-02-01 20:32 [PATCH] fscrypt: properly declare on-stack completion Richard Weinberger
  2017-02-01 20:34 ` Richard Weinberger
@ 2017-02-07  4:21 ` Theodore Ts'o
  1 sibling, 0 replies; 3+ messages in thread
From: Theodore Ts'o @ 2017-02-07  4:21 UTC (permalink / raw)
  To: Richard Weinberger
  Cc: linux-fsdevel, linux-kernel, jaegeuk, tytso, ebiggers, david

On Wed, Feb 01, 2017 at 09:32:09PM +0100, Richard Weinberger wrote:
> When a completion is declared on-stack we have to use
> COMPLETION_INITIALIZER_ONSTACK().
> 
> Fixes: 0b81d07790726 ("fs crypto: move per-file encryption from f2fs
> tree to fs/crypto")
> Signed-off-by: Richard Weinberger <richard@nod.at>

Thanks, applied.

					- Ted

^ permalink raw reply	[flat|nested] 3+ messages in thread

end of thread, other threads:[~2017-02-07  4:21 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2017-02-01 20:32 [PATCH] fscrypt: properly declare on-stack completion Richard Weinberger
2017-02-01 20:34 ` Richard Weinberger
2017-02-07  4:21 ` Theodore Ts'o

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.