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;
From:   Qi Han <hanqi@vivo.com>
To:     jaegeuk@kernel.org, chao@kernel.org
Cc:     linux-f2fs-devel@lists.sourceforge.net,
        linux-kernel@vger.kernel.org, Qi Han <hanqi@vivo.com>
Subject: [PATCH] f2fs: allows setting compress option to files which marked with FI_MMAP_FILE
Date:   Wed, 24 May 2023 19:41:59 +0800
Message-Id: <20230524114159.8999-1-hanqi@vivo.com>
Content-Transfer-Encoding: 8bit
Content-Type: text/plain
MIME-Version: 1.0
X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1
X-MS-Exchange-AntiSpam-MessageData-0: =?us-ascii?Q?lxyU28GXWTB+aLgU+8feDpUFTcmLXzFxsof6VuVGCnzjR7Z9XaJMcQEZ9m9G?=
 =?us-ascii?Q?vr7rlg95CvmWP46MkuNhBW+d0aOpePJHaPSi0GZb0bkFa9O7kwv2aTA8pHTH?=
 =?us-ascii?Q?DJ7NkQeu0RgBdVXciermo5nOCTkKlMR1sRq6+/8Q8VGXknmHwLSv7uQ/e8C4?=
 =?us-ascii?Q?7ySi2CXb5SXsABdwNdlXYyV5f9Tg+fel8oDEQIFsGO80WkxsmJthh2s6RE3a?=
 =?us-ascii?Q?tJvMVCakLfZs8VXJmEiN+QAR7xcoh4BaZY/4oU6BshIeYuhEGqJ+ICOrWmEG?=
 =?us-ascii?Q?xksum/qo2lM83t23jNVM5dJcdO+QPvI+ufJPOWQKYkz6yKFCZaEL6mDgnAPl?=
 =?us-ascii?Q?W8vBqryF0qD7utxtLsuGwWPCxrTfpH+qEbiLfhqjNG4+km7eNmoLc/7awSNd?=
 =?us-ascii?Q?W11jXmY60ShhYIDqc6xg4dO4Ho+gTDLTBGogM3zlfT3qjWoYfbL4sAkJH6bZ?=
 =?us-ascii?Q?XBB7RZ8a7RdO4YDHvRWJUMOmXpOHz4jofaDfXeEW8WYHLCCYKMLOpbBWuWxm?=
 =?us-ascii?Q?8a/QyB0/26rpGWJ6nZubSYUDtBcjz6JMFxH/Vgzy1xmuVnW1aYxfc2coTxDl?=
 =?us-ascii?Q?fb/UnXkv0ST+VspfqV9PGTlLxbdSGwQaNgVR/sX2ll5FImUsTcLh0WPzrChw?=
 =?us-ascii?Q?5kP+PumOf0hGgWkDuGDn7XTj6YBBdAVzjRgXcu5SgmIGjxm/VREpUIWEbKwZ?=
 =?us-ascii?Q?OXmMibTM9RYf60fB3vGiCMghnbI26wf5LzVuA5T3VvWaN060WfyXBAL1F3+x?=
 =?us-ascii?Q?wwvreRX9hGqmq85wkvNwJMeHgu2GH8Z0+ye8uQP8kJaN1jufUpfxQbjyW4Ar?=
 =?us-ascii?Q?1xtM5PkxHJmJedFm8av5Xtr84nXU1Z25nWGb/wWnxYE/zN27/DffTXhOKLVB?=
 =?us-ascii?Q?dm9tF5UrheyUhr62+cjk28NY7rZ1ldoFx3RzCoUs3chky0EvVmV6+mPungHf?=
 =?us-ascii?Q?NWUFWU1Tt2I+vnJB1vd4yba5mx354bAxWW7DXXmxC3kst8xyccb2XIKWCR2Z?=
 =?us-ascii?Q?GF6brutMGmE/NVIG/du+c0buHVSDp8n3ybVZDSdhmXbkOuLXg0jomAoSO5jB?=
 =?us-ascii?Q?lMlro8pTlzCNFiZZmg9Im0WFw+zyxDcGXHEoks8wj/0FeqSNmek3chhkHZGm?=
 =?us-ascii?Q?HRbbwAXARE936QHJIyz69UWiK4YaBsWapjSmOJGnhXZt1JJh6lM3thtprjq3?=
 =?us-ascii?Q?Mm39KJyBWg0/kHOA5WglUYfeq73ZpubbyBPigmMITJ5cy3bGn+/VA/HEX8gq?=
 =?us-ascii?Q?IJCSM0jTGww9wz4AWz7E9I8XVWPTYycWwzWmqnbJXoUJu44k3aKY5n+XFFyc?=
 =?us-ascii?Q?MW9aR9Ai/WE+I/rcKYZRgMZqPYqSM+vKn4YvsVs32YckKy/PjD6wb8y7EUbM?=
 =?us-ascii?Q?D7DLF9HD9irCaj3zk9HcbrsRoUR9P1LySD/R7DKiM9VEsKO8KEAEYkhJX9dZ?=
 =?us-ascii?Q?BU2p2rUCUZI5i2xMdn6uxLO1RmB8QiHH/e5SZTaZQvdBwdPEa6GmEMGuubNZ?=
 =?us-ascii?Q?zjlSk4tGfE/4rMltk0pEDutDCrFKjUsWoJ0uoa1JP18AhlUGBqtyGcDWYpd1?=
 =?us-ascii?Q?jtR2IaMzn+zxLuzifpLiiAH89Ye36EayN4ZosQc5?=
