NetBSD Problem Report #43504

From www@NetBSD.org  Thu Jun 24 19:32:24 2010
Return-Path: <www@NetBSD.org>
Received: from mail.netbsd.org (mail.netbsd.org [204.152.190.11])
	by www.NetBSD.org (Postfix) with ESMTP id 0428163BA59
	for <gnats-bugs@gnats.NetBSD.org>; Thu, 24 Jun 2010 19:32:24 +0000 (UTC)
Message-Id: <20100624193223.C521D63BA54@www.NetBSD.org>
Date: Thu, 24 Jun 2010 19:32:23 +0000 (UTC)
From: demelier.david@gmail.com
Reply-To: demelier.david@gmail.com
To: gnats-bugs@NetBSD.org
Subject: No brightness adjusting support on HP Probook
X-Send-Pr-Version: www-1.0

>Number:         43504
>Category:       kern
>Synopsis:       No brightness adjusting support on HP Probook
>Confidential:   no
>Severity:       critical
>Priority:       medium
>Responsible:    kern-bug-people
>State:          open
>Class:          sw-bug
>Submitter-Id:   net
>Arrival-Date:   Thu Jun 24 19:35:00 +0000 2010
>Last-Modified:  Wed Jan 23 16:20:01 +0000 2019
>Originator:     David DEMELIER
>Release:        -current
>Organization:
>Environment:
NetBSD Melon.malikania.fr 5.99.31 NetBSD 5.99.31 (Melon) #0: Thu Jun 24 18:55:01 CEST 2010  root@Melon.malikania.fr:/usr/obj/sys/arch/amd64/compile/Melon amd64
>Description:
Like all HP laptops (almost) I can't adjust the laptop brightness using fn + f7 or f8 keys.

If I boot without ACPI, I can adjust it.
>How-To-Repeat:

>Fix:

>Release-Note:

>Audit-Trail:
From: Jukka Ruohonen <jruohonen@iki.fi>
To: gnats-bugs@NetBSD.org
Cc: 
Subject: Re: kern/43504: No brightness adjusting support on HP Probook
Date: Fri, 25 Jun 2010 00:00:26 +0300

 On Thu, Jun 24, 2010 at 07:35:00PM +0000, demelier.david@gmail.com wrote:
 > Like all HP laptops (almost) I can't adjust the laptop brightness using fn
 > + f7 or f8 keys.
 > 
 > If I boot without ACPI, I can adjust it.

 To rule one out: can you adjust the brightness if you enable the wmihp(4)
 driver? I.e.

 wmihp*         at acpiwmibus?          # HP WMI mappings

 This is commented out by default on GENERIC kernels. Be sure to comment out
 the alternative in turn:

 #hpqlb*          at acpi?                # HP Quick Launch Buttons

 Otherwise we'll revisit this once the code for ACPI display devices has been
 merged (hopefully soon).

From: David DEMELIER <demelier.david@gmail.com>
To: gnats-bugs@netbsd.org
Cc: kern-bug-people@netbsd.org, gnats-admin@netbsd.org, netbsd-bugs@netbsd.org
Subject: Re: kern/43504: No brightness adjusting support on HP Probook
Date: Thu, 24 Jun 2010 23:29:59 +0200

 2010/6/24 Jukka Ruohonen <jruohonen@iki.fi>:
 > The following reply was made to PR kern/43504; it has been noted by GNATS=
 .
 >
 > From: Jukka Ruohonen <jruohonen@iki.fi>
 > To: gnats-bugs@NetBSD.org
 > Cc:
 > Subject: Re: kern/43504: No brightness adjusting support on HP Probook
 > Date: Fri, 25 Jun 2010 00:00:26 +0300
 >
 > =C2=A0On Thu, Jun 24, 2010 at 07:35:00PM +0000, demelier.david@gmail.com =
 wrote:
 > =C2=A0> Like all HP laptops (almost) I can't adjust the laptop brightness=
  using fn
 > =C2=A0> + f7 or f8 keys.
 > =C2=A0>
 > =C2=A0> If I boot without ACPI, I can adjust it.
 >
 > =C2=A0To rule one out: can you adjust the brightness if you enable the wm=
 ihp(4)
 > =C2=A0driver? I.e.
 >
 > =C2=A0wmihp* =C2=A0 =C2=A0 =C2=A0 =C2=A0 at acpiwmibus? =C2=A0 =C2=A0 =C2=
 =A0 =C2=A0 =C2=A0# HP WMI mappings
 >
 > =C2=A0This is commented out by default on GENERIC kernels. Be sure to com=
 ment out
 > =C2=A0the alternative in turn:
 >
 > =C2=A0#hpqlb* =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0at acpi? =C2=A0 =C2=A0 =
 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0# HP Quick Launch Buttons
 >
 > =C2=A0Otherwise we'll revisit this once the code for ACPI display devices=
  has been
 > =C2=A0merged (hopefully soon).
 >
 >

 Hi, I just added wmihp* at acpiwmibus? in the kernel config (in
 current hpqlb is uncommented) and it results in a panic :

 Because I can't save the backtrace can I attach a picture take from a
 digital camera ? (480K)

 --=20
 Demelier David

