From: Herman Sheremetyev <mlists-6MNMYbGzEYJWk0Htik3J/w@public.gmane.org>
To: ML ACPI-devel
<acpi-devel-5NWGOfrQmneRv+LV9MX5uipxlwaOVQ5f@public.gmane.org>
Subject: Re: PBLK length again
Date: Mon, 21 Jun 2004 16:40:04 -0400 [thread overview]
Message-ID: <1087850404.2501.43.camel@camel> (raw)
In-Reply-To: <40D73F9E.4030805-wlebWZzHoyE@public.gmane.org>
Hi Luca,
Thanks for the response,
> on 06/21/04 00:20, Herman Sheremetyev wrote:
> > I have an Asus M6N and the Bios DSDT provides a PBLK length of 7 which
> > causes ACPI to always use the C1 processor state. This laptop
> > definitely supports C2 and most people have just used custom DSDT's
> > and changed the value to 6. I don't like using a custom DSDT so I
> > made a quick kernel patch (attached) to accept 7 as a good value.
> AFAIK 7 isn't an allowed value from the ACPI-CA specs (but I'm not an
> expert).
>From the research I've done I am fairly certain that 7 is not a legal
value but Asus insists on breaking the spec by sticking 5's and 7's in
PBLK length even though they mean 6, and Windows doesn't seem to mind.
There should be a way for Linux to deal with this too I think.
> BTW, I've an ASUS M6N, too, please refers to this kernel bug:
> http://bugme.osdl.org/show_bug.cgi?id=1958
> I made some tests in the past with the provided Bruno Ducrot's patch (as
> you can read), but it seems that now the patch is broken.
>
> ASAP I'll try your patch, just to confirm that it works on different
> machines.
Cool, I run a "Linux on the M6N" forum so you can post your results
there as it probably doesn't need to be on this list:
http://mrhammy2.ath.cx:81/forum/viewtopic.php?t=85
> > My question is, if I make a nicer patch that allows for either strict
> > interpretation versus a relaxed one, is there a chance that it will
> > get included? It seems there is enough interest out there to make it
> > worth the effort.
> You should wait for someone other's advice, but if PBLK = 7 isn't
> correct for the ACPI-CA specs, IMHO the solution shouldn't be a
> workaround like your.
What this patch does is really no different from using a custom DSDT, in
the end you're still overriding the value the BIOS provides with one
that make sense to the OSPM system. I'm definitely no expert on this
stuff but I would think a kernel based solution (granted it should be
nicer than my initial patch) is better than a DSDT based one. Having
either a compile-time or boot-time option to accept other values or
maybe override the PBLK length would probably benefit a lot of people
with broken BIOS's.
-Herman
-------------------------------------------------------
This SF.Net email sponsored by Black Hat Briefings & Training.
Attend Black Hat Briefings & Training, Las Vegas July 24-29 -
digital self defense, top technical experts, no vendor pitches,
unmatched networking opportunities. Visit www.blackhat.com
next prev parent reply other threads:[~2004-06-21 20:40 UTC|newest]
Thread overview: 10+ messages / expand[flat|nested] mbox.gz Atom feed top
2004-06-20 2:55 patch & a question about the serial driver Dino Klein
[not found] ` <BAY16-F112kKIBXEHt30004aded-PkbjNfxxIARBDgjK7y7TUQ@public.gmane.org>
2004-06-20 15:47 ` Matthew Wilcox
[not found] ` <20040620154726.GT20511-+pPCBgu9SkPzIGdyhVEDUDl5KyyQGfY2kSSpQ9I8OhVaa/9Udqfwiw@public.gmane.org>
2004-06-20 22:20 ` PBLK length again Herman Sheremetyev
[not found] ` <1087770015.23371.27.camel-l85cmlzfk8I@public.gmane.org>
2004-06-21 20:05 ` Luca Capello
[not found] ` <40D73F9E.4030805-wlebWZzHoyE@public.gmane.org>
2004-06-21 20:40 ` Herman Sheremetyev [this message]
[not found] ` <1087850404.2501.43.camel-O4LVqDAXoJg@public.gmane.org>
2004-06-23 13:32 ` Stefan Seyfried
[not found] ` <20040623133208.GA3152-l0tNAEGuAhhzZ8+rp42Dbp9+tswZ0GTaehPwdyo5hKaELgA04lAiVw@public.gmane.org>
2004-06-24 16:21 ` Karol Kozimor
[not found] ` <20040624162117.GA697-DETuoxkZsSqrDJvtcaxF/A@public.gmane.org>
2004-06-24 18:08 ` Herman Sheremetyev
2004-06-24 21:44 ` Stefan Seyfried
2004-06-25 11:51 ` Luca Capello
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=1087850404.2501.43.camel@camel \
--to=mlists-6mnmybgzeyjwk0htik3j/w@public.gmane.org \
--cc=acpi-devel-5NWGOfrQmneRv+LV9MX5uipxlwaOVQ5f@public.gmane.org \
/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