On 2018/4/5 11:58, Gao Xiang wrote:
> Keep in line with the common case since it is some weird
> to take the address of an array again.

I encounter compile error after applying this patch:

super.c: In function ‘f2fs_fill_super’:
super.c:2711:2: error: incompatible type for argument 1 of ‘__builtin_memcpy’
  memcpy(sb->s_uuid, raw_super->uuid, sizeof(raw_super->uuid));
  ^
super.c:2711:2: note: expected ‘void *’ but argument is of type ‘uuid_t’

Anyway, we need '&' due to sb::s_uuid is a structure object instead of an array.

Thanks,

> 
> Signed-off-by: Gao Xiang <gaoxian...@huawei.com>
> ---
> fix auto-wrapping of email client
> 
>  fs/f2fs/super.c | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/fs/f2fs/super.c b/fs/f2fs/super.c
> index 9587ca0..4d467c7 100644
> --- a/fs/f2fs/super.c
> +++ b/fs/f2fs/super.c
> @@ -2565,7 +2565,7 @@ static int f2fs_fill_super(struct super_block *sb, void 
> *data, int silent)
>       sb->s_time_gran = 1;
>       sb->s_flags = (sb->s_flags & ~SB_POSIXACL) |
>               (test_opt(sbi, POSIX_ACL) ? SB_POSIXACL : 0);
> -     memcpy(&sb->s_uuid, raw_super->uuid, sizeof(raw_super->uuid));
> +     memcpy(sb->s_uuid, raw_super->uuid, sizeof(raw_super->uuid));
>       sb->s_iflags |= SB_I_CGROUPWB;
>  
>       /* init f2fs-specific super block info */
> 

------------------------------------------------------------------------------
Check out the vibrant tech community on one of the world's most
engaging tech sites, Slashdot.org! http://sdm.link/slashdot
_______________________________________________
Linux-f2fs-devel mailing list
Linux-f2fs-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/linux-f2fs-devel

Reply via email to