* PM_TRACE is no longer experimental
@ 2008-08-14 15:18 Pavel Machek
2008-08-15 17:11 ` Tilman Schmidt
0 siblings, 1 reply; 7+ messages in thread
From: Pavel Machek @ 2008-08-14 15:18 UTC (permalink / raw)
To: Andrew Morton, kernel list, Rafael J. Wysocki, Trivial patch monkey
PM_TRACE is stable for quite long. I don't think it needs to depend on
experimental.
Signed-off-by: Pavel Machek <pavel@suse.cz>
---
commit 710f4e9070c453e743fcfa1a6017075026c4967b
tree 7f3fe656004f2d21b6a9cbe1264f9f2236572456
parent ea977ecb4f994b0dc2e09cab3dbba87aad030d99
author Pavel <pavel@amd.ucw.cz> Thu, 14 Aug 2008 17:18:07 +0200
committer Pavel <pavel@amd.ucw.cz> Thu, 14 Aug 2008 17:18:07 +0200
kernel/power/Kconfig | 2 +-
1 files changed, 1 insertions(+), 1 deletions(-)
diff --git a/kernel/power/Kconfig b/kernel/power/Kconfig
index dcd165f..1800674 100644
--- a/kernel/power/Kconfig
+++ b/kernel/power/Kconfig
@@ -36,7 +36,7 @@ config PM_VERBOSE
config CAN_PM_TRACE
def_bool y
- depends on PM_DEBUG && PM_SLEEP && EXPERIMENTAL
+ depends on PM_DEBUG && PM_SLEEP
config PM_TRACE
bool
--
(english) http://www.livejournal.com/~pavelmachek
(cesky, pictures) http://atrey.karlin.mff.cuni.cz/~pavel/picture/horses/blog.html
^ permalink raw reply related [flat|nested] 7+ messages in thread
* Re: PM_TRACE is no longer experimental
2008-08-14 15:18 PM_TRACE is no longer experimental Pavel Machek
@ 2008-08-15 17:11 ` Tilman Schmidt
2008-08-15 20:34 ` Dave Jones
` (2 more replies)
0 siblings, 3 replies; 7+ messages in thread
From: Tilman Schmidt @ 2008-08-15 17:11 UTC (permalink / raw)
To: Pavel Machek
Cc: Andrew Morton, kernel list, Rafael J. Wysocki, Trivial patch monkey
[-- Attachment #1: Type: text/plain, Size: 303 bytes --]
On 14.08.2008 17:18 Pavel Machek wrote:
> PM_TRACE is stable for quite long. I don't think it needs to depend on
> experimental.
It might be stable, but its effect of trashing the RTC makes it unsuitable
for non-experimental use. Therefore I'd prefer the dependency be kept.
Thanks,
Tilman
[-- Attachment #2: OpenPGP digital signature --]
[-- Type: application/pgp-signature, Size: 254 bytes --]
^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: PM_TRACE is no longer experimental
2008-08-15 17:11 ` Tilman Schmidt
@ 2008-08-15 20:34 ` Dave Jones
2008-08-15 22:34 ` Nigel Cunningham
2008-08-17 9:33 ` Pavel Machek
2 siblings, 0 replies; 7+ messages in thread
From: Dave Jones @ 2008-08-15 20:34 UTC (permalink / raw)
To: Tilman Schmidt
Cc: Pavel Machek, Andrew Morton, kernel list, Rafael J. Wysocki,
Trivial patch monkey
On Fri, Aug 15, 2008 at 07:11:36PM +0200, Tilman Schmidt wrote:
> On 14.08.2008 17:18 Pavel Machek wrote:
> > PM_TRACE is stable for quite long. I don't think it needs to depend on
> > experimental.
>
> It might be stable, but its effect of trashing the RTC makes it unsuitable
> for non-experimental use. Therefore I'd prefer the dependency be kept.
It only does that if you boot with command line options to enable it.
Besides, CONFIG_EXPERIMENTAL was always about code maturity, not a
safety net for people who don't read the descriptions.
Pavels change makes sense to me.
Dave
--
http://www.codemonkey.org.uk
^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: PM_TRACE is no longer experimental
2008-08-15 17:11 ` Tilman Schmidt
2008-08-15 20:34 ` Dave Jones
@ 2008-08-15 22:34 ` Nigel Cunningham
2008-08-17 9:33 ` Pavel Machek
2 siblings, 0 replies; 7+ messages in thread
From: Nigel Cunningham @ 2008-08-15 22:34 UTC (permalink / raw)
To: Tilman Schmidt
Cc: Pavel Machek, Andrew Morton, kernel list, Rafael J. Wysocki,
Trivial patch monkey
Hi.
On Fri, 2008-08-15 at 19:11 +0200, Tilman Schmidt wrote:
> On 14.08.2008 17:18 Pavel Machek wrote:
> > PM_TRACE is stable for quite long. I don't think it needs to depend on
> > experimental.
>
> It might be stable, but its effect of trashing the RTC makes it unsuitable
> for non-experimental use. Therefore I'd prefer the dependency be kept.
I think you're confusing 'experimental' and 'debugging'. You certainly
wouldn't want to use it routinely, but that doesn't mean it's still an
experimental feature.
Regards,
Nigel
^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: PM_TRACE is no longer experimental
2008-08-15 17:11 ` Tilman Schmidt
2008-08-15 20:34 ` Dave Jones
2008-08-15 22:34 ` Nigel Cunningham
@ 2008-08-17 9:33 ` Pavel Machek
2008-08-17 10:23 ` Rafael J. Wysocki
2 siblings, 1 reply; 7+ messages in thread
From: Pavel Machek @ 2008-08-17 9:33 UTC (permalink / raw)
To: Tilman Schmidt
Cc: Andrew Morton, kernel list, Rafael J. Wysocki, Trivial patch monkey
> On 14.08.2008 17:18 Pavel Machek wrote:
> > PM_TRACE is stable for quite long. I don't think it needs to depend on
> > experimental.
>
> It might be stable, but its effect of trashing the RTC makes it unsuitable
> for non-experimental use. Therefore I'd prefer the dependency be kept.
Given that it is enabled in both 32-bit and 64-bit defconfig, I don't
think we can call it experimental.
Plus IIRC it needs command line option to thrash RTC.
Pavel
--
(english) http://www.livejournal.com/~pavelmachek
(cesky, pictures) http://atrey.karlin.mff.cuni.cz/~pavel/picture/horses/blog.html
^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: PM_TRACE is no longer experimental
2008-08-17 9:33 ` Pavel Machek
@ 2008-08-17 10:23 ` Rafael J. Wysocki
2008-08-17 15:16 ` Tilman Schmidt
0 siblings, 1 reply; 7+ messages in thread
From: Rafael J. Wysocki @ 2008-08-17 10:23 UTC (permalink / raw)
To: Pavel Machek
Cc: Tilman Schmidt, Andrew Morton, kernel list, Trivial patch monkey
On Sunday, 17 of August 2008, Pavel Machek wrote:
> > On 14.08.2008 17:18 Pavel Machek wrote:
> > > PM_TRACE is stable for quite long. I don't think it needs to depend on
> > > experimental.
> >
> > It might be stable, but its effect of trashing the RTC makes it unsuitable
> > for non-experimental use. Therefore I'd prefer the dependency be kept.
>
> Given that it is enabled in both 32-bit and 64-bit defconfig, I don't
> think we can call it experimental.
I agree.
> Plus IIRC it needs command line option to thrash RTC.
That's correct.
Thanks,
Rafael
^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: PM_TRACE is no longer experimental
2008-08-17 10:23 ` Rafael J. Wysocki
@ 2008-08-17 15:16 ` Tilman Schmidt
0 siblings, 0 replies; 7+ messages in thread
From: Tilman Schmidt @ 2008-08-17 15:16 UTC (permalink / raw)
To: Rafael J. Wysocki
Cc: Pavel Machek, Andrew Morton, kernel list, Trivial patch monkey
[-- Attachment #1: Type: text/plain, Size: 342 bytes --]
Rafael J. Wysocki schrieb:
> On Sunday, 17 of August 2008, Pavel Machek wrote:
>> Given that it is enabled in both 32-bit and 64-bit defconfig, I don't
>> think we can call it experimental.
>
> I agree.
>
>> Plus IIRC it needs command line option to thrash RTC.
>
> That's correct.
Alright, I withdraw my objection.
T.
[-- Attachment #2: OpenPGP digital signature --]
[-- Type: application/pgp-signature, Size: 254 bytes --]
^ permalink raw reply [flat|nested] 7+ messages in thread
end of thread, other threads:[~2008-08-18 15:52 UTC | newest]
Thread overview: 7+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2008-08-14 15:18 PM_TRACE is no longer experimental Pavel Machek
2008-08-15 17:11 ` Tilman Schmidt
2008-08-15 20:34 ` Dave Jones
2008-08-15 22:34 ` Nigel Cunningham
2008-08-17 9:33 ` Pavel Machek
2008-08-17 10:23 ` Rafael J. Wysocki
2008-08-17 15:16 ` Tilman Schmidt
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).