Received-SPF: pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) client-ip=209.132.180.67;
Date:   Thu, 1 Aug 2019 08:55:42 -0700
From:   Matthew Wilcox <willy@infradead.org>
To:     Joel Savitz <jsavitz@redhat.com>
Cc:     linux-kernel@vger.kernel.org,
        Andrew Morton <akpm@linux-foundation.org>,
        Mel Gorman <mgorman@techsingularity.net>,
        Vlastimil Babka <vbabka@suse.cz>,
        Randy Dunlap <rdunlap@infradead.org>,
        Yu Zhao <yuzhao@google.com>, trivial@kernel.org
Subject: Re: [PATCH] mm: fix typo in comment
Message-ID: <20190801155542.GJ4700@bombadil.infradead.org>
References: <1564674454-22469-1-git-send-email-jsavitz@redhat.com>
MIME-Version: 1.0
Content-Type: text/plain; charset=us-ascii
Content-Disposition: inline
In-Reply-To: <1564674454-22469-1-git-send-email-jsavitz@redhat.com>
User-Agent: Mutt/1.11.4 (2019-03-13)
Sender: linux-kernel-owner@vger.kernel.org
Precedence: bulk

On Thu, Aug 01, 2019 at 11:47:34AM -0400, Joel Savitz wrote:
> Fix spelling of successful (currently spelled successfull in kernel
> source)

If you're going to bother fixing the spelling, may as well fix the grammar
at the same time.

>  	/*
> -	 * The full zone was compacted scanned but wasn't successfull to compact
> +	 * The full zone was compacted scanned but wasn't successful to compact
>  	 * suitable pages.
>  	 */

... actually, I don't know what is meant here.  Could it mean:

	/*
	 * The full zone was scanned for compaction, but didn't produce
	 * suitable pages.
	 */

>  	/*
> -	 * direct compaction has scanned part of the zone but wasn't successfull
> +	 * direct compaction has scanned part of the zone but wasn't successful
>  	 * to compact suitable pages.

	 * Zone compaction had to stop before scanning the full zone, and
	 * no suitable pages were produced.