From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org Received: from lists.ozlabs.org (lists.ozlabs.org [112.213.38.117]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.lore.kernel.org (Postfix) with ESMTPS id 57990C001DF for ; Wed, 2 Aug 2023 09:18:30 +0000 (UTC) Received: from boromir.ozlabs.org (localhost [IPv6:::1]) by lists.ozlabs.org (Postfix) with ESMTP id 4RG5xd0cTJz3c7g for ; Wed, 2 Aug 2023 19:18:29 +1000 (AEST) Authentication-Results: lists.ozlabs.org; spf=pass (sender SPF authorized) smtp.mailfrom=linux.alibaba.com (client-ip=115.124.30.130; helo=out30-130.freemail.mail.aliyun.com; envelope-from=jefflexu@linux.alibaba.com; receiver=lists.ozlabs.org) Received: from out30-130.freemail.mail.aliyun.com (out30-130.freemail.mail.aliyun.com [115.124.30.130]) (using TLSv1.3 with cipher TLS_AES_256_GCM_SHA384 (256/256 bits) key-exchange X25519 server-signature RSA-PSS (2048 bits) server-digest SHA256) (No client certificate requested) by lists.ozlabs.org (Postfix) with ESMTPS id 4RG5x93PnGz3bVJ for ; Wed, 2 Aug 2023 19:18:05 +1000 (AEST) X-Alimail-AntiSpam: AC=PASS;BC=-1|-1;BR=01201311R441e4;CH=green;DM=||false|;DS=||;FP=0|-1|-1|-1|0|-1|-1|-1;HT=ay29a033018045170;MF=jefflexu@linux.alibaba.com;NM=1;PH=DS;RN=2;SR=0;TI=SMTPD_---0VouOMIo_1690967879; Received: from localhost(mailfrom:jefflexu@linux.alibaba.com fp:SMTPD_---0VouOMIo_1690967879) by smtp.aliyun-inc.com; Wed, 02 Aug 2023 17:18:00 +0800 From: Jingbo Xu To: xiang@kernel.org, linux-erofs@lists.ozlabs.org Subject: [PATCH v2 09/16] erofs-utils: add erofs_rebuild_dump_tree() helper Date: Wed, 2 Aug 2023 17:17:43 +0800 Message-Id: <20230802091750.74181-9-jefflexu@linux.alibaba.com> X-Mailer: git-send-email 2.19.1.6.gb485710b In-Reply-To: <20230802091750.74181-1-jefflexu@linux.alibaba.com> References: <20230802091750.74181-1-jefflexu@linux.alibaba.com> MIME-Version: 1.0 Content-Transfer-Encoding: 8bit X-BeenThere: linux-erofs@lists.ozlabs.org X-Mailman-Version: 2.1.29 Precedence: list List-Id: Development of Linux EROFS file system List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: linux-erofs-bounces+linux-erofs=archiver.kernel.org@lists.ozlabs.org Sender: "Linux-erofs" Enhance tarerofs_dump_tree() so that it could optionally skip whiteout files. Rename it to erofs_rebuild_dump_tree(). Signed-off-by: Jingbo Xu --- include/erofs/inode.h | 2 +- include/erofs/internal.h | 2 ++ lib/inode.c | 23 ++++++++++++++++++++--- lib/tar.c | 2 -- mkfs/main.c | 2 +- 5 files changed, 24 insertions(+), 7 deletions(-) diff --git a/include/erofs/inode.h b/include/erofs/inode.h index 1c602a8..a43cb8f 100644 --- a/include/erofs/inode.h +++ b/include/erofs/inode.h @@ -32,7 +32,7 @@ unsigned int erofs_iput(struct erofs_inode *inode); erofs_nid_t erofs_lookupnid(struct erofs_inode *inode); struct erofs_dentry *erofs_d_alloc(struct erofs_inode *parent, const char *name); -int tarerofs_dump_tree(struct erofs_inode *dir); +int erofs_rebuild_dump_tree(struct erofs_inode *dir, bool skip_whout); int erofs_init_empty_dir(struct erofs_inode *dir); struct erofs_inode *erofs_new_inode(void); struct erofs_inode *erofs_mkfs_build_tree_from_path(const char *path); diff --git a/include/erofs/internal.h b/include/erofs/internal.h index 83a2e22..be84dc1 100644 --- a/include/erofs/internal.h +++ b/include/erofs/internal.h @@ -415,6 +415,8 @@ static inline u32 erofs_crc32c(u32 crc, const u8 *in, size_t len) return crc; } +#define EROFS_WHITEOUT_DEV 0 + #ifdef __cplusplus } #endif diff --git a/lib/inode.c b/lib/inode.c index b967aab..dae6c11 100644 --- a/lib/inode.c +++ b/lib/inode.c @@ -1323,9 +1323,14 @@ struct erofs_inode *erofs_mkfs_build_special_from_fd(int fd, const char *name) return inode; } -int tarerofs_dump_tree(struct erofs_inode *dir) +static bool erofs_inode_is_whiteout(struct erofs_inode *inode) { - struct erofs_dentry *d; + return S_ISCHR(inode->i_mode) && inode->u.i_rdev == EROFS_WHITEOUT_DEV; +} + +int erofs_rebuild_dump_tree(struct erofs_inode *dir, bool skip_whout) +{ + struct erofs_dentry *d, *n; unsigned int nr_subdirs; int ret; @@ -1368,6 +1373,18 @@ int tarerofs_dump_tree(struct erofs_inode *dir) return 0; } + if (skip_whout) { + list_for_each_entry_safe(d, n, &dir->i_subdirs, d_child) { + if (erofs_inode_is_whiteout(d->inode)) { + erofs_dbg("skip whiteout %s", d->inode->i_srcpath); + erofs_d_invalidate(d); + list_del(&d->d_child); + free(d); + continue; + } + } + } + nr_subdirs = 0; list_for_each_entry(d, &dir->i_subdirs, d_child) ++nr_subdirs; @@ -1391,7 +1408,7 @@ int tarerofs_dump_tree(struct erofs_inode *dir) continue; inode = erofs_igrab(d->inode); - ret = tarerofs_dump_tree(inode); + ret = erofs_rebuild_dump_tree(inode, skip_whout); dir->i_nlink += (erofs_mode_to_ftype(inode->i_mode) == EROFS_FT_DIR); erofs_iput(inode); if (ret) diff --git a/lib/tar.c b/lib/tar.c index 3180ee4..5382c54 100644 --- a/lib/tar.c +++ b/lib/tar.c @@ -19,8 +19,6 @@ #include "erofs/xattr.h" #include "erofs/blobchunk.h" -#define EROFS_WHITEOUT_DEV 0 - static char erofs_libbuf[16384]; struct tar_header { diff --git a/mkfs/main.c b/mkfs/main.c index 3809c71..b495a54 100644 --- a/mkfs/main.c +++ b/mkfs/main.c @@ -947,7 +947,7 @@ int main(int argc, char **argv) if (err < 0) goto exit; - err = tarerofs_dump_tree(root_inode); + err = erofs_rebuild_dump_tree(root_inode, false); if (err < 0) goto exit; } -- 2.19.1.6.gb485710b