Hi Chao and Jaegeuk, On 2018/4/8 20:16, Chao Yu wrote: > 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, > Sorry, I'm developing another feature related to it. But I checked the source code about sb->uuid from 3.x to 4.9. https://elixir.bootlin.com/linux/v4.9.93/source/include/linux/fs.h#L1382
It seems that the latest kernel changed from u8 s_uuid[16] to uuid_t s_uuid. Sorry for annoying, please ignore this patch. 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
