linux-kernel.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
* 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).