* [Kernel 5.1] ACPI_DEBUG messages without CONFIG_ACPI_DEBUG being set
@ 2019-05-06 23:34 Gabriel C
2019-05-07 7:01 ` Rafael J. Wysocki
0 siblings, 1 reply; 9+ messages in thread
From: Gabriel C @ 2019-05-06 23:34 UTC (permalink / raw)
To: linux-acpi; +Cc: LKML
Hello,
while testing kernel-5.1 I get on one of my Lenovo Laptops very
strange 'ACPI Debug:' messages.
After some grepping I realized these are Debug messages from DSDT ,
however my kernel does
not have ACPI_DEBUG enabled.
I found out the module triggering this, on this Laptop is
ideapad_laptop , but looking at the code
I cannot see what would causes that.
Also on the same Laptop with any 5.0.X kernels I cannot see these.
~$ grep -i ACPI_DEBUG /boot/config-5.1-fw1
# CONFIG_ACPI_DEBUGGER is not set
# CONFIG_ACPI_DEBUG is not set
# CONFIG_THINKPAD_ACPI_DEBUGFACILITIES is not set
# CONFIG_THINKPAD_ACPI_DEBUG is not set
.. dmesg ..
...
[ 68.020812] calling ideapad_acpi_driver_init+0x0/0x1000
[ideapad_laptop] @ 1322
[ 68.026708] input: Ideapad extra buttons as
/devices/pci0000:00/0000:00:1f.0/PNP0C09:00/VPC2004:00/input/input16
[ 68.038236] ACPI Debug: "=====QUERY_64====="
[ 68.050232] ACPI Debug: "=====QUERY_65====="
[ 68.060218] ACPI Debug: "=====QUERY_64====="
[ 68.092216] probe of VPC2004:00 returned 1 after 71386 usecs
[ 68.092245] initcall ideapad_acpi_driver_init+0x0/0x1000
[ideapad_laptop] returned 0 after 69751 usecssg
...
These =====QUERY_XX===== messages are from DSDT:
~/acpi$ grep QUERY dsdt.dsl
Debug = "=====QUERY_11====="
Debug = "=====QUERY_12====="
Debug = "=====QUERY_24====="
Debug = "=====QUERY_25====="
Debug = "=====QUERY_37====="
Debug = "=====QUERY_38====="
Debug = "=====QUERY_64====="
Debug = "=====QUERY_65====="
Also this is the code from DSDT for QUERY 64 and 65:
...
Method (_Q64, 0, NotSerialized) // _Qxx: EC Query
{
Debug = "=====QUERY_64====="
If ((OSYS == 0x07D9))
{
If (((WLEX == One) & (WLAT == One)))
{
SGOV (0x02040005, One)
}
Else
{
SGOV (0x02040005, Zero)
}
}
}
Method (_Q65, 0, NotSerialized) // _Qxx: EC Query
{
Debug = "=====QUERY_65====="
If ((OSYS == 0x07D9))
{
If (((BTEX == One) & (BTAT == One)))
{
SGOV (0x0202000B, One)
}
Else
{
SGOV (0x0202000B, Zero)
}
}
}
...
Any idea what would cause this ?
BR,
Gabriel C
^ permalink raw reply [flat|nested] 9+ messages in thread
* Re: [Kernel 5.1] ACPI_DEBUG messages without CONFIG_ACPI_DEBUG being set
2019-05-06 23:34 [Kernel 5.1] ACPI_DEBUG messages without CONFIG_ACPI_DEBUG being set Gabriel C
@ 2019-05-07 7:01 ` Rafael J. Wysocki
2019-05-07 7:32 ` Gabriel C
0 siblings, 1 reply; 9+ messages in thread
From: Rafael J. Wysocki @ 2019-05-07 7:01 UTC (permalink / raw)
To: Gabriel C; +Cc: ACPI Devel Maling List, LKML, Schmauss, Erik
+Erik
On Tue, May 7, 2019 at 1:33 AM Gabriel C <nix.or.die@gmail.com> wrote:
>
> Hello,
>
> while testing kernel-5.1 I get on one of my Lenovo Laptops very
> strange 'ACPI Debug:' messages.
>
> After some grepping I realized these are Debug messages from DSDT ,
> however my kernel does
> not have ACPI_DEBUG enabled.
>
> I found out the module triggering this, on this Laptop is
> ideapad_laptop , but looking at the code
> I cannot see what would causes that.
>
> Also on the same Laptop with any 5.0.X kernels I cannot see these.
>
>
> ~$ grep -i ACPI_DEBUG /boot/config-5.1-fw1
> # CONFIG_ACPI_DEBUGGER is not set
> # CONFIG_ACPI_DEBUG is not set
> # CONFIG_THINKPAD_ACPI_DEBUGFACILITIES is not set
> # CONFIG_THINKPAD_ACPI_DEBUG is not set
>
> .. dmesg ..
> ...
> [ 68.020812] calling ideapad_acpi_driver_init+0x0/0x1000
> [ideapad_laptop] @ 1322
> [ 68.026708] input: Ideapad extra buttons as
> /devices/pci0000:00/0000:00:1f.0/PNP0C09:00/VPC2004:00/input/input16
> [ 68.038236] ACPI Debug: "=====QUERY_64====="
> [ 68.050232] ACPI Debug: "=====QUERY_65====="
> [ 68.060218] ACPI Debug: "=====QUERY_64====="
> [ 68.092216] probe of VPC2004:00 returned 1 after 71386 usecs
> [ 68.092245] initcall ideapad_acpi_driver_init+0x0/0x1000
> [ideapad_laptop] returned 0 after 69751 usecssg
>
> ...
>
> These =====QUERY_XX===== messages are from DSDT:
>
> ~/acpi$ grep QUERY dsdt.dsl
> Debug = "=====QUERY_11====="
> Debug = "=====QUERY_12====="
> Debug = "=====QUERY_24====="
> Debug = "=====QUERY_25====="
> Debug = "=====QUERY_37====="
> Debug = "=====QUERY_38====="
> Debug = "=====QUERY_64====="
> Debug = "=====QUERY_65====="
>
> Also this is the code from DSDT for QUERY 64 and 65:
>
> ...
> Method (_Q64, 0, NotSerialized) // _Qxx: EC Query
> {
> Debug = "=====QUERY_64====="
> If ((OSYS == 0x07D9))
> {
> If (((WLEX == One) & (WLAT == One)))
> {
> SGOV (0x02040005, One)
> }
> Else
> {
> SGOV (0x02040005, Zero)
> }
> }
> }
>
> Method (_Q65, 0, NotSerialized) // _Qxx: EC Query
> {
> Debug = "=====QUERY_65====="
> If ((OSYS == 0x07D9))
> {
> If (((BTEX == One) & (BTAT == One)))
> {
> SGOV (0x0202000B, One)
> }
> Else
> {
> SGOV (0x0202000B, Zero)
> }
> }
> }
>
> ...
>
>
> Any idea what would cause this ?
>
> BR,
>
> Gabriel C
^ permalink raw reply [flat|nested] 9+ messages in thread
* Re: [Kernel 5.1] ACPI_DEBUG messages without CONFIG_ACPI_DEBUG being set
2019-05-07 7:01 ` Rafael J. Wysocki
@ 2019-05-07 7:32 ` Gabriel C
2019-05-07 8:34 ` Rafael J. Wysocki
0 siblings, 1 reply; 9+ messages in thread
From: Gabriel C @ 2019-05-07 7:32 UTC (permalink / raw)
To: Rafael J. Wysocki; +Cc: ACPI Devel Maling List, LKML, Schmauss, Erik
Am Di., 7. Mai 2019 um 09:01 Uhr schrieb Rafael J. Wysocki <rafael@kernel.org>:
>
Hello Rafael , Erik
> +Erik
>
> On Tue, May 7, 2019 at 1:33 AM Gabriel C <nix.or.die@gmail.com> wrote:
> >
> > Hello,
> >
> > while testing kernel-5.1 I get on one of my Lenovo Laptops very
> > strange 'ACPI Debug:' messages.
> >
> > After some grepping I realized these are Debug messages from DSDT ,
> > however my kernel does
> > not have ACPI_DEBUG enabled.
> >
> > I found out the module triggering this, on this Laptop is
> > ideapad_laptop , but looking at the code
> > I cannot see what would causes that.
> >
> > Also on the same Laptop with any 5.0.X kernels I cannot see these.
> >
> >
> > ~$ grep -i ACPI_DEBUG /boot/config-5.1-fw1
> > # CONFIG_ACPI_DEBUGGER is not set
> > # CONFIG_ACPI_DEBUG is not set
> > # CONFIG_THINKPAD_ACPI_DEBUGFACILITIES is not set
> > # CONFIG_THINKPAD_ACPI_DEBUG is not set
> >
> > .. dmesg ..
> > ...
> > [ 68.020812] calling ideapad_acpi_driver_init+0x0/0x1000
> > [ideapad_laptop] @ 1322
> > [ 68.026708] input: Ideapad extra buttons as
> > /devices/pci0000:00/0000:00:1f.0/PNP0C09:00/VPC2004:00/input/input16
> > [ 68.038236] ACPI Debug: "=====QUERY_64====="
> > [ 68.050232] ACPI Debug: "=====QUERY_65====="
> > [ 68.060218] ACPI Debug: "=====QUERY_64====="
> > [ 68.092216] probe of VPC2004:00 returned 1 after 71386 usecs
> > [ 68.092245] initcall ideapad_acpi_driver_init+0x0/0x1000
> > [ideapad_laptop] returned 0 after 69751 usecssg
> >
> > ...
> >
> > These =====QUERY_XX===== messages are from DSDT:
> >
> > ~/acpi$ grep QUERY dsdt.dsl
> > Debug = "=====QUERY_11====="
> > Debug = "=====QUERY_12====="
> > Debug = "=====QUERY_24====="
> > Debug = "=====QUERY_25====="
> > Debug = "=====QUERY_37====="
> > Debug = "=====QUERY_38====="
> > Debug = "=====QUERY_64====="
> > Debug = "=====QUERY_65====="
> >
> > Also this is the code from DSDT for QUERY 64 and 65:
> >
> > ...
> > Method (_Q64, 0, NotSerialized) // _Qxx: EC Query
> > {
> > Debug = "=====QUERY_64====="
> > If ((OSYS == 0x07D9))
> > {
> > If (((WLEX == One) & (WLAT == One)))
> > {
> > SGOV (0x02040005, One)
> > }
> > Else
> > {
> > SGOV (0x02040005, Zero)
> > }
> > }
> > }
> >
> > Method (_Q65, 0, NotSerialized) // _Qxx: EC Query
> > {
> > Debug = "=====QUERY_65====="
> > If ((OSYS == 0x07D9))
> > {
> > If (((BTEX == One) & (BTAT == One)))
> > {
> > SGOV (0x0202000B, One)
> > }
> > Else
> > {
> > SGOV (0x0202000B, Zero)
> > }
> > }
> > }
> >
> > ...
> >
> >
> > Any idea what would cause this ?
I think I found what is causing that.
Commit aa9aaa4d61c0048d3faad056893cd7860bbc084c is moving
definition of Linux's ACPI_DEBUG_DEFAULT to incude/acpi/platform/aclinux.h ,
however include/acpi/acpi.h didn't got updated and ACPI_DEBUG_DEFAULT now
being always defined as (ACPI_LV_INIT | ACPI_LV_DEBUG_OBJECT |
ACPI_LV_EVALUATION | ACPI_LV_REPAIR)
BR,
Gabriel C
^ permalink raw reply [flat|nested] 9+ messages in thread
* Re: [Kernel 5.1] ACPI_DEBUG messages without CONFIG_ACPI_DEBUG being set
2019-05-07 7:32 ` Gabriel C
@ 2019-05-07 8:34 ` Rafael J. Wysocki
2019-05-07 9:33 ` Gabriel C
0 siblings, 1 reply; 9+ messages in thread
From: Rafael J. Wysocki @ 2019-05-07 8:34 UTC (permalink / raw)
To: Gabriel C; +Cc: Rafael J. Wysocki, ACPI Devel Maling List, LKML, Schmauss, Erik
On Tue, May 7, 2019 at 9:31 AM Gabriel C <nix.or.die@gmail.com> wrote:
>
> Am Di., 7. Mai 2019 um 09:01 Uhr schrieb Rafael J. Wysocki <rafael@kernel.org>:
> >
> Hello Rafael , Erik
>
> > +Erik
> >
> > On Tue, May 7, 2019 at 1:33 AM Gabriel C <nix.or.die@gmail.com> wrote:
> > >
> > > Hello,
> > >
> > > while testing kernel-5.1 I get on one of my Lenovo Laptops very
> > > strange 'ACPI Debug:' messages.
> > >
> > > After some grepping I realized these are Debug messages from DSDT ,
> > > however my kernel does
> > > not have ACPI_DEBUG enabled.
> > >
> > > I found out the module triggering this, on this Laptop is
> > > ideapad_laptop , but looking at the code
> > > I cannot see what would causes that.
> > >
> > > Also on the same Laptop with any 5.0.X kernels I cannot see these.
> > >
> > >
> > > ~$ grep -i ACPI_DEBUG /boot/config-5.1-fw1
> > > # CONFIG_ACPI_DEBUGGER is not set
> > > # CONFIG_ACPI_DEBUG is not set
> > > # CONFIG_THINKPAD_ACPI_DEBUGFACILITIES is not set
> > > # CONFIG_THINKPAD_ACPI_DEBUG is not set
> > >
> > > .. dmesg ..
> > > ...
> > > [ 68.020812] calling ideapad_acpi_driver_init+0x0/0x1000
> > > [ideapad_laptop] @ 1322
> > > [ 68.026708] input: Ideapad extra buttons as
> > > /devices/pci0000:00/0000:00:1f.0/PNP0C09:00/VPC2004:00/input/input16
> > > [ 68.038236] ACPI Debug: "=====QUERY_64====="
> > > [ 68.050232] ACPI Debug: "=====QUERY_65====="
> > > [ 68.060218] ACPI Debug: "=====QUERY_64====="
> > > [ 68.092216] probe of VPC2004:00 returned 1 after 71386 usecs
> > > [ 68.092245] initcall ideapad_acpi_driver_init+0x0/0x1000
> > > [ideapad_laptop] returned 0 after 69751 usecssg
> > >
> > > ...
> > >
> > > These =====QUERY_XX===== messages are from DSDT:
> > >
> > > ~/acpi$ grep QUERY dsdt.dsl
> > > Debug = "=====QUERY_11====="
> > > Debug = "=====QUERY_12====="
> > > Debug = "=====QUERY_24====="
> > > Debug = "=====QUERY_25====="
> > > Debug = "=====QUERY_37====="
> > > Debug = "=====QUERY_38====="
> > > Debug = "=====QUERY_64====="
> > > Debug = "=====QUERY_65====="
> > >
> > > Also this is the code from DSDT for QUERY 64 and 65:
> > >
> > > ...
> > > Method (_Q64, 0, NotSerialized) // _Qxx: EC Query
> > > {
> > > Debug = "=====QUERY_64====="
> > > If ((OSYS == 0x07D9))
> > > {
> > > If (((WLEX == One) & (WLAT == One)))
> > > {
> > > SGOV (0x02040005, One)
> > > }
> > > Else
> > > {
> > > SGOV (0x02040005, Zero)
> > > }
> > > }
> > > }
> > >
> > > Method (_Q65, 0, NotSerialized) // _Qxx: EC Query
> > > {
> > > Debug = "=====QUERY_65====="
> > > If ((OSYS == 0x07D9))
> > > {
> > > If (((BTEX == One) & (BTAT == One)))
> > > {
> > > SGOV (0x0202000B, One)
> > > }
> > > Else
> > > {
> > > SGOV (0x0202000B, Zero)
> > > }
> > > }
> > > }
> > >
> > > ...
> > >
> > >
> > > Any idea what would cause this ?
>
> I think I found what is causing that.
>
> Commit aa9aaa4d61c0048d3faad056893cd7860bbc084c is moving
> definition of Linux's ACPI_DEBUG_DEFAULT to incude/acpi/platform/aclinux.h ,
> however include/acpi/acpi.h didn't got updated and ACPI_DEBUG_DEFAULT now
> being always defined as (ACPI_LV_INIT | ACPI_LV_DEBUG_OBJECT |
> ACPI_LV_EVALUATION | ACPI_LV_REPAIR)
I'm not sure what you mean by this.
include/acpi/acpi.h should get the right definitions through
include/acpi/platform/acenv.h and include/acpi/acoutput.h that are
included by it.
It looks like incude/acpi/platform/aclinux.h has not been included by
include/acpi/platform/acenv.h for some reason when building this
module.
^ permalink raw reply [flat|nested] 9+ messages in thread
* Re: [Kernel 5.1] ACPI_DEBUG messages without CONFIG_ACPI_DEBUG being set
2019-05-07 8:34 ` Rafael J. Wysocki
@ 2019-05-07 9:33 ` Gabriel C
2019-05-07 18:46 ` Schmauss, Erik
0 siblings, 1 reply; 9+ messages in thread
From: Gabriel C @ 2019-05-07 9:33 UTC (permalink / raw)
To: Rafael J. Wysocki; +Cc: ACPI Devel Maling List, LKML, Schmauss, Erik
Am Di., 7. Mai 2019 um 10:35 Uhr schrieb Rafael J. Wysocki <rafael@kernel.org>:
>
> On Tue, May 7, 2019 at 9:31 AM Gabriel C <nix.or.die@gmail.com> wrote:
> >
> > Am Di., 7. Mai 2019 um 09:01 Uhr schrieb Rafael J. Wysocki <rafael@kernel.org>:
> > >
> > Hello Rafael , Erik
> >
> > > +Erik
> > >
> > > On Tue, May 7, 2019 at 1:33 AM Gabriel C <nix.or.die@gmail.com> wrote:
> > > >
> > > > Hello,
> > > >
> > > > while testing kernel-5.1 I get on one of my Lenovo Laptops very
> > > > strange 'ACPI Debug:' messages.
> > > >
> > > > After some grepping I realized these are Debug messages from DSDT ,
> > > > however my kernel does
> > > > not have ACPI_DEBUG enabled.
> > > >
> > > > I found out the module triggering this, on this Laptop is
> > > > ideapad_laptop , but looking at the code
> > > > I cannot see what would causes that.
> > > >
> > > > Also on the same Laptop with any 5.0.X kernels I cannot see these.
> > > >
> > > >
> > > > ~$ grep -i ACPI_DEBUG /boot/config-5.1-fw1
> > > > # CONFIG_ACPI_DEBUGGER is not set
> > > > # CONFIG_ACPI_DEBUG is not set
> > > > # CONFIG_THINKPAD_ACPI_DEBUGFACILITIES is not set
> > > > # CONFIG_THINKPAD_ACPI_DEBUG is not set
> > > >
> > > > .. dmesg ..
> > > > ...
> > > > [ 68.020812] calling ideapad_acpi_driver_init+0x0/0x1000
> > > > [ideapad_laptop] @ 1322
> > > > [ 68.026708] input: Ideapad extra buttons as
> > > > /devices/pci0000:00/0000:00:1f.0/PNP0C09:00/VPC2004:00/input/input16
> > > > [ 68.038236] ACPI Debug: "=====QUERY_64====="
> > > > [ 68.050232] ACPI Debug: "=====QUERY_65====="
> > > > [ 68.060218] ACPI Debug: "=====QUERY_64====="
> > > > [ 68.092216] probe of VPC2004:00 returned 1 after 71386 usecs
> > > > [ 68.092245] initcall ideapad_acpi_driver_init+0x0/0x1000
> > > > [ideapad_laptop] returned 0 after 69751 usecssg
> > > >
> > > > ...
> > > >
> > > > These =====QUERY_XX===== messages are from DSDT:
> > > >
> > > > ~/acpi$ grep QUERY dsdt.dsl
> > > > Debug = "=====QUERY_11====="
> > > > Debug = "=====QUERY_12====="
> > > > Debug = "=====QUERY_24====="
> > > > Debug = "=====QUERY_25====="
> > > > Debug = "=====QUERY_37====="
> > > > Debug = "=====QUERY_38====="
> > > > Debug = "=====QUERY_64====="
> > > > Debug = "=====QUERY_65====="
> > > >
> > > > Also this is the code from DSDT for QUERY 64 and 65:
> > > >
> > > > ...
> > > > Method (_Q64, 0, NotSerialized) // _Qxx: EC Query
> > > > {
> > > > Debug = "=====QUERY_64====="
> > > > If ((OSYS == 0x07D9))
> > > > {
> > > > If (((WLEX == One) & (WLAT == One)))
> > > > {
> > > > SGOV (0x02040005, One)
> > > > }
> > > > Else
> > > > {
> > > > SGOV (0x02040005, Zero)
> > > > }
> > > > }
> > > > }
> > > >
> > > > Method (_Q65, 0, NotSerialized) // _Qxx: EC Query
> > > > {
> > > > Debug = "=====QUERY_65====="
> > > > If ((OSYS == 0x07D9))
> > > > {
> > > > If (((BTEX == One) & (BTAT == One)))
> > > > {
> > > > SGOV (0x0202000B, One)
> > > > }
> > > > Else
> > > > {
> > > > SGOV (0x0202000B, Zero)
> > > > }
> > > > }
> > > > }
> > > >
> > > > ...
> > > >
> > > >
> > > > Any idea what would cause this ?
> >
> > I think I found what is causing that.
> >
> > Commit aa9aaa4d61c0048d3faad056893cd7860bbc084c is moving
> > definition of Linux's ACPI_DEBUG_DEFAULT to incude/acpi/platform/aclinux.h ,
> > however include/acpi/acpi.h didn't got updated and ACPI_DEBUG_DEFAULT now
> > being always defined as (ACPI_LV_INIT | ACPI_LV_DEBUG_OBJECT |
> > ACPI_LV_EVALUATION | ACPI_LV_REPAIR)
>
> I'm not sure what you mean by this.
>
> include/acpi/acpi.h should get the right definitions through
> include/acpi/platform/acenv.h and include/acpi/acoutput.h that are
> included by it.
>
Hmm yes should , I missed that include. However it does not work.
I've just put back old code back into drivers/acpi/bus.c to test
and all working now fine without any other changes.
> It looks like incude/acpi/platform/aclinux.h has not been included by
> include/acpi/platform/acenv.h for some reason when building this
> module.
If this one getting it wrong all will do so , they all just include
<linux/acpi.h>.
Probably not so much users will notice that , I just did because my
BIOS vendor forgot DEBUG around.
^ permalink raw reply [flat|nested] 9+ messages in thread
* RE: [Kernel 5.1] ACPI_DEBUG messages without CONFIG_ACPI_DEBUG being set
2019-05-07 9:33 ` Gabriel C
@ 2019-05-07 18:46 ` Schmauss, Erik
2019-05-07 19:06 ` Gabriel C
0 siblings, 1 reply; 9+ messages in thread
From: Schmauss, Erik @ 2019-05-07 18:46 UTC (permalink / raw)
To: Gabriel C, Rafael J. Wysocki; +Cc: ACPI Devel Maling List, LKML, Moore, Robert
> -----Original Message-----
> From: Gabriel C [mailto:nix.or.die@gmail.com]
> Sent: Tuesday, May 7, 2019 2:33 AM
> To: Rafael J. Wysocki <rafael@kernel.org>
> Cc: ACPI Devel Maling List <linux-acpi@vger.kernel.org>; LKML <linux-
> kernel@vger.kernel.org>; Schmauss, Erik <erik.schmauss@intel.com>
> Subject: Re: [Kernel 5.1] ACPI_DEBUG messages without
> CONFIG_ACPI_DEBUG being set
>
> Am Di., 7. Mai 2019 um 10:35 Uhr schrieb Rafael J. Wysocki
> <rafael@kernel.org>:
> >
> > On Tue, May 7, 2019 at 9:31 AM Gabriel C <nix.or.die@gmail.com> wrote:
> > >
> > > Am Di., 7. Mai 2019 um 09:01 Uhr schrieb Rafael J. Wysocki
> <rafael@kernel.org>:
> > > >
> > > Hello Rafael , Erik
> > >
> > > > +Erik
> > > >
> > > > On Tue, May 7, 2019 at 1:33 AM Gabriel C <nix.or.die@gmail.com>
> wrote:
> > > > >
> > > > > Hello,
> > > > >
> > > > > while testing kernel-5.1 I get on one of my Lenovo Laptops very
> > > > > strange 'ACPI Debug:' messages.
> > > > >
> > > > > After some grepping I realized these are Debug messages from
> > > > > DSDT , however my kernel does not have ACPI_DEBUG enabled.
> > > > >
> > > > > I found out the module triggering this, on this Laptop is
> > > > > ideapad_laptop , but looking at the code I cannot see what would
> > > > > causes that.
> > > > >
> > > > > Also on the same Laptop with any 5.0.X kernels I cannot see these.
> > > > >
> > > > >
> > > > > ~$ grep -i ACPI_DEBUG /boot/config-5.1-fw1 #
> > > > > CONFIG_ACPI_DEBUGGER is not set # CONFIG_ACPI_DEBUG is not
> set #
> > > > > CONFIG_THINKPAD_ACPI_DEBUGFACILITIES is not set #
> > > > > CONFIG_THINKPAD_ACPI_DEBUG is not set
> > > > >
> > > > > .. dmesg ..
> > > > > ...
> > > > > [ 68.020812] calling ideapad_acpi_driver_init+0x0/0x1000
> > > > > [ideapad_laptop] @ 1322
> > > > > [ 68.026708] input: Ideapad extra buttons as
> > > > >
> /devices/pci0000:00/0000:00:1f.0/PNP0C09:00/VPC2004:00/input/input16
> > > > > [ 68.038236] ACPI Debug: "=====QUERY_64====="
> > > > > [ 68.050232] ACPI Debug: "=====QUERY_65====="
> > > > > [ 68.060218] ACPI Debug: "=====QUERY_64====="
> > > > > [ 68.092216] probe of VPC2004:00 returned 1 after 71386 usecs
> > > > > [ 68.092245] initcall ideapad_acpi_driver_init+0x0/0x1000
> > > > > [ideapad_laptop] returned 0 after 69751 usecssg
> > > > >
> > > > > ...
> > > > >
> > > > > These =====QUERY_XX===== messages are from DSDT:
> > > > >
> > > > > ~/acpi$ grep QUERY dsdt.dsl
> > > > > Debug = "=====QUERY_11====="
> > > > > Debug = "=====QUERY_12====="
> > > > > Debug = "=====QUERY_24====="
> > > > > Debug = "=====QUERY_25====="
> > > > > Debug = "=====QUERY_37====="
> > > > > Debug = "=====QUERY_38====="
> > > > > Debug = "=====QUERY_64====="
> > > > > Debug = "=====QUERY_65====="
> > > > >
> > > > > Also this is the code from DSDT for QUERY 64 and 65:
> > > > >
> > > > > ...
> > > > > Method (_Q64, 0, NotSerialized) // _Qxx: EC Query
> > > > > {
> > > > > Debug = "=====QUERY_64====="
> > > > > If ((OSYS == 0x07D9))
> > > > > {
> > > > > If (((WLEX == One) & (WLAT == One)))
> > > > > {
> > > > > SGOV (0x02040005, One)
> > > > > }
> > > > > Else
> > > > > {
> > > > > SGOV (0x02040005, Zero)
> > > > > }
> > > > > }
> > > > > }
> > > > >
> > > > > Method (_Q65, 0, NotSerialized) // _Qxx: EC Query
> > > > > {
> > > > > Debug = "=====QUERY_65====="
> > > > > If ((OSYS == 0x07D9))
> > > > > {
> > > > > If (((BTEX == One) & (BTAT == One)))
> > > > > {
> > > > > SGOV (0x0202000B, One)
> > > > > }
> > > > > Else
> > > > > {
> > > > > SGOV (0x0202000B, Zero)
> > > > > }
> > > > > }
> > > > > }
> > > > >
> > > > > ...
> > > > >
> > > > >
> > > > > Any idea what would cause this ?
> > >
> > > I think I found what is causing that.
> > >
> > > Commit aa9aaa4d61c0048d3faad056893cd7860bbc084c is moving
> definition
> > > of Linux's ACPI_DEBUG_DEFAULT to incude/acpi/platform/aclinux.h ,
> > > however include/acpi/acpi.h didn't got updated and
> > > ACPI_DEBUG_DEFAULT now being always defined as (ACPI_LV_INIT |
> > > ACPI_LV_DEBUG_OBJECT | ACPI_LV_EVALUATION | ACPI_LV_REPAIR)
> >
> > I'm not sure what you mean by this.
> >
> > include/acpi/acpi.h should get the right definitions through
> > include/acpi/platform/acenv.h and include/acpi/acoutput.h that are
> > included by it.
> >
>
> Hmm yes should , I missed that include. However it does not work.
>
> I've just put back old code back into drivers/acpi/bus.c to test and all working
> now fine without any other changes.
>
> > It looks like incude/acpi/platform/aclinux.h has not been included by
> > include/acpi/platform/acenv.h for some reason when building this
> > module.
>
> If this one getting it wrong all will do so , they all just include <linux/acpi.h>.
>
> Probably not so much users will notice that , I just did because my BIOS
> vendor forgot DEBUG around.
Do you still get this issue after doing make clean and rebuilding your kernel?
^ permalink raw reply [flat|nested] 9+ messages in thread
* Re: [Kernel 5.1] ACPI_DEBUG messages without CONFIG_ACPI_DEBUG being set
2019-05-07 18:46 ` Schmauss, Erik
@ 2019-05-07 19:06 ` Gabriel C
2019-05-07 20:17 ` Schmauss, Erik
0 siblings, 1 reply; 9+ messages in thread
From: Gabriel C @ 2019-05-07 19:06 UTC (permalink / raw)
To: Schmauss, Erik
Cc: Rafael J. Wysocki, ACPI Devel Maling List, LKML, Moore, Robert
Am Di., 7. Mai 2019 um 20:46 Uhr schrieb Schmauss, Erik
<erik.schmauss@intel.com>:
>
>
>
> > -----Original Message-----
> > From: Gabriel C [mailto:nix.or.die@gmail.com]
> > Sent: Tuesday, May 7, 2019 2:33 AM
> > To: Rafael J. Wysocki <rafael@kernel.org>
> > Cc: ACPI Devel Maling List <linux-acpi@vger.kernel.org>; LKML <linux-
> > kernel@vger.kernel.org>; Schmauss, Erik <erik.schmauss@intel.com>
> > Subject: Re: [Kernel 5.1] ACPI_DEBUG messages without
> > CONFIG_ACPI_DEBUG being set
> >
> > Am Di., 7. Mai 2019 um 10:35 Uhr schrieb Rafael J. Wysocki
> > <rafael@kernel.org>:
> > >
> > > On Tue, May 7, 2019 at 9:31 AM Gabriel C <nix.or.die@gmail.com> wrote:
> > > >
> > > > Am Di., 7. Mai 2019 um 09:01 Uhr schrieb Rafael J. Wysocki
> > <rafael@kernel.org>:
> > > > >
> > > > Hello Rafael , Erik
> > > >
> > > > > +Erik
> > > > >
> > > > > On Tue, May 7, 2019 at 1:33 AM Gabriel C <nix.or.die@gmail.com>
> > wrote:
> > > > > >
> > > > > > Hello,
> > > > > >
> > > > > > while testing kernel-5.1 I get on one of my Lenovo Laptops very
> > > > > > strange 'ACPI Debug:' messages.
> > > > > >
> > > > > > After some grepping I realized these are Debug messages from
> > > > > > DSDT , however my kernel does not have ACPI_DEBUG enabled.
> > > > > >
> > > > > > I found out the module triggering this, on this Laptop is
> > > > > > ideapad_laptop , but looking at the code I cannot see what would
> > > > > > causes that.
> > > > > >
> > > > > > Also on the same Laptop with any 5.0.X kernels I cannot see these.
> > > > > >
> > > > > >
> > > > > > ~$ grep -i ACPI_DEBUG /boot/config-5.1-fw1 #
> > > > > > CONFIG_ACPI_DEBUGGER is not set # CONFIG_ACPI_DEBUG is not
> > set #
> > > > > > CONFIG_THINKPAD_ACPI_DEBUGFACILITIES is not set #
> > > > > > CONFIG_THINKPAD_ACPI_DEBUG is not set
> > > > > >
> > > > > > .. dmesg ..
> > > > > > ...
> > > > > > [ 68.020812] calling ideapad_acpi_driver_init+0x0/0x1000
> > > > > > [ideapad_laptop] @ 1322
> > > > > > [ 68.026708] input: Ideapad extra buttons as
> > > > > >
> > /devices/pci0000:00/0000:00:1f.0/PNP0C09:00/VPC2004:00/input/input16
> > > > > > [ 68.038236] ACPI Debug: "=====QUERY_64====="
> > > > > > [ 68.050232] ACPI Debug: "=====QUERY_65====="
> > > > > > [ 68.060218] ACPI Debug: "=====QUERY_64====="
> > > > > > [ 68.092216] probe of VPC2004:00 returned 1 after 71386 usecs
> > > > > > [ 68.092245] initcall ideapad_acpi_driver_init+0x0/0x1000
> > > > > > [ideapad_laptop] returned 0 after 69751 usecssg
> > > > > >
> > > > > > ...
> > > > > >
> > > > > > These =====QUERY_XX===== messages are from DSDT:
> > > > > >
> > > > > > ~/acpi$ grep QUERY dsdt.dsl
> > > > > > Debug = "=====QUERY_11====="
> > > > > > Debug = "=====QUERY_12====="
> > > > > > Debug = "=====QUERY_24====="
> > > > > > Debug = "=====QUERY_25====="
> > > > > > Debug = "=====QUERY_37====="
> > > > > > Debug = "=====QUERY_38====="
> > > > > > Debug = "=====QUERY_64====="
> > > > > > Debug = "=====QUERY_65====="
> > > > > >
> > > > > > Also this is the code from DSDT for QUERY 64 and 65:
> > > > > >
> > > > > > ...
> > > > > > Method (_Q64, 0, NotSerialized) // _Qxx: EC Query
> > > > > > {
> > > > > > Debug = "=====QUERY_64====="
> > > > > > If ((OSYS == 0x07D9))
> > > > > > {
> > > > > > If (((WLEX == One) & (WLAT == One)))
> > > > > > {
> > > > > > SGOV (0x02040005, One)
> > > > > > }
> > > > > > Else
> > > > > > {
> > > > > > SGOV (0x02040005, Zero)
> > > > > > }
> > > > > > }
> > > > > > }
> > > > > >
> > > > > > Method (_Q65, 0, NotSerialized) // _Qxx: EC Query
> > > > > > {
> > > > > > Debug = "=====QUERY_65====="
> > > > > > If ((OSYS == 0x07D9))
> > > > > > {
> > > > > > If (((BTEX == One) & (BTAT == One)))
> > > > > > {
> > > > > > SGOV (0x0202000B, One)
> > > > > > }
> > > > > > Else
> > > > > > {
> > > > > > SGOV (0x0202000B, Zero)
> > > > > > }
> > > > > > }
> > > > > > }
> > > > > >
> > > > > > ...
> > > > > >
> > > > > >
> > > > > > Any idea what would cause this ?
> > > >
> > > > I think I found what is causing that.
> > > >
> > > > Commit aa9aaa4d61c0048d3faad056893cd7860bbc084c is moving
> > definition
> > > > of Linux's ACPI_DEBUG_DEFAULT to incude/acpi/platform/aclinux.h ,
> > > > however include/acpi/acpi.h didn't got updated and
> > > > ACPI_DEBUG_DEFAULT now being always defined as (ACPI_LV_INIT |
> > > > ACPI_LV_DEBUG_OBJECT | ACPI_LV_EVALUATION | ACPI_LV_REPAIR)
> > >
> > > I'm not sure what you mean by this.
> > >
> > > include/acpi/acpi.h should get the right definitions through
> > > include/acpi/platform/acenv.h and include/acpi/acoutput.h that are
> > > included by it.
> > >
> >
> > Hmm yes should , I missed that include. However it does not work.
> >
> > I've just put back old code back into drivers/acpi/bus.c to test and all working
> > now fine without any other changes.
> >
> > > It looks like incude/acpi/platform/aclinux.h has not been included by
> > > include/acpi/platform/acenv.h for some reason when building this
> > > module.
> >
> > If this one getting it wrong all will do so , they all just include <linux/acpi.h>.
> >
> > Probably not so much users will notice that , I just did because my BIOS
> > vendor forgot DEBUG around.
>
> Do you still get this issue after doing make clean and rebuilding your kernel?
A clean build from tarball has this issue for me. I don't think is a
building problem ,
any <5.1 kernel is fine , kernel 5.1 with your commit reverted is fine too ,
kernel 5.1 with your commit in but the removed hunk in bus.c re-added
is fine also.
^ permalink raw reply [flat|nested] 9+ messages in thread
* RE: [Kernel 5.1] ACPI_DEBUG messages without CONFIG_ACPI_DEBUG being set
2019-05-07 19:06 ` Gabriel C
@ 2019-05-07 20:17 ` Schmauss, Erik
2019-05-07 21:39 ` Gabriel C
0 siblings, 1 reply; 9+ messages in thread
From: Schmauss, Erik @ 2019-05-07 20:17 UTC (permalink / raw)
To: Gabriel C; +Cc: Rafael J. Wysocki, ACPI Devel Maling List, LKML, Moore, Robert
> -----Original Message-----
> From: Gabriel C [mailto:nix.or.die@gmail.com]
> Sent: Tuesday, May 7, 2019 12:06 PM
> To: Schmauss, Erik <erik.schmauss@intel.com>
> Cc: Rafael J. Wysocki <rafael@kernel.org>; ACPI Devel Maling List <linux-
> acpi@vger.kernel.org>; LKML <linux-kernel@vger.kernel.org>; Moore,
> Robert <robert.moore@intel.com>
> Subject: Re: [Kernel 5.1] ACPI_DEBUG messages without
> CONFIG_ACPI_DEBUG being set
>
> Am Di., 7. Mai 2019 um 20:46 Uhr schrieb Schmauss, Erik
> <erik.schmauss@intel.com>:
> >
> >
> >
> > > -----Original Message-----
> > > From: Gabriel C [mailto:nix.or.die@gmail.com]
> > > Sent: Tuesday, May 7, 2019 2:33 AM
> > > To: Rafael J. Wysocki <rafael@kernel.org>
> > > Cc: ACPI Devel Maling List <linux-acpi@vger.kernel.org>; LKML
> > > <linux- kernel@vger.kernel.org>; Schmauss, Erik
> > > <erik.schmauss@intel.com>
> > > Subject: Re: [Kernel 5.1] ACPI_DEBUG messages without
> > > CONFIG_ACPI_DEBUG being set
> > >
> > > Am Di., 7. Mai 2019 um 10:35 Uhr schrieb Rafael J. Wysocki
> > > <rafael@kernel.org>:
> > > >
> > > > On Tue, May 7, 2019 at 9:31 AM Gabriel C <nix.or.die@gmail.com>
> wrote:
> > > > >
> > > > > Am Di., 7. Mai 2019 um 09:01 Uhr schrieb Rafael J. Wysocki
> > > <rafael@kernel.org>:
> > > > > >
> > > > > Hello Rafael , Erik
> > > > >
> > > > > > +Erik
> > > > > >
> > > > > > On Tue, May 7, 2019 at 1:33 AM Gabriel C
> > > > > > <nix.or.die@gmail.com>
> > > wrote:
> > > > > > >
> > > > > > > Hello,
> > > > > > >
> > > > > > > while testing kernel-5.1 I get on one of my Lenovo Laptops
> > > > > > > very strange 'ACPI Debug:' messages.
> > > > > > >
> > > > > > > After some grepping I realized these are Debug messages from
> > > > > > > DSDT , however my kernel does not have ACPI_DEBUG enabled.
> > > > > > >
> > > > > > > I found out the module triggering this, on this Laptop is
> > > > > > > ideapad_laptop , but looking at the code I cannot see what
> > > > > > > would causes that.
> > > > > > >
> > > > > > > Also on the same Laptop with any 5.0.X kernels I cannot see these.
> > > > > > >
> > > > > > >
> > > > > > > ~$ grep -i ACPI_DEBUG /boot/config-5.1-fw1 #
> > > > > > > CONFIG_ACPI_DEBUGGER is not set # CONFIG_ACPI_DEBUG is
> not
> > > set #
> > > > > > > CONFIG_THINKPAD_ACPI_DEBUGFACILITIES is not set #
> > > > > > > CONFIG_THINKPAD_ACPI_DEBUG is not set
> > > > > > >
> > > > > > > .. dmesg ..
> > > > > > > ...
> > > > > > > [ 68.020812] calling ideapad_acpi_driver_init+0x0/0x1000
> > > > > > > [ideapad_laptop] @ 1322
> > > > > > > [ 68.026708] input: Ideapad extra buttons as
> > > > > > >
> > > /devices/pci0000:00/0000:00:1f.0/PNP0C09:00/VPC2004:00/input/input16
> > > > > > > [ 68.038236] ACPI Debug: "=====QUERY_64====="
> > > > > > > [ 68.050232] ACPI Debug: "=====QUERY_65====="
> > > > > > > [ 68.060218] ACPI Debug: "=====QUERY_64====="
> > > > > > > [ 68.092216] probe of VPC2004:00 returned 1 after 71386 usecs
> > > > > > > [ 68.092245] initcall ideapad_acpi_driver_init+0x0/0x1000
> > > > > > > [ideapad_laptop] returned 0 after 69751 usecssg
> > > > > > >
> > > > > > > ...
> > > > > > >
> > > > > > > These =====QUERY_XX===== messages are from DSDT:
> > > > > > >
> > > > > > > ~/acpi$ grep QUERY dsdt.dsl
> > > > > > > Debug = "=====QUERY_11====="
> > > > > > > Debug = "=====QUERY_12====="
> > > > > > > Debug = "=====QUERY_24====="
> > > > > > > Debug = "=====QUERY_25====="
> > > > > > > Debug = "=====QUERY_37====="
> > > > > > > Debug = "=====QUERY_38====="
> > > > > > > Debug = "=====QUERY_64====="
> > > > > > > Debug = "=====QUERY_65====="
> > > > > > >
> > > > > > > Also this is the code from DSDT for QUERY 64 and 65:
> > > > > > >
> > > > > > > ...
> > > > > > > Method (_Q64, 0, NotSerialized) // _Qxx: EC Query
> > > > > > > {
> > > > > > > Debug = "=====QUERY_64====="
> > > > > > > If ((OSYS == 0x07D9))
> > > > > > > {
> > > > > > > If (((WLEX == One) & (WLAT == One)))
> > > > > > > {
> > > > > > > SGOV (0x02040005, One)
> > > > > > > }
> > > > > > > Else
> > > > > > > {
> > > > > > > SGOV (0x02040005, Zero)
> > > > > > > }
> > > > > > > }
> > > > > > > }
> > > > > > >
> > > > > > > Method (_Q65, 0, NotSerialized) // _Qxx: EC Query
> > > > > > > {
> > > > > > > Debug = "=====QUERY_65====="
> > > > > > > If ((OSYS == 0x07D9))
> > > > > > > {
> > > > > > > If (((BTEX == One) & (BTAT == One)))
> > > > > > > {
> > > > > > > SGOV (0x0202000B, One)
> > > > > > > }
> > > > > > > Else
> > > > > > > {
> > > > > > > SGOV (0x0202000B, Zero)
> > > > > > > }
> > > > > > > }
> > > > > > > }
> > > > > > >
> > > > > > > ...
> > > > > > >
> > > > > > >
> > > > > > > Any idea what would cause this ?
> > > > >
> > > > > I think I found what is causing that.
> > > > >
> > > > > Commit aa9aaa4d61c0048d3faad056893cd7860bbc084c is moving
> > > definition
> > > > > of Linux's ACPI_DEBUG_DEFAULT to incude/acpi/platform/aclinux.h
> > > > > , however include/acpi/acpi.h didn't got updated and
> > > > > ACPI_DEBUG_DEFAULT now being always defined as (ACPI_LV_INIT
> |
> > > > > ACPI_LV_DEBUG_OBJECT | ACPI_LV_EVALUATION |
> ACPI_LV_REPAIR)
> > > >
> > > > I'm not sure what you mean by this.
> > > >
> > > > include/acpi/acpi.h should get the right definitions through
> > > > include/acpi/platform/acenv.h and include/acpi/acoutput.h that are
> > > > included by it.
> > > >
> > >
> > > Hmm yes should , I missed that include. However it does not work.
> > >
> > > I've just put back old code back into drivers/acpi/bus.c to test and
> > > all working now fine without any other changes.
> > >
> > > > It looks like incude/acpi/platform/aclinux.h has not been included
> > > > by include/acpi/platform/acenv.h for some reason when building
> > > > this module.
> > >
> > > If this one getting it wrong all will do so , they all just include
> <linux/acpi.h>.
> > >
> > > Probably not so much users will notice that , I just did because my
> > > BIOS vendor forgot DEBUG around.
> >
> > Do you still get this issue after doing make clean and rebuilding your
> kernel?
>
> A clean build from tarball has this issue for me. I don't think is a building
> problem , any <5.1 kernel is fine , kernel 5.1 with your commit reverted is fine
> too , kernel 5.1 with your commit in but the removed hunk in bus.c re-added
> is fine also.
I took a closer look. This was an obvious mistake on my part. Thanks for catching it.
The following should fix it:
diff --git a/include/acpi/platform/aclinux.h b/include/acpi/platform/aclinux.h
index 624b90b34085..70b55b8fe48b 100644
--- a/include/acpi/platform/aclinux.h
+++ b/include/acpi/platform/aclinux.h
@@ -66,6 +66,12 @@
#define ACPI_INIT_FUNCTION __init
+
+/* Use a specific bugging default separate from ACPICA */
+
+#undef ACPI_DEBUG_DEFAULT
+#define ACPI_DEBUG_DEFAULT (ACPI_LV_INFO | ACPI_LV_REPAIR)
+
#ifndef CONFIG_ACPI
/* External globals for __KERNEL__, stubs is needed */
@@ -82,11 +88,6 @@
#define ACPI_NO_ERROR_MESSAGES
#undef ACPI_DEBUG_OUTPUT
-/* Use a specific bugging default separate from ACPICA */
-
-#undef ACPI_DEBUG_DEFAULT
-#define ACPI_DEBUG_DEFAULT (ACPI_LV_INFO | ACPI_LV_REPAIR)
-
/* External interface for __KERNEL__, stub is needed */
#define ACPI_EXTERNAL_RETURN_STATUS(prototype) \
^ permalink raw reply related [flat|nested] 9+ messages in thread
* Re: [Kernel 5.1] ACPI_DEBUG messages without CONFIG_ACPI_DEBUG being set
2019-05-07 20:17 ` Schmauss, Erik
@ 2019-05-07 21:39 ` Gabriel C
0 siblings, 0 replies; 9+ messages in thread
From: Gabriel C @ 2019-05-07 21:39 UTC (permalink / raw)
To: Schmauss, Erik
Cc: Rafael J. Wysocki, ACPI Devel Maling List, LKML, Moore, Robert
Am Di., 7. Mai 2019 um 22:17 Uhr schrieb Schmauss, Erik
<erik.schmauss@intel.com>:
>
>
>
> > -----Original Message-----
> > From: Gabriel C [mailto:nix.or.die@gmail.com]
> > Sent: Tuesday, May 7, 2019 12:06 PM
> > To: Schmauss, Erik <erik.schmauss@intel.com>
> > Cc: Rafael J. Wysocki <rafael@kernel.org>; ACPI Devel Maling List <linux-
> > acpi@vger.kernel.org>; LKML <linux-kernel@vger.kernel.org>; Moore,
> > Robert <robert.moore@intel.com>
> > Subject: Re: [Kernel 5.1] ACPI_DEBUG messages without
> > CONFIG_ACPI_DEBUG being set
> >
> > Am Di., 7. Mai 2019 um 20:46 Uhr schrieb Schmauss, Erik
> > <erik.schmauss@intel.com>:
> > >
> > >
> > >
> > > > -----Original Message-----
> > > > From: Gabriel C [mailto:nix.or.die@gmail.com]
> > > > Sent: Tuesday, May 7, 2019 2:33 AM
> > > > To: Rafael J. Wysocki <rafael@kernel.org>
> > > > Cc: ACPI Devel Maling List <linux-acpi@vger.kernel.org>; LKML
> > > > <linux- kernel@vger.kernel.org>; Schmauss, Erik
> > > > <erik.schmauss@intel.com>
> > > > Subject: Re: [Kernel 5.1] ACPI_DEBUG messages without
> > > > CONFIG_ACPI_DEBUG being set
> > > >
> > > > Am Di., 7. Mai 2019 um 10:35 Uhr schrieb Rafael J. Wysocki
> > > > <rafael@kernel.org>:
> > > > >
> > > > > On Tue, May 7, 2019 at 9:31 AM Gabriel C <nix.or.die@gmail.com>
> > wrote:
> > > > > >
> > > > > > Am Di., 7. Mai 2019 um 09:01 Uhr schrieb Rafael J. Wysocki
> > > > <rafael@kernel.org>:
> > > > > > >
> > > > > > Hello Rafael , Erik
> > > > > >
> > > > > > > +Erik
> > > > > > >
> > > > > > > On Tue, May 7, 2019 at 1:33 AM Gabriel C
> > > > > > > <nix.or.die@gmail.com>
> > > > wrote:
> > > > > > > >
> > > > > > > > Hello,
> > > > > > > >
> > > > > > > > while testing kernel-5.1 I get on one of my Lenovo Laptops
> > > > > > > > very strange 'ACPI Debug:' messages.
> > > > > > > >
> > > > > > > > After some grepping I realized these are Debug messages from
> > > > > > > > DSDT , however my kernel does not have ACPI_DEBUG enabled.
> > > > > > > >
> > > > > > > > I found out the module triggering this, on this Laptop is
> > > > > > > > ideapad_laptop , but looking at the code I cannot see what
> > > > > > > > would causes that.
> > > > > > > >
> > > > > > > > Also on the same Laptop with any 5.0.X kernels I cannot see these.
> > > > > > > >
> > > > > > > >
> > > > > > > > ~$ grep -i ACPI_DEBUG /boot/config-5.1-fw1 #
> > > > > > > > CONFIG_ACPI_DEBUGGER is not set # CONFIG_ACPI_DEBUG is
> > not
> > > > set #
> > > > > > > > CONFIG_THINKPAD_ACPI_DEBUGFACILITIES is not set #
> > > > > > > > CONFIG_THINKPAD_ACPI_DEBUG is not set
> > > > > > > >
> > > > > > > > .. dmesg ..
> > > > > > > > ...
> > > > > > > > [ 68.020812] calling ideapad_acpi_driver_init+0x0/0x1000
> > > > > > > > [ideapad_laptop] @ 1322
> > > > > > > > [ 68.026708] input: Ideapad extra buttons as
> > > > > > > >
> > > > /devices/pci0000:00/0000:00:1f.0/PNP0C09:00/VPC2004:00/input/input16
> > > > > > > > [ 68.038236] ACPI Debug: "=====QUERY_64====="
> > > > > > > > [ 68.050232] ACPI Debug: "=====QUERY_65====="
> > > > > > > > [ 68.060218] ACPI Debug: "=====QUERY_64====="
> > > > > > > > [ 68.092216] probe of VPC2004:00 returned 1 after 71386 usecs
> > > > > > > > [ 68.092245] initcall ideapad_acpi_driver_init+0x0/0x1000
> > > > > > > > [ideapad_laptop] returned 0 after 69751 usecssg
> > > > > > > >
> > > > > > > > ...
> > > > > > > >
> > > > > > > > These =====QUERY_XX===== messages are from DSDT:
> > > > > > > >
> > > > > > > > ~/acpi$ grep QUERY dsdt.dsl
> > > > > > > > Debug = "=====QUERY_11====="
> > > > > > > > Debug = "=====QUERY_12====="
> > > > > > > > Debug = "=====QUERY_24====="
> > > > > > > > Debug = "=====QUERY_25====="
> > > > > > > > Debug = "=====QUERY_37====="
> > > > > > > > Debug = "=====QUERY_38====="
> > > > > > > > Debug = "=====QUERY_64====="
> > > > > > > > Debug = "=====QUERY_65====="
> > > > > > > >
> > > > > > > > Also this is the code from DSDT for QUERY 64 and 65:
> > > > > > > >
> > > > > > > > ...
> > > > > > > > Method (_Q64, 0, NotSerialized) // _Qxx: EC Query
> > > > > > > > {
> > > > > > > > Debug = "=====QUERY_64====="
> > > > > > > > If ((OSYS == 0x07D9))
> > > > > > > > {
> > > > > > > > If (((WLEX == One) & (WLAT == One)))
> > > > > > > > {
> > > > > > > > SGOV (0x02040005, One)
> > > > > > > > }
> > > > > > > > Else
> > > > > > > > {
> > > > > > > > SGOV (0x02040005, Zero)
> > > > > > > > }
> > > > > > > > }
> > > > > > > > }
> > > > > > > >
> > > > > > > > Method (_Q65, 0, NotSerialized) // _Qxx: EC Query
> > > > > > > > {
> > > > > > > > Debug = "=====QUERY_65====="
> > > > > > > > If ((OSYS == 0x07D9))
> > > > > > > > {
> > > > > > > > If (((BTEX == One) & (BTAT == One)))
> > > > > > > > {
> > > > > > > > SGOV (0x0202000B, One)
> > > > > > > > }
> > > > > > > > Else
> > > > > > > > {
> > > > > > > > SGOV (0x0202000B, Zero)
> > > > > > > > }
> > > > > > > > }
> > > > > > > > }
> > > > > > > >
> > > > > > > > ...
> > > > > > > >
> > > > > > > >
> > > > > > > > Any idea what would cause this ?
> > > > > >
> > > > > > I think I found what is causing that.
> > > > > >
> > > > > > Commit aa9aaa4d61c0048d3faad056893cd7860bbc084c is moving
> > > > definition
> > > > > > of Linux's ACPI_DEBUG_DEFAULT to incude/acpi/platform/aclinux.h
> > > > > > , however include/acpi/acpi.h didn't got updated and
> > > > > > ACPI_DEBUG_DEFAULT now being always defined as (ACPI_LV_INIT
> > |
> > > > > > ACPI_LV_DEBUG_OBJECT | ACPI_LV_EVALUATION |
> > ACPI_LV_REPAIR)
> > > > >
> > > > > I'm not sure what you mean by this.
> > > > >
> > > > > include/acpi/acpi.h should get the right definitions through
> > > > > include/acpi/platform/acenv.h and include/acpi/acoutput.h that are
> > > > > included by it.
> > > > >
> > > >
> > > > Hmm yes should , I missed that include. However it does not work.
> > > >
> > > > I've just put back old code back into drivers/acpi/bus.c to test and
> > > > all working now fine without any other changes.
> > > >
> > > > > It looks like incude/acpi/platform/aclinux.h has not been included
> > > > > by include/acpi/platform/acenv.h for some reason when building
> > > > > this module.
> > > >
> > > > If this one getting it wrong all will do so , they all just include
> > <linux/acpi.h>.
> > > >
> > > > Probably not so much users will notice that , I just did because my
> > > > BIOS vendor forgot DEBUG around.
> > >
> > > Do you still get this issue after doing make clean and rebuilding your
> > kernel?
> >
> > A clean build from tarball has this issue for me. I don't think is a building
> > problem , any <5.1 kernel is fine , kernel 5.1 with your commit reverted is fine
> > too , kernel 5.1 with your commit in but the removed hunk in bus.c re-added
> > is fine also.
>
> I took a closer look. This was an obvious mistake on my part. Thanks for catching it.
>
> The following should fix it:
>
I can confirm it does fix the bug , thx.
> diff --git a/include/acpi/platform/aclinux.h b/include/acpi/platform/aclinux.h
> index 624b90b34085..70b55b8fe48b 100644
> --- a/include/acpi/platform/aclinux.h
> +++ b/include/acpi/platform/aclinux.h
> @@ -66,6 +66,12 @@
>
> #define ACPI_INIT_FUNCTION __init
>
> +
> +/* Use a specific bugging default separate from ACPICA */
> +
> +#undef ACPI_DEBUG_DEFAULT
> +#define ACPI_DEBUG_DEFAULT (ACPI_LV_INFO | ACPI_LV_REPAIR)
> +
> #ifndef CONFIG_ACPI
>
> /* External globals for __KERNEL__, stubs is needed */
> @@ -82,11 +88,6 @@
> #define ACPI_NO_ERROR_MESSAGES
> #undef ACPI_DEBUG_OUTPUT
>
> -/* Use a specific bugging default separate from ACPICA */
> -
> -#undef ACPI_DEBUG_DEFAULT
> -#define ACPI_DEBUG_DEFAULT (ACPI_LV_INFO | ACPI_LV_REPAIR)
> -
> /* External interface for __KERNEL__, stub is needed */
>
> #define ACPI_EXTERNAL_RETURN_STATUS(prototype) \
^ permalink raw reply [flat|nested] 9+ messages in thread
end of thread, other threads:[~2019-05-07 21:39 UTC | newest]
Thread overview: 9+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2019-05-06 23:34 [Kernel 5.1] ACPI_DEBUG messages without CONFIG_ACPI_DEBUG being set Gabriel C
2019-05-07 7:01 ` Rafael J. Wysocki
2019-05-07 7:32 ` Gabriel C
2019-05-07 8:34 ` Rafael J. Wysocki
2019-05-07 9:33 ` Gabriel C
2019-05-07 18:46 ` Schmauss, Erik
2019-05-07 19:06 ` Gabriel C
2019-05-07 20:17 ` Schmauss, Erik
2019-05-07 21:39 ` Gabriel C
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.