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=-3.7 required=3.0 tests=BAYES_00, HEADER_FROM_DIFFERENT_DOMAINS,MAILING_LIST_MULTI,SPF_HELO_NONE,SPF_PASS, UNPARSEABLE_RELAY autolearn=no 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 F0A62C4320A for ; Mon, 2 Aug 2021 12:15:47 +0000 (UTC) Received: from vger.kernel.org (vger.kernel.org [23.128.96.18]) by mail.kernel.org (Postfix) with ESMTP id D77F360EFD for ; Mon, 2 Aug 2021 12:15:47 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S233577AbhHBMPz (ORCPT ); Mon, 2 Aug 2021 08:15:55 -0400 Received: from out30-57.freemail.mail.aliyun.com ([115.124.30.57]:55492 "EHLO out30-57.freemail.mail.aliyun.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S233498AbhHBMPy (ORCPT ); Mon, 2 Aug 2021 08:15:54 -0400 X-Alimail-AntiSpam: AC=PASS;BC=-1|-1;BR=01201311R571e4;CH=green;DM=||false|;DS=||;FP=0|-1|-1|-1|0|-1|-1|-1;HT=e01e04400;MF=hsiangkao@linux.alibaba.com;NM=1;PH=DS;RN=9;SR=0;TI=SMTPD_---0UhmnTvb_1627906541; Received: from B-P7TQMD6M-0146.local(mailfrom:hsiangkao@linux.alibaba.com fp:SMTPD_---0UhmnTvb_1627906541) by smtp.aliyun-inc.com(127.0.0.1); Mon, 02 Aug 2021 20:15:42 +0800 Date: Mon, 2 Aug 2021 20:15:41 +0800 From: Gao Xiang To: Andreas Gruenbacher , "Darrick J . Wong" Cc: linux-erofs@lists.ozlabs.org, linux-fsdevel , LKML , Huang Jianan , Joseph Qi , Christoph Hellwig , Matthew Wilcox Subject: Re: [PATCH v9] iomap: Support file tail packing Message-ID: Mail-Followup-To: Andreas Gruenbacher , "Darrick J . Wong" , linux-erofs@lists.ozlabs.org, linux-fsdevel , LKML , Huang Jianan , Joseph Qi , Christoph Hellwig , Matthew Wilcox References: <20210727025956.80684-1-hsiangkao@linux.alibaba.com> MIME-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Disposition: inline In-Reply-To: Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Hi Andreas, On Sun, Aug 01, 2021 at 02:03:33PM +0200, Andreas Gruenbacher wrote: > On Tue, Jul 27, 2021 at 5:00 AM Gao Xiang wrote: ... > > +static int iomap_write_begin_inline(struct inode *inode, > > + struct page *page, struct iomap *srcmap) > > +{ > > + /* needs more work for the tailpacking case, disable for now */ > > Nit: the comma should be a semicolon or period here. Sorry for some delay (busy in other things...) Yeah, that's fine to me, in English contexts it'd be better like that (there is some punctuation rule difference between languages.) Hi Darrick, Should I resend v10 for this punctuation change or could you kindly help revise this? (btw, would you mind set up a for-next iomap branch so I could rebase other EROFS patches on iomap for-next, thank you very much!) Thanks, Gao Xiang