Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1755017AbZCCBW6 (ORCPT ); Mon, 2 Mar 2009 20:22:58 -0500 Received: (majordomo@vger.kernel.org) by vger.kernel.org id S1750810AbZCCBWu (ORCPT ); Mon, 2 Mar 2009 20:22:50 -0500 Received: from mail-gx0-f174.google.com ([209.85.217.174]:51542 "EHLO mail-gx0-f174.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1750745AbZCCBWt convert rfc822-to-8bit (ORCPT ); Mon, 2 Mar 2009 20:22:49 -0500 MIME-Version: 1.0 In-Reply-To: <200903011020.53130.rjw@sisk.pl> References: <200902192215.18365.rjw@sisk.pl> <200902282353.39763.rjw@sisk.pl> <200903011020.53130.rjw@sisk.pl> Date: Mon, 2 Mar 2009 17:22:46 -0800 Message-ID: Subject: Re: [RFD] Automatic suspend From: =?ISO-8859-1?Q?Arve_Hj=F8nnev=E5g?= To: "Rafael J. Wysocki" Cc: Pavel Machek , Alan Stern , "Woodruff, Richard" , Arjan van de Ven , Kyle Moffett , Oliver Neukum , Benjamin Herrenschmidt , pm list , LKML , Nigel Cunningham , Matthew Garrett , mark gross , Uli Luckas , Igor Stoppa , Brian Swetland , Len Brown Content-Type: text/plain; charset=ISO-8859-1 Content-Transfer-Encoding: 8BIT Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 3540 Lines: 72 On Sun, Mar 1, 2009 at 1:20 AM, Rafael J. Wysocki wrote: >> >> >> I do provide an option to turn off the wakelock stats, which makes >> >> >> wake_lock/unlock significantly faster, but we never run with wakelock >> >> >> stats off. Also, it pushes timeout handling to the drivers. I know may >> >> >> of you don't like timeout support, but ignoring the problem is not a >> >> >> solution. If each driver that needs timeouts uses its own timer, then >> >> >> you will often wakeup from idle just to unlock a wakelock that will >> >> >> not trigger suspend. This wakeup is a thousand times as costly on the >> >> >> msm platform as a wakelock/unlock pair (with wakelock stats enabled). >> >> > >> >> > Well, at least a couple of people told you that the timeouts are hardly >> >> > acceptable and they told you why. ?Please stop repeating the same arguments you >> >> > have given already for a couple of times. ?They're not convincing. >> >> >> >> And you keep ignoring them. >> > >> > Not ignoring, but considering them as insufficient. ?And since they've already >> > been considered as insufficient, there's no point repeating them over and over >> > again. ?That doesn't make them any better. >> >> The problem is that what you consider insufficient is what allows us >> to ship a product. > > This doesn't matter a whit, because the mainline kernel is not just your > product. Unless you are saying that changes in the mainline kernel does not need be usable in practice, then it does matter. If we remove the feature that allows us to interact with existing code, it will take much longer before it is usable by anyone. > By the same rule you could say that every working vendor driver is worth > merging into the mainline kernel, which clearly is not the case. Some people seem to think they are. > >> >> > I do realize that you consider your current solution as the best thing since >> >> > the sliced bread, but please accept the fact that the other people think >> >> > differently. >> >> >> >> I certainly do not think my current solution is the best, it is very >> >> invasive. I do however think your proposed solution is worse. The only >> >> proposed alternative that we could actually ship a product on today is >> >> to not use suspend at all. >> > >> > Well, I'm sure your code is useful for the Android platform, but the question >> > is whether we want this code in the mainline kernel. ?For now, the answer is >> > "no, we don't". ?Moreover, since you're the one who wants the code to be >> > merged, it's your burden to make it acceptable for us. ?However you're going >> > to do it is up to you, but certainly trying to force your current code on us >> > is not going to work. >> >> I don't think I am the only one who want this code in the mainline >> kernel. Many people want to use the android platform, and support in >> the mainline kernel would be beneficial to some of them. I made many >> requested changes to my code that provides no benefit to us, but I >> have not made any changes that breaks our own use. > > OK, please resubmit the patches, then. I submitted them three weeks ago. I'll submit a new set after I rename the api (presumably to suspend_block(er)) but I would like more agreement on the timeout issue first. -- Arve Hj?nnev?g -- 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/