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 X-Spam-Level: X-Spam-Status: No, score=-18.8 required=3.0 tests=BAYES_00,DKIM_SIGNED, DKIM_VALID,DKIM_VALID_AU,HEADER_FROM_DIFFERENT_DOMAINS,INCLUDES_CR_TRAILER, INCLUDES_PATCH,MAILING_LIST_MULTI,MSGID_FROM_MTA_HEADER,SPF_HELO_NONE, SPF_PASS,USER_AGENT_GIT autolearn=ham autolearn_force=no version=3.4.0 Received: from mail.kernel.org (mail.kernel.org [198.145.29.99]) by smtp.lore.kernel.org (Postfix) with ESMTP id D0D75C433B4 for ; Mon, 26 Apr 2021 22:07:36 +0000 (UTC) Received: from vger.kernel.org (vger.kernel.org [23.128.96.18]) by mail.kernel.org (Postfix) with ESMTP id A84DF60FF1 for ; Mon, 26 Apr 2021 22:07:36 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S235509AbhDZWIR (ORCPT ); Mon, 26 Apr 2021 18:08:17 -0400 Received: from aserp2130.oracle.com ([141.146.126.79]:49410 "EHLO aserp2130.oracle.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S235538AbhDZWIN (ORCPT ); Mon, 26 Apr 2021 18:08:13 -0400 Received: from pps.filterd (aserp2130.oracle.com [127.0.0.1]) by aserp2130.oracle.com (8.16.0.42/8.16.0.42) with SMTP id 13QM0VgC115653; Mon, 26 Apr 2021 22:07:29 GMT DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=oracle.com; h=from : to : cc : subject : date : message-id : in-reply-to : references : content-transfer-encoding : content-type : mime-version; s=corp-2020-01-29; bh=laIkdfWVNsl4N4zcL8amigZZXhtf0a9Umv8mzGbfcQY=; b=Uw64mhEFu7m+ORUTBDU+Qsw+ChjbnucSYxRlFB/hwN2EERsKwa5RYzd6UtBgQ2Tq1xfW QjOxc3/IyFGTNtNRpYXGH1nEFQua0HYj5oXY553JpUWKdSfcIKnQ5wkgT1xAt3igsKoV GraF5s3OcIv8MZfBrfrP868tjitbP5oJcdbc59B3Zm7DGbOmE1ma9ZXd8LZqE1xBj9u9 NS8UJQVmW6ZmFPS2bh0Rt6K3I07XYZ/G28SlU/Qv3hitUc6a8VbbBRw5rM/FejofvamK 3OR0dYTZGue8J1dRVbNTInDcBJlu+I6ijP/vTCuc0tu7IVsRxvXCDqTtwqquV+RnL3DI dQ== Received: from userp3020.oracle.com (userp3020.oracle.com [156.151.31.79]) by aserp2130.oracle.com with ESMTP id 385afpuksg-1 (version=TLSv1.2 cipher=ECDHE-RSA-AES256-GCM-SHA384 bits=256 verify=OK); Mon, 26 Apr 2021 22:07:29 +0000 Received: from pps.filterd (userp3020.oracle.com [127.0.0.1]) by userp3020.oracle.com (8.16.0.42/8.16.0.42) with SMTP id 13QM6Klg105544; Mon, 26 Apr 2021 22:07:28 GMT Received: from nam04-bn3-obe.outbound.protection.outlook.com (mail-bn3nam04lp2054.outbound.protection.outlook.com [104.47.46.54]) by userp3020.oracle.com with ESMTP id 384w3s65pu-4 (version=TLSv1.2 cipher=ECDHE-RSA-AES256-GCM-SHA384 bits=256 verify=OK); Mon, 26 Apr 2021 22:07:28 +0000 ARC-Seal: i=1; a=rsa-sha256; s=arcselector9901; d=microsoft.com; cv=none; b=DaX++mq0H+xsROZL7Suo0MLmII26XFKqOr8NwmJgAxEtvFgMlc22sg9PR7WAeR7Bn6RhyUgFa534dvlw127O7eQ8ZVPmTnQD9GP9zokuxpJvi+x84aAHSTDiGwqJAYHYuk2+SaTOSIHwmX/QXCSqLInjG3OZNnGqTZfngN6BYgADBrXCoC7BAZU1ttPbVDl5HIwXL6VlW5uwIaHntEJr0B1/o8TCq7DinvicdAIrAGWXWgKm5PV5uIulWPKvOs/tsTxChb3lQi+6hQ8fzhyZR8OVDZ77StJv5PpBYbS0EuVfypTObZqjKWNw5U/vk5EDIucL4sSBFKK6ZewbkwkYaw== 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-SenderADCheck; bh=laIkdfWVNsl4N4zcL8amigZZXhtf0a9Umv8mzGbfcQY=; b=NW4wQvIK414gU4fvbBWbFyO6pq8Jj8IEuDHFS3fDjEuRlkMN4msJaansGD+dPg1plfJ0qN6C9x/05izczfcrLl6G6KbUXanVIh5jmCO+VuWm3txW022Maa6MKh73s4DnupIQzMpY4+4grLI3aipT+vJ3XiLtfDf3qL7mvpY//Vu2zx2oNQ4AFwCz1z2L5JFAzMho2moYXYPpJI8vnqNmDqc5wNRIpIik0rdcW5y9xsNSO7LXnMARCwnDxXsHFj9hlUsIWLnRDT1hBSM7fcCUxgXFtFWyxcBKWlzde4xHkFzYzFjjESShoONbpepX+2l0GDdqJqek2IQqpUxLGRZDyQ== ARC-Authentication-Results: i=1; mx.microsoft.com 1; spf=pass smtp.mailfrom=oracle.com; dmarc=pass action=none header.from=oracle.com; dkim=pass header.d=oracle.com; arc=none DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=oracle.onmicrosoft.com; s=selector2-oracle-onmicrosoft-com; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-SenderADCheck; bh=laIkdfWVNsl4N4zcL8amigZZXhtf0a9Umv8mzGbfcQY=; b=UXrhHhL/H6fhHOQbuv5wUXdbfw8jnBMMiqmLb2VW3sQFD7CKBLFggXE44wjlwOS70/H4094PRD3CnKJQ8YkT/0OncMIi7b6EBZMC6Hk0KUVBAZQUsSSxZMuwHhboEhsBoeIEoWFhAEZdm+qvEGhvckUkUOmJYydFWAaYAwXvqaE= Authentication-Results: oss.oracle.com; dkim=none (message not signed) header.d=none;oss.oracle.com; dmarc=none action=none header.from=oracle.com; Received: from SJ0PR10MB4752.namprd10.prod.outlook.com (2603:10b6:a03:2d7::19) by BY5PR10MB4307.namprd10.prod.outlook.com (2603:10b6:a03:212::18) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.4042.19; Mon, 26 Apr 2021 22:07:25 +0000 Received: from SJ0PR10MB4752.namprd10.prod.outlook.com ([fe80::7865:7d35:9cee:363f]) by SJ0PR10MB4752.namprd10.prod.outlook.com ([fe80::7865:7d35:9cee:363f%5]) with mapi id 15.20.4065.027; Mon, 26 Apr 2021 22:07:25 +0000 From: Junxiao Bi To: ocfs2-devel@oss.oracle.com, cluster-devel@redhat.com, linux-fsdevel@vger.kernel.org Cc: junxiao.bi@oracle.com Subject: [PATCH 2/3] ocfs2: allow writing back pages out of inode size Date: Mon, 26 Apr 2021 15:05:51 -0700 Message-Id: <20210426220552.45413-2-junxiao.bi@oracle.com> X-Mailer: git-send-email 2.24.3 (Apple Git-128) In-Reply-To: <20210426220552.45413-1-junxiao.bi@oracle.com> References: <20210426220552.45413-1-junxiao.bi@oracle.com> Content-Transfer-Encoding: 8bit Content-Type: text/plain X-Originating-IP: [73.231.9.254] X-ClientProxiedBy: SN6PR05CA0026.namprd05.prod.outlook.com (2603:10b6:805:de::39) To SJ0PR10MB4752.namprd10.prod.outlook.com (2603:10b6:a03:2d7::19) MIME-Version: 1.0 X-MS-Exchange-MessageSentRepresentingType: 1 Received: from dhcp-10-159-151-113.vpn.oracle.com (73.231.9.254) by SN6PR05CA0026.namprd05.prod.outlook.com (2603:10b6:805:de::39) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.4087.16 via Frontend Transport; Mon, 26 Apr 2021 22:07:24 +0000 X-MS-PublicTrafficType: Email X-MS-Office365-Filtering-Correlation-Id: 2a8e2d37-4503-4b06-f203-08d908ffac6e X-MS-TrafficTypeDiagnostic: BY5PR10MB4307: X-MS-Exchange-Transport-Forked: True X-Microsoft-Antispam-PRVS: X-MS-Oob-TLC-OOBClassifiers: OLM:8273; X-MS-Exchange-SenderADCheck: 1 X-Microsoft-Antispam: BCL:0; X-Microsoft-Antispam-Message-Info: zHVKO/nMN4+N7PsK3S0CfHI9lMCaKNw+qUiO0/JaVU5UZTXPnKCdvQ4RI0oigmwJsxE5uRJiUyB0ki7vFK3rbNqdaHxh+/uUwX/epj9Xi4sS6rY6LqYRg1HaLcqpK6h2Ir4wwtV02yMfo7YOCu2c7Ic8JxtEtwtps0RBMIsSnEYXH93vw/zri+vlYec8SSyHT5IjGJwOKcwcF5rCe6XCE9uTjx13Mmn+R2Zt0q/S19/Zgl8XpTps3Gg7luvn1ctgE5kqhzowQDm8zzCEyEdsHISxl4FA47EdoZDhLn8R0GNEC8w9kFlzyMoAXKfnTFLTEVz8h71g69SmHgoN7adJmCM4NwQ2ewr3ScLpfWLEImwHpdzGHSLBgsaCTQ8uAzFDyu1LAJKFuAEBwKLNEqpLW4Fo4PQczRJyIq37NDORikn8iqVPNPYgTI1EWijTAaPckrFTU27RPpAJYFFjEel9JMs9Azg7Q3VqteQvMRvL9dxgt/TkknKqwZvF9vZUkSPRyXe7b6D2RkCwHsLMaUHr5yiUK/I/T0M9TXjkMpU8si8EQv/pOLcdyQqh9NNEsv6uM6jCttp0eJKFhoLMdOu/LeOLanHRlKhH5Kse8tpm8971G6bM19jFFSyr0ZSN0UzLhs3wDbgFpOhWzeslG5FLzjZLDKspDnJyA3HIkFt/DL4= X-Forefront-Antispam-Report: CIP:255.255.255.255;CTRY:;LANG:en;SCL:1;SRV:;IPV:NLI;SFV:NSPM;H:SJ0PR10MB4752.namprd10.prod.outlook.com;PTR:;CAT:NONE;SFS:(396003)(376002)(346002)(39860400002)(366004)(136003)(6486002)(6666004)(44832011)(5660300002)(478600001)(2906002)(66556008)(52116002)(4326008)(36756003)(83380400001)(38350700002)(86362001)(107886003)(26005)(956004)(2616005)(316002)(8676002)(16526019)(1076003)(38100700002)(8936002)(7696005)(66476007)(66946007)(186003);DIR:OUT;SFP:1101; X-MS-Exchange-AntiSpam-MessageData: =?us-ascii?Q?cFNHjP5FUdIFbj93xG43IRskHQROlJVnJgqZgABWdzpeVVLyKudfwwNrkjXX?= =?us-ascii?Q?vMnbDxNE+WilVTtJKSnnlYTBj3PyM2yUhZc0lxrr2qjo6kWMeMF/6GFWzxiK?= =?us-ascii?Q?IoJC9VA2GU3SZwgcQp3/9x30F57eNI1Y8pSW8HmdrAeLQV7TUrs9FDwDrWEm?= =?us-ascii?Q?oT4Qu2n/Wgo9t9buM5HKAA9jnqzaAWlRU3PJ+i0ATORiJKZ8K90ShiAAnDml?= =?us-ascii?Q?ttBqjz1yjS2DgdtZegwWbIXg8CAK+gRntczX6QFujD9avHo+2hlNi6WmH67s?= =?us-ascii?Q?s7GrLJ2ehqqx3JiRzrfTO8rhhwFXH+I5thKrBxFCmrYFd30o9eHsrPvyHcZV?= =?us-ascii?Q?KxrP494ol5MpLAsS0x45aImU2hWGqqnPlIYHNplcQ/CfEHtNB8dwkupcRgbL?= =?us-ascii?Q?LazIPVuIwdYiQ0aILgeWi083oERej/5H7WLis5P189Swdt1tkx/AH2odZ6S3?= =?us-ascii?Q?xK89xkC6aDHFQFWs+p+vZMvDKYoZnk2G8uDZJ5yM9VWDNIwmVPd+sszva3Fq?= =?us-ascii?Q?uZns0SMeA3BPMP48abMSj4/2VjRqeD4QYpI39PjCPOx4N8dvK3oCuXF0l/2y?= =?us-ascii?Q?dB4lPy9/i995eHdBLLsRsbLTKyL//oF8n3au/p1qBUZTgU7Z9yweYwW8ZF7B?= =?us-ascii?Q?5+TXUYiY3JIax/Uhm4tddcWNRn60jVA4ur+AYrza2QrwxyC/bM4K+r9bbGL2?= =?us-ascii?Q?Kl8JowjKAlXPhDYA/h+j2cd5rdGpvK2ChM2g05PXdqSF7LtdRcSFB+q6uIJr?= =?us-ascii?Q?GAjtH276FNkCGlKi7PHRTVIQKkTlaRhA8zKFXBwIRoHyK0mg9NOcAsOpHKPC?= =?us-ascii?Q?xMDj4kznibm36BzgV9ajGP4mA/PG74ZC6ltDKHJemeItmdl6Y1cxLwLt6oz3?= =?us-ascii?Q?WRiksc39IydX1wdEL1xZ4XlaP9Qe/ZtebH9v1AMqMD5Y/DawyBlQbGl/sKnA?= =?us-ascii?Q?PFmpZR1uo2Hl6SELXKDHWjCrLwOPSo+HP8g19m7gy9gZYpSNaXk52mdwrD1D?= =?us-ascii?Q?pg9Rv4KyABLkKEju6dzco0pjl+3q0onCnFulkL58MOujxV6rlVWGEpVfnRCx?= =?us-ascii?Q?Dg3seHewzCW5JYlLM5G4c1I1zonI41IIjzq9Io2qBX41CPeCAQ1edwWnhvk0?= =?us-ascii?Q?DiiuPq1nIAl726TVsOqCgDHEnfL3oiwZy+OHjf0fNGDeVC1qV/wjewaPeLZR?= =?us-ascii?Q?rva4GcOWfghHig66w9bgTP4DsSDnYlOsKFQa46C3H6T3AvmlDnJ/f0ZIlFtT?= =?us-ascii?Q?w1eToh+bKIXv/BM9JQIAOt17W+xpZhO40lpu3uc23cezr7La2XqO2sKswqWn?= =?us-ascii?Q?YOwachgzb2B/fllPXKBbcbol?= X-OriginatorOrg: oracle.com X-MS-Exchange-CrossTenant-Network-Message-Id: 2a8e2d37-4503-4b06-f203-08d908ffac6e X-MS-Exchange-CrossTenant-AuthSource: SJ0PR10MB4752.namprd10.prod.outlook.com X-MS-Exchange-CrossTenant-AuthAs: Internal X-MS-Exchange-CrossTenant-OriginalArrivalTime: 26 Apr 2021 22:07:25.3863 (UTC) X-MS-Exchange-CrossTenant-FromEntityHeader: Hosted X-MS-Exchange-CrossTenant-Id: 4e2c6054-71cb-48f1-bd6c-3a9705aca71b X-MS-Exchange-CrossTenant-MailboxType: HOSTED X-MS-Exchange-CrossTenant-UserPrincipalName: rv8xsTrN/O+NCzFaHs19l22ZD6yrK3wBAt7Blb2gKtFX1jW2QeHUoTvhSFEp+rVtP9Bomygel5Js5XujDAd3rQ== X-MS-Exchange-Transport-CrossTenantHeadersStamped: BY5PR10MB4307 X-Proofpoint-Virus-Version: vendor=nai engine=6200 definitions=9966 signatures=668683 X-Proofpoint-Spam-Details: rule=notspam policy=default score=0 adultscore=0 malwarescore=0 mlxscore=0 mlxlogscore=999 bulkscore=0 phishscore=0 spamscore=0 suspectscore=0 classifier=spam adjust=0 reason=mlx scancount=1 engine=8.12.0-2104060000 definitions=main-2104260167 X-Proofpoint-ORIG-GUID: BnD2L5VtgzJF3N2jLH86hRjYErerrUaw X-Proofpoint-GUID: BnD2L5VtgzJF3N2jLH86hRjYErerrUaw X-Proofpoint-Virus-Version: vendor=nai engine=6200 definitions=9966 signatures=668683 X-Proofpoint-Spam-Details: rule=notspam policy=default score=0 bulkscore=0 spamscore=0 phishscore=0 clxscore=1015 suspectscore=0 lowpriorityscore=0 mlxlogscore=999 mlxscore=0 adultscore=0 malwarescore=0 impostorscore=0 priorityscore=1501 classifier=spam adjust=0 reason=mlx scancount=1 engine=8.12.0-2104060000 definitions=main-2104260166 Precedence: bulk List-ID: X-Mailing-List: linux-fsdevel@vger.kernel.org When fallocate/truncate extend inode size, if the original isize is in the middle of last cluster, then the part from isize to the end of the cluster needs to be zeroed with buffer write, at that time isize is not yet updated to match the new size, if writeback is kicked in, it will invoke ocfs2_writepage()->block_write_full_page() where the pages out of inode size will be dropped. That will cause file corruption. Running the following command with qemu-image 4.2.1 can get a corrupted coverted image file easily. qemu-img convert -p -t none -T none -f qcow2 $qcow_image \ -O qcow2 -o compat=1.1 $qcow_image.conv Cc: Signed-off-by: Junxiao Bi --- fs/ocfs2/aops.c | 19 ++++++++++++++++++- 1 file changed, 18 insertions(+), 1 deletion(-) diff --git a/fs/ocfs2/aops.c b/fs/ocfs2/aops.c index ad20403b383f..7a3e3d59f6a9 100644 --- a/fs/ocfs2/aops.c +++ b/fs/ocfs2/aops.c @@ -402,11 +402,28 @@ static void ocfs2_readahead(struct readahead_control *rac) */ static int ocfs2_writepage(struct page *page, struct writeback_control *wbc) { + struct inode * const inode = page->mapping->host; + loff_t i_size = i_size_read(inode); + const pgoff_t end_index = i_size >> PAGE_SHIFT; + unsigned int offset; + trace_ocfs2_writepage( (unsigned long long)OCFS2_I(page->mapping->host)->ip_blkno, page->index); - return block_write_full_page(page, ocfs2_get_block, wbc); + /* + * The page straddles i_size. It must be zeroed out on each and every + * writepage invocation because it may be mmapped. "A file is mapped + * in multiples of the page size. For a file that is not a multiple of + * the page size, the remaining memory is zeroed when mapped, and + * writes to that region are not written out to the file." + */ + offset = i_size & (PAGE_SIZE-1); + if (page->index == end_index && offset) + zero_user_segment(page, offset, PAGE_SIZE); + + return __block_write_full_page_eof(inode, page, ocfs2_get_block, wbc, + end_buffer_async_write, true); } /* Taken from ext3. We don't necessarily need the full blown -- 2.24.3 (Apple Git-128)