From: David Brownell <david-b@pacbell.net>
To: "Zhang, Rui" <rui.zhang@intel.com>
Cc: "Zhao, Yakui" <yakui.zhao@intel.com>,
linux-acpi@vger.kernel.org, lenb@kernel.org,
Alessandro Zummo <alessandro.zummo@towertech.it>
Subject: Re: [RFC] [PATCH 1/3] Allow to override the RTC alarm time
Date: Wed, 9 Apr 2008 01:27:34 -0700 [thread overview]
Message-ID: <200804090127.35037.david-b@pacbell.net> (raw)
In-Reply-To: <1207727167.27304.20.camel@acpi-hp-zz.sh.intel.com>
On Wednesday 09 April 2008, Zhang, Rui wrote:
> > > In current kernel when RTC alarm time is set and not fired,
> > > it is impossible to set RTC alarm again. But it is more
> > > reasonable that the RTC alarm time can be overrided.
> >
> > I'll disagree. The problem is that if some task is waiting
> > for the alarm at the specified time, you've just trashed the
> > alarm it was relying on.
>
> Hmm, what if the wakealarm is set by mistake?
> is there any chance that user can fix it?
Certainly; there are several ways to turn an alarm off.
With sysfs, just write a time in the past, as I recall...
If you're concerned about accidents, think of it this way:
the proposed patch would make it *REALLY EASY* to have
accidents with sysfs that bork the alarm state, which can
have been set by some task that's relying on it.
The current behavior doesn't prevent such borkage; it
can't, really, because of the alarm model. What it does
instead is force whatever shell script is doing that to be
explicit about *intending* such borkage ... requiring it
to disable existing alarms, instead of allowing accidents.
- Dave
--
To unsubscribe from this list: send the line "unsubscribe linux-acpi" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
next prev parent reply other threads:[~2008-04-09 8:27 UTC|newest]
Thread overview: 13+ messages / expand[flat|nested] mbox.gz Atom feed top
2008-04-09 6:34 [RFC] [PATCH 1/3] Allow to override the RTC alarm time Zhao Yakui
2008-04-09 7:34 ` David Brownell
2008-04-09 7:46 ` Zhang, Rui
2008-04-09 8:27 ` David Brownell [this message]
2008-04-09 16:55 ` Zhao Yakui
2008-04-09 10:13 ` David Brownell
2008-04-10 11:08 ` Zhao Yakui
2008-04-11 5:13 ` David Brownell
2008-04-11 11:04 ` Zhao Yakui
2008-04-11 3:46 ` David Brownell
2008-04-11 12:44 ` Zhao Yakui
2008-04-11 5:12 ` David Brownell
2008-04-11 6:30 ` Alessandro Zummo
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=200804090127.35037.david-b@pacbell.net \
--to=david-b@pacbell.net \
--cc=alessandro.zummo@towertech.it \
--cc=lenb@kernel.org \
--cc=linux-acpi@vger.kernel.org \
--cc=rui.zhang@intel.com \
--cc=yakui.zhao@intel.com \
/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 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.