From patchwork Fri Nov 11 10:08:29 2022 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Sheng Yong X-Patchwork-Id: 18645 Return-Path: Delivered-To: ouuuleilei@gmail.com Received: by 2002:a5d:6687:0:0:0:0:0 with SMTP id l7csp655927wru; Fri, 11 Nov 2022 02:18:01 -0800 (PST) X-Google-Smtp-Source: AA0mqf5xxPvVAoP2IQ/7s1YNOyzdfEHJbLzLBR7mlI3VIa9LUavbPJLMpDLQLYeF3r3DBU0rnr18 X-Received: by 2002:a50:ff04:0:b0:462:709:9f7b with SMTP id a4-20020a50ff04000000b0046207099f7bmr891583edu.263.1668161881245; Fri, 11 Nov 2022 02:18:01 -0800 (PST) ARC-Seal: i=2; a=rsa-sha256; t=1668161881; cv=pass; d=google.com; s=arc-20160816; b=QqFGeUHOa5NCaYCBTrqZ9ly+N/jyBhlaQxpEgl8KY0lP6B08WksUuoeP6s20vnE5sU AgWiBdwvRGTsjn19ejDaD3dpKVSQVeAHIdjq8F/hVe7xdD9f7wltR33aIHc28xnt0zej YZVNmoa9QmJoEs1xEWiWVLOchka/HF0AFRi27hVGkY8D7TxAlgv9EVbG/qOWIgS9Lcs3 0e/d6hlDhsVgOcpLKmg4AfJ5HOd4qC/3vmmEG42/O9pb6ViRxFX0e7/7LzSvJVGl25as o0lxcM/82STCQNsULyWChyxB4rj939we8mjo+7lZhXKDVKtfKKL1cQtDaBdRyoODpiUh Knjg== ARC-Message-Signature: i=2; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:mime-version:content-transfer-encoding :message-id:date:subject:cc:to:from:dkim-signature; bh=ffos0/zYHGurDgmO9NUd5jUgkWs+Vhq7VDzZxBNU4Vk=; b=aqfZ5wfGpYkzIhu5Fs992Tr95x+DzJH9ZhIS+Gu86Tx2M/SEaL6dxFNeffWoK/RsBT KybTYBoKCjjqpusGmFg7tKRyrsRZhpSIfq35BxknacsjpzJSEyVuwK07a8UAnAuhv3VY rWtecoau92SVcAbycBwRMkxF5d08f4kVjEjPjtLVBNQbmqAQHm+1GG1+yPkbd3gEq16C l9uuuKocKHuCiWc6oqMEnXqhVHmTnP3udJtTUt5kaCVpPdbVnHBArI7ng7VuQlJ6qX9M W254bb3iAhC7SO/cZ1iFIDwN7itwCgutB6Duq+mtpjc0NvMcj5QdtPusovduenRZUyRH XHfA== ARC-Authentication-Results: i=2; mx.google.com; dkim=pass header.i=@oppo.com header.s=selector1 header.b=QpTkgwn6; arc=pass (i=1 spf=pass spfdomain=oppo.com dkim=pass dkdomain=oppo.com dmarc=pass fromdomain=oppo.com); spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=pass (p=QUARANTINE sp=QUARANTINE dis=NONE) header.from=oppo.com Received: from out1.vger.email (out1.vger.email. [2620:137:e000::1:20]) by mx.google.com with ESMTP id h9-20020a50c389000000b00461c314f79fsi1758835edf.284.2022.11.11.02.17.37; Fri, 11 Nov 2022 02:18:01 -0800 (PST) Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) client-ip=2620:137:e000::1:20; Authentication-Results: mx.google.com; dkim=pass header.i=@oppo.com header.s=selector1 header.b=QpTkgwn6; arc=pass (i=1 spf=pass spfdomain=oppo.com dkim=pass dkdomain=oppo.com dmarc=pass fromdomain=oppo.com); spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=pass (p=QUARANTINE sp=QUARANTINE dis=NONE) header.from=oppo.com Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S232803AbiKKKJQ (ORCPT + 99 others); Fri, 11 Nov 2022 05:09:16 -0500 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:49004 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S233552AbiKKKJI (ORCPT ); Fri, 11 Nov 2022 05:09:08 -0500 Received: from APC01-SG2-obe.outbound.protection.outlook.com (mail-sgaapc01on2077.outbound.protection.outlook.com [40.107.215.77]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id 6A15E7BE44 for ; Fri, 11 Nov 2022 02:08:59 -0800 (PST) ARC-Seal: i=1; a=rsa-sha256; s=arcselector9901; d=microsoft.com; cv=none; b=EbyGjDTc8yR5Od+5oekXdBQ+Bx7CJ4ud0gc5B8pSD0UPpOOhHl/F+nXkHROOSXJGGfbSn+8d+U7xDXZGNKjtLAoy7nE/arUI30CWToz4BDhLhqzXRhhjrUXpnkQuHbIKHiofJpGYdJC7AVXQ5vVDHX/STxHykqNXpeaifQyMmq2hPZMWMpychGXJDiO1+06W/DfpU8Ana1c3Q0dGF9+zUlz713ePIlrGBgS/8AywI6Ge/ve7HcliYj9lN1vcwCdAhCJUJMO+AaCfaIcdQkvHy6UUuI2V45yLpjQaV26N0waciAy4EHqPWgkpMX7oF44F/M+3wkTqyavghyT4QjcP6g== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=microsoft.com; s=arcselector9901; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-AntiSpam-MessageData-ChunkCount:X-MS-Exchange-AntiSpam-MessageData-0:X-MS-Exchange-AntiSpam-MessageData-1; bh=ffos0/zYHGurDgmO9NUd5jUgkWs+Vhq7VDzZxBNU4Vk=; b=JoCcTg7AtTkKXYCH4kjj5/YkDNZtubmH0DbZgIb7SUV4m7/Uir6EF42+euizYQ5moYsvzAw3oRfKberQWjOgtM5Z34OXukYvFW8+Soz/PAmZDRd1kWY3wUnznI/bVVCboSkR6coyXoZR2UNf6To9GQ12LCQKeoMMNriClX6DYLO9wUN1DQlcDpsZ/vaPHu1suUFAtRSEN3VAppW0/HJhiUDku4bHjJ+mGa2vG0Hz8aG/RLIfoO99hgd9SqVpRhSe0fvJ4RgfKXCeduEbH4ADZ9fHqFYDCiR1llUzlM8HnPOLT77y69fmKGiBXFrZeF7STTFVGnG7Let6AczhVFWLGg== ARC-Authentication-Results: i=1; mx.microsoft.com 1; spf=pass smtp.mailfrom=oppo.com; dmarc=pass action=none header.from=oppo.com; dkim=pass header.d=oppo.com; arc=none DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=oppo.com; s=selector1; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-SenderADCheck; bh=ffos0/zYHGurDgmO9NUd5jUgkWs+Vhq7VDzZxBNU4Vk=; b=QpTkgwn6rI5p9Xj4rRAj6N3sd4KVCp6hTNNy6SA+JwzA5SyCI5xmsYWyVGMHKGNZMEImzbFyu77A3duQCJmBPuwjNE8iZLUSLApmKrsZSqj7S0+fPucVVvEWkKC6XDcNn2qUmrQKF2JNtKKkqYCHaYoZXKZuTnqccfntRHvN/4g= Authentication-Results: dkim=none (message not signed) header.d=none;dmarc=none action=none header.from=oppo.com; Received: from SI2PR02MB5148.apcprd02.prod.outlook.com (2603:1096:4:153::6) by SG2PR02MB5901.apcprd02.prod.outlook.com (2603:1096:4:1d2::11) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.5791.27; Fri, 11 Nov 2022 10:08:56 +0000 Received: from SI2PR02MB5148.apcprd02.prod.outlook.com ([fe80::2954:8e56:dcd1:452c]) by SI2PR02MB5148.apcprd02.prod.outlook.com ([fe80::2954:8e56:dcd1:452c%3]) with mapi id 15.20.5813.013; Fri, 11 Nov 2022 10:08:56 +0000 From: Sheng Yong To: jaegeuk@kernel.org, chao@kernel.org Cc: linux-f2fs-devel@lists.sourceforge.net, linux-kernel@vger.kernel.org, Sheng Yong Subject: [PATCH v2 1/2] f2fs: fix to enable compress for newly created file if extension matches Date: Fri, 11 Nov 2022 18:08:29 +0800 Message-Id: <20221111100830.953733-1-shengyong@oppo.com> X-Mailer: git-send-email 2.25.1 X-ClientProxiedBy: TYAPR01CA0135.jpnprd01.prod.outlook.com (2603:1096:404:2d::27) To SI2PR02MB5148.apcprd02.prod.outlook.com (2603:1096:4:153::6) MIME-Version: 1.0 X-MS-PublicTrafficType: Email X-MS-TrafficTypeDiagnostic: SI2PR02MB5148:EE_|SG2PR02MB5901:EE_ X-MS-Office365-Filtering-Correlation-Id: c80cbf0b-b008-44ed-b9b1-08dac3ccbeb2 X-MS-Exchange-SenderADCheck: 1 X-MS-Exchange-AntiSpam-Relay: 0 X-Microsoft-Antispam: BCL:0; X-Microsoft-Antispam-Message-Info: vLma93glbT3V4aUZYG+Yqurs9nk3DvwvbIkXP3CSLtbRFCDql6u8M0lycGb2NZQgczVWnvWXhislAz3jbg7UEaUqxsM8DvcFryMYgl1itqDUSwSn8CIho64eH2ZvRBs4HYlFYlIkXV+TiX6T7JGhNCOqrhoyvv69EANkVPE+61Db4nv5D6wAb6ziaJKdHL67R2IKqvIJm07jGaXISd36XRvyKSCFb/3nLTwn0I0ISERdT67Gt0WVGoP6NwBIp+yQbf9IWR4dNVdAzQR41HjXtsKGQ1bbEuV2CL7mrPPzZoHBHhAl/xuJplEcTMQqLSMv8ht0ckIpghzEWhskRMAv1so7Sq2JOweX+sBaq7gaceX8Pb+Uxe9sYNg67kA4ssq+jqxrmUuQubXRF8QCfDo1AugKRhBgfJD74a8bAWj07NkxD+gg2H3FsCSeguS1TS7jJqYNsT6rZzNsrQC3xm1gkqluOpwgnfJct+P0Nwed8hEFo7SpiVldfunSY4hrTAM+FZjBbLpV5qNiJs5ax9wGCwUPE8HXCKQdS/lAuryOMH139QkcHaPHZYSGFF+3FxPX9RtpNY/81FTehBSz47kBNAxLDoh/RXjmItHJZo4a8gX5/upvp9gOG+AQku51XeZ5tyVHfhmj4PYGdICW1l3YTtlBn4dFbdJHcW8FEI+UFMX2wf5P0j61Kga8jO9A6RQc2LydtoXguD3nTPiyHhVQVkHizyjGEaYGlpCUZxpsSy4ykz+jam40EIWGNq13jUQI X-Forefront-Antispam-Report: CIP:255.255.255.255;CTRY:;LANG:en;SCL:1;SRV:;IPV:NLI;SFV:NSPM;H:SI2PR02MB5148.apcprd02.prod.outlook.com;PTR:;CAT:NONE;SFS:(13230022)(4636009)(39860400002)(376002)(366004)(346002)(136003)(396003)(451199015)(36756003)(38350700002)(6512007)(83380400001)(86362001)(478600001)(6486002)(186003)(2906002)(38100700002)(1076003)(52116002)(26005)(6506007)(8936002)(4326008)(107886003)(66946007)(5660300002)(6666004)(316002)(66556008)(41300700001)(2616005)(66476007)(8676002);DIR:OUT;SFP:1101; X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1 X-MS-Exchange-AntiSpam-MessageData-0: TNr7BtdM/EkOZlm+N5Ydf9/pMz1F7G4zXHmDLmnsTL4SjTcvsrTyUMUD/MUWQi7+oh1N9EL/XfyteglnJX1ucYBO6rjK1lIOJjkl4ct3UkOObYiFu6qMQeu4hm2RCDj9jd7aTPiW1JeRIO8ZO3pAqHpWjxf5/jqQDpOOBoGoG+5z1o1plPd+ILhRc7T+NzIEwh1zKQHDn3Wxx3XY43JaMN3+9oG7cPpfw6HNFfklyaWc4i3eMh8Rlqvp8M47A2tpnshZtF7rvKNpwpKsCAh7vkSMuO0vo+ztLKjGTHxQW0m6xb4hOy2rwddYLlglbTNveY5rvQEyb8T5roBxpX2cXVM+bMUb1UtOUIhaUn44HbYciIh7bDqxIG96Dnv9vGYQ1MV/1SXiQrg0XXF14IgmHy4x29WMJgSCMJvGEbfTR5/jvMHwbIpQALT4sY0+4jYnhVzdonwy3XH61rddwGMXpylmu0TU5ReNAIJ0HuYf+GVIqUF++uEXoREnIfFGk2AMdC57xS7OymkuymXa2fn9JMDM9DNxeadG1YD4WeRmk1FH/atmY/30drSRaVYMCt+Vf+gAab8QHlKr47OSCGiVL3NuvYBIDvl4a2/eRObGXGJisKpLHlpZgPDcmkr8qNjdi3FnMo8lMo0OFJ/X1Ip+PcDAMFJ2c75hJkjBUVHxANlRvwJfPGeIKxEPGzTFbD1FTlO0ju65lJ7Nt3hltdPfJAGz0Ix65emwsNp6jHdnX+i4hSf9Z71A/a5TZUiNb+JqcNnYZqDGDVYIjkDQ4hO2TKxcAxRAWuqnYc02cvrecHLU2/E0VsAy1CH04ANobkcet109jqaZF1VTOJgNeOAeC00hPAb3AM91+WEBRvhWb6VKHZTIWz+wlKV/ZpS6aL+Ip5UU46QE5tmTcuiZFOrA9v4wgpZTplNSpwhbbke+cLtOSafNXXEcAuw4kTkHrQ4FDFDs9bNwi2stJn5AIWQUuBhgt9QhSStiw8fVyE0WSpPyuagxEQIDTi1+xzUdcdh8qZLNg8PeTGxkxl36Eh/SoJfyRTBhczGBYiCq2z/26nA96ICphgb0onxwxHDJU1PSktbi22yvuKbiLrtxH/aqVH6YwViMG0P8NK0jIoFvad/rpDo1PLQoWZfGP/5RpeIJOQTgvcv7kbIEAhPi9w1WDh1EXotLpWNI3W3bQcTrRZuxITRZlQ0QEmqsU92RqcdonU9bbWAWrpae/HxW8LrKutyj6xq5o52nh2V5yTBIQbTgdqMsxfP80gYf1FAVGUCQiibxT5Q1KQEQAH3msA1301iGLZw49kj82RcU/MDZAkB9yCFRLTbeGjnTOK0MBCGQ+uuckTi7ozLO68nE5MEQXN5iO6Rbn2V3MGvnjAYsiYwQ0NmkLQl6bksjqvGFFa1XCR+ClV868Z5iNmzm0tCzx0JDQssGfvNniamiD46YxTNd20zEE0+YEvQOHww8EZxlQyKovF5bHQpWjVITfrWX6t5FzD2sSHS1ztrMhP4uhJt2+93x+qRX1iJJlLZjbqTS40pUny72JHfHjXngFeWdaqnrQZkGl5xgrmkEok0JqnaiD3III5y7M8o91iYtnZeg X-OriginatorOrg: oppo.com X-MS-Exchange-CrossTenant-Network-Message-Id: c80cbf0b-b008-44ed-b9b1-08dac3ccbeb2 X-MS-Exchange-CrossTenant-AuthSource: SI2PR02MB5148.apcprd02.prod.outlook.com X-MS-Exchange-CrossTenant-AuthAs: Internal X-MS-Exchange-CrossTenant-OriginalArrivalTime: 11 Nov 2022 10:08:56.8662 (UTC) X-MS-Exchange-CrossTenant-FromEntityHeader: Hosted X-MS-Exchange-CrossTenant-Id: f1905eb1-c353-41c5-9516-62b4a54b5ee6 X-MS-Exchange-CrossTenant-MailboxType: HOSTED X-MS-Exchange-CrossTenant-UserPrincipalName: 0D3k/01lkudxfA06Q2b1iVRBhDNSO4rL7dY7stqHBk2NV2+x6ikthZMoNdrin+Y2/agVgpOWJSh4Ie9HhhZi/Q== X-MS-Exchange-Transport-CrossTenantHeadersStamped: SG2PR02MB5901 X-Spam-Status: No, score=-2.1 required=5.0 tests=BAYES_00,DKIM_SIGNED, DKIM_VALID,DKIM_VALID_AU,DKIM_VALID_EF,RCVD_IN_DNSWL_NONE, RCVD_IN_MSPIKE_H2,SPF_HELO_PASS,SPF_PASS autolearn=ham autolearn_force=no version=3.4.6 X-Spam-Checker-Version: SpamAssassin 3.4.6 (2021-04-09) on lindbergh.monkeyblade.net Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org X-getmail-retrieved-from-mailbox: =?utf-8?q?INBOX?= X-GMAIL-THRID: =?utf-8?q?1749194512625242599?= X-GMAIL-MSGID: =?utf-8?q?1749194512625242599?= If compress_extension is set, and a newly created file matches the extension, the file could be marked as compression file. However, if inline_data is also enabled, there is no chance to check its extension since f2fs_should_compress() always returns false. This patch moves set_compress_inode(), which do extension check, in f2fs_should_compress() to check extensions before setting inline data flag. Fixes: 7165841d578e ("f2fs: fix to check inline_data during compressed inode conversion") Signed-off-by: Sheng Yong Reviewed-by: Chao Yu Signed-off-by: Sheng Yong Signed-off-by: Jaegeuk Kim --- fs/f2fs/namei.c | 27 +++++++++++++-------------- 1 file changed, 13 insertions(+), 14 deletions(-) --- v1->v2: add filename parameter for f2fs_new_inode, and move set_compress_inode into f2fs_new_inode diff --git a/fs/f2fs/namei.c b/fs/f2fs/namei.c index e104409c3a0e5..36e251f438568 100644 --- a/fs/f2fs/namei.c +++ b/fs/f2fs/namei.c @@ -22,8 +22,12 @@ #include "acl.h" #include +static void set_compress_inode(struct f2fs_sb_info *sbi, struct inode *inode, + const unsigned char *name); + static struct inode *f2fs_new_inode(struct user_namespace *mnt_userns, - struct inode *dir, umode_t mode) + struct inode *dir, umode_t mode, + const char *name) { struct f2fs_sb_info *sbi = F2FS_I_SB(dir); nid_t ino; @@ -119,6 +123,8 @@ static struct inode *f2fs_new_inode(struct user_namespace *mnt_userns, if ((F2FS_I(dir)->i_flags & F2FS_COMPR_FL) && f2fs_may_compress(inode)) set_compress_context(inode); + if (name) + set_compress_inode(sbi, inode, name); } /* Should enable inline_data after compression set */ @@ -293,8 +299,7 @@ static void set_compress_inode(struct f2fs_sb_info *sbi, struct inode *inode, unsigned char noext_cnt = F2FS_OPTION(sbi).nocompress_ext_cnt; int i, cold_count, hot_count; - if (!f2fs_sb_has_compression(sbi) || - F2FS_I(inode)->i_flags & F2FS_NOCOMP_FL || + if (F2FS_I(inode)->i_flags & F2FS_NOCOMP_FL || !f2fs_may_compress(inode) || (!ext_cnt && !noext_cnt)) return; @@ -326,10 +331,6 @@ static void set_compress_inode(struct f2fs_sb_info *sbi, struct inode *inode, for (i = 0; i < ext_cnt; i++) { if (!is_extension_exist(name, ext[i], false)) continue; - - /* Do not use inline_data with compression */ - stat_dec_inline_inode(inode); - clear_inode_flag(inode, FI_INLINE_DATA); set_compress_context(inode); return; } @@ -352,15 +353,13 @@ static int f2fs_create(struct user_namespace *mnt_userns, struct inode *dir, if (err) return err; - inode = f2fs_new_inode(mnt_userns, dir, mode); + inode = f2fs_new_inode(mnt_userns, dir, mode, dentry->d_name.name); if (IS_ERR(inode)) return PTR_ERR(inode); if (!test_opt(sbi, DISABLE_EXT_IDENTIFY)) set_file_temperature(sbi, inode, dentry->d_name.name); - set_compress_inode(sbi, inode, dentry->d_name.name); - inode->i_op = &f2fs_file_inode_operations; inode->i_fop = &f2fs_file_operations; inode->i_mapping->a_ops = &f2fs_dblock_aops; @@ -689,7 +688,7 @@ static int f2fs_symlink(struct user_namespace *mnt_userns, struct inode *dir, if (err) return err; - inode = f2fs_new_inode(mnt_userns, dir, S_IFLNK | S_IRWXUGO); + inode = f2fs_new_inode(mnt_userns, dir, S_IFLNK | S_IRWXUGO, NULL); if (IS_ERR(inode)) return PTR_ERR(inode); @@ -760,7 +759,7 @@ static int f2fs_mkdir(struct user_namespace *mnt_userns, struct inode *dir, if (err) return err; - inode = f2fs_new_inode(mnt_userns, dir, S_IFDIR | mode); + inode = f2fs_new_inode(mnt_userns, dir, S_IFDIR | mode, NULL); if (IS_ERR(inode)) return PTR_ERR(inode); @@ -817,7 +816,7 @@ static int f2fs_mknod(struct user_namespace *mnt_userns, struct inode *dir, if (err) return err; - inode = f2fs_new_inode(mnt_userns, dir, mode); + inode = f2fs_new_inode(mnt_userns, dir, mode, NULL); if (IS_ERR(inode)) return PTR_ERR(inode); @@ -856,7 +855,7 @@ static int __f2fs_tmpfile(struct user_namespace *mnt_userns, struct inode *dir, if (err) return err; - inode = f2fs_new_inode(mnt_userns, dir, mode); + inode = f2fs_new_inode(mnt_userns, dir, mode, NULL); if (IS_ERR(inode)) return PTR_ERR(inode); From patchwork Fri Nov 11 10:08:30 2022 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Sheng Yong X-Patchwork-Id: 18644 Return-Path: Delivered-To: ouuuleilei@gmail.com Received: by 2002:a5d:6687:0:0:0:0:0 with SMTP id l7csp655830wru; Fri, 11 Nov 2022 02:17:49 -0800 (PST) X-Google-Smtp-Source: AA0mqf4IzrxwWiRACp0DulPkodVj6Xf23J6z0+bFdNJWprndm0LG1CORisVjmsRv4GSUgFTy+QTK X-Received: by 2002:a17:906:6581:b0:7ac:2e16:eb05 with SMTP id x1-20020a170906658100b007ac2e16eb05mr1444106ejn.26.1668161868949; Fri, 11 Nov 2022 02:17:48 -0800 (PST) ARC-Seal: i=2; a=rsa-sha256; t=1668161868; cv=pass; d=google.com; s=arc-20160816; b=zswsZpzPd/IHVRYy9AdHREvK06cRS+l/DJ5onWeqFfu8JUvOEXzNARR3AQ3vJkO5Ff Ib27MAnpPKskgJuHO+b66bPJ0nEnL7cMRP3Hc1oeEK2WPe+kEAwbYF4ozC8bIqyeomqT 522CiNRSIFcb5xkApTwW6yWuUUuTlKppeJbIpUXerMs6+4qxj5ZgitziVazlQrl2Xs00 n7BE8oBmkS7Yh+2xFRJMIerN+Y3DCEEhg8jZIOMWLr3DTd24Mud7xlBrFIZYEqZ60HR/ 6vPzegiht8S9N2ihFwbyorDFDGfMeYw1AT2VsMDst/Md21R0ewPgn4mIyyj9On/nUCMU 3Wbw== ARC-Message-Signature: i=2; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:mime-version:content-transfer-encoding :references:in-reply-to:message-id:date:subject:cc:to:from :dkim-signature; bh=shef+udH1+23RYxNjD9guO1Ey2dJ6Hr/P05JP2ik1qQ=; b=tqes+qOU7IFI1N/MaIIhyVib/1NXXAw/GUxQtonXC/nRr2VUE6GSZHNdASpTRLXNEn MbOUaTBaMPbOUN92LTsHAAaaz6jVE0wPeruuQuWI8XBcx0ky+7K5Sua8kjiixjGUqNYP ZMzQWdx0b47fe+O3axqMhVQUYz3o4s6E+JU71lk1n9Tj6R7BzFxan6xrlHThWogXpCOX 808skg5dHR3eijQLov0jS+CnExU16amRKsJwzCcOWU8gm4E4cnaw3Ja11m0FCae3cQD1 uHB3VQMYTJ/l4YNffPirMSsdTR1utJMvyRkVwGKB0LxxZBz15IOrdXbj1Z70Tn1v2Lpd wR5w== ARC-Authentication-Results: i=2; mx.google.com; dkim=pass header.i=@oppo.com header.s=selector1 header.b=c7uk3IW4; arc=pass (i=1 spf=pass spfdomain=oppo.com dkim=pass dkdomain=oppo.com dmarc=pass fromdomain=oppo.com); spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=pass (p=QUARANTINE sp=QUARANTINE dis=NONE) header.from=oppo.com Received: from out1.vger.email (out1.vger.email. [2620:137:e000::1:20]) by mx.google.com with ESMTP id ds18-20020a170907725200b007940226fd21si1666658ejc.903.2022.11.11.02.17.24; Fri, 11 Nov 2022 02:17:48 -0800 (PST) Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) client-ip=2620:137:e000::1:20; Authentication-Results: mx.google.com; dkim=pass header.i=@oppo.com header.s=selector1 header.b=c7uk3IW4; arc=pass (i=1 spf=pass spfdomain=oppo.com dkim=pass dkdomain=oppo.com dmarc=pass fromdomain=oppo.com); spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=pass (p=QUARANTINE sp=QUARANTINE dis=NONE) header.from=oppo.com Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S233423AbiKKKJN (ORCPT + 99 others); Fri, 11 Nov 2022 05:09:13 -0500 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:49202 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S233553AbiKKKJI (ORCPT ); Fri, 11 Nov 2022 05:09:08 -0500 Received: from APC01-SG2-obe.outbound.protection.outlook.com (mail-sgaapc01on2077.outbound.protection.outlook.com [40.107.215.77]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id 25D267C8C4 for ; Fri, 11 Nov 2022 02:09:02 -0800 (PST) ARC-Seal: i=1; a=rsa-sha256; s=arcselector9901; d=microsoft.com; cv=none; b=FpR8QGh3eEc47M8LpxwZdqf5ukmHcz9cHCAUnGdUnlWDaSfSVFqQB77t7Ve0r0hnF76pYBYI/zLTbghGlNlvmWX3lkh+SV6GS6MgbtwZc5NGg5kVbqKsC2mJ6u2+tYUX+i8wqEB2sX+Z7Vq2TX2lPj1SuYnX5MYIzZ884NwwDWIxdBTI9Feb5ANp10XObBGocJzaFUkBXhN06UqEYGi7BGDM/8yYKmUtedJvh7WXi4Sd8+uFP0IiQU6NIzd4nRNIcgO90hXRWHtix0aHbTOASA5lLpAUaWnj7MCNzDM+WK5otCmM9Hj5ARDmB0xGK+WxeVXMdyUxPLg0k/lMq5Gw3w== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=microsoft.com; s=arcselector9901; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-AntiSpam-MessageData-ChunkCount:X-MS-Exchange-AntiSpam-MessageData-0:X-MS-Exchange-AntiSpam-MessageData-1; bh=shef+udH1+23RYxNjD9guO1Ey2dJ6Hr/P05JP2ik1qQ=; b=K9RGKp6ahLSBSXkvYMP0DIw60aq8894Hzo8RqRv1bZ1vKuKqDGp081xMcSuaQNgnG0ngSdsq0yLwxLP8Sp/B9UVhOzZCMFofGvFBNiA3Z08d57tk2W1ptO40FSl0BDWtqNFo25Zh/ynOC3wr7j3lKHlwXioHEaE3dKRCVvUxNFdqR6qXWjh5beoIBzTz2hGOjNb1WsYKm5XSLeymm/rVew4O3sNsfxB+ufFt71JwtiJPV27VGaMiiRcbF6umOebr6ta1B9Qf0ROgj6uQQcxgEZOJYMK54VzHmFKeFzZgKRJjP/z+6jVF7WKS96wIlV6B6IxAPkV6xlu2pWte0d5ugg== ARC-Authentication-Results: i=1; mx.microsoft.com 1; spf=pass smtp.mailfrom=oppo.com; dmarc=pass action=none header.from=oppo.com; dkim=pass header.d=oppo.com; arc=none DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=oppo.com; s=selector1; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-SenderADCheck; bh=shef+udH1+23RYxNjD9guO1Ey2dJ6Hr/P05JP2ik1qQ=; b=c7uk3IW4nqi7RZj4g10AC3cU6bwUTiNWQLt4d+w78Ju0kDX6CoJ90D3m+/UBuigw3LGGBRkhtFEsgwxEVf1Smsuu8oPK3pI0mbT/Dp6ggXy0BL6g18niX0+bG1JEoSvdrnUK4z9iMhZOE7TNRfOKHRZqHwOJKk8PbTi24Uy1urc= Authentication-Results: dkim=none (message not signed) header.d=none;dmarc=none action=none header.from=oppo.com; Received: from SI2PR02MB5148.apcprd02.prod.outlook.com (2603:1096:4:153::6) by SG2PR02MB5901.apcprd02.prod.outlook.com (2603:1096:4:1d2::11) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.5791.27; Fri, 11 Nov 2022 10:08:58 +0000 Received: from SI2PR02MB5148.apcprd02.prod.outlook.com ([fe80::2954:8e56:dcd1:452c]) by SI2PR02MB5148.apcprd02.prod.outlook.com ([fe80::2954:8e56:dcd1:452c%3]) with mapi id 15.20.5813.013; Fri, 11 Nov 2022 10:08:58 +0000 From: Sheng Yong To: jaegeuk@kernel.org, chao@kernel.org Cc: linux-f2fs-devel@lists.sourceforge.net, linux-kernel@vger.kernel.org, Sheng Yong Subject: [PATCH v2 2/2] f2fs: move set_file_temperature into f2fs_new_inode Date: Fri, 11 Nov 2022 18:08:30 +0800 Message-Id: <20221111100830.953733-2-shengyong@oppo.com> X-Mailer: git-send-email 2.25.1 In-Reply-To: <20221111100830.953733-1-shengyong@oppo.com> References: <20221111100830.953733-1-shengyong@oppo.com> X-ClientProxiedBy: TYAPR01CA0135.jpnprd01.prod.outlook.com (2603:1096:404:2d::27) To SI2PR02MB5148.apcprd02.prod.outlook.com (2603:1096:4:153::6) MIME-Version: 1.0 X-MS-PublicTrafficType: Email X-MS-TrafficTypeDiagnostic: SI2PR02MB5148:EE_|SG2PR02MB5901:EE_ X-MS-Office365-Filtering-Correlation-Id: d5863c15-af71-428b-1134-08dac3ccbfa6 X-MS-Exchange-SenderADCheck: 1 X-MS-Exchange-AntiSpam-Relay: 0 X-Microsoft-Antispam: BCL:0; X-Microsoft-Antispam-Message-Info: YG6ZgsYbBuHgd7WV3dQXvyb9M56o8pGGPQD0Dn3Be0UouybyrbLqb5jppWrT10w7oeyY0ooqXUfwXiuEThF3r1SJyX2Khd+QF8NurcCjet2MDUZDM69TOC1WljC7zx6YxpTR5OfumzAgc3vMe9+hBilsP2rY0T0BjTH5+8OOqqHZpEWiBHJ7stsmH0T13oRC4JYBS/B2IaVhZUNnvZehnZGb1JA0mdGyjtMlkNuN5xTH0m+6Hana0LSSIWLFJoMJKC0ee5aH2iiIxbnCdrStn8/pS/EzH/Ya5GdYxapclPmhkrrNmXR6vt2Ugw2G+CHURWeWjuCioZaG45dF+M82JjlepIUCbHEjRrysEyXvO8MDKbDHmao0w/ZiIt4YZ9KRCTPoC/X9bnlZBCgavYYQFLW43raFwT+4pvhHtkZtVd9VkydBoK0q+LWqIiCxsuCdtxF+R6gBUqQ0JOjRoZ1UzgnPJ7ET6OLyf71yX30fgo7S7KDXomnI8fMM4T/1NktE1p0ue8ES3H1zE7NwMxzpb3VDfR0mirkPOUIr9z0z9mmG66c4Swt2dr/I35oPbTd2AVpi7Q9JpgW9UorW3SJ0wyRoXRc5cXlGWv+V1rugvEbEyK2nV244ArqjWWhTTT7Xdhqhyx/XiDTEYwZiTjgsssm5DxQ86THt/ux2082I20yp8+MKwh7rp4kwO4zLtxCbyTylCHexe33ovqzPrSsf9uZHy02sdKTqcDFa/PtmpYVSuWp4jF/lyKnJSAU6dQD6koC02HG1oZuNpp5vfK0I6Q== X-Forefront-Antispam-Report: CIP:255.255.255.255;CTRY:;LANG:en;SCL:1;SRV:;IPV:NLI;SFV:NSPM;H:SI2PR02MB5148.apcprd02.prod.outlook.com;PTR:;CAT:NONE;SFS:(13230022)(4636009)(39860400002)(376002)(366004)(346002)(136003)(396003)(451199015)(36756003)(38350700002)(6512007)(83380400001)(86362001)(478600001)(6486002)(186003)(2906002)(38100700002)(1076003)(52116002)(26005)(6506007)(8936002)(4326008)(107886003)(66946007)(5660300002)(6666004)(316002)(66556008)(41300700001)(2616005)(66476007)(8676002);DIR:OUT;SFP:1101; X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1 X-MS-Exchange-AntiSpam-MessageData-0: 7zeBI5g3YeQPF4DF/RaXB4y48FMDk5xXozC+XwjzLChbomAYQ1vbdxHhbliDd/KO2/JHeCfilj5JsYtIYDYoJNeUKmV/IOv89m82tx4wdWzzYN9ovkWKS2k9AiYiBJApzVA1xsC5UlgRLzI82bXWL3oAizxRQn7y1z5eCxnsMbX+bfJq50u7XTFAUvI10j1eyWxOw5YkgD+sAzdZk13lTo6umA8AUt5gdCF73e+adqVZb1kUrO2yV3bHCYSOkRfpHFm1wfAEceFKrrcgrZ/ou0IIfTkfgneAHu2SPHM7ZRY3b2zrGwJarl50l5mxVjV2XC3ch81r1cdXqemUe1Jgryc6bdJ7ljwcw+1f9gfSyuoNjYHbjNs1tpE03xpQyICAvm63aGT0S92dIWEtcfUqpEYhnyU1+MO3RC6F5crZ/1t33h+RW4hePKMeA1g9etpUjrCzDDx2icyjw1hV+Hp7MfMr2DewWUH8bEuvgnIpOXqqYqZDd09aHMrfoa0r6PH9y/sFp4Ch2CkHh502GoCsrihN2Ap3wIrf5fi1HKqQlcFUd5s8S1/OMg0ITzlkzUrWgnnwLyg28D4alp2Oqdp+ORlqUsoU0dGd1vjKONFi6zeIFx4TfXC6phYeukrAZEFHqMJzzfkjfmOwFnGKiKUJUFWkkrgUw/xvaSx499mDhjRyRYSE16k3vscRqpSy+Z9llcH7hcv8KiwNwaQwiUQg1XwY09slPDsizbCyQJFC2YleGzkr7FhImY42Yp20YqFc+wHvZPiGzXs9vvUKIn7xgpJ6ZRBzmjbvuxRzfrrHMRjfbwq6IQm+xeJ2Lx542iMDnPLcHZaViQE8fJGnhDmFnmOTbCwYAS5SEgq2s2lahyTpybR6EXKXYzHHrOl7BlkWINKR2nNE6lwqpCDi2pY472Tppko5hT7iS3FogFnIlh9UqbQ77Hjcq3qmtMoZc3U64TsJBLVTXaWVP+mozoSau3yCx6t92smko8JdhOJhKrIL9oI71NxihqppMU6f8WsY6QJzCdIqmTdD0UpZy9j8iWgNGCWX1j9QnmzZD2DccKlLIqaw5jR+WTU8GVbd7iitThxZrxOv8nui872pGjzZj2BAaDRxh3M454N2CkQ3CsycjPMk9gtLmZKm1yWTB/KIA18t1/+L9a3YqxK7Ea1xXllp7hYa85AAFt2GAB4Op3DUGSrCGr220pkctZC7jDQoAj4BwmyDYbQ5p+j6VZsTZdtDaI39TiFOmEPrv9d/wtfD/hdzn9mGOV8c+MVp9Qh60lntXazNehWJ9CzCXhYsH83omj07sYMcYWJhlgdTYrQ94s3Q/7Ppi6LY/tvY6JIcl7am22M206e3Rf7uOYEdepelBS2aRa640TjuqlmaoJxCBOQxNK7YoQnTO2WuX0cQf6A/RF9XOAugMdkBW+dgPLsp1x0bx/medTWMkJU2FBTquHGt/+qIi9C64yqX46LSIC9ADisS6Z50adH7LH1AHFiK7XFhVgkNv4vfMEEiE7ty/16M3KyT0qiKMIjPQK1ydlI2VfXaXkUtBcOqcPkF18qa7DPCqknJanqH0LWt8s77n2u2ZgpZWj5FuY57cDZM X-OriginatorOrg: oppo.com X-MS-Exchange-CrossTenant-Network-Message-Id: d5863c15-af71-428b-1134-08dac3ccbfa6 X-MS-Exchange-CrossTenant-AuthSource: SI2PR02MB5148.apcprd02.prod.outlook.com X-MS-Exchange-CrossTenant-AuthAs: Internal X-MS-Exchange-CrossTenant-OriginalArrivalTime: 11 Nov 2022 10:08:58.4922 (UTC) X-MS-Exchange-CrossTenant-FromEntityHeader: Hosted X-MS-Exchange-CrossTenant-Id: f1905eb1-c353-41c5-9516-62b4a54b5ee6 X-MS-Exchange-CrossTenant-MailboxType: HOSTED X-MS-Exchange-CrossTenant-UserPrincipalName: zQHIdZ2XP/W/pBENT11fEA535s5f9F1oy5pAQPyzcAYIeKbPhivUmpDH4nxmXVccTXSp+VDpnusRWC+dvMF/aw== X-MS-Exchange-Transport-CrossTenantHeadersStamped: SG2PR02MB5901 X-Spam-Status: No, score=-2.1 required=5.0 tests=BAYES_00,DKIM_SIGNED, DKIM_VALID,DKIM_VALID_AU,DKIM_VALID_EF,RCVD_IN_DNSWL_NONE, RCVD_IN_MSPIKE_H2,SPF_HELO_PASS,SPF_PASS autolearn=ham autolearn_force=no version=3.4.6 X-Spam-Checker-Version: SpamAssassin 3.4.6 (2021-04-09) on lindbergh.monkeyblade.net Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org X-getmail-retrieved-from-mailbox: =?utf-8?q?INBOX?= X-GMAIL-THRID: =?utf-8?q?1749194500206558722?= X-GMAIL-MSGID: =?utf-8?q?1749194500206558722?= Since the file name has already passed to f2fs_new_inode(), let's move set_file_temperature() into f2fs_new_inode(). Signed-off-by: Sheng Yong Reviewed-by: Chao Yu Signed-off-by: Sheng Yong Reviewed-by: Chao Yu Signed-off-by: Jaegeuk Kim --- fs/f2fs/namei.c | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/fs/f2fs/namei.c b/fs/f2fs/namei.c index 36e251f438568..04bc7ec5e29e9 100644 --- a/fs/f2fs/namei.c +++ b/fs/f2fs/namei.c @@ -131,6 +131,9 @@ static struct inode *f2fs_new_inode(struct user_namespace *mnt_userns, if (test_opt(sbi, INLINE_DATA) && f2fs_may_inline_data(inode)) set_inode_flag(inode, FI_INLINE_DATA); + if (name && !test_opt(sbi, DISABLE_EXT_IDENTIFY)) + set_file_temperature(sbi, inode, name); + stat_inc_inline_xattr(inode); stat_inc_inline_inode(inode); stat_inc_inline_dir(inode); @@ -357,9 +360,6 @@ static int f2fs_create(struct user_namespace *mnt_userns, struct inode *dir, if (IS_ERR(inode)) return PTR_ERR(inode); - if (!test_opt(sbi, DISABLE_EXT_IDENTIFY)) - set_file_temperature(sbi, inode, dentry->d_name.name); - inode->i_op = &f2fs_file_inode_operations; inode->i_fop = &f2fs_file_operations; inode->i_mapping->a_ops = &f2fs_dblock_aops;