All of lore.kernel.org
 help / color / mirror / Atom feed
From: Henrique de Moraes Holschuh <hmh-N3TV7GIv+o9fyO9Q7EP/yw@public.gmane.org>
To: Mark Stosberg <mark-WmyhgDpj2fCHT8/ATrE1yw@public.gmane.org>
Cc: ibm-acpi-devel-5NWGOfrQmneRv+LV9MX5uipxlwaOVQ5f@public.gmane.org,
	linux-acpi-u79uwXL29TY76Z2rM5mHXA@public.gmane.org
Subject: Re: remove HKEY disable functionality (triggers Ubuntu Karmic regression)
Date: Thu, 29 Oct 2009 20:46:34 -0200	[thread overview]
Message-ID: <20091029224634.GA15435@khazad-dum.debian.net> (raw)
In-Reply-To: <20091028215838.38b362ad@mark-x41>

On Wed, 28 Oct 2009, Mark Stosberg wrote:
> With the release of Karmic upon us, perhaps there a short term fix that 
> can be made, like making the warning level less severe.

If the userspace in Ubuntu is triggering the warning, and you don't have
enough time to fix the issue properly (i.e. remove any code that
attempts to mess with hotkey_enable), the Ubuntu kernel team will have
to remove the WARN from the driver code, yes.

However, I recommend trying to fix the real bug, first.  There are
absolutely no known-valid reasons for any program to access the
hotkey_enable attribute.


Please ask someone who can reproduce the warning to check for debug output
from thinkpad-acpi, it will log to the kernel log the PID of the process
trying to access the deprecated attributes.  The user should check the PID
and use the ps command to find out what process corresponts to that PID:

Run this in a shell window:

tail -f /var/log/kern.log | grep "thinkpad_acpi"

then run whatever application causes the warnings to happen, or if they
are common, just wait for one to happen.  Thinkpad-acpi will log the PID
of the offending processes using a phrase like "access by process with
PID...".

I would be interested in the full log output by thinkpad-acpi showing the
problem.  In fact, I find it surprising that your automated bug report tool
doesn't attach to the bug report the last 100 lines of the kernel log or
somesuch...

Anyway, now that the user knows the PID of the process attempting to access
the deprecated attributes, he should immediately use the "ps PID", where PID
is the numerical value of the PID logged by thinkpad-acpi, to find out the
name and parameters of the process who accessed the deprecated attribute.

Maybe the user should also use "pstree -p" to help locate the parent of the
process if the PID resultued in something unhelpful (e.g. bash).

That could give us a clue about exactly why that attribute is being
accessed, especially if it is happening when someone is trying to test their
ALSA setup...

-- 
  "One disk to rule them all, One disk to find them. One disk to bring
  them all and in the darkness grind them. In the Land of Redmond
  where the shadows lie." -- The Silicon Valley Tarot
  Henrique Holschuh

------------------------------------------------------------------------------
Come build with us! The BlackBerry(R) Developer Conference in SF, CA
is the only developer event you need to attend this year. Jumpstart your
developing skills, take BlackBerry mobile applications to market and stay 
ahead of the curve. Join us from November 9 - 12, 2009. Register now!
http://p.sf.net/sfu/devconference

  reply	other threads:[~2009-10-29 22:46 UTC|newest]

Thread overview: 16+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2009-04-04  4:25 [GIT PATCH] thinkpad-acpi queue for 2.6.30 Henrique de Moraes Holschuh
2009-04-04  4:25 ` [PATCH 01/12] thinkpad-acpi: update copyright notices Henrique de Moraes Holschuh
2009-04-04  4:25 ` [PATCH 02/12] thinkpad-acpi: drop ibm-acpi alias Henrique de Moraes Holschuh
2009-04-04  4:25 ` [PATCH 03/12] thinkpad-acpi: documentation cleanup Henrique de Moraes Holschuh
2009-04-04  4:25 ` [PATCH 04/12] thinkpad-acpi: cleanup debug helpers Henrique de Moraes Holschuh
2009-04-04  4:25 ` [PATCH 05/12] thinkpad-acpi: add missing log levels Henrique de Moraes Holschuh
2009-04-04  4:25 ` [PATCH 06/12] thinkpad-acpi: add new debug helpers and warn of deprecated atts Henrique de Moraes Holschuh
2009-04-04  4:25 ` [PATCH 07/12] thinkpad-acpi: remove HKEY disable functionality Henrique de Moraes Holschuh
2009-10-29  6:58   ` remove HKEY disable functionality (triggers Ubuntu Karmic regression) Mark Stosberg
2009-10-29 22:46     ` Henrique de Moraes Holschuh [this message]
2009-04-04  4:25 ` [PATCH 08/12] thinkpad-acpi: restrict access to some firmware LEDs Henrique de Moraes Holschuh
2009-04-04  4:25 ` [PATCH 09/12] thinkpad-acpi: enhanced debugging messages for rfkill subdrivers Henrique de Moraes Holschuh
2009-04-04  4:25 ` [PATCH 10/12] thinkpad-acpi: enhanced debugging messages for the hotkey subdriver Henrique de Moraes Holschuh
2009-04-04  4:25 ` [PATCH 11/12] thinkpad-acpi: enhanced debugging messages for the fan subdriver Henrique de Moraes Holschuh
     [not found] ` <1238819153-16004-1-git-send-email-hmh-N3TV7GIv+o9fyO9Q7EP/yw@public.gmane.org>
2009-04-04  4:25   ` [PATCH 12/12] thinkpad-acpi: rework brightness support Henrique de Moraes Holschuh
2009-04-04  7:36 ` [GIT PATCH] thinkpad-acpi queue for 2.6.30 Len Brown

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=20091029224634.GA15435@khazad-dum.debian.net \
    --to=hmh-n3tv7giv+o9fyo9q7ep/yw@public.gmane.org \
    --cc=ibm-acpi-devel-5NWGOfrQmneRv+LV9MX5uipxlwaOVQ5f@public.gmane.org \
    --cc=linux-acpi-u79uwXL29TY76Z2rM5mHXA@public.gmane.org \
    --cc=mark-WmyhgDpj2fCHT8/ATrE1yw@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 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.