Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1751580AbaAMMht (ORCPT ); Mon, 13 Jan 2014 07:37:49 -0500 Received: from gw-1.arm.linux.org.uk ([78.32.30.217]:35525 "EHLO pandora.arm.linux.org.uk" rhost-flags-OK-OK-OK-FAIL) by vger.kernel.org with ESMTP id S1751362AbaAMMhq (ORCPT ); Mon, 13 Jan 2014 07:37:46 -0500 Date: Mon, 13 Jan 2014 12:37:33 +0000 From: Russell King - ARM Linux To: Santosh Shilimkar Cc: Andrew Morton , Tejun Heo , linux-mm@kvack.org, linux-kernel@vger.kernel.org, Sergei Shtylyov , linux-arm-kernel@lists.infradead.org Subject: Re: [PATCH] mm: nobootmem: avoid type warning about alignment value Message-ID: <20140113123733.GU15937@n2100.arm.linux.org.uk> References: <1385249326-9089-1-git-send-email-santosh.shilimkar@ti.com> <529217C7.6030304@cogentembedded.com> <52935762.1080409@ti.com> <20131209165044.cf7de2edb8f4205d5ac02ab0@linux-foundation.org> <20131210005454.GX4360@n2100.arm.linux.org.uk> <52A66826.7060204@ti.com> <20140112105958.GA9791@n2100.arm.linux.org.uk> <52D2B7C8.4060103@ti.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <52D2B7C8.4060103@ti.com> User-Agent: Mutt/1.5.19 (2009-01-05) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Sun, Jan 12, 2014 at 10:42:00AM -0500, Santosh Shilimkar wrote: > On Sunday 12 January 2014 05:59 AM, Russell King - ARM Linux wrote: > > On Mon, Dec 09, 2013 at 08:02:30PM -0500, Santosh Shilimkar wrote: > >> On Monday 09 December 2013 07:54 PM, Russell King - ARM Linux wrote: > >>> The underlying reason is that - as I've already explained - ARM's __ffs() > >>> differs from other architectures in that it ends up being an int, whereas > >>> almost everyone else is unsigned long. > >>> > >>> The fix is to fix ARMs __ffs() to conform to other architectures. > >>> > >> I was just about to cross-post your reply here. Obviously I didn't think > >> this far when I made $subject fix. > >> > >> So lets ignore the $subject patch which is not correct. Sorry for noise > > > > Well, here we are, a month on, and this still remains unfixed despite > > my comments pointing to what the problem is. So, here's a patch to fix > > this problem the correct way. I took the time to add some comments to > > these functions as I find that I wonder about their return values, and > > these comments make the patch a little larger than it otherwise would be. > > > The $subject warning fix [1] is already picked by Andrew with your ack > and its in his queue [2] > > > This patch makes their types match exactly with x86's definitions of > > the same, which is the basic problem: on ARM, they all took "int" values > > and returned "int"s, which leads to min() in nobootmem.c complaining. > > > Not sure if you missed the thread but the right fix was picked. Ofcourse > you do have additional clz optimisation in updated patch and some comments > on those functions. The problem here is that the patch fixing this is going via akpm's tree (why?) yet you want the code which introduces the warning to be merged via my tree. It seems to me to be absolutely silly to have code introduce a warning yet push the fix for the warning via a completely different tree... -- FTTC broadband for 0.8mile line: 5.8Mbps down 500kbps up. Estimation in database were 13.1 to 19Mbit for a good line, about 7.5+ for a bad. Estimate before purchase was "up to 13.2Mbit". -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/