From: Alan Cox <alan@lxorguk.ukuu.org.uk> To: tytso@mit.edu Cc: markgross@thegnar.org, "Alan Stern" <stern@rowland.harvard.edu>, "Rafael J. Wysocki" <rjw@sisk.pl>, "Linux-pm mailing list" <linux-pm@lists.linux-foundation.org>, "Matthew Garrett" <mjg59@srcf.ucam.org>, "Linux Kernel Mailing List" <linux-kernel@vger.kernel.org>, "Dmitry Torokhov" <dmitry.torokhov@gmail.com>, "Arve Hjønnevåg" <arve@android.com>, "Neil Brown" <neilb@suse.de>, "mark gross" <640e9920@gmail.com> Subject: Re: [RFC][PATCH] PM: Avoid losing wakeup events during suspend Date: Mon, 21 Jun 2010 13:22:34 +0100 [thread overview] Message-ID: <20100621132234.5eb880b0@lxorguk.ukuu.org.uk> (raw) In-Reply-To: <20100621121058.GF6199@thunk.org> > There's been one proposal that we simply merge in a set of no-op > inline functions for suspend blockers, just so we can get let the > drivers go in (assuming that Greg K-H believes this is still a > problem), but with an automatical removal of N months (N to be > decided, say 9 or 12 or 18 months). If you automatically remove the suspend blocker wrappers in 12 months then you can keep the drivers in tree. The drivers are generally not a problem (ok some of them like the binder are interesting little trips) its just those hooks, and we'd all benefit somewhat even if the only bit google are patching back in are their hooks. > down the road for N months. Another approach is to simply merge in > no-op functions and not leave any kind of deprecation schedule. > That's sort of an implicit admission of the fact that we may not reach > consensus on this issue. Or we could simply ship the patches as-is to Very bad precedent. What happens when every other vendor does the same ? Keep the upstream code clean. > Linus after he gets back from vacation and ask him for a thumbs up or > thumbs down vote, which might settle things once and for all. You seem desperate to just throw it at Linus - you have been all along before the discussion, during it and now: but I don't understand why ? It's as if you don't want progress to be made by other means ? > How do we go forward from here? PM QoS seems to be evolving nicely. As for merging stuff - if its new code then it should get submitted with the hooks left out anyway, just as all the other vendors are doing with weird little custom hooks of their own. The hooks can still easily be patched back in as a tiny easy to maintain vendor patch. This works - the code still gets updated and seen by people, only the little hook patch has to be maintained and generally it looks after itself except for the odd .rej when something changes right up by the merge point. Alan
WARNING: multiple messages have this Message-ID (diff)
From: Alan Cox <alan@lxorguk.ukuu.org.uk> To: tytso@mit.edu Cc: mark gross <640e9920@gmail.com>, markgross@thegnar.org, Neil Brown <neilb@suse.de>, Dmitry Torokhov <dmitry.torokhov@gmail.com>, Linux Kernel Mailing List <linux-kernel@vger.kernel.org>, Linux-pm mailing list <linux-pm@lists.linux-foundation.org> Subject: Re: [RFC][PATCH] PM: Avoid losing wakeup events during suspend Date: Mon, 21 Jun 2010 13:22:34 +0100 [thread overview] Message-ID: <20100621132234.5eb880b0@lxorguk.ukuu.org.uk> (raw) In-Reply-To: <20100621121058.GF6199@thunk.org> > There's been one proposal that we simply merge in a set of no-op > inline functions for suspend blockers, just so we can get let the > drivers go in (assuming that Greg K-H believes this is still a > problem), but with an automatical removal of N months (N to be > decided, say 9 or 12 or 18 months). If you automatically remove the suspend blocker wrappers in 12 months then you can keep the drivers in tree. The drivers are generally not a problem (ok some of them like the binder are interesting little trips) its just those hooks, and we'd all benefit somewhat even if the only bit google are patching back in are their hooks. > down the road for N months. Another approach is to simply merge in > no-op functions and not leave any kind of deprecation schedule. > That's sort of an implicit admission of the fact that we may not reach > consensus on this issue. Or we could simply ship the patches as-is to Very bad precedent. What happens when every other vendor does the same ? Keep the upstream code clean. > Linus after he gets back from vacation and ask him for a thumbs up or > thumbs down vote, which might settle things once and for all. You seem desperate to just throw it at Linus - you have been all along before the discussion, during it and now: but I don't understand why ? It's as if you don't want progress to be made by other means ? > How do we go forward from here? PM QoS seems to be evolving nicely. As for merging stuff - if its new code then it should get submitted with the hooks left out anyway, just as all the other vendors are doing with weird little custom hooks of their own. The hooks can still easily be patched back in as a tiny easy to maintain vendor patch. This works - the code still gets updated and seen by people, only the little hook patch has to be maintained and generally it looks after itself except for the odd .rej when something changes right up by the merge point. Alan
next prev parent reply other threads:[~2010-06-21 12:20 UTC|newest] Thread overview: 151+ messages / expand[flat|nested] mbox.gz Atom feed top 2010-06-19 22:05 [RFC][PATCH] PM: Avoid losing wakeup events during suspend Rafael J. Wysocki 2010-06-20 5:52 ` mark gross 2010-06-20 5:52 ` mark gross 2010-06-20 12:49 ` Rafael J. Wysocki 2010-06-20 23:13 ` mark gross 2010-06-20 23:13 ` mark gross 2010-06-20 12:49 ` Rafael J. Wysocki 2010-06-20 16:28 ` Alan Stern 2010-06-20 16:28 ` Alan Stern 2010-06-20 21:50 ` Rafael J. Wysocki 2010-06-21 2:23 ` Alan Stern 2010-06-21 2:23 ` Alan Stern 2010-06-21 5:32 ` Florian Mickler 2010-06-21 5:32 ` Florian Mickler 2010-06-21 15:23 ` Alan Stern 2010-06-21 20:38 ` Florian Mickler 2010-06-21 22:18 ` Alan Stern 2010-06-21 22:18 ` Alan Stern 2010-06-21 22:40 ` Rafael J. Wysocki 2010-06-21 22:48 ` Rafael J. Wysocki 2010-06-21 22:48 ` Rafael J. Wysocki 2010-06-22 0:50 ` Arve Hjønnevåg 2010-06-22 0:50 ` Arve Hjønnevåg 2010-06-22 10:21 ` Rafael J. Wysocki 2010-06-22 10:21 ` Rafael J. Wysocki 2010-06-22 14:35 ` Alan Stern 2010-06-22 15:35 ` Rafael J. Wysocki 2010-06-22 19:55 ` Alan Stern 2010-06-22 19:55 ` Alan Stern 2010-06-22 20:58 ` Rafael J. Wysocki 2010-06-22 20:58 ` Rafael J. Wysocki 2010-06-22 19:59 ` [update] " Rafael J. Wysocki 2010-06-24 14:16 ` Andy Lutomirski 2010-06-24 14:16 ` Andy Lutomirski 2010-06-24 14:45 ` Alan Stern 2010-06-24 14:45 ` Alan Stern 2010-06-24 14:48 ` Rafael J. Wysocki 2010-06-24 15:21 ` Andy Lutomirski 2010-06-24 15:21 ` Andy Lutomirski 2010-06-24 14:48 ` Rafael J. Wysocki 2010-06-22 19:59 ` Rafael J. Wysocki 2010-06-22 20:34 ` Alan Stern 2010-06-22 20:34 ` Alan Stern 2010-06-22 21:41 ` Rafael J. Wysocki 2010-06-23 2:12 ` Alan Stern 2010-06-23 2:12 ` Alan Stern 2010-06-23 10:09 ` Rafael J. Wysocki 2010-06-23 10:09 ` Rafael J. Wysocki 2010-06-23 15:21 ` Alan Stern 2010-06-23 15:21 ` Alan Stern 2010-06-23 22:17 ` Rafael J. Wysocki 2010-06-23 22:17 ` Rafael J. Wysocki 2010-06-24 13:13 ` [update 2] " Rafael J. Wysocki 2010-06-24 15:06 ` Rafael J. Wysocki 2010-06-24 15:35 ` Alan Stern 2010-06-24 15:35 ` Alan Stern 2010-06-24 23:00 ` [update 3] " Rafael J. Wysocki 2010-06-24 23:00 ` Rafael J. Wysocki 2010-06-25 14:42 ` Alan Stern 2010-06-25 14:42 ` Alan Stern 2010-06-25 20:33 ` Rafael J. Wysocki 2010-06-25 20:33 ` Rafael J. Wysocki 2010-06-24 15:06 ` [update 2] " Rafael J. Wysocki 2010-06-24 15:44 ` Alan Stern 2010-06-24 15:44 ` Alan Stern 2010-06-24 16:19 ` Rafael J. Wysocki 2010-06-24 17:09 ` Alan Stern 2010-06-24 23:06 ` Rafael J. Wysocki 2010-06-24 23:06 ` Rafael J. Wysocki 2010-06-25 15:09 ` Alan Stern 2010-06-25 15:09 ` Alan Stern 2010-06-25 20:37 ` Rafael J. Wysocki 2010-06-25 20:57 ` Alan Stern 2010-06-25 20:57 ` Alan Stern 2010-06-25 20:37 ` Rafael J. Wysocki 2010-06-25 6:40 ` Florian Mickler 2010-06-25 6:40 ` Florian Mickler 2010-06-25 13:28 ` Rafael J. Wysocki 2010-06-25 13:28 ` Rafael J. Wysocki 2010-06-24 17:09 ` Alan Stern 2010-06-24 16:19 ` Rafael J. Wysocki 2010-06-24 13:13 ` Rafael J. Wysocki 2010-06-22 21:41 ` [update] " Rafael J. Wysocki 2010-06-22 15:35 ` Rafael J. Wysocki 2010-06-22 14:35 ` Alan Stern 2010-06-22 23:00 ` mark gross 2010-06-22 23:00 ` mark gross 2010-06-21 22:40 ` Rafael J. Wysocki 2010-06-21 20:38 ` Florian Mickler 2010-06-21 15:23 ` Alan Stern 2010-06-21 16:54 ` Alan Stern 2010-06-21 20:40 ` Florian Mickler 2010-06-21 20:40 ` Florian Mickler 2010-06-21 21:18 ` Rafael J. Wysocki 2010-06-21 21:18 ` Rafael J. Wysocki 2010-06-21 22:27 ` Alan Stern 2010-06-21 22:27 ` Alan Stern 2010-06-21 16:54 ` Alan Stern 2010-06-21 6:13 ` mark gross 2010-06-21 6:13 ` mark gross 2010-06-21 12:10 ` tytso 2010-06-21 12:10 ` tytso 2010-06-21 12:22 ` Alan Cox [this message] 2010-06-21 12:22 ` Alan Cox 2010-06-21 12:26 ` Florian Mickler 2010-06-21 12:26 ` Florian Mickler 2010-06-21 12:26 ` Florian Mickler 2010-06-21 13:42 ` tytso 2010-06-21 14:01 ` Alan Cox 2010-06-21 14:01 ` Alan Cox 2010-06-21 13:42 ` tytso 2010-06-22 1:07 ` mark gross 2010-06-22 1:07 ` mark gross 2010-06-21 16:01 ` Alan Stern 2010-06-21 16:01 ` Alan Stern 2010-06-22 1:25 ` mark gross 2010-06-22 2:24 ` Alan Stern 2010-06-22 2:24 ` Alan Stern 2010-06-22 1:25 ` mark gross 2010-06-21 21:58 ` Rafael J. Wysocki 2010-06-21 21:58 ` Rafael J. Wysocki 2010-06-20 21:50 ` Rafael J. Wysocki 2010-06-20 22:58 ` mark gross 2010-06-21 2:33 ` Alan Stern 2010-06-21 2:33 ` Alan Stern 2010-06-21 4:04 ` [linux-pm] " David Brownell 2010-06-21 4:04 ` David Brownell 2010-06-21 6:02 ` [linux-pm] " David Brownell 2010-06-21 6:02 ` David Brownell 2010-06-21 15:06 ` Alan Stern 2010-06-21 15:06 ` [linux-pm] " Alan Stern 2010-06-21 5:55 ` mark gross 2010-06-21 12:39 ` Florian Mickler 2010-06-21 12:39 ` Florian Mickler 2010-06-21 15:57 ` Alan Stern 2010-06-21 15:57 ` Alan Stern 2010-06-22 1:58 ` mark gross 2010-06-22 2:46 ` Alan Stern 2010-06-22 2:46 ` Alan Stern 2010-06-22 9:24 ` Rafael J. Wysocki 2010-06-22 9:24 ` Rafael J. Wysocki 2010-06-22 6:18 ` Florian Mickler 2010-06-22 6:18 ` Florian Mickler 2010-06-22 23:22 ` mark gross 2010-06-22 23:22 ` mark gross 2010-06-22 9:29 ` Rafael J. Wysocki 2010-06-22 9:29 ` Rafael J. Wysocki 2010-06-22 1:58 ` mark gross 2010-06-21 5:55 ` mark gross 2010-06-20 22:58 ` mark gross 2010-06-19 22:05 Rafael J. Wysocki
Reply instructions: You may reply publicly to this message via plain-text email using any one of the following methods: * Save the following mbox file, import it into your mail client, and reply-to-all from there: mbox Avoid top-posting and favor interleaved quoting: https://en.wikipedia.org/wiki/Posting_style#Interleaved_style * Reply using the --to, --cc, and --in-reply-to switches of git-send-email(1): git send-email \ --in-reply-to=20100621132234.5eb880b0@lxorguk.ukuu.org.uk \ --to=alan@lxorguk.ukuu.org.uk \ --cc=640e9920@gmail.com \ --cc=arve@android.com \ --cc=dmitry.torokhov@gmail.com \ --cc=linux-kernel@vger.kernel.org \ --cc=linux-pm@lists.linux-foundation.org \ --cc=markgross@thegnar.org \ --cc=mjg59@srcf.ucam.org \ --cc=neilb@suse.de \ --cc=rjw@sisk.pl \ --cc=stern@rowland.harvard.edu \ --cc=tytso@mit.edu \ /path/to/YOUR_REPLY https://kernel.org/pub/software/scm/git/docs/git-send-email.html * If your mail client supports setting the In-Reply-To header via mailto: links, try the mailto: linkBe sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes, see mirroring instructions on how to clone and mirror all data and code used by this external index.