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=-7.0 required=3.0 tests=HEADER_FROM_DIFFERENT_DOMAINS, INCLUDES_PATCH,MAILING_LIST_MULTI,SIGNED_OFF_BY,SPF_PASS 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 CBE92C43441 for ; Sun, 11 Nov 2018 00:54:28 +0000 (UTC) Received: from vger.kernel.org (vger.kernel.org [209.132.180.67]) by mail.kernel.org (Postfix) with ESMTP id 8AB8420892 for ; Sun, 11 Nov 2018 00:54:28 +0000 (UTC) DMARC-Filter: OpenDMARC Filter v1.3.2 mail.kernel.org 8AB8420892 Authentication-Results: mail.kernel.org; dmarc=none (p=none dis=none) header.from=perches.com Authentication-Results: mail.kernel.org; spf=none smtp.mailfrom=linux-kernel-owner@vger.kernel.org Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1727281AbeKKKlT (ORCPT ); Sun, 11 Nov 2018 05:41:19 -0500 Received: from smtprelay0149.hostedemail.com ([216.40.44.149]:56714 "EHLO smtprelay.hostedemail.com" rhost-flags-OK-OK-OK-FAIL) by vger.kernel.org with ESMTP id S1727037AbeKKKlS (ORCPT ); Sun, 11 Nov 2018 05:41:18 -0500 Received: from filter.hostedemail.com (clb03-v110.bra.tucows.net [216.40.38.60]) by smtprelay08.hostedemail.com (Postfix) with ESMTP id 3D9DF182CED34; Sun, 11 Nov 2018 00:54:25 +0000 (UTC) X-Session-Marker: 6A6F6540706572636865732E636F6D X-HE-Tag: apple02_60aa6b4ffe923 X-Filterd-Recvd-Size: 7699 Received: from XPS-9350.home (unknown [47.151.153.53]) (Authenticated sender: joe@perches.com) by omf07.hostedemail.com (Postfix) with ESMTPA; Sun, 11 Nov 2018 00:54:24 +0000 (UTC) Message-ID: Subject: Re: [PATCH] Fix coding style issue in xfs_acl.c and xfs_aops.c From: Joe Perches To: hmsjwzb , darrick.wong@oracle.com Cc: linux-xfs@vger.kernel.org, linux-kernel@vger.kernel.org Date: Sat, 10 Nov 2018 16:54:22 -0800 In-Reply-To: <20181111003603.7598-1-weizhefix@gmail.com> References: <20181111003603.7598-1-weizhefix@gmail.com> Content-Type: text/plain; charset="ISO-8859-1" User-Agent: Evolution 3.30.1-1build1 Mime-Version: 1.0 Content-Transfer-Encoding: 7bit Sender: linux-kernel-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Sun, 2018-11-11 at 08:36 +0800, hmsjwzb wrote: > Possible unwrapped commit description (prefer a maximum 75 chars per line) This commit message makes no sense. Do say what you do to the code in the commit description. > Signed-off-by: hmsjwzb > --- > fs/xfs/xfs_acl.c | 4 +-- > fs/xfs/xfs_aops.c | 73 ++++++++++++++++++++++++----------------------- > 2 files changed, 39 insertions(+), 38 deletions(-) > > diff --git a/fs/xfs/xfs_acl.c b/fs/xfs/xfs_acl.c > index 8039e35147dd..5c779c161727 100644 > --- a/fs/xfs/xfs_acl.c > +++ b/fs/xfs/xfs_acl.c > @@ -19,8 +19,8 @@ > > /* > * Locking scheme: > - * - all ACL updates are protected by inode->i_mutex, which is taken before > - * calling into this file. > + * - all ACL updates are protected by inode->i_mutex, > + * which is taken before calling into this file. > */ > > STATIC struct posix_acl * > diff --git a/fs/xfs/xfs_aops.c b/fs/xfs/xfs_aops.c > index 338b9d9984e0..1a6cb88ffdb7 100644 > --- a/fs/xfs/xfs_aops.c > +++ b/fs/xfs/xfs_aops.c > @@ -81,8 +81,8 @@ xfs_finish_page_writeback( > > /* > * We're now finished for good with this ioend structure. Update the page > - * state, release holds on bios, and finally free up memory. Do not use the > - * ioend after this. > + * state, release holds on bios, and finally free up memory. > + * Do not use the ioend after this. > */ > STATIC void > xfs_destroy_ioend( > @@ -464,18 +464,18 @@ xfs_map_blocks( > } > > /* > - * Submit the bio for an ioend. We are passed an ioend with a bio attached to > - * it, and we submit that bio. The ioend may be used for multiple bio > - * submissions, so we only want to allocate an append transaction for the ioend > - * once. In the case of multiple bio submission, each bio will take an IO > - * reference to the ioend to ensure that the ioend completion is only done once > - * all bios have been submitted and the ioend is really done. > + * Submit the bio for an ioend. We are passed an ioend with a bio attached > + * to it, and we submit that bio. The ioend may be used for multiple bio > + * submissions, so we only want to allocate an append transaction for the > + * ioend once. In the case of multiple bio submission, each bio will take > + * an IO reference to the ioend to ensure that the ioend completion is only > + * done once all bios have been submitted and the ioend is really done. > * > - * If @fail is non-zero, it means that we have a situation where some part of > - * the submission process has failed after we have marked paged for writeback > - * and unlocked them. In this situation, we need to fail the bio and ioend > - * rather than submit it to IO. This typically only happens on a filesystem > - * shutdown. > + * If @fail is non-zero, it means that we have a situation where some part > + * of the submission process has failed after we have marked paged for > + * writeback and unlocked them. In this situation, we need to fail the bio > + * and ioend rather than submit it to IO. This typically only happens > + * on a filesystem shutdown. > */ > STATIC int > xfs_submit_ioend( > @@ -583,8 +583,8 @@ xfs_chain_bio( > } > > /* > - * Test to see if we have an existing ioend structure that we could append to > - * first, otherwise finish off the current ioend and start another. > + * Test to see if we have an existing ioend structure that we could append > + * to first, otherwise finish off the current ioend and start another. > */ > STATIC void > xfs_add_to_ioend( > @@ -637,15 +637,16 @@ xfs_vm_invalidatepage( > } > > /* > - * If the page has delalloc blocks on it, we need to punch them out before we > - * invalidate the page. If we don't, we leave a stale delalloc mapping on the > - * inode that can trip up a later direct I/O read operation on the same region. > + * If the page has delalloc blocks on it, we need to punch them out before > + * we invalidate the page. If we don't, we leave a stale delalloc mapping > + * on the inode that can trip up a later direct I/O read operation on > + * the same region. > * > - * We prevent this by truncating away the delalloc regions on the page. Because > - * they are delalloc, we can do this without needing a transaction. Indeed - if > - * we get ENOSPC errors, we have to be able to do this truncation without a > - * transaction as there is no space left for block reservation (typically why we > - * see a ENOSPC in writeback). > + * We prevent this by truncating away the delalloc regions on the page. > + * Because they are delalloc, we can do this without needing a transaction. > + * Indeed - if we get ENOSPC errors, we have to be able to do this > + * truncation without a transaction as there is no space left for block > + * reservation (typically why we see a ENOSPC in writeback). > */ > STATIC void > xfs_aops_discard_page( > @@ -674,20 +675,20 @@ xfs_aops_discard_page( > } > > /* > - * We implement an immediate ioend submission policy here to avoid needing to > - * chain multiple ioends and hence nest mempool allocations which can violate > - * forward progress guarantees we need to provide. The current ioend we are > - * adding blocks to is cached on the writepage context, and if the new block > - * does not append to the cached ioend it will create a new ioend and cache that > - * instead. > + * We implement an immediate ioend submission policy here to avoid needing > + * to chain multiple ioends and hence nest mempool allocations which can > + * violate forward progress guarantees we need to provide. The current > + * ioend we are adding blocks to is cached on the writepage context, > + * and if the new block does not append to the cached ioend it will create > + * a new ioend and cache that instead. > * > - * If a new ioend is created and cached, the old ioend is returned and queued > - * locally for submission once the entire page is processed or an error has been > - * detected. While ioends are submitted immediately after they are completed, > - * batching optimisations are provided by higher level block plugging. > - * > - * At the end of a writeback pass, there will be a cached ioend remaining on the > - * writepage context that the caller will need to submit. > + * If a new ioend is created and cached, the old ioend is returned and > + * queued locally for submission once the entire page is processed or an > + * error has been detected. While ioends are submitted immediately > + * after they are completed, batching optimisations are provided by higher > + * level block plugging. > + * At the end of a writeback pass, there will be a cached ioend remaining > + * on the writepage context that the caller will need to submit. > */ > static int > xfs_writepage_map(