Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1759594Ab1FATqq (ORCPT ); Wed, 1 Jun 2011 15:46:46 -0400 Received: from www.linutronix.de ([62.245.132.108]:55676 "EHLO Galois.linutronix.de" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1759552Ab1FATqp (ORCPT ); Wed, 1 Jun 2011 15:46:45 -0400 Date: Wed, 1 Jun 2011 21:46:37 +0200 (CEST) From: Thomas Gleixner To: David Rientjes cc: Russell King - ARM Linux , Dmitry Eremin-Solenikov , KOSAKI Motohiro , linux-mm@kvack.org, linux-kernel@vger.kernel.org, Mel Gorman , KAMEZAWA Hiroyuki , Rik van Riel , Andrew Morton Subject: Re: [PATCH] Make GFP_DMA allocations w/o ZONE_DMA emit a warning instead of failing In-Reply-To: Message-ID: References: <1306922672-9012-1-git-send-email-dbaryshkov@gmail.com> <20110601181918.GO3660@n2100.arm.linux.org.uk> User-Agent: Alpine 2.02 (LFD 1266 2009-07-14) MIME-Version: 1.0 Content-Type: TEXT/PLAIN; charset=US-ASCII X-Linutronix-Spam-Score: -1.0 X-Linutronix-Spam-Level: - X-Linutronix-Spam-Status: No , -1.0 points, 5.0 required, ALL_TRUSTED=-1,SHORTCIRCUIT=-0.0001 Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 1967 Lines: 59 On Wed, 1 Jun 2011, David Rientjes wrote: > On Wed, 1 Jun 2011, Thomas Gleixner wrote: > > > > That is NOT an unreasonable request, but it seems that its far too much > > > to ask of you. > > > > Full ack. > > > > David, > > > > stop that nonsense already. You changed the behaviour and broke stuff > > which was working fine before for whatever reason. That behaviour was > > in the kernel for ages and we tolerated the abuse. > > > > Did I nack this patch and not realize it? No, you did not realize anything. > Does my patch fix the warning for pxaficp_ir that would still be emitted > with this patch? If the driver uses GFP_DMA and nobody from the arm side Your patch does not fix anything. It papers over the problem and that's the f@&^%%@^#ing wrong approach. And just to be clear. You CANNOT fix a warning. You can fix the code which causes the warning, but that's not what your patch is doing. Your patch HIDES the problem. > is prepared to remove it yet, then I'd suggest merging my patch until that > can be determined. Otherwise, you have no guarantees about where the > memory is actually coming from. Did you actually try to understand what I wrote? You decided that it's a BUG just because it should not be allowed. So you changed the behaviour, which was perfectly fine before. Now you try to paper over the problem by selecting ZONE_DMA and refuse to give a grace period of _ONE_ kernel release. IOW, you are preventing that the abusers of GFP_DMA are fixed properly. I can see that you neither have the bandwidth nor the knowledge to analyse each user of GFP_DMA. And that should tell you something. If you cannot fix it yourself, then f*(&!@$#ng not break it. Thanks, tglx -- 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/