public inbox for linux-acpi@vger.kernel.org
 help / color / mirror / Atom feed
From: Richard <tuxbox.guru@gmail.com>
To: trenn@suse.de
Cc: Andrew Morton <akpm@linux-foundation.org>,
	Len Brown <lenb@kernel.org>,
	linux-acpi@vger.kernel.org, Adam M Belay <abelay@mit.edu>,
	Li Shaohua <shaohua.li@intel.com>,
	Matthieu Castet <castet.matthieu@free.fr>,
	Jaroslav Kysela <perex@perex.cz>,
	andreas.herrmann3@amd.com
Subject: Re: 2.6.25 pci=noacpi
Date: Thu, 08 May 2008 12:38:14 +0100	[thread overview]
Message-ID: <4822E626.3070401@gmail.com> (raw)
In-Reply-To: <1210244763.26999.49.camel@queen.suse.de>

Thomas Renninger wrote:
> On Thu, 2008-05-08 at 08:23 +0100, Richard wrote:
>   
>> Thomas Renninger wrote:
>>     
> ...
>
>   
>>>>>> thanks - the idle=poll works to fix the problem.
>>>>>>         
>>>>>>             
>>>>> Well, that's hardly a "fix".  The power consumption will be very high.
>>>>>
>>>>>           
>>>> Hehehe, its a start. A booting kernel is definitely a good start (even 
>>>> one what drinks battery). I have some avenues in the source to 
>>>> investigate, but being a ACPI n00bie its trying to get my head around 
>>>> the mass of code thats a daunting task.
>>>>     
>>>>         
>>> Richard, could you try processor.max_cstate=2 (or even 1), pls (without
>>> noapictimer).
>>> Or you can double check whether you have cstates at all first(when
>>> booting with noapictimer):
>>> cat /proc/acpi/processor/*/power
>>>
>>> Vendors often invalidate deeper sleep states with an invalid latency.
>>> Maybe this has been forgotten here, Windows does not use deeper C-state
>>> or it works there for some reason.
>>> If this works for you, please send dmidecode output.
>>> IMO we should then blacklist this machine in
>>> drivers/acpi/processor_idle.c to not use the offending sleep state.
>>> There already is a blacklist.
>>>
>>> Thanks,
>>>
>>>    Thomas
>>>
>>>
>>>   
>>>       
>> Hi Thomas (and all)
>>
>> I tried the processor.max_cstate=2 to no avail.   on a 23, 24 and a .25 
>> kernel :-P   The machine shuts  down just after INIT  starts.
>>     
> Could you also try processor.max_cstate=1 (one kernel is enough, just a
> quick test... )
> The culprit is in the idle routine and there just the C-states are
> triggered..., it should be related to C-states.
>
>   
>> ------------- This is reported when booted with noapictimer ----------------
>> cat /proc/acpi/processor/*/power
>> active state:            C0
>> max_cstate:              C8
>> bus master activity:     00000000
>> maximum allowed latency: 6666 usec
>> states:
>>     C1:                  type[C1] promotion[--] demotion[--] 
>> latency[000] usage[00082490] duration[00000000000000000000]
>>     C2:                  type[C2] promotion[--] demotion[--] 
>> latency[018] usage[00000000] duration[00000000000000000000]
>>     
> I wonder why even C2 is not entered with noapictimer...
>   
>>     C3:                  type[C3] promotion[--] demotion[--] 
>> latency[083] usage[00000000] duration[00000000000000000000]
>> ----------------------------------------------------------------------------------
>>     
>
>
>   Thomas
>
>
>   
Hi there Thomas,

processor.max_cstate=1 didnt make any difference.   I can video the 
bootup sequence so you can see exactly what I am looking at?

Richard


  reply	other threads:[~2008-05-08 11:38 UTC|newest]