X-OriginatorOrg: vivo.com
X-MS-Exchange-CrossTenant-Network-Message-Id: 51de8b13-a18f-48bc-ca18-08db5c4beb0d
X-MS-Exchange-CrossTenant-AuthSource: SL2PR06MB3227.apcprd06.prod.outlook.com
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-OriginalArrivalTime: 24 May 2023 11:42:14.3129
 (UTC)
X-MS-Exchange-CrossTenant-FromEntityHeader: Hosted
X-MS-Exchange-CrossTenant-Id: 923e42dc-48d5-4cbe-b582-1a797a6412ed
X-MS-Exchange-CrossTenant-MailboxType: HOSTED
X-MS-Exchange-CrossTenant-UserPrincipalName: Il22ncSA2qiEekmF08qPuuKkMZtgDUoAKbzzPQVx3T4fJsTmqtFwK3C6LN+Ia72mvb6bgKTJz53XJ/g+xuKJ3A==
X-MS-Exchange-Transport-CrossTenantHeadersStamped: TYUPR06MB6079
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,T_SCC_BODY_TEXT_LINE
        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: <linux-kernel.vger.kernel.org>
X-Mailing-List: linux-kernel@vger.kernel.org

Allow to compress mmap files in commit e3c548323d32 ("f2fs: let's
allow compression for mmap files"). However, we cannot set the
compress option to the mmap file. To keep the same concept in both
compress_modes, f2fs_ioc_set_compress_option should also allow it.

Let's remove checking mmap files in f2fs_ioc_set_compress_option so
that we can set compress option to the mmap files.

Signed-off-by: Qi Han <hanqi@vivo.com>
---
 fs/f2fs/file.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/fs/f2fs/file.c b/fs/f2fs/file.c
index 5ac53d2627d2..3a97b753a074 100644
--- a/fs/f2fs/file.c
+++ b/fs/f2fs/file.c
@@ -3957,7 +3957,7 @@ static int f2fs_ioc_set_compress_option(struct file *filp, unsigned long arg)
 	file_start_write(filp);
 	inode_lock(inode);
 
-	if (f2fs_is_mmap_file(inode) || get_dirty_pages(inode)) {
+	if (get_dirty_pages(inode)) {
 		ret = -EBUSY;
 		goto out;
 	}
-- 
2.39.0