Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1752416AbcL2Heb (ORCPT ); Thu, 29 Dec 2016 02:34:31 -0500 Received: from mail-pg0-f68.google.com ([74.125.83.68]:35507 "EHLO mail-pg0-f68.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751689AbcL2He3 (ORCPT ); Thu, 29 Dec 2016 02:34:29 -0500 Date: Thu, 29 Dec 2016 16:34:03 +0900 From: Sergey Senozhatsky To: Minchan Kim Cc: Sergey Senozhatsky , Xishi Qiu , ngupta@vflare.org, Linux MM , LKML , Andrew Morton Subject: Re: mm: fix typo of cache_alloc_zspage() Message-ID: <20161229073403.GB3892@jagdpanzerIV.localdomain> References: <58646FB7.2040502@huawei.com> <20161229064457.GD1815@bbox> <20161229065205.GA3892@jagdpanzerIV.localdomain> <20161229065935.GE1815@bbox> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20161229065935.GE1815@bbox> User-Agent: Mutt/1.7.2 (2016-11-26) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 2463 Lines: 78 Hello, On (12/29/16 15:59), Minchan Kim wrote: [..] > > I don't know... do we want to have it as a separate patch? > > may be we can fold it into some other patch someday later. > > Xishi spent his time to make the patch(review,create/send). And I want to > give a credit to him. :) sure, I didn't mean "let's seize the credit" :) my reasoning was that that patch hardly can be counted even as trivial. per documentation: : Trivial patches must qualify for one of the following rules: : : - Spelling fixes in documentation : - Spelling fixes for errors which could break :manpage:`grep(1)` : - Warning fixes (cluttering with useless warnings is bad) : - Compilation fixes (only if they are actually correct) : - Runtime fixes (only if they actually fix things) : - Removing use of deprecated functions/macros : - Contact detail and documentation fixes : - Non-portable code replaced by portable code (even in arch-specific, : since people copy, as long as it's trivial) : - Any fix by the author/maintainer of the file (ie. patch monkey : in re-transmission mode) hence was my question. we can have it as "p.s. in this patch we also remove XYZ reported by Xishi Qiu". but up to you. for instance, we can have Xishi's fix up as part of this "fix documentation typos" patch. which can be counted in as trivial. --- diff --git a/mm/zsmalloc.c b/mm/zsmalloc.c index 9cc3c0b2c2c1..af7cd90c26f7 100644 --- a/mm/zsmalloc.c +++ b/mm/zsmalloc.c @@ -25,7 +25,7 @@ * Usage of struct page flags: * PG_private: identifies the first component page * PG_private2: identifies the last component page - * PG_owner_priv_1: indentifies the huge component page + * PG_owner_priv_1: identifies the huge component page * */ @@ -65,7 +65,7 @@ #define ZS_ALIGN 8 /* - * A single 'zspage' is composed of up to 2^N discontiguous 0-order (single) + * A single 'zspage' is composed of up to 2^N discontinuous 0-order (single) * pages. ZS_MAX_ZSPAGE_ORDER defines upper limit on N. */ #define ZS_MAX_ZSPAGE_ORDER 2 @@ -2383,7 +2383,7 @@ struct zs_pool *zs_create_pool(const char *name) goto err; /* - * Iterate reversly, because, size of size_class that we want to use + * Iterate reversely, because, size of size_class that we want to use * for merging should be larger or equal to current size. */ for (i = zs_size_classes - 1; i >= 0; i--) { --- -ss