Re: [patch] suspend/resume self-test

Previous message: [thread] [date] [author]
Next message: [thread] [date] [author]
From: Ingo Molnar
Date: Wednesday, February 20, 2008 - 3:15 am

* David Brownell <david-b@pacbell.net> wrote:


i think you dont understand what "default enabled" means in this 
context. Default enabled means the Kconfig entry has a "default y" line 
- which it did not have in the patch i sent.

if a tester enables the .config option then it is very much expected to 
be triggered, just like the other .config debug options. Of course, the 
tester specifically enabled it, so that's what we want to happen. Not 
some other "yes_i_really_meant_it" boot option ... There can also be 
_another_ .config option that turns off the 'run by default' property - 
but if someone selects the _default off_ feature and enables it, just 
like i did, then the feature is very much expected to run.

you really seem to have deep misunderstandings about how testers use 
Linux, what it takes to build up a proper debug infrastructure and how 
to utilize all these things to make Linux better. You come from the 
embedded world where everything is configured together specifically, 
where people know what they are using and were every feature is well 
accounted for. But you seem to have little insight into how 
distributions utilize kernel features and what it takes to accept the 
help of testers who dont know the field that well but still want to help 
out - as long as we let them.

These are well-established practices we use in all other debug features 
that help Linux today, and what you are asking for is weird, unusual and 
wrong. Please take a look at how all the other debug stuff works. Rule
#1: testers dont have to beg the kernel for 1 hour and have to write 2
patches to let the test code be run finally ... ;-) This isnt even about 
any nuances where reasonable people might disagree, this is really basic 
testing-101 stuff.

	Ingo
--
Previous message: [thread] [date] [author]
Next message: [thread] [date] [author]

