* [PATCH 0/5] erofs-utils: fix checkpatch.pl complains @ 2021-08-31 16:51 Huang Jianan 2021-08-31 16:51 ` [PATCH 1/5] erofs-utils: remove filename in the file Huang Jianan ` (5 more replies) 0 siblings, 6 replies; 21+ messages in thread From: Huang Jianan @ 2021-08-31 16:51 UTC (permalink / raw) To: linux-erofs From: Huang Jianan <huangjianan@oppo.com> This patchset fix most of checkpatch.pl complains in erofs-utils, some problems that also exist in the kernel haven't been fixed. Huang Jianan (5): erofs-utils: remove filename in the file erofs-utils: fix SPDX comment style erofs-utils: fix general style problem erofs-utils: remove unnecessary codes and comments erofs-utils: fix print style Makefile.am | 1 - fuse/Makefile.am | 1 - fuse/dir.c | 2 -- fuse/macosx.h | 1 + fuse/main.c | 7 +------ include/erofs/block_list.h | 8 +++----- include/erofs/cache.h | 2 -- include/erofs/compress.h | 2 -- include/erofs/config.h | 2 -- include/erofs/decompress.h | 2 -- include/erofs/defs.h | 2 -- include/erofs/err.h | 2 -- include/erofs/exclude.h | 2 -- include/erofs/hashtable.h | 2 -- include/erofs/inode.h | 2 -- include/erofs/internal.h | 2 -- include/erofs/io.h | 2 -- include/erofs/list.h | 2 -- include/erofs/print.h | 2 -- include/erofs/trace.h | 2 -- include/erofs/xattr.h | 4 +--- include/erofs_fs.h | 1 - lib/Makefile.am | 1 - lib/block_list.c | 7 +------ lib/cache.c | 2 -- lib/compress.c | 11 ++++------- lib/compressor.c | 4 +--- lib/compressor.h | 2 -- lib/compressor_lz4.c | 2 -- lib/compressor_lz4hc.c | 2 -- lib/config.c | 2 -- lib/data.c | 2 -- lib/decompress.c | 2 -- lib/exclude.c | 2 -- lib/inode.c | 9 +-------- lib/io.c | 5 +---- lib/namei.c | 5 ++--- lib/super.c | 2 -- lib/xattr.c | 4 +--- lib/zmap.c | 6 +----- man/Makefile.am | 1 - mkfs/Makefile.am | 1 - mkfs/main.c | 4 +--- 43 files changed, 19 insertions(+), 110 deletions(-) -- 2.25.1 ^ permalink raw reply [flat|nested] 21+ messages in thread
* [PATCH 1/5] erofs-utils: remove filename in the file 2021-08-31 16:51 [PATCH 0/5] erofs-utils: fix checkpatch.pl complains Huang Jianan @ 2021-08-31 16:51 ` Huang Jianan 2021-09-03 3:09 ` Gao Xiang 2021-08-31 16:51 ` [PATCH 2/5] erofs-utils: fix SPDX comment style Huang Jianan ` (4 subsequent siblings) 5 siblings, 1 reply; 21+ messages in thread From: Huang Jianan @ 2021-08-31 16:51 UTC (permalink / raw) To: linux-erofs From: Huang Jianan <huangjianan@oppo.com> It's generally not useful to have the filename in the file. Signed-off-by: Huang Jianan <huangjianan@oppo.com> --- Makefile.am | 1 - fuse/Makefile.am | 1 - fuse/dir.c | 2 -- fuse/main.c | 2 -- include/erofs/block_list.h | 2 -- include/erofs/cache.h | 2 -- include/erofs/compress.h | 2 -- include/erofs/config.h | 2 -- include/erofs/decompress.h | 2 -- include/erofs/defs.h | 2 -- include/erofs/err.h | 2 -- include/erofs/exclude.h | 2 -- include/erofs/hashtable.h | 2 -- include/erofs/inode.h | 2 -- include/erofs/internal.h | 2 -- include/erofs/io.h | 2 -- include/erofs/list.h | 2 -- include/erofs/print.h | 2 -- include/erofs/trace.h | 2 -- include/erofs/xattr.h | 2 -- include/erofs_fs.h | 1 - lib/Makefile.am | 1 - lib/block_list.c | 2 -- lib/cache.c | 2 -- lib/compress.c | 2 -- lib/compressor.c | 2 -- lib/compressor.h | 2 -- lib/compressor_lz4.c | 2 -- lib/compressor_lz4hc.c | 2 -- lib/config.c | 2 -- lib/data.c | 2 -- lib/decompress.c | 2 -- lib/exclude.c | 2 -- lib/inode.c | 2 -- lib/io.c | 2 -- lib/namei.c | 2 -- lib/super.c | 2 -- lib/xattr.c | 2 -- lib/zmap.c | 2 -- man/Makefile.am | 1 - mkfs/Makefile.am | 1 - mkfs/main.c | 2 -- 42 files changed, 78 deletions(-) diff --git a/Makefile.am b/Makefile.am index b804aa9..9a49efc 100644 --- a/Makefile.am +++ b/Makefile.am @@ -1,5 +1,4 @@ # SPDX-License-Identifier: GPL-2.0+ -# Makefile.am ACLOCAL_AMFLAGS = -I m4 diff --git a/fuse/Makefile.am b/fuse/Makefile.am index e7757bc..962b467 100644 --- a/fuse/Makefile.am +++ b/fuse/Makefile.am @@ -1,5 +1,4 @@ # SPDX-License-Identifier: GPL-2.0+ -# Makefile.am AUTOMAKE_OPTIONS = foreign bin_PROGRAMS = erofsfuse diff --git a/fuse/dir.c b/fuse/dir.c index e16fda1..9d95ec6 100644 --- a/fuse/dir.c +++ b/fuse/dir.c @@ -1,7 +1,5 @@ // SPDX-License-Identifier: GPL-2.0+ /* - * erofs-utils/fuse/dir.c - * * Created by Li Guifu <blucerlee@gmail.com> */ #include <fuse.h> diff --git a/fuse/main.c b/fuse/main.c index 5552480..34a9b7a 100644 --- a/fuse/main.c +++ b/fuse/main.c @@ -1,7 +1,5 @@ // SPDX-License-Identifier: GPL-2.0+ /* - * erofs-utils/fuse/main.c - * * Created by Li Guifu <blucerlee@gmail.com> */ #include <stdlib.h> diff --git a/include/erofs/block_list.h b/include/erofs/block_list.h index 7756d8a..5127b23 100644 --- a/include/erofs/block_list.h +++ b/include/erofs/block_list.h @@ -1,7 +1,5 @@ // SPDX-License-Identifier: GPL-2.0+ /* - * erofs-utils/include/erofs/block_list.h - * * Copyright (C), 2021, Coolpad Group Limited. * Created by Yue Hu <huyue2@yulong.com> */ diff --git a/include/erofs/cache.h b/include/erofs/cache.h index 611ca5b..a249d30 100644 --- a/include/erofs/cache.h +++ b/include/erofs/cache.h @@ -1,7 +1,5 @@ /* SPDX-License-Identifier: GPL-2.0+ */ /* - * erofs-utils/include/erofs/cache.h - * * Copyright (C) 2018 HUAWEI, Inc. * http://www.huawei.com/ * Created by Miao Xie <miaoxie@huawei.com> diff --git a/include/erofs/compress.h b/include/erofs/compress.h index d234e8b..47fd489 100644 --- a/include/erofs/compress.h +++ b/include/erofs/compress.h @@ -1,7 +1,5 @@ /* SPDX-License-Identifier: GPL-2.0+ */ /* - * erofs-utils/include/erofs/compress.h - * * Copyright (C) 2019 HUAWEI, Inc. * http://www.huawei.com/ * Created by Gao Xiang <gaoxiang25@huawei.com> diff --git a/include/erofs/config.h b/include/erofs/config.h index 8124f3b..896049b 100644 --- a/include/erofs/config.h +++ b/include/erofs/config.h @@ -1,7 +1,5 @@ /* SPDX-License-Identifier: GPL-2.0+ */ /* - * erofs-utils/include/erofs/config.h - * * Copyright (C) 2018-2019 HUAWEI, Inc. * http://www.huawei.com/ * Created by Li Guifu <bluce.liguifu@huawei.com> diff --git a/include/erofs/decompress.h b/include/erofs/decompress.h index beaac35..0ba2b08 100644 --- a/include/erofs/decompress.h +++ b/include/erofs/decompress.h @@ -1,7 +1,5 @@ /* SPDX-License-Identifier: GPL-2.0+ */ /* - * erofs-utils/include/erofs/decompress.h - * * Copyright (C), 2008-2020, OPPO Mobile Comm Corp., Ltd. * Created by Huang Jianan <huangjianan@oppo.com> */ diff --git a/include/erofs/defs.h b/include/erofs/defs.h index 5410685..6e0a777 100644 --- a/include/erofs/defs.h +++ b/include/erofs/defs.h @@ -1,7 +1,5 @@ /* SPDX-License-Identifier: GPL-2.0+ */ /* - * erofs-utils/include/erofs/defs.h - * * Copyright (C) 2018 HUAWEI, Inc. * http://www.huawei.com/ * Created by Li Guifu <bluce.liguifu@huawei.com> diff --git a/include/erofs/err.h b/include/erofs/err.h index da3b681..2ff77a4 100644 --- a/include/erofs/err.h +++ b/include/erofs/err.h @@ -1,7 +1,5 @@ /* SPDX-License-Identifier: GPL-2.0+ */ /* - * erofs-utils/include/erofs/err.h - * * Copyright (C) 2018 HUAWEI, Inc. * http://www.huawei.com/ * Created by Li Guifu <bluce.liguifu@huawei.com> diff --git a/include/erofs/exclude.h b/include/erofs/exclude.h index 88c55d7..98217e4 100644 --- a/include/erofs/exclude.h +++ b/include/erofs/exclude.h @@ -1,7 +1,5 @@ /* SPDX-License-Identifier: GPL-2.0+ */ /* - * erofs-utils/include/erofs/exclude.h - * * Created by Li Guifu <bluce.lee@aliyun.com> */ #ifndef __EROFS_EXCLUDE_H diff --git a/include/erofs/hashtable.h b/include/erofs/hashtable.h index 7e47189..a71cb00 100644 --- a/include/erofs/hashtable.h +++ b/include/erofs/hashtable.h @@ -1,7 +1,5 @@ /* SPDX-License-Identifier: GPL-2.0 */ /* - * erofs-utils/include/erofs/hashtable.h - * * Original code taken from 'linux/include/linux/hash{,table}.h' */ #ifndef __EROFS_HASHTABLE_H diff --git a/include/erofs/inode.h b/include/erofs/inode.h index 5a7f5f1..a736762 100644 --- a/include/erofs/inode.h +++ b/include/erofs/inode.h @@ -1,7 +1,5 @@ /* SPDX-License-Identifier: GPL-2.0+ */ /* - * erofs-utils/include/erofs/inode.h - * * Copyright (C) 2018-2019 HUAWEI, Inc. * http://www.huawei.com/ * Created by Li Guifu <bluce.liguifu@huawei.com> diff --git a/include/erofs/internal.h b/include/erofs/internal.h index 5583861..7dc5ff0 100644 --- a/include/erofs/internal.h +++ b/include/erofs/internal.h @@ -1,7 +1,5 @@ /* SPDX-License-Identifier: GPL-2.0+ */ /* - * erofs-utils/include/erofs/internal.h - * * Copyright (C) 2019 HUAWEI, Inc. * http://www.huawei.com/ * Created by Gao Xiang <gaoxiang25@huawei.com> diff --git a/include/erofs/io.h b/include/erofs/io.h index 5574245..20b25d0 100644 --- a/include/erofs/io.h +++ b/include/erofs/io.h @@ -1,7 +1,5 @@ /* SPDX-License-Identifier: GPL-2.0+ */ /* - * erofs-utils/include/erofs/io.h - * * Copyright (C) 2018-2019 HUAWEI, Inc. * http://www.huawei.com/ * Created by Li Guifu <bluce.liguifu@huawei.com> diff --git a/include/erofs/list.h b/include/erofs/list.h index 3572726..d2bc704 100644 --- a/include/erofs/list.h +++ b/include/erofs/list.h @@ -1,7 +1,5 @@ /* SPDX-License-Identifier: GPL-2.0+ */ /* - * erofs-utils/include/erofs/list.h - * * Copyright (C) 2018 HUAWEI, Inc. * http://www.huawei.com/ * Created by Li Guifu <bluce.liguifu@huawei.com> diff --git a/include/erofs/print.h b/include/erofs/print.h index 6b79074..9c08a50 100644 --- a/include/erofs/print.h +++ b/include/erofs/print.h @@ -1,7 +1,5 @@ /* SPDX-License-Identifier: GPL-2.0+ */ /* - * erofs-utils/include/erofs/print.h - * * Copyright (C) 2018-2019 HUAWEI, Inc. * http://www.huawei.com/ * Created by Li Guifu <bluce.liguifu@huawei.com> diff --git a/include/erofs/trace.h b/include/erofs/trace.h index 5a12da7..5412ded 100644 --- a/include/erofs/trace.h +++ b/include/erofs/trace.h @@ -1,7 +1,5 @@ /* SPDX-License-Identifier: GPL-2.0+ */ /* - * erofs-utils/include/erofs/trace.h - * * Copyright (C) 2020 Gao Xiang <hsiangkao@aol.com> */ #ifndef __EROFS_TRACE_H diff --git a/include/erofs/xattr.h b/include/erofs/xattr.h index 197fe25..5086b54 100644 --- a/include/erofs/xattr.h +++ b/include/erofs/xattr.h @@ -1,7 +1,5 @@ // SPDX-License-Identifier: GPL-2.0+ /* - * erofs-utils/include/erofs/xattr.h - * * Originally contributed by an anonymous person, * heavily changed by Li Guifu <blucerlee@gmail.com> * and Gao Xiang <xiang@kernel.org> diff --git a/include/erofs_fs.h b/include/erofs_fs.h index 18fc182..f9cdd71 100644 --- a/include/erofs_fs.h +++ b/include/erofs_fs.h @@ -1,6 +1,5 @@ /* SPDX-License-Identifier: GPL-2.0-only OR Apache-2.0 */ /* - * erofs-utils/include/erofs_fs.h * EROFS (Enhanced ROM File System) on-disk format definition * * Copyright (C) 2017-2018 HUAWEI, Inc. diff --git a/lib/Makefile.am b/lib/Makefile.am index b12e2c1..87e6411 100644 --- a/lib/Makefile.am +++ b/lib/Makefile.am @@ -1,5 +1,4 @@ # SPDX-License-Identifier: GPL-2.0+ -# Makefile.am noinst_LTLIBRARIES = liberofs.la noinst_HEADERS = $(top_srcdir)/include/erofs_fs.h \ diff --git a/lib/block_list.c b/lib/block_list.c index 3be0992..73c1bde 100644 --- a/lib/block_list.c +++ b/lib/block_list.c @@ -1,7 +1,5 @@ // SPDX-License-Identifier: GPL-2.0+ /* - * erofs-utils/lib/block_list.c - * * Copyright (C), 2021, Coolpad Group Limited. * Created by Yue Hu <huyue2@yulong.com> */ diff --git a/lib/cache.c b/lib/cache.c index 340dcdd..0a1c0cf 100644 --- a/lib/cache.c +++ b/lib/cache.c @@ -1,7 +1,5 @@ // SPDX-License-Identifier: GPL-2.0+ /* - * erofs-utils/lib/cache.c - * * Copyright (C) 2018-2019 HUAWEI, Inc. * http://www.huawei.com/ * Created by Miao Xie <miaoxie@huawei.com> diff --git a/lib/compress.c b/lib/compress.c index a8ebbc1..2b12d67 100644 --- a/lib/compress.c +++ b/lib/compress.c @@ -1,7 +1,5 @@ // SPDX-License-Identifier: GPL-2.0+ /* - * erofs-utils/lib/compress.c - * * Copyright (C) 2018-2019 HUAWEI, Inc. * http://www.huawei.com/ * Created by Miao Xie <miaoxie@huawei.com> diff --git a/lib/compressor.c b/lib/compressor.c index 8836e0c..846a836 100644 --- a/lib/compressor.c +++ b/lib/compressor.c @@ -1,7 +1,5 @@ // SPDX-License-Identifier: GPL-2.0+ /* - * erofs-utils/lib/compressor.c - * * Copyright (C) 2018-2019 HUAWEI, Inc. * http://www.huawei.com/ * Created by Gao Xiang <gaoxiang25@huawei.com> diff --git a/lib/compressor.h b/lib/compressor.h index b2471c4..132bd65 100644 --- a/lib/compressor.h +++ b/lib/compressor.h @@ -1,7 +1,5 @@ /* SPDX-License-Identifier: GPL-2.0+ */ /* - * erofs-utils/lib/compressor.h - * * Copyright (C) 2018-2019 HUAWEI, Inc. * http://www.huawei.com/ * Created by Gao Xiang <gaoxiang25@huawei.com> diff --git a/lib/compressor_lz4.c b/lib/compressor_lz4.c index 292d0f2..ce8c472 100644 --- a/lib/compressor_lz4.c +++ b/lib/compressor_lz4.c @@ -1,7 +1,5 @@ // SPDX-License-Identifier: GPL-2.0+ /* - * erofs-utils/lib/compressor-lz4.c - * * Copyright (C) 2018-2019 HUAWEI, Inc. * http://www.huawei.com/ * Created by Gao Xiang <gaoxiang25@huawei.com> diff --git a/lib/compressor_lz4hc.c b/lib/compressor_lz4hc.c index 14c3a71..e345f70 100644 --- a/lib/compressor_lz4hc.c +++ b/lib/compressor_lz4hc.c @@ -1,7 +1,5 @@ // SPDX-License-Identifier: GPL-2.0+ /* - * erofs-utils/lib/compressor-lz4hc.c - * * Copyright (C) 2018-2019 HUAWEI, Inc. * http://www.huawei.com/ * Created by Gao Xiang <gaoxiang25@huawei.com> diff --git a/lib/config.c b/lib/config.c index 99fcf49..aa328c4 100644 --- a/lib/config.c +++ b/lib/config.c @@ -1,7 +1,5 @@ // SPDX-License-Identifier: GPL-2.0+ /* - * erofs-utils/lib/config.c - * * Copyright (C) 2018-2019 HUAWEI, Inc. * http://www.huawei.com/ * Created by Li Guifu <bluce.liguifu@huawei.com> diff --git a/lib/data.c b/lib/data.c index 42b4904..32d4c00 100644 --- a/lib/data.c +++ b/lib/data.c @@ -1,7 +1,5 @@ // SPDX-License-Identifier: GPL-2.0+ /* - * erofs-utils/lib/data.c - * * Copyright (C) 2020 Gao Xiang <hsiangkao@aol.com> * Compression support by Huang Jianan <huangjianan@oppo.com> */ diff --git a/lib/decompress.c b/lib/decompress.c index 490c4bc..2ee1439 100644 --- a/lib/decompress.c +++ b/lib/decompress.c @@ -1,7 +1,5 @@ // SPDX-License-Identifier: GPL-2.0+ /* - * erofs-utils/lib/decompress.c - * * Copyright (C), 2008-2020, OPPO Mobile Comm Corp., Ltd. * Created by Huang Jianan <huangjianan@oppo.com> */ diff --git a/lib/exclude.c b/lib/exclude.c index 73b3720..e9e8b9b 100644 --- a/lib/exclude.c +++ b/lib/exclude.c @@ -1,7 +1,5 @@ // SPDX-License-Identifier: GPL-2.0+ /* - * erofs-utils/lib/exclude.c - * * Created by Li Guifu <bluce.lee@aliyun.com> */ #include <string.h> diff --git a/lib/inode.c b/lib/inode.c index 6871d2b..62047d3 100644 --- a/lib/inode.c +++ b/lib/inode.c @@ -1,7 +1,5 @@ // SPDX-License-Identifier: GPL-2.0+ /* - * erofs-utils/lib/inode.c - * * Copyright (C) 2018-2019 HUAWEI, Inc. * http://www.huawei.com/ * Created by Li Guifu <bluce.liguifu@huawei.com> diff --git a/lib/io.c b/lib/io.c index 6067041..b053137 100644 --- a/lib/io.c +++ b/lib/io.c @@ -1,7 +1,5 @@ // SPDX-License-Identifier: GPL-2.0+ /* - * erofs-utils/lib/io.c - * * Copyright (C) 2018 HUAWEI, Inc. * http://www.huawei.com/ * Created by Li Guifu <bluce.liguifu@huawei.com> diff --git a/lib/namei.c b/lib/namei.c index b572d17..f4094a1 100644 --- a/lib/namei.c +++ b/lib/namei.c @@ -1,7 +1,5 @@ // SPDX-License-Identifier: GPL-2.0+ /* - * erofs-utils/lib/namei.c - * * Created by Li Guifu <blucerlee@gmail.com> */ #include <sys/types.h> diff --git a/lib/super.c b/lib/super.c index 11405ec..c4a67f3 100644 --- a/lib/super.c +++ b/lib/super.c @@ -1,7 +1,5 @@ // SPDX-License-Identifier: GPL-2.0+ /* - * erofs-utils/lib/super.c - * * Created by Li Guifu <blucerlee@gmail.com> */ #include <string.h> diff --git a/lib/xattr.c b/lib/xattr.c index aff3d67..39d4a96 100644 --- a/lib/xattr.c +++ b/lib/xattr.c @@ -1,7 +1,5 @@ // SPDX-License-Identifier: GPL-2.0+ /* - * erofs-utils/lib/xattr.c - * * Originally contributed by an anonymous person, * heavily changed by Li Guifu <blucerlee@gmail.com> * and Gao Xiang <hsiangkao@aol.com> diff --git a/lib/zmap.c b/lib/zmap.c index 1084faa..fdc84af 100644 --- a/lib/zmap.c +++ b/lib/zmap.c @@ -1,7 +1,5 @@ // SPDX-License-Identifier: GPL-2.0+ /* - * erofs-utils/lib/zmap.c - * * (a large amount of code was adapted from Linux kernel. ) * * Copyright (C) 2018-2019 HUAWEI, Inc. diff --git a/man/Makefile.am b/man/Makefile.am index 0df947b..d62d6e2 100644 --- a/man/Makefile.am +++ b/man/Makefile.am @@ -1,5 +1,4 @@ # SPDX-License-Identifier: GPL-2.0+ -# Makefile.am dist_man_MANS = mkfs.erofs.1 diff --git a/mkfs/Makefile.am b/mkfs/Makefile.am index 8b8e051..bcef7e1 100644 --- a/mkfs/Makefile.am +++ b/mkfs/Makefile.am @@ -1,5 +1,4 @@ # SPDX-License-Identifier: GPL-2.0+ -# Makefile.am AUTOMAKE_OPTIONS = foreign bin_PROGRAMS = mkfs.erofs diff --git a/mkfs/main.c b/mkfs/main.c index 89f2310..debb754 100644 --- a/mkfs/main.c +++ b/mkfs/main.c @@ -1,7 +1,5 @@ // SPDX-License-Identifier: GPL-2.0+ /* - * mkfs/main.c - * * Copyright (C) 2018-2019 HUAWEI, Inc. * http://www.huawei.com/ * Created by Li Guifu <bluce.liguifu@huawei.com> -- 2.25.1 ^ permalink raw reply related [flat|nested] 21+ messages in thread
* Re: [PATCH 1/5] erofs-utils: remove filename in the file 2021-08-31 16:51 ` [PATCH 1/5] erofs-utils: remove filename in the file Huang Jianan @ 2021-09-03 3:09 ` Gao Xiang 0 siblings, 0 replies; 21+ messages in thread From: Gao Xiang @ 2021-09-03 3:09 UTC (permalink / raw) To: Huang Jianan; +Cc: linux-erofs On Wed, Sep 01, 2021 at 12:51:12AM +0800, Huang Jianan wrote: > From: Huang Jianan <huangjianan@oppo.com> > > It's generally not useful to have the filename in the file. > > Signed-off-by: Huang Jianan <huangjianan@oppo.com> If we decide to clean up this stuff, could we remove trailing newline like the commit below? https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git/commit/fs/erofs?id=c5fcb51111b85323cafe3f02784f7f0bf6a7cf07 Thanks, Gao Xiang > --- > Makefile.am | 1 - > fuse/Makefile.am | 1 - > fuse/dir.c | 2 -- > fuse/main.c | 2 -- > include/erofs/block_list.h | 2 -- > include/erofs/cache.h | 2 -- > include/erofs/compress.h | 2 -- > include/erofs/config.h | 2 -- > include/erofs/decompress.h | 2 -- > include/erofs/defs.h | 2 -- > include/erofs/err.h | 2 -- > include/erofs/exclude.h | 2 -- > include/erofs/hashtable.h | 2 -- > include/erofs/inode.h | 2 -- > include/erofs/internal.h | 2 -- > include/erofs/io.h | 2 -- > include/erofs/list.h | 2 -- > include/erofs/print.h | 2 -- > include/erofs/trace.h | 2 -- > include/erofs/xattr.h | 2 -- > include/erofs_fs.h | 1 - > lib/Makefile.am | 1 - > lib/block_list.c | 2 -- > lib/cache.c | 2 -- > lib/compress.c | 2 -- > lib/compressor.c | 2 -- > lib/compressor.h | 2 -- > lib/compressor_lz4.c | 2 -- > lib/compressor_lz4hc.c | 2 -- > lib/config.c | 2 -- > lib/data.c | 2 -- > lib/decompress.c | 2 -- > lib/exclude.c | 2 -- > lib/inode.c | 2 -- > lib/io.c | 2 -- > lib/namei.c | 2 -- > lib/super.c | 2 -- > lib/xattr.c | 2 -- > lib/zmap.c | 2 -- > man/Makefile.am | 1 - > mkfs/Makefile.am | 1 - > mkfs/main.c | 2 -- > 42 files changed, 78 deletions(-) > > diff --git a/Makefile.am b/Makefile.am > index b804aa9..9a49efc 100644 > --- a/Makefile.am > +++ b/Makefile.am > @@ -1,5 +1,4 @@ > # SPDX-License-Identifier: GPL-2.0+ > -# Makefile.am > > ACLOCAL_AMFLAGS = -I m4 > > diff --git a/fuse/Makefile.am b/fuse/Makefile.am > index e7757bc..962b467 100644 > --- a/fuse/Makefile.am > +++ b/fuse/Makefile.am > @@ -1,5 +1,4 @@ > # SPDX-License-Identifier: GPL-2.0+ > -# Makefile.am > > AUTOMAKE_OPTIONS = foreign > bin_PROGRAMS = erofsfuse > diff --git a/fuse/dir.c b/fuse/dir.c > index e16fda1..9d95ec6 100644 > --- a/fuse/dir.c > +++ b/fuse/dir.c > @@ -1,7 +1,5 @@ > // SPDX-License-Identifier: GPL-2.0+ > /* > - * erofs-utils/fuse/dir.c > - * > * Created by Li Guifu <blucerlee@gmail.com> > */ > #include <fuse.h> > diff --git a/fuse/main.c b/fuse/main.c > index 5552480..34a9b7a 100644 > --- a/fuse/main.c > +++ b/fuse/main.c > @@ -1,7 +1,5 @@ > // SPDX-License-Identifier: GPL-2.0+ > /* > - * erofs-utils/fuse/main.c > - * > * Created by Li Guifu <blucerlee@gmail.com> > */ > #include <stdlib.h> > diff --git a/include/erofs/block_list.h b/include/erofs/block_list.h > index 7756d8a..5127b23 100644 > --- a/include/erofs/block_list.h > +++ b/include/erofs/block_list.h > @@ -1,7 +1,5 @@ > // SPDX-License-Identifier: GPL-2.0+ > /* > - * erofs-utils/include/erofs/block_list.h > - * > * Copyright (C), 2021, Coolpad Group Limited. > * Created by Yue Hu <huyue2@yulong.com> > */ > diff --git a/include/erofs/cache.h b/include/erofs/cache.h > index 611ca5b..a249d30 100644 > --- a/include/erofs/cache.h > +++ b/include/erofs/cache.h > @@ -1,7 +1,5 @@ > /* SPDX-License-Identifier: GPL-2.0+ */ > /* > - * erofs-utils/include/erofs/cache.h > - * > * Copyright (C) 2018 HUAWEI, Inc. > * http://www.huawei.com/ > * Created by Miao Xie <miaoxie@huawei.com> > diff --git a/include/erofs/compress.h b/include/erofs/compress.h > index d234e8b..47fd489 100644 > --- a/include/erofs/compress.h > +++ b/include/erofs/compress.h > @@ -1,7 +1,5 @@ > /* SPDX-License-Identifier: GPL-2.0+ */ > /* > - * erofs-utils/include/erofs/compress.h > - * > * Copyright (C) 2019 HUAWEI, Inc. > * http://www.huawei.com/ > * Created by Gao Xiang <gaoxiang25@huawei.com> > diff --git a/include/erofs/config.h b/include/erofs/config.h > index 8124f3b..896049b 100644 > --- a/include/erofs/config.h > +++ b/include/erofs/config.h > @@ -1,7 +1,5 @@ > /* SPDX-License-Identifier: GPL-2.0+ */ > /* > - * erofs-utils/include/erofs/config.h > - * > * Copyright (C) 2018-2019 HUAWEI, Inc. > * http://www.huawei.com/ > * Created by Li Guifu <bluce.liguifu@huawei.com> > diff --git a/include/erofs/decompress.h b/include/erofs/decompress.h > index beaac35..0ba2b08 100644 > --- a/include/erofs/decompress.h > +++ b/include/erofs/decompress.h > @@ -1,7 +1,5 @@ > /* SPDX-License-Identifier: GPL-2.0+ */ > /* > - * erofs-utils/include/erofs/decompress.h > - * > * Copyright (C), 2008-2020, OPPO Mobile Comm Corp., Ltd. > * Created by Huang Jianan <huangjianan@oppo.com> > */ > diff --git a/include/erofs/defs.h b/include/erofs/defs.h > index 5410685..6e0a777 100644 > --- a/include/erofs/defs.h > +++ b/include/erofs/defs.h > @@ -1,7 +1,5 @@ > /* SPDX-License-Identifier: GPL-2.0+ */ > /* > - * erofs-utils/include/erofs/defs.h > - * > * Copyright (C) 2018 HUAWEI, Inc. > * http://www.huawei.com/ > * Created by Li Guifu <bluce.liguifu@huawei.com> > diff --git a/include/erofs/err.h b/include/erofs/err.h > index da3b681..2ff77a4 100644 > --- a/include/erofs/err.h > +++ b/include/erofs/err.h > @@ -1,7 +1,5 @@ > /* SPDX-License-Identifier: GPL-2.0+ */ > /* > - * erofs-utils/include/erofs/err.h > - * > * Copyright (C) 2018 HUAWEI, Inc. > * http://www.huawei.com/ > * Created by Li Guifu <bluce.liguifu@huawei.com> > diff --git a/include/erofs/exclude.h b/include/erofs/exclude.h > index 88c55d7..98217e4 100644 > --- a/include/erofs/exclude.h > +++ b/include/erofs/exclude.h > @@ -1,7 +1,5 @@ > /* SPDX-License-Identifier: GPL-2.0+ */ > /* > - * erofs-utils/include/erofs/exclude.h > - * > * Created by Li Guifu <bluce.lee@aliyun.com> > */ > #ifndef __EROFS_EXCLUDE_H > diff --git a/include/erofs/hashtable.h b/include/erofs/hashtable.h > index 7e47189..a71cb00 100644 > --- a/include/erofs/hashtable.h > +++ b/include/erofs/hashtable.h > @@ -1,7 +1,5 @@ > /* SPDX-License-Identifier: GPL-2.0 */ > /* > - * erofs-utils/include/erofs/hashtable.h > - * > * Original code taken from 'linux/include/linux/hash{,table}.h' > */ > #ifndef __EROFS_HASHTABLE_H > diff --git a/include/erofs/inode.h b/include/erofs/inode.h > index 5a7f5f1..a736762 100644 > --- a/include/erofs/inode.h > +++ b/include/erofs/inode.h > @@ -1,7 +1,5 @@ > /* SPDX-License-Identifier: GPL-2.0+ */ > /* > - * erofs-utils/include/erofs/inode.h > - * > * Copyright (C) 2018-2019 HUAWEI, Inc. > * http://www.huawei.com/ > * Created by Li Guifu <bluce.liguifu@huawei.com> > diff --git a/include/erofs/internal.h b/include/erofs/internal.h > index 5583861..7dc5ff0 100644 > --- a/include/erofs/internal.h > +++ b/include/erofs/internal.h > @@ -1,7 +1,5 @@ > /* SPDX-License-Identifier: GPL-2.0+ */ > /* > - * erofs-utils/include/erofs/internal.h > - * > * Copyright (C) 2019 HUAWEI, Inc. > * http://www.huawei.com/ > * Created by Gao Xiang <gaoxiang25@huawei.com> > diff --git a/include/erofs/io.h b/include/erofs/io.h > index 5574245..20b25d0 100644 > --- a/include/erofs/io.h > +++ b/include/erofs/io.h > @@ -1,7 +1,5 @@ > /* SPDX-License-Identifier: GPL-2.0+ */ > /* > - * erofs-utils/include/erofs/io.h > - * > * Copyright (C) 2018-2019 HUAWEI, Inc. > * http://www.huawei.com/ > * Created by Li Guifu <bluce.liguifu@huawei.com> > diff --git a/include/erofs/list.h b/include/erofs/list.h > index 3572726..d2bc704 100644 > --- a/include/erofs/list.h > +++ b/include/erofs/list.h > @@ -1,7 +1,5 @@ > /* SPDX-License-Identifier: GPL-2.0+ */ > /* > - * erofs-utils/include/erofs/list.h > - * > * Copyright (C) 2018 HUAWEI, Inc. > * http://www.huawei.com/ > * Created by Li Guifu <bluce.liguifu@huawei.com> > diff --git a/include/erofs/print.h b/include/erofs/print.h > index 6b79074..9c08a50 100644 > --- a/include/erofs/print.h > +++ b/include/erofs/print.h > @@ -1,7 +1,5 @@ > /* SPDX-License-Identifier: GPL-2.0+ */ > /* > - * erofs-utils/include/erofs/print.h > - * > * Copyright (C) 2018-2019 HUAWEI, Inc. > * http://www.huawei.com/ > * Created by Li Guifu <bluce.liguifu@huawei.com> > diff --git a/include/erofs/trace.h b/include/erofs/trace.h > index 5a12da7..5412ded 100644 > --- a/include/erofs/trace.h > +++ b/include/erofs/trace.h > @@ -1,7 +1,5 @@ > /* SPDX-License-Identifier: GPL-2.0+ */ > /* > - * erofs-utils/include/erofs/trace.h > - * > * Copyright (C) 2020 Gao Xiang <hsiangkao@aol.com> > */ > #ifndef __EROFS_TRACE_H > diff --git a/include/erofs/xattr.h b/include/erofs/xattr.h > index 197fe25..5086b54 100644 > --- a/include/erofs/xattr.h > +++ b/include/erofs/xattr.h > @@ -1,7 +1,5 @@ > // SPDX-License-Identifier: GPL-2.0+ > /* > - * erofs-utils/include/erofs/xattr.h > - * > * Originally contributed by an anonymous person, > * heavily changed by Li Guifu <blucerlee@gmail.com> > * and Gao Xiang <xiang@kernel.org> > diff --git a/include/erofs_fs.h b/include/erofs_fs.h > index 18fc182..f9cdd71 100644 > --- a/include/erofs_fs.h > +++ b/include/erofs_fs.h > @@ -1,6 +1,5 @@ > /* SPDX-License-Identifier: GPL-2.0-only OR Apache-2.0 */ > /* > - * erofs-utils/include/erofs_fs.h > * EROFS (Enhanced ROM File System) on-disk format definition > * > * Copyright (C) 2017-2018 HUAWEI, Inc. > diff --git a/lib/Makefile.am b/lib/Makefile.am > index b12e2c1..87e6411 100644 > --- a/lib/Makefile.am > +++ b/lib/Makefile.am > @@ -1,5 +1,4 @@ > # SPDX-License-Identifier: GPL-2.0+ > -# Makefile.am > > noinst_LTLIBRARIES = liberofs.la > noinst_HEADERS = $(top_srcdir)/include/erofs_fs.h \ > diff --git a/lib/block_list.c b/lib/block_list.c > index 3be0992..73c1bde 100644 > --- a/lib/block_list.c > +++ b/lib/block_list.c > @@ -1,7 +1,5 @@ > // SPDX-License-Identifier: GPL-2.0+ > /* > - * erofs-utils/lib/block_list.c > - * > * Copyright (C), 2021, Coolpad Group Limited. > * Created by Yue Hu <huyue2@yulong.com> > */ > diff --git a/lib/cache.c b/lib/cache.c > index 340dcdd..0a1c0cf 100644 > --- a/lib/cache.c > +++ b/lib/cache.c > @@ -1,7 +1,5 @@ > // SPDX-License-Identifier: GPL-2.0+ > /* > - * erofs-utils/lib/cache.c > - * > * Copyright (C) 2018-2019 HUAWEI, Inc. > * http://www.huawei.com/ > * Created by Miao Xie <miaoxie@huawei.com> > diff --git a/lib/compress.c b/lib/compress.c > index a8ebbc1..2b12d67 100644 > --- a/lib/compress.c > +++ b/lib/compress.c > @@ -1,7 +1,5 @@ > // SPDX-License-Identifier: GPL-2.0+ > /* > - * erofs-utils/lib/compress.c > - * > * Copyright (C) 2018-2019 HUAWEI, Inc. > * http://www.huawei.com/ > * Created by Miao Xie <miaoxie@huawei.com> > diff --git a/lib/compressor.c b/lib/compressor.c > index 8836e0c..846a836 100644 > --- a/lib/compressor.c > +++ b/lib/compressor.c > @@ -1,7 +1,5 @@ > // SPDX-License-Identifier: GPL-2.0+ > /* > - * erofs-utils/lib/compressor.c > - * > * Copyright (C) 2018-2019 HUAWEI, Inc. > * http://www.huawei.com/ > * Created by Gao Xiang <gaoxiang25@huawei.com> > diff --git a/lib/compressor.h b/lib/compressor.h > index b2471c4..132bd65 100644 > --- a/lib/compressor.h > +++ b/lib/compressor.h > @@ -1,7 +1,5 @@ > /* SPDX-License-Identifier: GPL-2.0+ */ > /* > - * erofs-utils/lib/compressor.h > - * > * Copyright (C) 2018-2019 HUAWEI, Inc. > * http://www.huawei.com/ > * Created by Gao Xiang <gaoxiang25@huawei.com> > diff --git a/lib/compressor_lz4.c b/lib/compressor_lz4.c > index 292d0f2..ce8c472 100644 > --- a/lib/compressor_lz4.c > +++ b/lib/compressor_lz4.c > @@ -1,7 +1,5 @@ > // SPDX-License-Identifier: GPL-2.0+ > /* > - * erofs-utils/lib/compressor-lz4.c > - * > * Copyright (C) 2018-2019 HUAWEI, Inc. > * http://www.huawei.com/ > * Created by Gao Xiang <gaoxiang25@huawei.com> > diff --git a/lib/compressor_lz4hc.c b/lib/compressor_lz4hc.c > index 14c3a71..e345f70 100644 > --- a/lib/compressor_lz4hc.c > +++ b/lib/compressor_lz4hc.c > @@ -1,7 +1,5 @@ > // SPDX-License-Identifier: GPL-2.0+ > /* > - * erofs-utils/lib/compressor-lz4hc.c > - * > * Copyright (C) 2018-2019 HUAWEI, Inc. > * http://www.huawei.com/ > * Created by Gao Xiang <gaoxiang25@huawei.com> > diff --git a/lib/config.c b/lib/config.c > index 99fcf49..aa328c4 100644 > --- a/lib/config.c > +++ b/lib/config.c > @@ -1,7 +1,5 @@ > // SPDX-License-Identifier: GPL-2.0+ > /* > - * erofs-utils/lib/config.c > - * > * Copyright (C) 2018-2019 HUAWEI, Inc. > * http://www.huawei.com/ > * Created by Li Guifu <bluce.liguifu@huawei.com> > diff --git a/lib/data.c b/lib/data.c > index 42b4904..32d4c00 100644 > --- a/lib/data.c > +++ b/lib/data.c > @@ -1,7 +1,5 @@ > // SPDX-License-Identifier: GPL-2.0+ > /* > - * erofs-utils/lib/data.c > - * > * Copyright (C) 2020 Gao Xiang <hsiangkao@aol.com> > * Compression support by Huang Jianan <huangjianan@oppo.com> > */ > diff --git a/lib/decompress.c b/lib/decompress.c > index 490c4bc..2ee1439 100644 > --- a/lib/decompress.c > +++ b/lib/decompress.c > @@ -1,7 +1,5 @@ > // SPDX-License-Identifier: GPL-2.0+ > /* > - * erofs-utils/lib/decompress.c > - * > * Copyright (C), 2008-2020, OPPO Mobile Comm Corp., Ltd. > * Created by Huang Jianan <huangjianan@oppo.com> > */ > diff --git a/lib/exclude.c b/lib/exclude.c > index 73b3720..e9e8b9b 100644 > --- a/lib/exclude.c > +++ b/lib/exclude.c > @@ -1,7 +1,5 @@ > // SPDX-License-Identifier: GPL-2.0+ > /* > - * erofs-utils/lib/exclude.c > - * > * Created by Li Guifu <bluce.lee@aliyun.com> > */ > #include <string.h> > diff --git a/lib/inode.c b/lib/inode.c > index 6871d2b..62047d3 100644 > --- a/lib/inode.c > +++ b/lib/inode.c > @@ -1,7 +1,5 @@ > // SPDX-License-Identifier: GPL-2.0+ > /* > - * erofs-utils/lib/inode.c > - * > * Copyright (C) 2018-2019 HUAWEI, Inc. > * http://www.huawei.com/ > * Created by Li Guifu <bluce.liguifu@huawei.com> > diff --git a/lib/io.c b/lib/io.c > index 6067041..b053137 100644 > --- a/lib/io.c > +++ b/lib/io.c > @@ -1,7 +1,5 @@ > // SPDX-License-Identifier: GPL-2.0+ > /* > - * erofs-utils/lib/io.c > - * > * Copyright (C) 2018 HUAWEI, Inc. > * http://www.huawei.com/ > * Created by Li Guifu <bluce.liguifu@huawei.com> > diff --git a/lib/namei.c b/lib/namei.c > index b572d17..f4094a1 100644 > --- a/lib/namei.c > +++ b/lib/namei.c > @@ -1,7 +1,5 @@ > // SPDX-License-Identifier: GPL-2.0+ > /* > - * erofs-utils/lib/namei.c > - * > * Created by Li Guifu <blucerlee@gmail.com> > */ > #include <sys/types.h> > diff --git a/lib/super.c b/lib/super.c > index 11405ec..c4a67f3 100644 > --- a/lib/super.c > +++ b/lib/super.c > @@ -1,7 +1,5 @@ > // SPDX-License-Identifier: GPL-2.0+ > /* > - * erofs-utils/lib/super.c > - * > * Created by Li Guifu <blucerlee@gmail.com> > */ > #include <string.h> > diff --git a/lib/xattr.c b/lib/xattr.c > index aff3d67..39d4a96 100644 > --- a/lib/xattr.c > +++ b/lib/xattr.c > @@ -1,7 +1,5 @@ > // SPDX-License-Identifier: GPL-2.0+ > /* > - * erofs-utils/lib/xattr.c > - * > * Originally contributed by an anonymous person, > * heavily changed by Li Guifu <blucerlee@gmail.com> > * and Gao Xiang <hsiangkao@aol.com> > diff --git a/lib/zmap.c b/lib/zmap.c > index 1084faa..fdc84af 100644 > --- a/lib/zmap.c > +++ b/lib/zmap.c > @@ -1,7 +1,5 @@ > // SPDX-License-Identifier: GPL-2.0+ > /* > - * erofs-utils/lib/zmap.c > - * > * (a large amount of code was adapted from Linux kernel. ) > * > * Copyright (C) 2018-2019 HUAWEI, Inc. > diff --git a/man/Makefile.am b/man/Makefile.am > index 0df947b..d62d6e2 100644 > --- a/man/Makefile.am > +++ b/man/Makefile.am > @@ -1,5 +1,4 @@ > # SPDX-License-Identifier: GPL-2.0+ > -# Makefile.am > > dist_man_MANS = mkfs.erofs.1 > > diff --git a/mkfs/Makefile.am b/mkfs/Makefile.am > index 8b8e051..bcef7e1 100644 > --- a/mkfs/Makefile.am > +++ b/mkfs/Makefile.am > @@ -1,5 +1,4 @@ > # SPDX-License-Identifier: GPL-2.0+ > -# Makefile.am > > AUTOMAKE_OPTIONS = foreign > bin_PROGRAMS = mkfs.erofs > diff --git a/mkfs/main.c b/mkfs/main.c > index 89f2310..debb754 100644 > --- a/mkfs/main.c > +++ b/mkfs/main.c > @@ -1,7 +1,5 @@ > // SPDX-License-Identifier: GPL-2.0+ > /* > - * mkfs/main.c > - * > * Copyright (C) 2018-2019 HUAWEI, Inc. > * http://www.huawei.com/ > * Created by Li Guifu <bluce.liguifu@huawei.com> > -- > 2.25.1 ^ permalink raw reply [flat|nested] 21+ messages in thread
* [PATCH 2/5] erofs-utils: fix SPDX comment style 2021-08-31 16:51 [PATCH 0/5] erofs-utils: fix checkpatch.pl complains Huang Jianan 2021-08-31 16:51 ` [PATCH 1/5] erofs-utils: remove filename in the file Huang Jianan @ 2021-08-31 16:51 ` Huang Jianan 2021-09-03 3:10 ` Gao Xiang 2021-08-31 16:51 ` [PATCH 3/5] erofs-utils: fix general style problem Huang Jianan ` (3 subsequent siblings) 5 siblings, 1 reply; 21+ messages in thread From: Huang Jianan @ 2021-08-31 16:51 UTC (permalink / raw) To: linux-erofs From: Huang Jianan <huangjianan@oppo.com> Signed-off-by: Huang Jianan <huangjianan@oppo.com> --- fuse/macosx.h | 1 + include/erofs/block_list.h | 2 +- include/erofs/xattr.h | 2 +- 3 files changed, 3 insertions(+), 2 deletions(-) diff --git a/fuse/macosx.h b/fuse/macosx.h index 372eba6..81ac47f 100644 --- a/fuse/macosx.h +++ b/fuse/macosx.h @@ -1,3 +1,4 @@ +/* SPDX-License-Identifier: GPL-2.0+ */ #ifdef __APPLE__ #undef LIST_HEAD #endif diff --git a/include/erofs/block_list.h b/include/erofs/block_list.h index 5127b23..fca476a 100644 --- a/include/erofs/block_list.h +++ b/include/erofs/block_list.h @@ -1,4 +1,4 @@ -// SPDX-License-Identifier: GPL-2.0+ +/* SPDX-License-Identifier: GPL-2.0+ */ /* * Copyright (C), 2021, Coolpad Group Limited. * Created by Yue Hu <huyue2@yulong.com> diff --git a/include/erofs/xattr.h b/include/erofs/xattr.h index 5086b54..f0c4c26 100644 --- a/include/erofs/xattr.h +++ b/include/erofs/xattr.h @@ -1,4 +1,4 @@ -// SPDX-License-Identifier: GPL-2.0+ +/* SPDX-License-Identifier: GPL-2.0+ */ /* * Originally contributed by an anonymous person, * heavily changed by Li Guifu <blucerlee@gmail.com> -- 2.25.1 ^ permalink raw reply related [flat|nested] 21+ messages in thread
* Re: [PATCH 2/5] erofs-utils: fix SPDX comment style 2021-08-31 16:51 ` [PATCH 2/5] erofs-utils: fix SPDX comment style Huang Jianan @ 2021-09-03 3:10 ` Gao Xiang 0 siblings, 0 replies; 21+ messages in thread From: Gao Xiang @ 2021-09-03 3:10 UTC (permalink / raw) To: Huang Jianan; +Cc: linux-erofs On Wed, Sep 01, 2021 at 12:51:13AM +0800, Huang Jianan wrote: > From: Huang Jianan <huangjianan@oppo.com> > > Signed-off-by: Huang Jianan <huangjianan@oppo.com> Reviewed-by: Gao Xiang <hsiangkao@linux.alibaba.com> Thanks, Gao Xiang ^ permalink raw reply [flat|nested] 21+ messages in thread
* [PATCH 3/5] erofs-utils: fix general style problem 2021-08-31 16:51 [PATCH 0/5] erofs-utils: fix checkpatch.pl complains Huang Jianan 2021-08-31 16:51 ` [PATCH 1/5] erofs-utils: remove filename in the file Huang Jianan 2021-08-31 16:51 ` [PATCH 2/5] erofs-utils: fix SPDX comment style Huang Jianan @ 2021-08-31 16:51 ` Huang Jianan 2021-09-03 3:12 ` Gao Xiang 2021-08-31 16:51 ` [PATCH 4/5] erofs-utils: remove unnecessary codes and comments Huang Jianan ` (2 subsequent siblings) 5 siblings, 1 reply; 21+ messages in thread From: Huang Jianan @ 2021-08-31 16:51 UTC (permalink / raw) To: linux-erofs From: Huang Jianan <huangjianan@oppo.com> Signed-off-by: Huang Jianan <huangjianan@oppo.com> --- fuse/main.c | 5 +---- include/erofs/block_list.h | 4 ++-- lib/block_list.c | 5 +---- lib/compress.c | 9 ++++----- lib/compressor.c | 2 +- lib/namei.c | 3 ++- lib/xattr.c | 2 +- mkfs/main.c | 2 +- 8 files changed, 13 insertions(+), 19 deletions(-) diff --git a/fuse/main.c b/fuse/main.c index 34a9b7a..197943a 100644 --- a/fuse/main.c +++ b/fuse/main.c @@ -109,16 +109,13 @@ static struct options { bool odebug; } fusecfg; -#define OPTION(t, p) \ - { t, offsetof(struct options, p), 1 } +#define OPTION(t, p) { t, offsetof(struct options, p), 1 } static const struct fuse_opt option_spec[] = { OPTION("--dbglevel=%u", debug_lvl), OPTION("--help", show_help), FUSE_OPT_END }; -#define OPTION(t, p) { t, offsetof(struct options, p), 1 } - static void usage(void) { struct fuse_args args = FUSE_ARGS_INIT(0, NULL); diff --git a/include/erofs/block_list.h b/include/erofs/block_list.h index fca476a..dcc0e50 100644 --- a/include/erofs/block_list.h +++ b/include/erofs/block_list.h @@ -18,8 +18,8 @@ void erofs_droid_blocklist_write_tail_end(struct erofs_inode *inode, #else static inline void erofs_droid_blocklist_write(struct erofs_inode *inode, erofs_blk_t blk_start, erofs_blk_t nblocks) {} -static inline -void erofs_droid_blocklist_write_tail_end(struct erofs_inode *inode, +static inline void +erofs_droid_blocklist_write_tail_end(struct erofs_inode *inode, erofs_blk_t blkaddr) {} #endif #endif diff --git a/lib/block_list.c b/lib/block_list.c index 73c1bde..15bb5cf 100644 --- a/lib/block_list.c +++ b/lib/block_list.c @@ -11,13 +11,10 @@ #define pr_fmt(fmt) "EROFS block_list: " FUNC_LINE_FMT fmt "\n" #include "erofs/print.h" -static FILE *block_list_fp = NULL; +static FILE *block_list_fp; int erofs_droid_blocklist_fopen(void) { - if (block_list_fp) - return 0; - block_list_fp = fopen(cfg.block_list_file, "w"); if (!block_list_fp) diff --git a/lib/compress.c b/lib/compress.c index 2b12d67..0530765 100644 --- a/lib/compress.c +++ b/lib/compress.c @@ -292,13 +292,12 @@ static void *write_compacted_indexes(u8 *out, bool update_blkaddr; erofs_blk_t blkaddr; - if (destsize == 4) { + if (destsize == 4) vcnt = 2; - } else if (destsize == 2 && logical_clusterbits == 12) { + else if (destsize == 2 && logical_clusterbits == 12) vcnt = 16; - } else { + else return ERR_PTR(-EINVAL); - } encodebits = (vcnt * destsize * 8 - 32) / vcnt; blkaddr = *blkaddr_ret; update_blkaddr = erofs_sb_has_big_pcluster(); @@ -467,8 +466,8 @@ int erofs_write_compressed_file(struct erofs_inode *inode) erofs_blk_t blkaddr, compressed_blocks; unsigned int legacymetasize; int ret, fd; - u8 *compressmeta = malloc(vle_compressmeta_capacity(inode->i_size)); + if (!compressmeta) return -ENOMEM; diff --git a/lib/compressor.c b/lib/compressor.c index 846a836..6bc5c4c 100644 --- a/lib/compressor.c +++ b/lib/compressor.c @@ -26,7 +26,7 @@ int erofs_compress_destsize(struct erofs_compress *c, void *dst, unsigned int dstsize) { - unsigned uncompressed_size; + unsigned int uncompressed_size; int ret; DBG_BUGON(!c->alg); diff --git a/lib/namei.c b/lib/namei.c index f4094a1..54c368a 100644 --- a/lib/namei.c +++ b/lib/namei.c @@ -243,7 +243,8 @@ static int link_path_walk(const char *name, struct nameidata *nd) name = p; /* Skip until no more slashes. */ - for (name = p; *name == '/'; ++name); + for (name = p; *name == '/'; ++name) + ; } return 0; } diff --git a/lib/xattr.c b/lib/xattr.c index 39d4a96..821e6c9 100644 --- a/lib/xattr.c +++ b/lib/xattr.c @@ -215,7 +215,7 @@ static struct xattr_item *erofs_get_selabel_xattr(const char *srcpath, erofs_fspath(srcpath)); else #endif - ret = asprintf(&fspath, "/%s", erofs_fspath(srcpath)); + ret = asprintf(&fspath, "/%s", erofs_fspath(srcpath)); if (ret <= 0) return ERR_PTR(-ENOMEM); diff --git a/mkfs/main.c b/mkfs/main.c index debb754..40ca94f 100644 --- a/mkfs/main.c +++ b/mkfs/main.c @@ -174,7 +174,7 @@ static int mkfs_parse_options_cfg(int argc, char *argv[]) char *endptr; int opt, i; - while((opt = getopt_long(argc, argv, "d:x:z:E:T:U:C:", + while ((opt = getopt_long(argc, argv, "d:x:z:E:T:U:C:", long_options, NULL)) != -1) { switch (opt) { case 'z': -- 2.25.1 ^ permalink raw reply related [flat|nested] 21+ messages in thread
* Re: [PATCH 3/5] erofs-utils: fix general style problem 2021-08-31 16:51 ` [PATCH 3/5] erofs-utils: fix general style problem Huang Jianan @ 2021-09-03 3:12 ` Gao Xiang 0 siblings, 0 replies; 21+ messages in thread From: Gao Xiang @ 2021-09-03 3:12 UTC (permalink / raw) To: Huang Jianan; +Cc: linux-erofs On Wed, Sep 01, 2021 at 12:51:14AM +0800, Huang Jianan wrote: > From: Huang Jianan <huangjianan@oppo.com> > > Signed-off-by: Huang Jianan <huangjianan@oppo.com> Reviewed-by: Gao Xiang <hsiangkao@linux.alibaba.com> Thanks, Gao Xiang ^ permalink raw reply [flat|nested] 21+ messages in thread
* [PATCH 4/5] erofs-utils: remove unnecessary codes and comments 2021-08-31 16:51 [PATCH 0/5] erofs-utils: fix checkpatch.pl complains Huang Jianan ` (2 preceding siblings ...) 2021-08-31 16:51 ` [PATCH 3/5] erofs-utils: fix general style problem Huang Jianan @ 2021-08-31 16:51 ` Huang Jianan 2021-09-03 3:17 ` Gao Xiang 2021-08-31 16:51 ` [PATCH 5/5] erofs-utils: fix print style Huang Jianan 2021-09-03 13:40 ` [PATCH V2 0/6] erofs-utils: fix checkpatch.pl complains Huang Jianan 5 siblings, 1 reply; 21+ messages in thread From: Huang Jianan @ 2021-08-31 16:51 UTC (permalink / raw) To: linux-erofs From: Huang Jianan <huangjianan@oppo.com> Signed-off-by: Huang Jianan <huangjianan@oppo.com> --- lib/inode.c | 4 ---- lib/zmap.c | 1 - 2 files changed, 5 deletions(-) diff --git a/lib/inode.c b/lib/inode.c index 62047d3..f001016 100644 --- a/lib/inode.c +++ b/lib/inode.c @@ -677,11 +677,7 @@ out: * Don't leave DATA buffers which were written in the global * buffer list. It will make balloc() slowly. */ -#if 0 - bh->op = &erofs_drop_directly_bhops; -#else erofs_bdrop(bh, false); -#endif inode->bh_data = NULL; } return 0; diff --git a/lib/zmap.c b/lib/zmap.c index fdc84af..88da515 100644 --- a/lib/zmap.c +++ b/lib/zmap.c @@ -423,7 +423,6 @@ static int z_erofs_get_extent_compressedlen(struct z_erofs_maprecorder *m, goto err_bonus_cblkcnt; if (m->compressedlcs) break; - /* fallthrough */ default: erofs_err("cannot found CBLKCNT @ lcn %lu of nid %llu", lcn, vi->nid | 0ULL); -- 2.25.1 ^ permalink raw reply related [flat|nested] 21+ messages in thread
* Re: [PATCH 4/5] erofs-utils: remove unnecessary codes and comments 2021-08-31 16:51 ` [PATCH 4/5] erofs-utils: remove unnecessary codes and comments Huang Jianan @ 2021-09-03 3:17 ` Gao Xiang 0 siblings, 0 replies; 21+ messages in thread From: Gao Xiang @ 2021-09-03 3:17 UTC (permalink / raw) To: Huang Jianan; +Cc: linux-erofs On Wed, Sep 01, 2021 at 12:51:15AM +0800, Huang Jianan wrote: > From: Huang Jianan <huangjianan@oppo.com> > > Signed-off-by: Huang Jianan <huangjianan@oppo.com> > --- > lib/inode.c | 4 ---- > lib/zmap.c | 1 - > 2 files changed, 5 deletions(-) > > diff --git a/lib/inode.c b/lib/inode.c > index 62047d3..f001016 100644 > --- a/lib/inode.c > +++ b/lib/inode.c > @@ -677,11 +677,7 @@ out: > * Don't leave DATA buffers which were written in the global > * buffer list. It will make balloc() slowly. > */ > -#if 0 > - bh->op = &erofs_drop_directly_bhops; > -#else > erofs_bdrop(bh, false); > -#endif > inode->bh_data = NULL; > } > return 0; > diff --git a/lib/zmap.c b/lib/zmap.c > index fdc84af..88da515 100644 > --- a/lib/zmap.c > +++ b/lib/zmap.c > @@ -423,7 +423,6 @@ static int z_erofs_get_extent_compressedlen(struct z_erofs_maprecorder *m, > goto err_bonus_cblkcnt; > if (m->compressedlcs) > break; > - /* fallthrough */ I don't think this should be removed. On the kernel side, we use "fallthrough;" instead. Could we add all the missing "/* fallthrough */"? Thanks, Gao Xiang > default: > erofs_err("cannot found CBLKCNT @ lcn %lu of nid %llu", > lcn, vi->nid | 0ULL); > -- > 2.25.1 ^ permalink raw reply [flat|nested] 21+ messages in thread
* [PATCH 5/5] erofs-utils: fix print style 2021-08-31 16:51 [PATCH 0/5] erofs-utils: fix checkpatch.pl complains Huang Jianan ` (3 preceding siblings ...) 2021-08-31 16:51 ` [PATCH 4/5] erofs-utils: remove unnecessary codes and comments Huang Jianan @ 2021-08-31 16:51 ` Huang Jianan 2021-09-03 3:18 ` Gao Xiang 2021-09-03 13:40 ` [PATCH V2 0/6] erofs-utils: fix checkpatch.pl complains Huang Jianan 5 siblings, 1 reply; 21+ messages in thread From: Huang Jianan @ 2021-08-31 16:51 UTC (permalink / raw) To: linux-erofs From: Huang Jianan <huangjianan@oppo.com> Fix warning "quoted string split across lines". Signed-off-by: Huang Jianan <huangjianan@oppo.com> --- lib/inode.c | 3 +-- lib/io.c | 3 +-- lib/zmap.c | 3 +-- 3 files changed, 3 insertions(+), 6 deletions(-) diff --git a/lib/inode.c b/lib/inode.c index f001016..76f5fb3 100644 --- a/lib/inode.c +++ b/lib/inode.c @@ -741,8 +741,7 @@ int erofs_droid_inode_fsconfig(struct erofs_inode *inode, cfg.target_out_path, &uid, &gid, &mode, &inode->capabilities); - erofs_dbg("/%s -> mode = 0x%x, uid = 0x%x, gid = 0x%x, " - "capabilities = 0x%" PRIx64 "\n", + erofs_dbg("/%s -> mode = 0x%x, uid = 0x%x, gid = 0x%x, capabilities = 0x%" PRIx64 "\n", fspath, mode, uid, gid, inode->capabilities); if (decorated) diff --git a/lib/io.c b/lib/io.c index b053137..620cb9c 100644 --- a/lib/io.c +++ b/lib/io.c @@ -242,8 +242,7 @@ int dev_read(void *buf, u64 offset, size_t len) } if (offset >= erofs_devsz || len > erofs_devsz || offset > erofs_devsz - len) { - erofs_err("read posion[%" PRIu64 ", %zd] is too large beyond" - "the end of device(%" PRIu64 ").", + erofs_err("read posion[%" PRIu64 ", %zd] is too large beyond the end of device(%" PRIu64 ").", offset, len, erofs_devsz); return -EINVAL; } diff --git a/lib/zmap.c b/lib/zmap.c index 88da515..e4306ce 100644 --- a/lib/zmap.c +++ b/lib/zmap.c @@ -57,8 +57,7 @@ static int z_erofs_fill_inode_lazy(struct erofs_inode *vi) if (vi->datalayout == EROFS_INODE_FLAT_COMPRESSION && !(vi->z_advise & Z_EROFS_ADVISE_BIG_PCLUSTER_1) ^ !(vi->z_advise & Z_EROFS_ADVISE_BIG_PCLUSTER_2)) { - erofs_err( -"big pcluster head1/2 of compact indexes should be consistent for nid %llu", + erofs_err("big pcluster head1/2 of compact indexes should be consistent for nid %llu", vi->nid * 1ULL); return -EFSCORRUPTED; } -- 2.25.1 ^ permalink raw reply related [flat|nested] 21+ messages in thread
* Re: [PATCH 5/5] erofs-utils: fix print style 2021-08-31 16:51 ` [PATCH 5/5] erofs-utils: fix print style Huang Jianan @ 2021-09-03 3:18 ` Gao Xiang 0 siblings, 0 replies; 21+ messages in thread From: Gao Xiang @ 2021-09-03 3:18 UTC (permalink / raw) To: Huang Jianan; +Cc: linux-erofs On Wed, Sep 01, 2021 at 12:51:16AM +0800, Huang Jianan wrote: > From: Huang Jianan <huangjianan@oppo.com> > > Fix warning "quoted string split across lines". > > Signed-off-by: Huang Jianan <huangjianan@oppo.com> Reviewed-by: Gao Xiang <hsiangkao@linux.alibaba.com> Thanks, Gao Xiang > --- > lib/inode.c | 3 +-- > lib/io.c | 3 +-- > lib/zmap.c | 3 +-- > 3 files changed, 3 insertions(+), 6 deletions(-) > > diff --git a/lib/inode.c b/lib/inode.c > index f001016..76f5fb3 100644 > --- a/lib/inode.c > +++ b/lib/inode.c > @@ -741,8 +741,7 @@ int erofs_droid_inode_fsconfig(struct erofs_inode *inode, > cfg.target_out_path, > &uid, &gid, &mode, &inode->capabilities); > > - erofs_dbg("/%s -> mode = 0x%x, uid = 0x%x, gid = 0x%x, " > - "capabilities = 0x%" PRIx64 "\n", > + erofs_dbg("/%s -> mode = 0x%x, uid = 0x%x, gid = 0x%x, capabilities = 0x%" PRIx64 "\n", > fspath, mode, uid, gid, inode->capabilities); > > if (decorated) > diff --git a/lib/io.c b/lib/io.c > index b053137..620cb9c 100644 > --- a/lib/io.c > +++ b/lib/io.c > @@ -242,8 +242,7 @@ int dev_read(void *buf, u64 offset, size_t len) > } > if (offset >= erofs_devsz || len > erofs_devsz || > offset > erofs_devsz - len) { > - erofs_err("read posion[%" PRIu64 ", %zd] is too large beyond" > - "the end of device(%" PRIu64 ").", > + erofs_err("read posion[%" PRIu64 ", %zd] is too large beyond the end of device(%" PRIu64 ").", > offset, len, erofs_devsz); > return -EINVAL; > } > diff --git a/lib/zmap.c b/lib/zmap.c > index 88da515..e4306ce 100644 > --- a/lib/zmap.c > +++ b/lib/zmap.c > @@ -57,8 +57,7 @@ static int z_erofs_fill_inode_lazy(struct erofs_inode *vi) > if (vi->datalayout == EROFS_INODE_FLAT_COMPRESSION && > !(vi->z_advise & Z_EROFS_ADVISE_BIG_PCLUSTER_1) ^ > !(vi->z_advise & Z_EROFS_ADVISE_BIG_PCLUSTER_2)) { > - erofs_err( > -"big pcluster head1/2 of compact indexes should be consistent for nid %llu", > + erofs_err("big pcluster head1/2 of compact indexes should be consistent for nid %llu", > vi->nid * 1ULL); > return -EFSCORRUPTED; > } > -- > 2.25.1 ^ permalink raw reply [flat|nested] 21+ messages in thread
* [PATCH V2 0/6] erofs-utils: fix checkpatch.pl complains 2021-08-31 16:51 [PATCH 0/5] erofs-utils: fix checkpatch.pl complains Huang Jianan ` (4 preceding siblings ...) 2021-08-31 16:51 ` [PATCH 5/5] erofs-utils: fix print style Huang Jianan @ 2021-09-03 13:40 ` Huang Jianan 2021-09-03 13:40 ` [PATCH V2 1/6] erofs-utils: clean up file headers & footers Huang Jianan ` (5 more replies) 5 siblings, 6 replies; 21+ messages in thread From: Huang Jianan @ 2021-09-03 13:40 UTC (permalink / raw) To: linux-erofs From: Huang Jianan <huangjianan@oppo.com> This patchset fix most of checkpatch.pl complains in erofs-utils, some problems that also exist in the kernel haven't been fixed. Changes since v1: - remove trailing newline (Gao Xiang); - add missing /* fallthrough */ (Gao Xiang); Huang Jianan (6): erofs-utils: clean up file headers & footers erofs-utils: fix SPDX comment style erofs-utils: fix general style problem erofs-utils: remove unnecessary codes erofs-utils: fix print style erofs-utils: add missing /* fallthrough */ AUTHORS | 1 - ChangeLog | 1 - Makefile.am | 2 -- README | 1 - autogen.sh | 1 - configure.ac | 1 - fuse/Makefile.am | 2 -- fuse/dir.c | 3 --- fuse/macosx.h | 1 + fuse/main.c | 8 +------- include/erofs/block_list.h | 8 +++----- include/erofs/cache.h | 3 --- include/erofs/compress.h | 3 --- include/erofs/config.h | 3 --- include/erofs/decompress.h | 2 -- include/erofs/defs.h | 2 -- include/erofs/err.h | 3 --- include/erofs/exclude.h | 3 --- include/erofs/hashtable.h | 2 -- include/erofs/inode.h | 2 -- include/erofs/internal.h | 2 -- include/erofs/io.h | 3 --- include/erofs/list.h | 2 -- include/erofs/print.h | 4 ---- include/erofs/trace.h | 3 --- include/erofs/xattr.h | 4 +--- include/erofs_fs.h | 2 -- lib/Makefile.am | 2 -- lib/block_list.c | 7 +------ lib/cache.c | 3 --- lib/compress.c | 12 ++++-------- lib/compressor.c | 5 +---- lib/compressor.h | 3 --- lib/compressor_lz4.c | 3 --- lib/compressor_lz4hc.c | 3 --- lib/config.c | 3 --- lib/data.c | 3 --- lib/decompress.c | 2 -- lib/exclude.c | 3 --- lib/inode.c | 10 +--------- lib/io.c | 5 +---- lib/namei.c | 6 ++---- lib/super.c | 3 --- lib/xattr.c | 5 +---- lib/zmap.c | 8 ++++---- man/Makefile.am | 1 - man/mkfs.erofs.1 | 1 - mkfs/Makefile.am | 2 -- mkfs/main.c | 4 +--- 49 files changed, 22 insertions(+), 144 deletions(-) -- 2.25.1 ^ permalink raw reply [flat|nested] 21+ messages in thread
* [PATCH V2 1/6] erofs-utils: clean up file headers & footers 2021-09-03 13:40 ` [PATCH V2 0/6] erofs-utils: fix checkpatch.pl complains Huang Jianan @ 2021-09-03 13:40 ` Huang Jianan 2021-09-03 15:03 ` Gao Xiang 2021-09-03 13:40 ` [PATCH V2 2/6] erofs-utils: fix SPDX comment style Huang Jianan ` (4 subsequent siblings) 5 siblings, 1 reply; 21+ messages in thread From: Huang Jianan @ 2021-09-03 13:40 UTC (permalink / raw) To: linux-erofs From: Huang Jianan <huangjianan@oppo.com> - Remove filename in the file since it's generally not useful. - Get rid of all unnecessary trailing newline. Signed-off-by: Huang Jianan <huangjianan@oppo.com> --- AUTHORS | 1 - ChangeLog | 1 - Makefile.am | 2 -- README | 1 - autogen.sh | 1 - configure.ac | 1 - fuse/Makefile.am | 2 -- fuse/dir.c | 3 --- fuse/main.c | 3 --- include/erofs/block_list.h | 2 -- include/erofs/cache.h | 3 --- include/erofs/compress.h | 3 --- include/erofs/config.h | 3 --- include/erofs/decompress.h | 2 -- include/erofs/defs.h | 2 -- include/erofs/err.h | 3 --- include/erofs/exclude.h | 3 --- include/erofs/hashtable.h | 2 -- include/erofs/inode.h | 2 -- include/erofs/internal.h | 2 -- include/erofs/io.h | 3 --- include/erofs/list.h | 2 -- include/erofs/print.h | 4 ---- include/erofs/trace.h | 3 --- include/erofs/xattr.h | 2 -- include/erofs_fs.h | 2 -- lib/Makefile.am | 2 -- lib/block_list.c | 2 -- lib/cache.c | 3 --- lib/compress.c | 3 --- lib/compressor.c | 3 --- lib/compressor.h | 3 --- lib/compressor_lz4.c | 3 --- lib/compressor_lz4hc.c | 3 --- lib/config.c | 3 --- lib/data.c | 3 --- lib/decompress.c | 2 -- lib/exclude.c | 3 --- lib/inode.c | 3 --- lib/io.c | 2 -- lib/namei.c | 3 --- lib/super.c | 3 --- lib/xattr.c | 3 --- lib/zmap.c | 2 -- man/Makefile.am | 1 - man/mkfs.erofs.1 | 1 - mkfs/Makefile.am | 2 -- mkfs/main.c | 2 -- 48 files changed, 113 deletions(-) diff --git a/AUTHORS b/AUTHORS index 3487d44..dacbdda 100644 --- a/AUTHORS +++ b/AUTHORS @@ -6,4 +6,3 @@ R: Gao Xiang <xiang@kernel.org> R: Chao Yu <yuchao0@huawei.com> S: Maintained L: linux-erofs@lists.ozlabs.org - diff --git a/ChangeLog b/ChangeLog index 6637bc3..4c0c941 100644 --- a/ChangeLog +++ b/ChangeLog @@ -58,4 +58,3 @@ erofs-utils (1.0-1) unstable; urgency=low - (mkfs.erofs) Posix ACL support; -- Gao Xiang <xiang@kernel.org> Thu, 24 Oct 2019 00:00:00 +0800 - diff --git a/Makefile.am b/Makefile.am index b804aa9..20f0f7d 100644 --- a/Makefile.am +++ b/Makefile.am @@ -1,5 +1,4 @@ # SPDX-License-Identifier: GPL-2.0+ -# Makefile.am ACLOCAL_AMFLAGS = -I m4 @@ -7,4 +6,3 @@ SUBDIRS = man lib mkfs if ENABLE_FUSE SUBDIRS += fuse endif - diff --git a/README b/README index af9cdf1..7b641f7 100644 --- a/README +++ b/README @@ -230,4 +230,3 @@ Comments https://github.com/lz4/lz4/issues/783 which is also resolved in lz4-1.9.3. - diff --git a/autogen.sh b/autogen.sh index 6816b11..fd81db4 100755 --- a/autogen.sh +++ b/autogen.sh @@ -7,4 +7,3 @@ autoconf && \ case `uname` in Darwin*) glibtoolize --copy ;; \ *) libtoolize --copy ;; esac && \ automake -a -c - diff --git a/configure.ac b/configure.ac index 3510609..a749db0 100644 --- a/configure.ac +++ b/configure.ac @@ -297,4 +297,3 @@ AC_CONFIG_FILES([Makefile mkfs/Makefile fuse/Makefile]) AC_OUTPUT - diff --git a/fuse/Makefile.am b/fuse/Makefile.am index e7757bc..6893a97 100644 --- a/fuse/Makefile.am +++ b/fuse/Makefile.am @@ -1,5 +1,4 @@ # SPDX-License-Identifier: GPL-2.0+ -# Makefile.am AUTOMAKE_OPTIONS = foreign bin_PROGRAMS = erofsfuse @@ -7,4 +6,3 @@ erofsfuse_SOURCES = dir.c main.c erofsfuse_CFLAGS = -Wall -Werror -I$(top_srcdir)/include erofsfuse_CFLAGS += -DFUSE_USE_VERSION=26 ${libfuse_CFLAGS} ${libselinux_CFLAGS} erofsfuse_LDADD = $(top_builddir)/lib/liberofs.la ${libfuse_LIBS} ${liblz4_LIBS} ${libselinux_LIBS} - diff --git a/fuse/dir.c b/fuse/dir.c index e16fda1..bc8735b 100644 --- a/fuse/dir.c +++ b/fuse/dir.c @@ -1,7 +1,5 @@ // SPDX-License-Identifier: GPL-2.0+ /* - * erofs-utils/fuse/dir.c - * * Created by Li Guifu <blucerlee@gmail.com> */ #include <fuse.h> @@ -100,4 +98,3 @@ int erofsfuse_readdir(const char *path, void *buf, fuse_fill_dir_t filler, } return 0; } - diff --git a/fuse/main.c b/fuse/main.c index 5552480..fca4d7f 100644 --- a/fuse/main.c +++ b/fuse/main.c @@ -1,7 +1,5 @@ // SPDX-License-Identifier: GPL-2.0+ /* - * erofs-utils/fuse/main.c - * * Created by Li Guifu <blucerlee@gmail.com> */ #include <stdlib.h> @@ -249,4 +247,3 @@ err: erofs_exit_configure(); return ret ? EXIT_FAILURE : EXIT_SUCCESS; } - diff --git a/include/erofs/block_list.h b/include/erofs/block_list.h index 7756d8a..5127b23 100644 --- a/include/erofs/block_list.h +++ b/include/erofs/block_list.h @@ -1,7 +1,5 @@ // SPDX-License-Identifier: GPL-2.0+ /* - * erofs-utils/include/erofs/block_list.h - * * Copyright (C), 2021, Coolpad Group Limited. * Created by Yue Hu <huyue2@yulong.com> */ diff --git a/include/erofs/cache.h b/include/erofs/cache.h index 611ca5b..e324d92 100644 --- a/include/erofs/cache.h +++ b/include/erofs/cache.h @@ -1,7 +1,5 @@ /* SPDX-License-Identifier: GPL-2.0+ */ /* - * erofs-utils/include/erofs/cache.h - * * Copyright (C) 2018 HUAWEI, Inc. * http://www.huawei.com/ * Created by Miao Xie <miaoxie@huawei.com> @@ -102,4 +100,3 @@ bool erofs_bflush(struct erofs_buffer_block *bb); void erofs_bdrop(struct erofs_buffer_head *bh, bool tryrevoke); #endif - diff --git a/include/erofs/compress.h b/include/erofs/compress.h index d234e8b..4434aaa 100644 --- a/include/erofs/compress.h +++ b/include/erofs/compress.h @@ -1,7 +1,5 @@ /* SPDX-License-Identifier: GPL-2.0+ */ /* - * erofs-utils/include/erofs/compress.h - * * Copyright (C) 2019 HUAWEI, Inc. * http://www.huawei.com/ * Created by Gao Xiang <gaoxiang25@huawei.com> @@ -24,4 +22,3 @@ int z_erofs_compress_exit(void); const char *z_erofs_list_available_compressors(unsigned int i); #endif - diff --git a/include/erofs/config.h b/include/erofs/config.h index 8124f3b..95fc23e 100644 --- a/include/erofs/config.h +++ b/include/erofs/config.h @@ -1,7 +1,5 @@ /* SPDX-License-Identifier: GPL-2.0+ */ /* - * erofs-utils/include/erofs/config.h - * * Copyright (C) 2018-2019 HUAWEI, Inc. * http://www.huawei.com/ * Created by Li Guifu <bluce.liguifu@huawei.com> @@ -89,4 +87,3 @@ static inline int erofs_selabel_open(const char *file_contexts) #endif #endif - diff --git a/include/erofs/decompress.h b/include/erofs/decompress.h index beaac35..0ba2b08 100644 --- a/include/erofs/decompress.h +++ b/include/erofs/decompress.h @@ -1,7 +1,5 @@ /* SPDX-License-Identifier: GPL-2.0+ */ /* - * erofs-utils/include/erofs/decompress.h - * * Copyright (C), 2008-2020, OPPO Mobile Comm Corp., Ltd. * Created by Huang Jianan <huangjianan@oppo.com> */ diff --git a/include/erofs/defs.h b/include/erofs/defs.h index 5410685..6e0a777 100644 --- a/include/erofs/defs.h +++ b/include/erofs/defs.h @@ -1,7 +1,5 @@ /* SPDX-License-Identifier: GPL-2.0+ */ /* - * erofs-utils/include/erofs/defs.h - * * Copyright (C) 2018 HUAWEI, Inc. * http://www.huawei.com/ * Created by Li Guifu <bluce.liguifu@huawei.com> diff --git a/include/erofs/err.h b/include/erofs/err.h index da3b681..a33bdd1 100644 --- a/include/erofs/err.h +++ b/include/erofs/err.h @@ -1,7 +1,5 @@ /* SPDX-License-Identifier: GPL-2.0+ */ /* - * erofs-utils/include/erofs/err.h - * * Copyright (C) 2018 HUAWEI, Inc. * http://www.huawei.com/ * Created by Li Guifu <bluce.liguifu@huawei.com> @@ -31,4 +29,3 @@ static inline long PTR_ERR(const void *ptr) } #endif - diff --git a/include/erofs/exclude.h b/include/erofs/exclude.h index 88c55d7..6930f2b 100644 --- a/include/erofs/exclude.h +++ b/include/erofs/exclude.h @@ -1,7 +1,5 @@ /* SPDX-License-Identifier: GPL-2.0+ */ /* - * erofs-utils/include/erofs/exclude.h - * * Created by Li Guifu <bluce.lee@aliyun.com> */ #ifndef __EROFS_EXCLUDE_H @@ -24,4 +22,3 @@ int erofs_parse_exclude_path(const char *args, bool is_regex); struct erofs_exclude_rule *erofs_is_exclude_path(const char *dir, const char *name); #endif - diff --git a/include/erofs/hashtable.h b/include/erofs/hashtable.h index 7e47189..a71cb00 100644 --- a/include/erofs/hashtable.h +++ b/include/erofs/hashtable.h @@ -1,7 +1,5 @@ /* SPDX-License-Identifier: GPL-2.0 */ /* - * erofs-utils/include/erofs/hashtable.h - * * Original code taken from 'linux/include/linux/hash{,table}.h' */ #ifndef __EROFS_HASHTABLE_H diff --git a/include/erofs/inode.h b/include/erofs/inode.h index 5a7f5f1..a736762 100644 --- a/include/erofs/inode.h +++ b/include/erofs/inode.h @@ -1,7 +1,5 @@ /* SPDX-License-Identifier: GPL-2.0+ */ /* - * erofs-utils/include/erofs/inode.h - * * Copyright (C) 2018-2019 HUAWEI, Inc. * http://www.huawei.com/ * Created by Li Guifu <bluce.liguifu@huawei.com> diff --git a/include/erofs/internal.h b/include/erofs/internal.h index 5583861..7dc5ff0 100644 --- a/include/erofs/internal.h +++ b/include/erofs/internal.h @@ -1,7 +1,5 @@ /* SPDX-License-Identifier: GPL-2.0+ */ /* - * erofs-utils/include/erofs/internal.h - * * Copyright (C) 2019 HUAWEI, Inc. * http://www.huawei.com/ * Created by Gao Xiang <gaoxiang25@huawei.com> diff --git a/include/erofs/io.h b/include/erofs/io.h index 5574245..0763baf 100644 --- a/include/erofs/io.h +++ b/include/erofs/io.h @@ -1,7 +1,5 @@ /* SPDX-License-Identifier: GPL-2.0+ */ /* - * erofs-utils/include/erofs/io.h - * * Copyright (C) 2018-2019 HUAWEI, Inc. * http://www.huawei.com/ * Created by Li Guifu <bluce.liguifu@huawei.com> @@ -41,4 +39,3 @@ static inline int blk_read(void *buf, erofs_blk_t start, } #endif - diff --git a/include/erofs/list.h b/include/erofs/list.h index 3572726..d2bc704 100644 --- a/include/erofs/list.h +++ b/include/erofs/list.h @@ -1,7 +1,5 @@ /* SPDX-License-Identifier: GPL-2.0+ */ /* - * erofs-utils/include/erofs/list.h - * * Copyright (C) 2018 HUAWEI, Inc. * http://www.huawei.com/ * Created by Li Guifu <bluce.liguifu@huawei.com> diff --git a/include/erofs/print.h b/include/erofs/print.h index 6b79074..57b6607 100644 --- a/include/erofs/print.h +++ b/include/erofs/print.h @@ -1,7 +1,5 @@ /* SPDX-License-Identifier: GPL-2.0+ */ /* - * erofs-utils/include/erofs/print.h - * * Copyright (C) 2018-2019 HUAWEI, Inc. * http://www.huawei.com/ * Created by Li Guifu <bluce.liguifu@huawei.com> @@ -71,6 +69,4 @@ enum { #define erofs_dump(fmt, ...) fprintf(stderr, fmt, ##__VA_ARGS__) - #endif - diff --git a/include/erofs/trace.h b/include/erofs/trace.h index 5a12da7..d70d236 100644 --- a/include/erofs/trace.h +++ b/include/erofs/trace.h @@ -1,7 +1,5 @@ /* SPDX-License-Identifier: GPL-2.0+ */ /* - * erofs-utils/include/erofs/trace.h - * * Copyright (C) 2020 Gao Xiang <hsiangkao@aol.com> */ #ifndef __EROFS_TRACE_H @@ -11,4 +9,3 @@ #define trace_erofs_map_blocks_flatmode_exit(inode, map, flags, ret) ((void)0) #endif - diff --git a/include/erofs/xattr.h b/include/erofs/xattr.h index 197fe25..5086b54 100644 --- a/include/erofs/xattr.h +++ b/include/erofs/xattr.h @@ -1,7 +1,5 @@ // SPDX-License-Identifier: GPL-2.0+ /* - * erofs-utils/include/erofs/xattr.h - * * Originally contributed by an anonymous person, * heavily changed by Li Guifu <blucerlee@gmail.com> * and Gao Xiang <xiang@kernel.org> diff --git a/include/erofs_fs.h b/include/erofs_fs.h index 18fc182..48934bb 100644 --- a/include/erofs_fs.h +++ b/include/erofs_fs.h @@ -1,6 +1,5 @@ /* SPDX-License-Identifier: GPL-2.0-only OR Apache-2.0 */ /* - * erofs-utils/include/erofs_fs.h * EROFS (Enhanced ROM File System) on-disk format definition * * Copyright (C) 2017-2018 HUAWEI, Inc. @@ -360,4 +359,3 @@ static inline void erofs_check_ondisk_layout_definitions(void) } #endif - diff --git a/lib/Makefile.am b/lib/Makefile.am index b12e2c1..b5127c4 100644 --- a/lib/Makefile.am +++ b/lib/Makefile.am @@ -1,5 +1,4 @@ # SPDX-License-Identifier: GPL-2.0+ -# Makefile.am noinst_LTLIBRARIES = liberofs.la noinst_HEADERS = $(top_srcdir)/include/erofs_fs.h \ @@ -30,4 +29,3 @@ if ENABLE_LZ4HC liberofs_la_SOURCES += compressor_lz4hc.c endif endif - diff --git a/lib/block_list.c b/lib/block_list.c index 3be0992..73c1bde 100644 --- a/lib/block_list.c +++ b/lib/block_list.c @@ -1,7 +1,5 @@ // SPDX-License-Identifier: GPL-2.0+ /* - * erofs-utils/lib/block_list.c - * * Copyright (C), 2021, Coolpad Group Limited. * Created by Yue Hu <huyue2@yulong.com> */ diff --git a/lib/cache.c b/lib/cache.c index 340dcdd..8016e38 100644 --- a/lib/cache.c +++ b/lib/cache.c @@ -1,7 +1,5 @@ // SPDX-License-Identifier: GPL-2.0+ /* - * erofs-utils/lib/cache.c - * * Copyright (C) 2018-2019 HUAWEI, Inc. * http://www.huawei.com/ * Created by Miao Xie <miaoxie@huawei.com> @@ -442,4 +440,3 @@ void erofs_bdrop(struct erofs_buffer_head *bh, bool tryrevoke) if (rollback) tail_blkaddr = blkaddr; } - diff --git a/lib/compress.c b/lib/compress.c index a8ebbc1..5ac9427 100644 --- a/lib/compress.c +++ b/lib/compress.c @@ -1,7 +1,5 @@ // SPDX-License-Identifier: GPL-2.0+ /* - * erofs-utils/lib/compress.c - * * Copyright (C) 2018-2019 HUAWEI, Inc. * http://www.huawei.com/ * Created by Miao Xie <miaoxie@huawei.com> @@ -680,4 +678,3 @@ int z_erofs_compress_exit(void) { return erofs_compressor_exit(&compresshandle); } - diff --git a/lib/compressor.c b/lib/compressor.c index 8836e0c..c14fc05 100644 --- a/lib/compressor.c +++ b/lib/compressor.c @@ -1,7 +1,5 @@ // SPDX-License-Identifier: GPL-2.0+ /* - * erofs-utils/lib/compressor.c - * * Copyright (C) 2018-2019 HUAWEI, Inc. * http://www.huawei.com/ * Created by Gao Xiang <gaoxiang25@huawei.com> @@ -91,4 +89,3 @@ int erofs_compressor_exit(struct erofs_compress *c) return c->alg->exit(c); return 0; } - diff --git a/lib/compressor.h b/lib/compressor.h index b2471c4..151c43d 100644 --- a/lib/compressor.h +++ b/lib/compressor.h @@ -1,7 +1,5 @@ /* SPDX-License-Identifier: GPL-2.0+ */ /* - * erofs-utils/lib/compressor.h - * * Copyright (C) 2018-2019 HUAWEI, Inc. * http://www.huawei.com/ * Created by Gao Xiang <gaoxiang25@huawei.com> @@ -53,4 +51,3 @@ int erofs_compressor_init(struct erofs_compress *c, char *alg_name); int erofs_compressor_exit(struct erofs_compress *c); #endif - diff --git a/lib/compressor_lz4.c b/lib/compressor_lz4.c index 292d0f2..f71252e 100644 --- a/lib/compressor_lz4.c +++ b/lib/compressor_lz4.c @@ -1,7 +1,5 @@ // SPDX-License-Identifier: GPL-2.0+ /* - * erofs-utils/lib/compressor-lz4.c - * * Copyright (C) 2018-2019 HUAWEI, Inc. * http://www.huawei.com/ * Created by Gao Xiang <gaoxiang25@huawei.com> @@ -48,4 +46,3 @@ struct erofs_compressor erofs_compressor_lz4 = { .exit = compressor_lz4_exit, .compress_destsize = lz4_compress_destsize, }; - diff --git a/lib/compressor_lz4hc.c b/lib/compressor_lz4hc.c index 14c3a71..0c912fb 100644 --- a/lib/compressor_lz4hc.c +++ b/lib/compressor_lz4hc.c @@ -1,7 +1,5 @@ // SPDX-License-Identifier: GPL-2.0+ /* - * erofs-utils/lib/compressor-lz4hc.c - * * Copyright (C) 2018-2019 HUAWEI, Inc. * http://www.huawei.com/ * Created by Gao Xiang <gaoxiang25@huawei.com> @@ -61,4 +59,3 @@ struct erofs_compressor erofs_compressor_lz4hc = { .exit = compressor_lz4hc_exit, .compress_destsize = lz4hc_compress_destsize, }; - diff --git a/lib/config.c b/lib/config.c index 99fcf49..4757dbb 100644 --- a/lib/config.c +++ b/lib/config.c @@ -1,7 +1,5 @@ // SPDX-License-Identifier: GPL-2.0+ /* - * erofs-utils/lib/config.c - * * Copyright (C) 2018-2019 HUAWEI, Inc. * http://www.huawei.com/ * Created by Li Guifu <bluce.liguifu@huawei.com> @@ -85,4 +83,3 @@ int erofs_selabel_open(const char *file_contexts) return 0; } #endif - diff --git a/lib/data.c b/lib/data.c index 42b4904..1a1005a 100644 --- a/lib/data.c +++ b/lib/data.c @@ -1,7 +1,5 @@ // SPDX-License-Identifier: GPL-2.0+ /* - * erofs-utils/lib/data.c - * * Copyright (C) 2020 Gao Xiang <hsiangkao@aol.com> * Compression support by Huang Jianan <huangjianan@oppo.com> */ @@ -217,4 +215,3 @@ int erofs_pread(struct erofs_inode *inode, char *buf, } return -EINVAL; } - diff --git a/lib/decompress.c b/lib/decompress.c index 490c4bc..2ee1439 100644 --- a/lib/decompress.c +++ b/lib/decompress.c @@ -1,7 +1,5 @@ // SPDX-License-Identifier: GPL-2.0+ /* - * erofs-utils/lib/decompress.c - * * Copyright (C), 2008-2020, OPPO Mobile Comm Corp., Ltd. * Created by Huang Jianan <huangjianan@oppo.com> */ diff --git a/lib/exclude.c b/lib/exclude.c index 73b3720..2f980a3 100644 --- a/lib/exclude.c +++ b/lib/exclude.c @@ -1,7 +1,5 @@ // SPDX-License-Identifier: GPL-2.0+ /* - * erofs-utils/lib/exclude.c - * * Created by Li Guifu <bluce.lee@aliyun.com> */ #include <string.h> @@ -129,4 +127,3 @@ struct erofs_exclude_rule *erofs_is_exclude_path(const char *dir, } return NULL; } - diff --git a/lib/inode.c b/lib/inode.c index 6871d2b..97ee2c9 100644 --- a/lib/inode.c +++ b/lib/inode.c @@ -1,7 +1,5 @@ // SPDX-License-Identifier: GPL-2.0+ /* - * erofs-utils/lib/inode.c - * * Copyright (C) 2018-2019 HUAWEI, Inc. * http://www.huawei.com/ * Created by Li Guifu <bluce.liguifu@huawei.com> @@ -1100,4 +1098,3 @@ struct erofs_inode *erofs_mkfs_build_tree_from_path(struct erofs_inode *parent, return erofs_mkfs_build_tree(inode); } - diff --git a/lib/io.c b/lib/io.c index 6067041..b053137 100644 --- a/lib/io.c +++ b/lib/io.c @@ -1,7 +1,5 @@ // SPDX-License-Identifier: GPL-2.0+ /* - * erofs-utils/lib/io.c - * * Copyright (C) 2018 HUAWEI, Inc. * http://www.huawei.com/ * Created by Li Guifu <bluce.liguifu@huawei.com> diff --git a/lib/namei.c b/lib/namei.c index b572d17..755a5ad 100644 --- a/lib/namei.c +++ b/lib/namei.c @@ -1,7 +1,5 @@ // SPDX-License-Identifier: GPL-2.0+ /* - * erofs-utils/lib/namei.c - * * Created by Li Guifu <blucerlee@gmail.com> */ #include <sys/types.h> @@ -262,4 +260,3 @@ int erofs_ilookup(const char *path, struct erofs_inode *vi) vi->nid = nd.nid; return erofs_read_inode_from_disk(vi); } - diff --git a/lib/super.c b/lib/super.c index 11405ec..0fa69ab 100644 --- a/lib/super.c +++ b/lib/super.c @@ -1,7 +1,5 @@ // SPDX-License-Identifier: GPL-2.0+ /* - * erofs-utils/lib/super.c - * * Created by Li Guifu <blucerlee@gmail.com> */ #include <string.h> @@ -71,4 +69,3 @@ int erofs_read_superblock(void) memcpy(&sbi.uuid, dsb->uuid, sizeof(dsb->uuid)); return 0; } - diff --git a/lib/xattr.c b/lib/xattr.c index aff3d67..ffc5f7a 100644 --- a/lib/xattr.c +++ b/lib/xattr.c @@ -1,7 +1,5 @@ // SPDX-License-Identifier: GPL-2.0+ /* - * erofs-utils/lib/xattr.c - * * Originally contributed by an anonymous person, * heavily changed by Li Guifu <blucerlee@gmail.com> * and Gao Xiang <hsiangkao@aol.com> @@ -687,4 +685,3 @@ char *erofs_export_xattr_ibody(struct list_head *ixattrs, unsigned int size) DBG_BUGON(p > size); return buf; } - diff --git a/lib/zmap.c b/lib/zmap.c index 1084faa..fdc84af 100644 --- a/lib/zmap.c +++ b/lib/zmap.c @@ -1,7 +1,5 @@ // SPDX-License-Identifier: GPL-2.0+ /* - * erofs-utils/lib/zmap.c - * * (a large amount of code was adapted from Linux kernel. ) * * Copyright (C) 2018-2019 HUAWEI, Inc. diff --git a/man/Makefile.am b/man/Makefile.am index 0df947b..d62d6e2 100644 --- a/man/Makefile.am +++ b/man/Makefile.am @@ -1,5 +1,4 @@ # SPDX-License-Identifier: GPL-2.0+ -# Makefile.am dist_man_MANS = mkfs.erofs.1 diff --git a/man/mkfs.erofs.1 b/man/mkfs.erofs.1 index d164fa5..bc0a10b 100644 --- a/man/mkfs.erofs.1 +++ b/man/mkfs.erofs.1 @@ -99,4 +99,3 @@ This manual page was written by Gao Xiang <xiang@kernel.org>. git://git.kernel.org/pub/scm/linux/kernel/git/xiang/erofs-utils.git. .SH SEE ALSO .BR mkfs (8). - diff --git a/mkfs/Makefile.am b/mkfs/Makefile.am index 8b8e051..e488f86 100644 --- a/mkfs/Makefile.am +++ b/mkfs/Makefile.am @@ -1,5 +1,4 @@ # SPDX-License-Identifier: GPL-2.0+ -# Makefile.am AUTOMAKE_OPTIONS = foreign bin_PROGRAMS = mkfs.erofs @@ -7,4 +6,3 @@ AM_CPPFLAGS = ${libuuid_CFLAGS} ${libselinux_CFLAGS} mkfs_erofs_SOURCES = main.c mkfs_erofs_CFLAGS = -Wall -Werror -I$(top_srcdir)/include mkfs_erofs_LDADD = ${libuuid_LIBS} $(top_builddir)/lib/liberofs.la ${libselinux_LIBS} ${liblz4_LIBS} - diff --git a/mkfs/main.c b/mkfs/main.c index 89f2310..debb754 100644 --- a/mkfs/main.c +++ b/mkfs/main.c @@ -1,7 +1,5 @@ // SPDX-License-Identifier: GPL-2.0+ /* - * mkfs/main.c - * * Copyright (C) 2018-2019 HUAWEI, Inc. * http://www.huawei.com/ * Created by Li Guifu <bluce.liguifu@huawei.com> -- 2.25.1 ^ permalink raw reply related [flat|nested] 21+ messages in thread
* Re: [PATCH V2 1/6] erofs-utils: clean up file headers & footers 2021-09-03 13:40 ` [PATCH V2 1/6] erofs-utils: clean up file headers & footers Huang Jianan @ 2021-09-03 15:03 ` Gao Xiang 0 siblings, 0 replies; 21+ messages in thread From: Gao Xiang @ 2021-09-03 15:03 UTC (permalink / raw) To: Huang Jianan; +Cc: linux-erofs On Fri, Sep 03, 2021 at 09:40:30PM +0800, Huang Jianan wrote: > From: Huang Jianan <huangjianan@oppo.com> > > - Remove filename in the file since it's generally not useful. > - Get rid of all unnecessary trailing newline. > > Signed-off-by: Huang Jianan <huangjianan@oppo.com> Reviewed-by: Gao Xiang <hsiangkao@linux.alibaba.com> Thanks, Gao Xiang ^ permalink raw reply [flat|nested] 21+ messages in thread
* [PATCH V2 2/6] erofs-utils: fix SPDX comment style 2021-09-03 13:40 ` [PATCH V2 0/6] erofs-utils: fix checkpatch.pl complains Huang Jianan 2021-09-03 13:40 ` [PATCH V2 1/6] erofs-utils: clean up file headers & footers Huang Jianan @ 2021-09-03 13:40 ` Huang Jianan 2021-09-03 13:40 ` [PATCH V2 3/6] erofs-utils: fix general style problem Huang Jianan ` (3 subsequent siblings) 5 siblings, 0 replies; 21+ messages in thread From: Huang Jianan @ 2021-09-03 13:40 UTC (permalink / raw) To: linux-erofs From: Huang Jianan <huangjianan@oppo.com> Signed-off-by: Huang Jianan <huangjianan@oppo.com> --- fuse/macosx.h | 1 + include/erofs/block_list.h | 2 +- include/erofs/xattr.h | 2 +- 3 files changed, 3 insertions(+), 2 deletions(-) diff --git a/fuse/macosx.h b/fuse/macosx.h index 372eba6..81ac47f 100644 --- a/fuse/macosx.h +++ b/fuse/macosx.h @@ -1,3 +1,4 @@ +/* SPDX-License-Identifier: GPL-2.0+ */ #ifdef __APPLE__ #undef LIST_HEAD #endif diff --git a/include/erofs/block_list.h b/include/erofs/block_list.h index 5127b23..fca476a 100644 --- a/include/erofs/block_list.h +++ b/include/erofs/block_list.h @@ -1,4 +1,4 @@ -// SPDX-License-Identifier: GPL-2.0+ +/* SPDX-License-Identifier: GPL-2.0+ */ /* * Copyright (C), 2021, Coolpad Group Limited. * Created by Yue Hu <huyue2@yulong.com> diff --git a/include/erofs/xattr.h b/include/erofs/xattr.h index 5086b54..f0c4c26 100644 --- a/include/erofs/xattr.h +++ b/include/erofs/xattr.h @@ -1,4 +1,4 @@ -// SPDX-License-Identifier: GPL-2.0+ +/* SPDX-License-Identifier: GPL-2.0+ */ /* * Originally contributed by an anonymous person, * heavily changed by Li Guifu <blucerlee@gmail.com> -- 2.25.1 ^ permalink raw reply related [flat|nested] 21+ messages in thread
* [PATCH V2 3/6] erofs-utils: fix general style problem 2021-09-03 13:40 ` [PATCH V2 0/6] erofs-utils: fix checkpatch.pl complains Huang Jianan 2021-09-03 13:40 ` [PATCH V2 1/6] erofs-utils: clean up file headers & footers Huang Jianan 2021-09-03 13:40 ` [PATCH V2 2/6] erofs-utils: fix SPDX comment style Huang Jianan @ 2021-09-03 13:40 ` Huang Jianan 2021-09-03 13:40 ` [PATCH V2 4/6] erofs-utils: remove unnecessary codes Huang Jianan ` (2 subsequent siblings) 5 siblings, 0 replies; 21+ messages in thread From: Huang Jianan @ 2021-09-03 13:40 UTC (permalink / raw) To: linux-erofs From: Huang Jianan <huangjianan@oppo.com> Signed-off-by: Huang Jianan <huangjianan@oppo.com> --- include/erofs/block_list.h | 4 ++-- lib/block_list.c | 5 +---- lib/compress.c | 9 ++++----- lib/compressor.c | 2 +- lib/namei.c | 3 ++- lib/xattr.c | 2 +- mkfs/main.c | 2 +- 7 files changed, 12 insertions(+), 15 deletions(-) diff --git a/include/erofs/block_list.h b/include/erofs/block_list.h index fca476a..dcc0e50 100644 --- a/include/erofs/block_list.h +++ b/include/erofs/block_list.h @@ -18,8 +18,8 @@ void erofs_droid_blocklist_write_tail_end(struct erofs_inode *inode, #else static inline void erofs_droid_blocklist_write(struct erofs_inode *inode, erofs_blk_t blk_start, erofs_blk_t nblocks) {} -static inline -void erofs_droid_blocklist_write_tail_end(struct erofs_inode *inode, +static inline void +erofs_droid_blocklist_write_tail_end(struct erofs_inode *inode, erofs_blk_t blkaddr) {} #endif #endif diff --git a/lib/block_list.c b/lib/block_list.c index 73c1bde..15bb5cf 100644 --- a/lib/block_list.c +++ b/lib/block_list.c @@ -11,13 +11,10 @@ #define pr_fmt(fmt) "EROFS block_list: " FUNC_LINE_FMT fmt "\n" #include "erofs/print.h" -static FILE *block_list_fp = NULL; +static FILE *block_list_fp; int erofs_droid_blocklist_fopen(void) { - if (block_list_fp) - return 0; - block_list_fp = fopen(cfg.block_list_file, "w"); if (!block_list_fp) diff --git a/lib/compress.c b/lib/compress.c index 5ac9427..6df30ea 100644 --- a/lib/compress.c +++ b/lib/compress.c @@ -292,13 +292,12 @@ static void *write_compacted_indexes(u8 *out, bool update_blkaddr; erofs_blk_t blkaddr; - if (destsize == 4) { + if (destsize == 4) vcnt = 2; - } else if (destsize == 2 && logical_clusterbits == 12) { + else if (destsize == 2 && logical_clusterbits == 12) vcnt = 16; - } else { + else return ERR_PTR(-EINVAL); - } encodebits = (vcnt * destsize * 8 - 32) / vcnt; blkaddr = *blkaddr_ret; update_blkaddr = erofs_sb_has_big_pcluster(); @@ -467,8 +466,8 @@ int erofs_write_compressed_file(struct erofs_inode *inode) erofs_blk_t blkaddr, compressed_blocks; unsigned int legacymetasize; int ret, fd; - u8 *compressmeta = malloc(vle_compressmeta_capacity(inode->i_size)); + if (!compressmeta) return -ENOMEM; diff --git a/lib/compressor.c b/lib/compressor.c index c14fc05..1f1a33d 100644 --- a/lib/compressor.c +++ b/lib/compressor.c @@ -26,7 +26,7 @@ int erofs_compress_destsize(struct erofs_compress *c, void *dst, unsigned int dstsize) { - unsigned uncompressed_size; + unsigned int uncompressed_size; int ret; DBG_BUGON(!c->alg); diff --git a/lib/namei.c b/lib/namei.c index 755a5ad..f96e400 100644 --- a/lib/namei.c +++ b/lib/namei.c @@ -243,7 +243,8 @@ static int link_path_walk(const char *name, struct nameidata *nd) name = p; /* Skip until no more slashes. */ - for (name = p; *name == '/'; ++name); + for (name = p; *name == '/'; ++name) + ; } return 0; } diff --git a/lib/xattr.c b/lib/xattr.c index ffc5f7a..196133a 100644 --- a/lib/xattr.c +++ b/lib/xattr.c @@ -215,7 +215,7 @@ static struct xattr_item *erofs_get_selabel_xattr(const char *srcpath, erofs_fspath(srcpath)); else #endif - ret = asprintf(&fspath, "/%s", erofs_fspath(srcpath)); + ret = asprintf(&fspath, "/%s", erofs_fspath(srcpath)); if (ret <= 0) return ERR_PTR(-ENOMEM); diff --git a/mkfs/main.c b/mkfs/main.c index debb754..40ca94f 100644 --- a/mkfs/main.c +++ b/mkfs/main.c @@ -174,7 +174,7 @@ static int mkfs_parse_options_cfg(int argc, char *argv[]) char *endptr; int opt, i; - while((opt = getopt_long(argc, argv, "d:x:z:E:T:U:C:", + while ((opt = getopt_long(argc, argv, "d:x:z:E:T:U:C:", long_options, NULL)) != -1) { switch (opt) { case 'z': -- 2.25.1 ^ permalink raw reply related [flat|nested] 21+ messages in thread
* [PATCH V2 4/6] erofs-utils: remove unnecessary codes 2021-09-03 13:40 ` [PATCH V2 0/6] erofs-utils: fix checkpatch.pl complains Huang Jianan ` (2 preceding siblings ...) 2021-09-03 13:40 ` [PATCH V2 3/6] erofs-utils: fix general style problem Huang Jianan @ 2021-09-03 13:40 ` Huang Jianan 2021-09-03 15:10 ` Gao Xiang 2021-09-03 13:40 ` [PATCH V2 5/6] erofs-utils: fix print style Huang Jianan 2021-09-03 13:40 ` [PATCH V2 6/6] erofs-utils: add missing /* fallthrough */ Huang Jianan 5 siblings, 1 reply; 21+ messages in thread From: Huang Jianan @ 2021-09-03 13:40 UTC (permalink / raw) To: linux-erofs From: Huang Jianan <huangjianan@oppo.com> Signed-off-by: Huang Jianan <huangjianan@oppo.com> --- fuse/main.c | 5 +---- lib/inode.c | 4 ---- 2 files changed, 1 insertion(+), 8 deletions(-) diff --git a/fuse/main.c b/fuse/main.c index fca4d7f..8137421 100644 --- a/fuse/main.c +++ b/fuse/main.c @@ -109,16 +109,13 @@ static struct options { bool odebug; } fusecfg; -#define OPTION(t, p) \ - { t, offsetof(struct options, p), 1 } +#define OPTION(t, p) { t, offsetof(struct options, p), 1 } static const struct fuse_opt option_spec[] = { OPTION("--dbglevel=%u", debug_lvl), OPTION("--help", show_help), FUSE_OPT_END }; -#define OPTION(t, p) { t, offsetof(struct options, p), 1 } - static void usage(void) { struct fuse_args args = FUSE_ARGS_INIT(0, NULL); diff --git a/lib/inode.c b/lib/inode.c index 97ee2c9..61dc802 100644 --- a/lib/inode.c +++ b/lib/inode.c @@ -677,11 +677,7 @@ out: * Don't leave DATA buffers which were written in the global * buffer list. It will make balloc() slowly. */ -#if 0 - bh->op = &erofs_drop_directly_bhops; -#else erofs_bdrop(bh, false); -#endif inode->bh_data = NULL; } return 0; -- 2.25.1 ^ permalink raw reply related [flat|nested] 21+ messages in thread
* Re: [PATCH V2 4/6] erofs-utils: remove unnecessary codes 2021-09-03 13:40 ` [PATCH V2 4/6] erofs-utils: remove unnecessary codes Huang Jianan @ 2021-09-03 15:10 ` Gao Xiang 0 siblings, 0 replies; 21+ messages in thread From: Gao Xiang @ 2021-09-03 15:10 UTC (permalink / raw) To: Huang Jianan; +Cc: linux-erofs On Fri, Sep 03, 2021 at 09:40:33PM +0800, Huang Jianan wrote: > From: Huang Jianan <huangjianan@oppo.com> > > Signed-off-by: Huang Jianan <huangjianan@oppo.com> Reviewed-by: Gao Xiang <hsiangkao@linux.alibaba.com> Thanks, Gao Xiang ^ permalink raw reply [flat|nested] 21+ messages in thread
* [PATCH V2 5/6] erofs-utils: fix print style 2021-09-03 13:40 ` [PATCH V2 0/6] erofs-utils: fix checkpatch.pl complains Huang Jianan ` (3 preceding siblings ...) 2021-09-03 13:40 ` [PATCH V2 4/6] erofs-utils: remove unnecessary codes Huang Jianan @ 2021-09-03 13:40 ` Huang Jianan 2021-09-03 13:40 ` [PATCH V2 6/6] erofs-utils: add missing /* fallthrough */ Huang Jianan 5 siblings, 0 replies; 21+ messages in thread From: Huang Jianan @ 2021-09-03 13:40 UTC (permalink / raw) To: linux-erofs From: Huang Jianan <huangjianan@oppo.com> Fix warning "quoted string split across lines". Signed-off-by: Huang Jianan <huangjianan@oppo.com> --- lib/inode.c | 3 +-- lib/io.c | 3 +-- lib/zmap.c | 3 +-- 3 files changed, 3 insertions(+), 6 deletions(-) diff --git a/lib/inode.c b/lib/inode.c index 61dc802..6024e8c 100644 --- a/lib/inode.c +++ b/lib/inode.c @@ -741,8 +741,7 @@ int erofs_droid_inode_fsconfig(struct erofs_inode *inode, cfg.target_out_path, &uid, &gid, &mode, &inode->capabilities); - erofs_dbg("/%s -> mode = 0x%x, uid = 0x%x, gid = 0x%x, " - "capabilities = 0x%" PRIx64 "\n", + erofs_dbg("/%s -> mode = 0x%x, uid = 0x%x, gid = 0x%x, capabilities = 0x%" PRIx64 "\n", fspath, mode, uid, gid, inode->capabilities); if (decorated) diff --git a/lib/io.c b/lib/io.c index b053137..620cb9c 100644 --- a/lib/io.c +++ b/lib/io.c @@ -242,8 +242,7 @@ int dev_read(void *buf, u64 offset, size_t len) } if (offset >= erofs_devsz || len > erofs_devsz || offset > erofs_devsz - len) { - erofs_err("read posion[%" PRIu64 ", %zd] is too large beyond" - "the end of device(%" PRIu64 ").", + erofs_err("read posion[%" PRIu64 ", %zd] is too large beyond the end of device(%" PRIu64 ").", offset, len, erofs_devsz); return -EINVAL; } diff --git a/lib/zmap.c b/lib/zmap.c index fdc84af..ce79601 100644 --- a/lib/zmap.c +++ b/lib/zmap.c @@ -57,8 +57,7 @@ static int z_erofs_fill_inode_lazy(struct erofs_inode *vi) if (vi->datalayout == EROFS_INODE_FLAT_COMPRESSION && !(vi->z_advise & Z_EROFS_ADVISE_BIG_PCLUSTER_1) ^ !(vi->z_advise & Z_EROFS_ADVISE_BIG_PCLUSTER_2)) { - erofs_err( -"big pcluster head1/2 of compact indexes should be consistent for nid %llu", + erofs_err("big pcluster head1/2 of compact indexes should be consistent for nid %llu", vi->nid * 1ULL); return -EFSCORRUPTED; } -- 2.25.1 ^ permalink raw reply related [flat|nested] 21+ messages in thread
* [PATCH V2 6/6] erofs-utils: add missing /* fallthrough */ 2021-09-03 13:40 ` [PATCH V2 0/6] erofs-utils: fix checkpatch.pl complains Huang Jianan ` (4 preceding siblings ...) 2021-09-03 13:40 ` [PATCH V2 5/6] erofs-utils: fix print style Huang Jianan @ 2021-09-03 13:40 ` Huang Jianan 2021-09-03 15:21 ` Gao Xiang 5 siblings, 1 reply; 21+ messages in thread From: Huang Jianan @ 2021-09-03 13:40 UTC (permalink / raw) To: linux-erofs From: Huang Jianan <huangjianan@oppo.com> Signed-off-by: Huang Jianan <huangjianan@oppo.com> --- lib/zmap.c | 3 +++ 1 file changed, 3 insertions(+) diff --git a/lib/zmap.c b/lib/zmap.c index ce79601..458030b 100644 --- a/lib/zmap.c +++ b/lib/zmap.c @@ -360,6 +360,7 @@ static int z_erofs_extent_lookback(struct z_erofs_maprecorder *m, return z_erofs_extent_lookback(m, m->delta[0]); case Z_EROFS_VLE_CLUSTER_TYPE_PLAIN: map->m_flags &= ~EROFS_MAP_ZIPPED; + /* fallthrough */ case Z_EROFS_VLE_CLUSTER_TYPE_HEAD: map->m_la = (lcn << lclusterbits) | m->clusterofs; break; @@ -479,6 +480,7 @@ int z_erofs_map_blocks_iter(struct erofs_inode *vi, case Z_EROFS_VLE_CLUSTER_TYPE_PLAIN: if (endoff >= m.clusterofs) map->m_flags &= ~EROFS_MAP_ZIPPED; + /* fallthrough */ case Z_EROFS_VLE_CLUSTER_TYPE_HEAD: if (endoff >= m.clusterofs) { map->m_la = (m.lcn << lclusterbits) | m.clusterofs; @@ -494,6 +496,7 @@ int z_erofs_map_blocks_iter(struct erofs_inode *vi, end = (m.lcn << lclusterbits) | m.clusterofs; map->m_flags |= EROFS_MAP_FULL_MAPPED; m.delta[0] = 1; + /* fallthrough */ case Z_EROFS_VLE_CLUSTER_TYPE_NONHEAD: /* get the correspoinding first chunk */ err = z_erofs_extent_lookback(&m, m.delta[0]); -- 2.25.1 ^ permalink raw reply related [flat|nested] 21+ messages in thread
* Re: [PATCH V2 6/6] erofs-utils: add missing /* fallthrough */ 2021-09-03 13:40 ` [PATCH V2 6/6] erofs-utils: add missing /* fallthrough */ Huang Jianan @ 2021-09-03 15:21 ` Gao Xiang 0 siblings, 0 replies; 21+ messages in thread From: Gao Xiang @ 2021-09-03 15:21 UTC (permalink / raw) To: Huang Jianan; +Cc: linux-erofs On Fri, Sep 03, 2021 at 09:40:35PM +0800, Huang Jianan wrote: > From: Huang Jianan <huangjianan@oppo.com> > > Signed-off-by: Huang Jianan <huangjianan@oppo.com> Reviewed-by: Gao Xiang <hsiangkao@linux.alibaba.com> Thanks, Gao Xiang ^ permalink raw reply [flat|nested] 21+ messages in thread
end of thread, other threads:[~2021-09-03 15:22 UTC | newest] Thread overview: 21+ messages (download: mbox.gz / follow: Atom feed) -- links below jump to the message on this page -- 2021-08-31 16:51 [PATCH 0/5] erofs-utils: fix checkpatch.pl complains Huang Jianan 2021-08-31 16:51 ` [PATCH 1/5] erofs-utils: remove filename in the file Huang Jianan 2021-09-03 3:09 ` Gao Xiang 2021-08-31 16:51 ` [PATCH 2/5] erofs-utils: fix SPDX comment style Huang Jianan 2021-09-03 3:10 ` Gao Xiang 2021-08-31 16:51 ` [PATCH 3/5] erofs-utils: fix general style problem Huang Jianan 2021-09-03 3:12 ` Gao Xiang 2021-08-31 16:51 ` [PATCH 4/5] erofs-utils: remove unnecessary codes and comments Huang Jianan 2021-09-03 3:17 ` Gao Xiang 2021-08-31 16:51 ` [PATCH 5/5] erofs-utils: fix print style Huang Jianan 2021-09-03 3:18 ` Gao Xiang 2021-09-03 13:40 ` [PATCH V2 0/6] erofs-utils: fix checkpatch.pl complains Huang Jianan 2021-09-03 13:40 ` [PATCH V2 1/6] erofs-utils: clean up file headers & footers Huang Jianan 2021-09-03 15:03 ` Gao Xiang 2021-09-03 13:40 ` [PATCH V2 2/6] erofs-utils: fix SPDX comment style Huang Jianan 2021-09-03 13:40 ` [PATCH V2 3/6] erofs-utils: fix general style problem Huang Jianan 2021-09-03 13:40 ` [PATCH V2 4/6] erofs-utils: remove unnecessary codes Huang Jianan 2021-09-03 15:10 ` Gao Xiang 2021-09-03 13:40 ` [PATCH V2 5/6] erofs-utils: fix print style Huang Jianan 2021-09-03 13:40 ` [PATCH V2 6/6] erofs-utils: add missing /* fallthrough */ Huang Jianan 2021-09-03 15:21 ` Gao Xiang
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.