From: Jukka Ruohonen <jruohonen@iki.fi>
To: gnats-bugs@NetBSD.org
Cc: 
Subject: Re: kern/43504: No brightness adjusting support on HP Probook
Date: Fri, 25 Jun 2010 08:10:15 +0300

 On Thu, Jun 24, 2010 at 11:29:59PM +0200, David DEMELIER wrote:
 > Hi, I just added wmihp* at acpiwmibus? in the kernel config (in
 > current hpqlb is uncommented) and it results in a panic :

 Oh. That was unexpected.

 > Because I can't save the backtrace can I attach a picture take from a
 > digital camera ? (480K)

 I am not sure if that is possible.... If you have any place to put the
 picture, an URL link is fine. If you have another machine, it is also fine
 to copy the backtrace by hand.

From: David DEMELIER <demelier.david@gmail.com>
To: gnats-bugs@netbsd.org
Cc: 
Subject: Re: kern/43504: No brightness adjusting support on HP Probook
Date: Fri, 25 Jun 2010 09:07:33 +0200

 2010/6/25 Jukka Ruohonen <jruohonen@iki.fi>:
 > The following reply was made to PR kern/43504; it has been noted by GNATS=
 .
 >
 > From: Jukka Ruohonen <jruohonen@iki.fi>
 > To: gnats-bugs@NetBSD.org
 > Cc:
 > Subject: Re: kern/43504: No brightness adjusting support on HP Probook
 > Date: Fri, 25 Jun 2010 08:10:15 +0300
 >
 > =C2=A0On Thu, Jun 24, 2010 at 11:29:59PM +0200, David DEMELIER wrote:
 > =C2=A0> Hi, I just added wmihp* at acpiwmibus? in the kernel config (in
 > =C2=A0> current hpqlb is uncommented) and it results in a panic :
 >
 > =C2=A0Oh. That was unexpected.
 >
 > =C2=A0> Because I can't save the backtrace can I attach a picture take fr=
 om a
 > =C2=A0> digital camera ? (480K)
 >
 > =C2=A0I am not sure if that is possible.... If you have any place to put =
 the
 > =C2=A0picture, an URL link is fine. If you have another machine, it is al=
 so fine
 > =C2=A0to copy the backtrace by hand.
 >

 Yes okay, I uploaded right here :

 http://img37.imageshack.us/i/dsc00418xr.jpg/

 Cheers.

 --=20
 Demelier David

Responsible-Changed-From-To: kern-bug-people->jruoho
Responsible-Changed-By: jruoho@NetBSD.org
Responsible-Changed-When: Wed, 30 Jun 2010 08:01:22 +0000
Responsible-Changed-Why:
Can you try the revision 1.2 of wmi_hp.c?


From: Jukka Ruohonen <jruoho@netbsd.org>
To: gnats-bugs@gnats.NetBSD.org
Cc: 
Subject: PR/43504 CVS commit: src/sys/dev/acpi/wmi
Date: Wed, 30 Jun 2010 07:58:11 +0000

 Module Name:	src
 Committed By:	jruoho
 Date:		Wed Jun 30 07:58:11 UTC 2010

 Modified Files:
 	src/sys/dev/acpi/wmi: wmi_dell.c wmi_hp.c

 Log Message:
 Initialize pointers to NULL. Otherwise we may try to free garbage.
 Tries to address a panic reported by David Demelier in PR kern/43504.


 To generate a diff of this commit:
 cvs rdiff -u -r1.3 -r1.4 src/sys/dev/acpi/wmi/wmi_dell.c
 cvs rdiff -u -r1.1 -r1.2 src/sys/dev/acpi/wmi/wmi_hp.c

 Please note that diffs are not public domain; they are subject to the
 copyright notices on the relevant files.

From: David DEMELIER <demelier.david@gmail.com>
To: gnats-bugs@netbsd.org
Cc: jruoho@netbsd.org, gnats-admin@netbsd.org, netbsd-bugs@netbsd.org
Subject: Re: PR/43504 CVS commit: src/sys/dev/acpi/wmi
Date: Wed, 30 Jun 2010 11:23:44 +0200

 2010/6/30 Jukka Ruohonen <jruoho@netbsd.org>:
 > The following reply was made to PR kern/43504; it has been noted by GNATS=
 .
 >
 > From: Jukka Ruohonen <jruoho@netbsd.org>
 > To: gnats-bugs@gnats.NetBSD.org
 > Cc:
 > Subject: PR/43504 CVS commit: src/sys/dev/acpi/wmi
 > Date: Wed, 30 Jun 2010 07:58:11 +0000
 >
 > =C2=A0Module Name: =C2=A0 src
 > =C2=A0Committed By: =C2=A0jruoho
 > =C2=A0Date: =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0Wed Jun 30 07:58:11 UTC 201=
 0
 >
 > =C2=A0Modified Files:
 > =C2=A0 =C2=A0 =C2=A0 =C2=A0src/sys/dev/acpi/wmi: wmi_dell.c wmi_hp.c
 >
 > =C2=A0Log Message:
 > =C2=A0Initialize pointers to NULL. Otherwise we may try to free garbage.
 > =C2=A0Tries to address a panic reported by David Demelier in PR kern/4350=
 4.
 >
 >
 > =C2=A0To generate a diff of this commit:
 > =C2=A0cvs rdiff -u -r1.3 -r1.4 src/sys/dev/acpi/wmi/wmi_dell.c
 > =C2=A0cvs rdiff -u -r1.1 -r1.2 src/sys/dev/acpi/wmi/wmi_hp.c
 >
 > =C2=A0Please note that diffs are not public domain; they are subject to t=
 he
 > =C2=A0copyright notices on the relevant files.
 >

 Tested, it does not panic anymore but it doesn't print any output and
 does not adjust brightness. I guess this laptop is impossible to
 maintain.

 --=20
 Demelier David