Messages in current thread:
sleepy linux self-test, Pavel Machek, (Wed Jan 30, 6:17 am)
Re: sleepy linux self-test, Ingo Molnar, (Wed Jan 30, 9:35 am)
Re: sleepy linux self-test, Pavel Machek, (Wed Jan 30, 9:39 am)
Re: sleepy linux self-test, Ingo Molnar, (Wed Jan 30, 12:36 pm)
Re: sleepy linux self-test, Pavel Machek, (Wed Jan 30, 4:26 pm)
Re: [linux-pm] sleepy linux self-test, David Brownell, (Thu Jan 31, 6:55 pm)
Re: sleepy linux self-test, Ingo Molnar, (Fri Feb 1, 7:22 am)
Re: sleepy linux self-test, Pavel Machek, (Sat Feb 2, 5:45 am)
Re: [linux-pm] sleepy linux self-test, Pavel Machek, (Sat Feb 2, 5:47 am)
Re: sleepy linux self-test, Ingo Molnar, (Sat Feb 2, 6:49 am)
Re: [linux-pm] sleepy linux self-test, Ingo Molnar, (Sat Feb 2, 6:50 am)
Re: sleepy linux self-test, Ingo Molnar, (Sat Feb 2, 6:51 am)
Re: [linux-pm] sleepy linux self-test, David Brownell, (Sat Feb 2, 10:31 am)
Re: [linux-pm] sleepy linux self-test, David Brownell, (Sat Feb 2, 10:49 am)
Re: [linux-pm] sleepy linux self-test, David Brownell, (Sat Feb 2, 10:51 am)
Re: [linux-pm] sleepy linux self-test, Ingo Molnar, (Sat Feb 2, 11:00 am)
Re: [linux-pm] sleepy linux self-test, Ingo Molnar, (Sat Feb 2, 11:06 am)
Re: [linux-pm] sleepy linux self-test, David Brownell, (Sat Feb 2, 12:13 pm)
Re: [linux-pm] sleepy linux self-test, Pavel Machek, (Sat Feb 2, 12:32 pm)
Re: [linux-pm] sleepy linux self-test, Ingo Molnar, (Sat Feb 2, 12:38 pm)
Re: [linux-pm] sleepy linux self-test, David Brownell, (Sat Feb 2, 12:47 pm)
Re: [linux-pm] sleepy linux self-test, Pavel Machek, (Sat Feb 2, 12:59 pm)
Re: [linux-pm] sleepy linux self-test, David Brownell, (Sat Feb 2, 7:37 pm)
Re: [linux-pm] sleepy linux self-test, Ingo Molnar, (Sat Feb 2, 10:05 pm)
Re: [linux-pm] sleepy linux self-test, Ingo Molnar, (Sat Feb 2, 10:14 pm)
Re: [linux-pm] sleepy linux self-test, Ingo Molnar, (Sat Feb 2, 10:19 pm)
Re: [linux-pm] sleepy linux self-test, Ingo Molnar, (Sat Feb 2, 10:35 pm)
Re: [linux-pm] sleepy linux self-test, Ingo Molnar, (Sat Feb 2, 10:54 pm)
Re: [linux-pm] sleepy linux self-test, Ingo Molnar, (Sun Feb 3, 12:05 am)
Re: [linux-pm] sleepy linux self-test, David Brownell, (Sun Feb 3, 12:18 am)
Re: [linux-pm] sleepy linux self-test, David Brownell, (Sun Feb 3, 12:32 am)
Re: [linux-pm] sleepy linux self-test, Sam Ravnborg, (Sun Feb 3, 12:51 am)
Re: [linux-pm] sleepy linux self-test, David Brownell, (Sun Feb 3, 1:26 am)
Re: [linux-pm] sleepy linux self-test, Rafael J. Wysocki, (Sun Feb 3, 5:21 am)
Re: [linux-pm] sleepy linux self-test, David Brownell, (Sun Feb 3, 6:16 am)
Re: [linux-pm] sleepy linux self-test, Rafael J. Wysocki, (Sun Feb 3, 2:29 pm)
Re: [linux-pm] sleepy linux self-test, David Brownell, (Sun Feb 3, 3:42 pm)
Re: [linux-pm] sleepy linux self-test, Rafael J. Wysocki, (Sun Feb 3, 3:43 pm)
Re: [linux-pm] sleepy linux self-test, Pavel Machek, (Sun Feb 3, 3:48 pm)
Re: [linux-pm] sleepy linux self-test, David Brownell, (Sun Feb 3, 4:08 pm)
Re: [linux-pm] sleepy linux self-test, Pavel Machek, (Sun Feb 10, 2:02 pm)
Re: [linux-pm] sleepy linux self-test, Pavel Machek, (Sun Feb 10, 2:03 pm)
Re: [linux-pm] sleepy linux self-test, Pavel Machek, (Mon Feb 18, 1:56 am)
[patch] suspend/resume self-test, Ingo Molnar, (Mon Feb 18, 2:46 am)
Re: [patch] suspend/resume self-test, Pavel Machek, (Mon Feb 18, 2:53 am)
Re: [patch] suspend/resume self-test, David Brownell, (Mon Feb 18, 3:40 am)
Re: [patch] suspend/resume self-test, Rafael J. Wysocki, (Mon Feb 18, 4:04 am)
Re: [patch] suspend/resume self-test, Rafael J. Wysocki, (Mon Feb 18, 4:06 am)
Re: [patch] suspend/resume self-test, Ingo Molnar, (Mon Feb 18, 6:09 am)
Re: [patch] suspend/resume self-test, David Brownell, (Mon Feb 18, 1:16 pm)
Re: [patch] suspend/resume self-test, Pavel Machek, (Tue Feb 19, 3:11 am)
Re: [patch] suspend/resume self-test, Ingo Molnar, (Tue Feb 19, 7:40 am)
Re: [patch] suspend/resume self-test, Ingo Molnar, (Tue Feb 19, 7:43 am)
Re: [patch] suspend/resume self-test, David Brownell, (Tue Feb 19, 12:12 pm)
Re: [patch] suspend/resume self-test, Ingo Molnar, (Wed Feb 20, 3:15 am)