Thread overview: 92+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2008-04-25 18:38 [patch 00/54] PNP cleanup, v4 Bjorn Helgaas
2008-04-25 18:38 ` [patch 01/54] PNP: turn on -DDEBUG when CONFIG_PNP_DEBUG is set Bjorn Helgaas
2008-04-25 18:38 ` [patch 02/54] ISAPNP: move config register addresses out of isapnp.h Bjorn Helgaas
2008-04-25 18:38 ` [patch 03/54] PNPACPI: continue after _CRS and _PRS errors Bjorn Helgaas
2008-04-25 18:38 ` [patch 04/54] PNP: make pnp_add_id() internal to PNP core Bjorn Helgaas
2008-04-25 18:38 ` [patch 05/54] PNP: change pnp_add_id() to allocate its own pnp_id structures Bjorn Helgaas
2008-04-25 18:38 ` [patch 06/54] PNP: add pnp_eisa_id_to_string() Bjorn Helgaas
2008-04-25 18:38 ` [patch 07/54] PNP: add pnp_alloc_dev() Bjorn Helgaas
2008-04-25 18:38 ` [patch 08/54] PNP: make pnp_add_card_id() internal to PNP core Bjorn Helgaas
2008-04-25 18:38 ` [patch 09/54] PNP: change pnp_add_card_id() to allocate its own pnp_id structures Bjorn Helgaas
2008-04-25 18:38 ` [patch 10/54] ISAPNP: pull pnp_add_card_id() out of isapnp_parse_card_id() Bjorn Helgaas
2008-04-25 18:38 ` [patch 11/54] PNP: add pnp_alloc_card() Bjorn Helgaas
2008-04-25 18:38 ` [patch 12/54] PNPACPI: pnpacpi_encode_ext_irq() wrongly set "irq" instead of "extended_irq" Bjorn Helgaas
2008-04-25 18:38 ` [patch 13/54] PNPACPI: use temporaries to reduce repetition Bjorn Helgaas
2008-04-25 18:38 ` [patch 14/54] PNPACPI: hoist dma_flags() out of pnpacpi_parse_allocated_dmaresource() Bjorn Helgaas
2008-04-25 18:38 ` [patch 15/54] PNPACPI: extend irq_flags() to set IORESOURCE_IRQ_SHAREABLE when appropriate Bjorn Helgaas
2008-04-25 18:38 ` [patch 16/54] PNPACPI: pass pnp_dev instead of acpi_handle Bjorn Helgaas
2008-04-25 18:38 ` [patch 17/54] PNP: add debug output to option registration Bjorn Helgaas
2008-04-25 18:38 ` [patch 18/54] PNP: remove pnp_resource_table from internal get/set interfaces Bjorn Helgaas
2008-04-25 18:38 ` [patch 19/54] PNP: remove more pnp_resource_table arguments Bjorn Helgaas
2008-04-25 18:38 ` [patch 20/54] PNP: add debug output to encoders Bjorn Helgaas
2008-04-25 18:38 ` [patch 21/54] PNP: add debug when assigning PNP resources Bjorn Helgaas
2008-04-26 21:26   ` Rene Herman
2008-04-28 15:25     ` Bjorn Helgaas
2008-04-25 18:38 ` [patch 22/54] PNP: add pnp_init_resources(struct pnp_dev *) interface Bjorn Helgaas
2008-04-25 18:38 ` [patch 23/54] PNP: remove pnp_resource_table from internal pnp_clean_resource_table interface Bjorn Helgaas
2008-04-25 18:38 ` [patch 24/54] PNP: remove unused interfaces using pnp_resource_table Bjorn Helgaas
2008-04-25 18:38 ` [patch 25/54] PNP: use dev_printk when possible Bjorn Helgaas
2008-04-25 18:38 ` [patch 26/54] PNP: factor pnp_init_resource_table() and pnp_clean_resource_table() Bjorn Helgaas
2008-04-26 21:33   ` Rene Herman
2008-04-28 17:15     ` Bjorn Helgaas
2008-04-25 18:38 ` [patch 27/54] PNP: add pnp_get_resource() interface Bjorn Helgaas
2008-04-25 18:38 ` [patch 28/54] PNP: remove pnp_mem_flags() as an lvalue Bjorn Helgaas
2008-04-25 18:38 ` [patch 29/54] PNP: convert resource accessors to use pnp_get_resource(), not pnp_resource_table Bjorn Helgaas
2008-04-26 21:42   ` Rene Herman
2008-04-28 17:19     ` Bjorn Helgaas
2008-04-25 18:38 ` [patch 30/54] PNP: use conventional "i" for loop indices Bjorn Helgaas
2008-04-25 18:38 ` [patch 31/54] PNP: reduce redundancy in pnp_assign_port() and others Bjorn Helgaas
2008-04-25 18:38 ` [patch 32/54] PNP: reduce redundancy in pnp_check_port() " Bjorn Helgaas
2008-04-25 18:38 ` [patch 33/54] PNP: reduce redundancy in pnp_set_current_resources() Bjorn Helgaas
2008-04-25 18:38 ` [patch 34/54] PNP: check for conflicts with all resources, not just earlier ones Bjorn Helgaas
2008-04-25 18:38 ` [patch 35/54] PNP: pass resources, not indexes, to pnp_check_port(), et al Bjorn Helgaas
2008-04-25 18:38 ` [patch 36/54] PNP: convert resource checks to use pnp_get_resource(), not pnp_resource_table Bjorn Helgaas
2008-04-25 18:38 ` [patch 37/54] PNP: convert encoders " Bjorn Helgaas
2008-04-25 18:38 ` [patch 38/54] PNP: remove PNP_MAX_* uses Bjorn Helgaas
2008-04-26 22:10   ` Rene Herman
2008-04-25 18:38 ` [patch 39/54] rtc: dont reference pnp_resource_table directly Bjorn Helgaas
2008-04-25 18:38 ` [patch 40/54] PNP: make pnp_resource_table private to PNP core Bjorn Helgaas
2008-04-25 18:38 ` [patch 41/54] PNP: remove pnp_resource_table references from resource decoders Bjorn Helgaas
2008-04-26 22:28   ` Rene Herman
2008-04-28 20:36     ` Bjorn Helgaas
2008-04-28 20:41       ` Rene Herman
2008-04-25 18:38 ` [patch 42/54] PNP: add struct pnp_resource Bjorn Helgaas
2008-04-25 18:38 ` [patch 43/54] PNP: add pnp_resource index for ISAPNP Bjorn Helgaas
2008-04-25 18:38 ` [patch 44/54] PNP: add pnp_new_resource() to find a new unset pnp_resource Bjorn Helgaas
2008-04-26 22:37   ` Rene Herman
2008-04-28 20:43     ` Bjorn Helgaas
2008-04-25 18:38 ` [patch 45/54] PNP: make generic pnp_add_irq_resource() Bjorn Helgaas
2008-04-25 18:38 ` [patch 46/54] PNP: make generic pnp_add_dma_resource() Bjorn Helgaas
2008-04-25 18:38 ` [patch 47/54] PNP: make generic pnp_add_io_resource() Bjorn Helgaas
2008-04-25 18:38 ` [patch 48/54] PNP: make generic pnp_add_mem_resource() Bjorn Helgaas
2008-04-25 18:38 ` [patch 49/54] PNP: use pnp_get_pnp_resource() in resource assignment functions Bjorn Helgaas
2008-04-25 18:38 ` [patch 50/54] ISAPNP: fold isapnp_read_resources() back into isapnp_get_resources() Bjorn Helgaas
2008-04-26 23:07   ` Rene Herman
2008-04-28 21:07     ` Bjorn Helgaas
2008-04-29  6:53       ` 2.6.25 pci=noacpi Richard
2008-04-29  7:04         ` Andrew Morton
2008-04-29  7:17           ` Richard
2008-04-29  7:27           ` Richard
2008-04-29 11:01           ` Richard
2008-05-01  5:10             ` Len Brown
2008-05-01  7:09               ` Richard
2008-05-01 15:59                 ` Len Brown
2008-05-02 11:23                   ` Richard
2008-05-02 12:47                     ` Andrew Morton
2008-05-02 13:11                       ` Richard
2008-05-07 10:19                         ` Thomas Renninger
2008-05-08  7:23                           ` Richard
2008-05-08 11:06                             ` Thomas Renninger
2008-05-08 11:38                               ` Richard [this message]
2008-05-08 12:37                                 ` Thomas Renninger
2008-05-04 19:23         ` Rene Herman
2008-05-08  6:59           ` Richard
2008-05-08  7:08           ` Richard
2008-05-08 11:50             ` Rene Herman
2008-04-25 18:38 ` [patch 51/54] PNPACPI: move _CRS/_PRS warnings closer to the action Bjorn Helgaas
2008-04-25 18:38 ` [patch 52/54] PNP: make interfaces private to the PNP core Bjorn Helgaas
2008-04-25 18:39 ` [patch 53/54] ISAPNP: remove unused pnp_dev->regs field Bjorn Helgaas
2008-04-25 18:39 ` [patch 54/54] PNPBIOS: remove include/linux/pnpbios.h Bjorn Helgaas
2008-04-26 23:34 ` [patch 00/54] PNP cleanup, v4 Rene Herman
2008-04-26 23:47   ` Rene Herman
2008-04-28 21:16     ` Bjorn Helgaas

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=4822E626.3070401@gmail.com \
    --to=tuxbox.guru@gmail.com \
    --cc=abelay@mit.edu \
    --cc=akpm@linux-foundation.org \
    --cc=andreas.herrmann3@amd.com \
    --cc=castet.matthieu@free.fr \
    --cc=lenb@kernel.org \
    --cc=linux-acpi@vger.kernel.org \
    --cc=perex@perex.cz \
    --cc=shaohua.li@intel.com \
    --cc=trenn@suse.de \
    /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