From: Nigel Cunningham <nigel@nigel.suspend2.net>
To: Pavel Machek <pavel@ucw.cz>
Cc: "Rafael J. Wysocki" <rjw@sisk.pl>,
Andrew Morton <akpm@linux-foundation.org>,
Christian Leber <christian@leber.de>,
kernel list <linux-kernel@vger.kernel.org>
Subject: Re: beeping patch for debugging acpi sleep
Date: Wed, 13 Jun 2007 21:32:33 +1000 [thread overview]
Message-ID: <1181734353.5012.12.camel@nigel.suspend2.net> (raw)
In-Reply-To: <20070613082448.GH13747@elf.ucw.cz>
[-- Attachment #1: Type: text/plain, Size: 1373 bytes --]
Hi.
On Wed, 2007-06-13 at 10:24 +0200, Pavel Machek wrote:
> Hi!
>
> > > > Wouldn't it be much more useful if it was unconditionally compiled in
> > > > and controlled instead by a sysfs entry? That way it will be far more
> > > > useful to $user who doesn't know or want to know how to compile and
> > > > install a kernel, but wants to do what they can to get provide helpful
> > > > debugging info and perhaps even get it going.
> > >
> > > I like this idea.
> > >
> > > > Yes, Pavel, I'll supply a patch if you (plural) agree.
> > >
> > > I agree. :-)
> >
> > Ok. I'll take Pavel's silence as agreement too. I'll be a little slow
> > (as usual, nowadays!), but will try to get it done next week. I think I
> > can in clear conscience do it on Redhat time if I don't manage it
> > beforehand.
>
> Well, everyone wants it so what can I do... Stefan basically posted 3
> liner to do that, and that one will probably work... so just test it.
As he admitted, it's a quick and dirty hack. I'd like to try something
nicer.
> Oh, and please keep the macro, so it can be moved around .S file for
> finding out where it crashes.
Are there other points you'd like to nominate? It would defeat the logic
for implementing a nice interface if you still have to recompile your
kernel to test beeping in another place.
Regards,
Nigel
[-- Attachment #2: This is a digitally signed message part --]
[-- Type: application/pgp-signature, Size: 189 bytes --]
next prev parent reply other threads:[~2007-06-13 11:32 UTC|newest]
Thread overview: 28+ messages / expand[flat|nested] mbox.gz Atom feed top
2007-05-18 21:37 [BUG] acpi double resume and fail Christian Leber
2007-05-18 22:45 ` Andrew Morton
2007-05-19 19:42 ` Christian Leber
2007-05-20 20:01 ` Pavel Machek
[not found] ` <20070602182014.GB29546@core>
2007-06-09 13:08 ` beeping patch for debugging acpi sleep Pavel Machek
2007-06-09 13:16 ` Jan-Benedict Glaw
2007-06-09 22:54 ` Pavel Machek
2007-06-09 23:27 ` Nigel Cunningham
2007-06-09 23:35 ` Pavel Machek
2007-06-11 8:56 ` Stefan Seyfried
2007-06-11 16:50 ` Pavel Machek
2007-06-11 19:00 ` Andrew Morton
2007-06-11 19:49 ` Matthieu CASTET
2007-06-12 15:12 ` Pavel Machek
2007-06-13 17:58 ` Stefan Seyfried
2007-06-11 21:26 ` Pavel Machek
2007-06-11 22:42 ` Nigel Cunningham
2007-06-12 12:11 ` Rafael J. Wysocki
2007-06-13 1:17 ` Nigel Cunningham
2007-06-13 8:24 ` Pavel Machek
2007-06-13 11:32 ` Nigel Cunningham [this message]
2007-06-13 12:28 ` Pavel Machek
2007-06-13 12:40 ` Rafael J. Wysocki
2007-06-16 19:46 ` Christian Leber
2007-06-24 20:46 ` Christian Leber
2007-06-28 14:08 ` Pavel Machek
2007-08-12 21:37 ` Christian Leber
2007-05-22 22:16 ` [BUG] acpi double resume and fail Mark Lord
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=1181734353.5012.12.camel@nigel.suspend2.net \
--to=nigel@nigel.suspend2.net \
--cc=akpm@linux-foundation.org \
--cc=christian@leber.de \
--cc=linux-kernel@vger.kernel.org \
--cc=pavel@ucw.cz \
--cc=rjw@sisk.pl \
/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: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
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).