From mboxrd@z Thu Jan 1 00:00:00 1970 From: David Laight Date: Wed, 05 Feb 2020 14:54:07 +0000 Subject: RE: [PATCH v6 08/10] mm/memory_hotplug: Don't check for "all holes" in shrink_zone_span() Message-Id: List-Id: References: <20191006085646.5768-1-david@redhat.com> <20191006085646.5768-9-david@redhat.com> <20200205095924.GC24162@richard> In-Reply-To: <20200205095924.GC24162@richard> MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: 'Wei Yang' , David Hildenbrand Cc: "linux-s390@vger.kernel.org" , Michal Hocko , "linux-ia64@vger.kernel.org" , Pavel Tatashin , "linux-sh@vger.kernel.org" , "x86@kernel.org" , "linux-kernel@vger.kernel.org" , "linux-mm@kvack.org" , Andrew Morton , "linuxppc-dev@lists.ozlabs.org" , Dan Williams , "linux-arm-kernel@lists.infradead.org" , Oscar Salvador From: Wei Yang > Sent: 05 February 2020 09:59 ... > If it is me, I would like to take out these two similar logic out. > > For example: > > if () { > } else if () { > } else { > goto out; > } I'm pretty sure the kernel layout rules disallow 'else if'. It is also pretty horrid unless the conditionals are all related (so it is almost a switch statement). David - Registered Address Lakeside, Bramley Road, Mount Farm, Milton Keynes, MK1 1PT, UK Registration No: 1397386 (Wales)