From: Jukka Ruohonen <jruohonen@iki.fi>
To: gnats-bugs@NetBSD.org
Cc: 
Subject: Re: PR/43504 CVS commit: src/sys/dev/acpi/wmi
Date: Wed, 30 Jun 2010 12:38:21 +0300

 On Wed, Jun 30, 2010 at 09:25:02AM +0000, David DEMELIER wrote:
 >  Tested, it does not panic anymore but it doesn't print any output and
 >  does not adjust brightness. I guess this laptop is impossible to
 >  maintain.

 Well this is sort-of expected as we do not yet have a driver for ACPI
 display devices.

 But if you have time and interest, boot with "-v" and "-x" (see
 boothowto(9)), press various hot and function keys, and if any output
 appears in dmesg, write the mappings down with a paper and pencil. That is,
 most of the hotkeys for wmihp(4) are still unknown, and this so-called "WMI"
 is pretty much undocumented, non-standardized, black magic.

 Some functionality may appear also in the envstat(8) output.

Responsible-Changed-From-To: jruoho->kern-bug-people
Responsible-Changed-By: dholland@NetBSD.org
Responsible-Changed-When: Sun, 20 Jan 2019 08:02:03 +0000
Responsible-Changed-Why:
Reset responsible field for retired developer.


From: David Demelier <demelier.david@gmail.com>
To: gnats-bugs@NetBSD.org, kern-bug-people@netbsd.org, jruoho@NetBSD.org,
 netbsd-bugs@netbsd.org, gnats-admin@netbsd.org, dholland@NetBSD.org
Cc: 
Subject: Re: kern/43504 (No brightness adjusting support on HP Probook)
Date: Wed, 23 Jan 2019 10:00:54 +0100

 Le 20/01/2019 à 09:02, dholland@NetBSD.org a écrit :
 > Synopsis: No brightness adjusting support on HP Probook
 > 
 > Responsible-Changed-From-To: jruoho->kern-bug-people
 > Responsible-Changed-By: dholland@NetBSD.org
 > Responsible-Changed-When: Sun, 20 Jan 2019 08:02:03 +0000
 > Responsible-Changed-Why:
 > Reset responsible field for retired developer.

 Hi,

 I don't have this machine running anymore. You can close this bug.

From: Ryo ONODERA <ryo@tetera.org>
To: gnats-bugs@NetBSD.org
Cc: 
Subject: Re: kern/43504 (No brightness adjusting support on HP Probook)
Date: Wed, 23 Jan 2019 21:48:19 +0900 (JST)

 Hi,

 I have HP machines that has same problem.
 I can provide any information about the machines.

 Could anyone take a look at this problem?

 --
 Ryo ONODERA // ryo@tetera.org
 PGP fingerprint = 82A2 DC91 76E0 A10A 8ABB  FD1B F404 27FA C7D1 15F3

From: David Holland <dholland-bugs@netbsd.org>
To: gnats-bugs@NetBSD.org
Cc: 
Subject: Re: kern/43504 (No brightness adjusting support on HP Probook)
Date: Wed, 23 Jan 2019 16:19:05 +0000

 On Wed, Jan 23, 2019 at 12:50:00PM +0000, Ryo ONODERA wrote:
  >  Hi,
  >  
  >  I have HP machines that has same problem.
  >  I can provide any information about the machines.
  >  
  >  Could anyone take a look at this problem?

 I know nothing about the innards but I'd guess that the list of
 mappings jruoho@ wanted would still be helpful.

 -- 
 David A. Holland
 dholland@netbsd.org

>Unformatted:

NetBSD Home
NetBSD PR Database Search

(Contact us) $NetBSD: query-full-pr,v 1.43 2018/01/16 07:36:43 maya Exp $
$NetBSD: gnats_config.sh,v 1.9 2014/08/02 14:16:04 spz Exp $
Copyright © 1994-2017 The NetBSD Foundation, Inc. ALL RIGHTS RESERVED.