linux-acpi.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
From: Hans de Goede <hdegoede@redhat.com>
To: "Martin Hundebøll" <martin@hundeboll.net>
Cc: linux-acpi <linux-acpi@vger.kernel.org>,
	Matthew Garrett <mjg59@srcf.ucam.org>
Subject: Re: ACPI errors on Dell Latitude E5430
Date: Mon, 08 Oct 2012 13:22:55 +0200	[thread overview]
Message-ID: <5072B78F.6040707@redhat.com> (raw)
In-Reply-To: <506C68BB.4030908@hundeboll.net>

Hi,

On 10/03/2012 06:32 PM, Martin Hundebøll wrote:
> On 2012-10-03 14:52, Hans de Goede wrote:
>> Hi,
>>
>> On 10/03/2012 01:48 PM, Martin Hundebøll wrote:
>>> On 2012-10-03 12:43, Hans de Goede wrote:
>>>> Hi,
>>>>
>>>> On 10/03/2012 08:36 AM, Martin Hundebøll wrote:
>>>>> On 2012-10-02 13:14, Hans de Goede wrote:
>>>>>> Hi,
>>>>>>
>>>>>> On 10/02/2012 11:22 AM, Martin Hundebøll wrote:
>>>>>>> Hi Hans,
>>>>>>>
>>>>>>> On 2012-10-01 20:38, Hans de Goede wrote:
>>>>>>>> 2) Even with a 3.6 which shows the problematic behavior, I can
>>>>>>>> make things work by adding acpi.ec_delay=2000 to the kernel cmdline,
>>>>>>>> the debug traces show that the EC takes it sweet time to respond
>>>>>>>> (sometime close to 2 seconds) but it does eventually respond in all
>>>>>>>> cases I've seen sofar.
>>>>>>>>
>>>>>>>> This also seems to fix a sporadic occurence of:
>>>>>>>> "ACPI: EC: input buffer is not empty aborting transaction"
>>>>>>>> Which I'm seeing with 3.5 / 3.6 in a non debug build.
>>>>>>>>
>>>>>>>> Martin, can you see if adding that helps you with the E5430 problems
>>>>>>>> too ?
>>>>>>>
>>>>>>> As far as I can tell from the attached dmesg, the errors are gone
>>>>>>> after setting acpi.ec_delay=2000. Thanks a lot!
>>>>>>>
>>>>>>> The only remaining issue is the fan control, which works as expected
>>>>>>> initially, but then fails to spin down after a few minutes.
>>>>>>
>>>>>> Hmm, I'm seeing the same thing, but only when charging my battery, and
>>>>>> then indeed
>>>>>> does not spin down completely, but it does step down from vacuum
>>>>>> cleaner
>>>>>> mode to
>>>>>> something more pleasant after the machine was loaded for a while and
>>>>>> returns back
>>>>>> to idle.
>>>>>>
>>>>>> Also it spins up to its slowest speed by jut using the charger, so I
>>>>>> think it is
>>>>>> just bleeding of heat caused by the charging hardware in this case
>>>>>> (for
>>>>>> me).
>>>>>
>>>>> For the record, my fan keeps spinning also when on battery (as
>>>>> reported by i8k/sensors):
>>>>>
>>>>>    i8k-virtual-0
>>>>>    Adapter: Virtual device
>>>>>    Right Fan:   84000 RPM
>>>>>    CPU:          +39.0C
>>>>>
>>>>> This is not a total show-stopper, but nevertheless quite annoying when
>>>>> the home-office is shared with the bedroom :)
>>>>
>>>> Ah right, you added acpi_enforce_resources=lax to get i8k to work,
>>>> right, hint:
>>>> DO NOT DO THAT!
>>>>
>>>> See:
>>>> http://lm-sensors.org/wiki/FAQ/Chapter3#Mysensorshavestoppedworkinginkernel2.6.31
>>>>
>>>
>>> Unfortunately, no. I only tried those paramters to see no effect and
>>> thus removed them again before sending my initial mail. The
>>> description in my previous mail was given with only:
>>>   cat /proc/cmdline
>>>   BOOT_IMAGE=/boot/vmlinuz-linux
>>> root=UUID=ef14a89c-443d-4d51-99b8-bff7a90f1015 ro quiet
>>> acpi.ec_delay=2000
>>>
>>
>> Hmm, so the i8k driver loads without acpi_enforce_resources=lax, ok. But
>> you do need
>> to load it manually right, or does it auto - load ?
>
> I was in the believe that i8k was of the good, so I loaded it manually, but it did also auto load...
>
>> Either way could you try without the i8k driver? Blacklisting it if
>> necessary ?
>
> It is now blacklisted/uninstalled and the fan is indeed unhearable! I will consider the case solved (with the increased timeout), unless something else appears within near future.

I'm glad to hear that for you things are solved, but it would be nice to make
things work out of the box. Can you please send in a bug report about the
i8k driver causing the fans to spin too fast? Please send it to both
the platform-drivers-x86 and the lm-sensors mailing lists:
platform-driver-x86@vger.kernel.org and lm-sensors@lm-sensors.org
note for the last list you probably need to be subscribed:
http://lists.lm-sensors.org/mailman/listinfo/lm-sensors

Also please put me in the CC :)

On a related note, since I've not heard anything from Dell about this,
I'll likely write a patch adding a new dmi based quirk to the ACPI-ec
code to make a delay of 2000 the default on both our laptops. Can you
do (as root):
dmidecode > dmi.log

And then send me the generated dmi.log, then I've the necessary info
to also enable the quirk for your machine.

Regards,

Hans
--
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

  reply	other threads:[~2012-10-08 11:21 UTC|newest]

Thread overview: 12+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2012-10-01 18:38 ACPI errors on Dell Latitude E5430 Hans de Goede
2012-10-02  9:22 ` Martin Hundebøll
2012-10-02 11:14   ` Hans de Goede
2012-10-03  6:36     ` Martin Hundebøll
2012-10-03 10:43       ` Hans de Goede
2012-10-03 11:48         ` Martin Hundebøll
2012-10-03 12:52           ` Hans de Goede
2012-10-03 16:32             ` Martin Hundebøll
2012-10-08 11:22               ` Hans de Goede [this message]
2012-10-03 12:53           ` Hans de Goede
  -- strict thread matches above, loose matches on Subject: below --
2012-09-21 12:10 Martin Hundebøll
2012-09-21 19:34 ` Matthew Garrett

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=5072B78F.6040707@redhat.com \
    --to=hdegoede@redhat.com \
    --cc=linux-acpi@vger.kernel.org \
    --cc=martin@hundeboll.net \
    --cc=mjg59@srcf.ucam.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;
as well as URLs for NNTP newsgroup(s).