2014-12-28 18:49:00

by Carlos Mafra

[permalink] [raw]
Subject: Bisected regression from 3.17 still present in 3.19-rc1


The laptop is a 2-year-old macbook Pro with retina display.

I use a dockapp called wmlaptop2 (http://repo.or.cz/w/wmlaptop2.git)
which, among other things, displays battery information (% of charge
remaining etc).

With the kernel v3.17 the battery information stopped working and it
still does not work in the latest v3.19-rc1.

Looking at what wmlaptop2 does I noticed that one entry inside
/sys/class/power_supply is not present in a bad kernel.

In a good kernel:

[mafra@linux-g29b:~]$ ls /sys/class/power_supply/
ADP1@ BAT0@

but in v3.17 onwards there's no BAT0@ entry:

[mafra@linux-g29b:~]$ ls /sys/class/power_supply/
ADP1@

I bisected the problem to this commit:

commit 7bc5a2bad0b8d9d1ac9f7b8b33150e4ddf197334
Author: Matthew Garrett <[email protected]>
Date: Sat Sep 20 13:19:47 2014 +0200

ACPI: Support _OSI("Darwin") correctly


Is there anything else I can do to help fixing this?

PS: the bisect log:

git bisect start
# bad: [f114040e3ea6e07372334ade75d1ee0775c355e1] Linux 3.18-rc1
git bisect bad f114040e3ea6e07372334ade75d1ee0775c355e1
# good: [bfe01a5ba2490f299e1d2d5508cbbbadd897bbe9] Linux 3.17
git bisect good bfe01a5ba2490f299e1d2d5508cbbbadd897bbe9
# good: [35a9ad8af0bb0fa3525e6d0d20e32551d226f38e] Merge git://git.kernel.org/pub/scm/linux/kernel/git/davem/net-next
git bisect good 35a9ad8af0bb0fa3525e6d0d20e32551d226f38e
# bad: [d6dd50e07c5bec00db2005969b1a01f8ca3d25ef] Merge branch 'core-rcu-for-linus' of git://git.kernel.org/pub/scm/linux/kernel/git/tip/tip
git bisect bad d6dd50e07c5bec00db2005969b1a01f8ca3d25ef
# bad: [bf65dea87e87c53ba4f97c6432761498bc977efd] Merge tag 'edac/v3.18-rc1' of git://git.kernel.org/pub/scm/linux/kernel/git/mchehab/linux-edac
git bisect bad bf65dea87e87c53ba4f97c6432761498bc977efd
# bad: [b211e9d7c861bdb37b86d6384da9edfb80949ceb] Merge branch 'for-3.18' of git://git.kernel.org/pub/scm/linux/kernel/git/tj/cgroup
git bisect bad b211e9d7c861bdb37b86d6384da9edfb80949ceb
# good: [80213c03c4151d900cf293ef0fc51f8d88495e14] Merge tag 'pci-v3.18-changes' of git://git.kernel.org/pub/scm/linux/kernel/git/helgaas/pci
git bisect good 80213c03c4151d900cf293ef0fc51f8d88495e14
# good: [7f8998c7aef3ac9c5f3f2943e083dfa6302e90d0] nosave: consolidate __nosave_{begin,end} in <asm/sections.h>
git bisect good 7f8998c7aef3ac9c5f3f2943e083dfa6302e90d0
# bad: [49a09c9ab012017c4673b86dbb28c616cf8f2381] Merge branch 'pm-domains'
git bisect bad 49a09c9ab012017c4673b86dbb28c616cf8f2381
# bad: [88b42a4883a7783972c8fc607e60bd3f027e24de] Merge branch 'pm-genirq'
git bisect bad 88b42a4883a7783972c8fc607e60bd3f027e24de
# bad: [a13f453140d542f9d5a0ee15601531c72e5401d7] Merge branch 'acpi-lpss'
git bisect bad a13f453140d542f9d5a0ee15601531c72e5401d7
# bad: [939558f2a4b7851c11ce8d08387730914a1e1f5f] Merge branch 'acpi-apple'
git bisect bad 939558f2a4b7851c11ce8d08387730914a1e1f5f
# good: [5a0b8deeeb19906b24a48d0078aa6b64dc0b4dab] ACPICA: Clear all non-wakeup GPEs in acpi_hw_enable_wakeup_gpe_block()
git bisect good 5a0b8deeeb19906b24a48d0078aa6b64dc0b4dab
# bad: [7bc5a2bad0b8d9d1ac9f7b8b33150e4ddf197334] ACPI: Support _OSI("Darwin") correctly
git bisect bad 7bc5a2bad0b8d9d1ac9f7b8b33150e4ddf197334
# good: [9faf6136ff4647452580b019f4b16f8c5082e589] ACPI / SBS: Disable smart battery manager on Apple
git bisect good 9faf6136ff4647452580b019f4b16f8c5082e589
# first bad commit: [7bc5a2bad0b8d9d1ac9f7b8b33150e4ddf197334] ACPI: Support _OSI("Darwin") correctly
[mafra@linux-g29b:linux-2.6]$


2014-12-29 21:27:36

by Rafael J. Wysocki

[permalink] [raw]
Subject: Re: Bisected regression from 3.17 still present in 3.19-rc1

On Sunday, December 28, 2014 06:54:09 PM Carlos R. Mafra wrote:
>
> The laptop is a 2-year-old macbook Pro with retina display.
>
> I use a dockapp called wmlaptop2 (http://repo.or.cz/w/wmlaptop2.git)
> which, among other things, displays battery information (% of charge
> remaining etc).
>
> With the kernel v3.17 the battery information stopped working and it
> still does not work in the latest v3.19-rc1.
>
> Looking at what wmlaptop2 does I noticed that one entry inside
> /sys/class/power_supply is not present in a bad kernel.
>
> In a good kernel:
>
> [mafra@linux-g29b:~]$ ls /sys/class/power_supply/
> ADP1@ BAT0@
>
> but in v3.17 onwards there's no BAT0@ entry:
>
> [mafra@linux-g29b:~]$ ls /sys/class/power_supply/
> ADP1@
>
> I bisected the problem to this commit:

Does it help if you revert this commit from 3.19-rc1 (or -rc2)?

> commit 7bc5a2bad0b8d9d1ac9f7b8b33150e4ddf197334
> Author: Matthew Garrett <[email protected]>
> Date: Sat Sep 20 13:19:47 2014 +0200
>
> ACPI: Support _OSI("Darwin") correctly
>
>
> Is there anything else I can do to help fixing this?
>
> PS: the bisect log:
>
> git bisect start
> # bad: [f114040e3ea6e07372334ade75d1ee0775c355e1] Linux 3.18-rc1
> git bisect bad f114040e3ea6e07372334ade75d1ee0775c355e1
> # good: [bfe01a5ba2490f299e1d2d5508cbbbadd897bbe9] Linux 3.17
> git bisect good bfe01a5ba2490f299e1d2d5508cbbbadd897bbe9
> # good: [35a9ad8af0bb0fa3525e6d0d20e32551d226f38e] Merge git://git.kernel.org/pub/scm/linux/kernel/git/davem/net-next
> git bisect good 35a9ad8af0bb0fa3525e6d0d20e32551d226f38e
> # bad: [d6dd50e07c5bec00db2005969b1a01f8ca3d25ef] Merge branch 'core-rcu-for-linus' of git://git.kernel.org/pub/scm/linux/kernel/git/tip/tip
> git bisect bad d6dd50e07c5bec00db2005969b1a01f8ca3d25ef
> # bad: [bf65dea87e87c53ba4f97c6432761498bc977efd] Merge tag 'edac/v3.18-rc1' of git://git.kernel.org/pub/scm/linux/kernel/git/mchehab/linux-edac
> git bisect bad bf65dea87e87c53ba4f97c6432761498bc977efd
> # bad: [b211e9d7c861bdb37b86d6384da9edfb80949ceb] Merge branch 'for-3.18' of git://git.kernel.org/pub/scm/linux/kernel/git/tj/cgroup
> git bisect bad b211e9d7c861bdb37b86d6384da9edfb80949ceb
> # good: [80213c03c4151d900cf293ef0fc51f8d88495e14] Merge tag 'pci-v3.18-changes' of git://git.kernel.org/pub/scm/linux/kernel/git/helgaas/pci
> git bisect good 80213c03c4151d900cf293ef0fc51f8d88495e14
> # good: [7f8998c7aef3ac9c5f3f2943e083dfa6302e90d0] nosave: consolidate __nosave_{begin,end} in <asm/sections.h>
> git bisect good 7f8998c7aef3ac9c5f3f2943e083dfa6302e90d0
> # bad: [49a09c9ab012017c4673b86dbb28c616cf8f2381] Merge branch 'pm-domains'
> git bisect bad 49a09c9ab012017c4673b86dbb28c616cf8f2381
> # bad: [88b42a4883a7783972c8fc607e60bd3f027e24de] Merge branch 'pm-genirq'
> git bisect bad 88b42a4883a7783972c8fc607e60bd3f027e24de
> # bad: [a13f453140d542f9d5a0ee15601531c72e5401d7] Merge branch 'acpi-lpss'
> git bisect bad a13f453140d542f9d5a0ee15601531c72e5401d7
> # bad: [939558f2a4b7851c11ce8d08387730914a1e1f5f] Merge branch 'acpi-apple'
> git bisect bad 939558f2a4b7851c11ce8d08387730914a1e1f5f
> # good: [5a0b8deeeb19906b24a48d0078aa6b64dc0b4dab] ACPICA: Clear all non-wakeup GPEs in acpi_hw_enable_wakeup_gpe_block()
> git bisect good 5a0b8deeeb19906b24a48d0078aa6b64dc0b4dab
> # bad: [7bc5a2bad0b8d9d1ac9f7b8b33150e4ddf197334] ACPI: Support _OSI("Darwin") correctly
> git bisect bad 7bc5a2bad0b8d9d1ac9f7b8b33150e4ddf197334
> # good: [9faf6136ff4647452580b019f4b16f8c5082e589] ACPI / SBS: Disable smart battery manager on Apple
> git bisect good 9faf6136ff4647452580b019f4b16f8c5082e589
> # first bad commit: [7bc5a2bad0b8d9d1ac9f7b8b33150e4ddf197334] ACPI: Support _OSI("Darwin") correctly
> [mafra@linux-g29b:linux-2.6]$
> --
> To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
> the body of a message to [email protected]
> More majordomo info at http://vger.kernel.org/majordomo-info.html
> Please read the FAQ at http://www.tux.org/lkml/

--
I speak only for myself.
Rafael J. Wysocki, Intel Open Source Technology Center.

2014-12-29 21:51:54

by Carlos R. Mafra

[permalink] [raw]
Subject: Re: Bisected regression from 3.17 still present in 3.19-rc1

On Mon, 29 Dec 2014 at 22:49:37 +0100, Rafael J. Wysocki wrote:
> On Sunday, December 28, 2014 06:54:09 PM Carlos R. Mafra wrote:
> >
> > The laptop is a 2-year-old macbook Pro with retina display.
> >
> > I use a dockapp called wmlaptop2 (http://repo.or.cz/w/wmlaptop2.git)
> > which, among other things, displays battery information (% of charge
> > remaining etc).
> >
> > With the kernel v3.17 the battery information stopped working and it
> > still does not work in the latest v3.19-rc1.

Correction: v3.18 is when it stopped working, not v3.17.

> > Looking at what wmlaptop2 does I noticed that one entry inside
> > /sys/class/power_supply is not present in a bad kernel.
> >
> > In a good kernel:
> >
> > [mafra@linux-g29b:~]$ ls /sys/class/power_supply/
> > ADP1@ BAT0@
> >
> > but in v3.17 onwards there's no BAT0@ entry:
> >
> > [mafra@linux-g29b:~]$ ls /sys/class/power_supply/
> > ADP1@
> >
> > I bisected the problem to this commit:
>
> Does it help if you revert this commit from 3.19-rc1 (or -rc2)?

Yes, I just did that now (it reverts cleanly) and reverting the
commit makes it work again.


> > commit 7bc5a2bad0b8d9d1ac9f7b8b33150e4ddf197334
> > Author: Matthew Garrett <[email protected]>
> > Date: Sat Sep 20 13:19:47 2014 +0200
> >
> > ACPI: Support _OSI("Darwin") correctly
> >
> >
> > Is there anything else I can do to help fixing this?
> >
> > PS: the bisect log:
> >
> > git bisect start
> > # bad: [f114040e3ea6e07372334ade75d1ee0775c355e1] Linux 3.18-rc1
> > git bisect bad f114040e3ea6e07372334ade75d1ee0775c355e1
> > # good: [bfe01a5ba2490f299e1d2d5508cbbbadd897bbe9] Linux 3.17
> > git bisect good bfe01a5ba2490f299e1d2d5508cbbbadd897bbe9
> > # good: [35a9ad8af0bb0fa3525e6d0d20e32551d226f38e] Merge git://git.kernel.org/pub/scm/linux/kernel/git/davem/net-next
> > git bisect good 35a9ad8af0bb0fa3525e6d0d20e32551d226f38e
> > # bad: [d6dd50e07c5bec00db2005969b1a01f8ca3d25ef] Merge branch 'core-rcu-for-linus' of git://git.kernel.org/pub/scm/linux/kernel/git/tip/tip
> > git bisect bad d6dd50e07c5bec00db2005969b1a01f8ca3d25ef
> > # bad: [bf65dea87e87c53ba4f97c6432761498bc977efd] Merge tag 'edac/v3.18-rc1' of git://git.kernel.org/pub/scm/linux/kernel/git/mchehab/linux-edac
> > git bisect bad bf65dea87e87c53ba4f97c6432761498bc977efd
> > # bad: [b211e9d7c861bdb37b86d6384da9edfb80949ceb] Merge branch 'for-3.18' of git://git.kernel.org/pub/scm/linux/kernel/git/tj/cgroup
> > git bisect bad b211e9d7c861bdb37b86d6384da9edfb80949ceb
> > # good: [80213c03c4151d900cf293ef0fc51f8d88495e14] Merge tag 'pci-v3.18-changes' of git://git.kernel.org/pub/scm/linux/kernel/git/helgaas/pci
> > git bisect good 80213c03c4151d900cf293ef0fc51f8d88495e14
> > # good: [7f8998c7aef3ac9c5f3f2943e083dfa6302e90d0] nosave: consolidate __nosave_{begin,end} in <asm/sections.h>
> > git bisect good 7f8998c7aef3ac9c5f3f2943e083dfa6302e90d0
> > # bad: [49a09c9ab012017c4673b86dbb28c616cf8f2381] Merge branch 'pm-domains'
> > git bisect bad 49a09c9ab012017c4673b86dbb28c616cf8f2381
> > # bad: [88b42a4883a7783972c8fc607e60bd3f027e24de] Merge branch 'pm-genirq'
> > git bisect bad 88b42a4883a7783972c8fc607e60bd3f027e24de
> > # bad: [a13f453140d542f9d5a0ee15601531c72e5401d7] Merge branch 'acpi-lpss'
> > git bisect bad a13f453140d542f9d5a0ee15601531c72e5401d7
> > # bad: [939558f2a4b7851c11ce8d08387730914a1e1f5f] Merge branch 'acpi-apple'
> > git bisect bad 939558f2a4b7851c11ce8d08387730914a1e1f5f
> > # good: [5a0b8deeeb19906b24a48d0078aa6b64dc0b4dab] ACPICA: Clear all non-wakeup GPEs in acpi_hw_enable_wakeup_gpe_block()
> > git bisect good 5a0b8deeeb19906b24a48d0078aa6b64dc0b4dab
> > # bad: [7bc5a2bad0b8d9d1ac9f7b8b33150e4ddf197334] ACPI: Support _OSI("Darwin") correctly
> > git bisect bad 7bc5a2bad0b8d9d1ac9f7b8b33150e4ddf197334
> > # good: [9faf6136ff4647452580b019f4b16f8c5082e589] ACPI / SBS: Disable smart battery manager on Apple
> > git bisect good 9faf6136ff4647452580b019f4b16f8c5082e589
> > # first bad commit: [7bc5a2bad0b8d9d1ac9f7b8b33150e4ddf197334] ACPI: Support _OSI("Darwin") correctly
> > [mafra@linux-g29b:linux-2.6]$
> > --
> > To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
> > the body of a message to [email protected]
> > More majordomo info at http://vger.kernel.org/majordomo-info.html
> > Please read the FAQ at http://www.tux.org/lkml/
>
> --
> I speak only for myself.
> Rafael J. Wysocki, Intel Open Source Technology Center.

2014-12-29 22:20:35

by Andreas Noever

[permalink] [raw]
Subject: Re: Bisected regression from 3.17 still present in 3.19-rc1

On Mon, Dec 29, 2014 at 10:57 PM, Carlos R. Mafra <[email protected]> wrote:
> On Mon, 29 Dec 2014 at 22:49:37 +0100, Rafael J. Wysocki wrote:
>> On Sunday, December 28, 2014 06:54:09 PM Carlos R. Mafra wrote:
>> >
>> > The laptop is a 2-year-old macbook Pro with retina display.
>> >
>> > I use a dockapp called wmlaptop2 (http://repo.or.cz/w/wmlaptop2.git)
>> > which, among other things, displays battery information (% of charge
>> > remaining etc).
>> >
>> > With the kernel v3.17 the battery information stopped working and it
>> > still does not work in the latest v3.19-rc1.
>
> Correction: v3.18 is when it stopped working, not v3.17.
>
>> > Looking at what wmlaptop2 does I noticed that one entry inside
>> > /sys/class/power_supply is not present in a bad kernel.
>> >
>> > In a good kernel:
>> >
>> > [mafra@linux-g29b:~]$ ls /sys/class/power_supply/
>> > ADP1@ BAT0@
>> >
>> > but in v3.17 onwards there's no BAT0@ entry:
>> >
>> > [mafra@linux-g29b:~]$ ls /sys/class/power_supply/
>> > ADP1@
>> >
>> > I bisected the problem to this commit:
>>
>> Does it help if you revert this commit from 3.19-rc1 (or -rc2)?
>
> Yes, I just did that now (it reverts cleanly) and reverting the
> commit makes it work again.
Could you try to revert 9faf6136ff4647452580b019f4b16f8c5082e589 (ACPI
/ SBS: Disable smart battery manager on Apple) and possibly also
3031cddea633ea5328162d3d712a582e4205dbed (ACPI / SBS: Don't assume the
existence of an SBS charger) without reverting "ACPI: Support
_OSI("Darwin") correctly"?

As far as I understand these two were added because the third patch
breaks battery support on some Apple systems. Maybe the fix is only
needed for some models (and makes your model fail)?


>
>> > commit 7bc5a2bad0b8d9d1ac9f7b8b33150e4ddf197334
>> > Author: Matthew Garrett <[email protected]>
>> > Date: Sat Sep 20 13:19:47 2014 +0200
>> >
>> > ACPI: Support _OSI("Darwin") correctly
>> >
>> >
>> > Is there anything else I can do to help fixing this?
>> >
>> > PS: the bisect log:
>> >
>> > git bisect start
>> > # bad: [f114040e3ea6e07372334ade75d1ee0775c355e1] Linux 3.18-rc1
>> > git bisect bad f114040e3ea6e07372334ade75d1ee0775c355e1
>> > # good: [bfe01a5ba2490f299e1d2d5508cbbbadd897bbe9] Linux 3.17
>> > git bisect good bfe01a5ba2490f299e1d2d5508cbbbadd897bbe9
>> > # good: [35a9ad8af0bb0fa3525e6d0d20e32551d226f38e] Merge git://git.kernel.org/pub/scm/linux/kernel/git/davem/net-next
>> > git bisect good 35a9ad8af0bb0fa3525e6d0d20e32551d226f38e
>> > # bad: [d6dd50e07c5bec00db2005969b1a01f8ca3d25ef] Merge branch 'core-rcu-for-linus' of git://git.kernel.org/pub/scm/linux/kernel/git/tip/tip
>> > git bisect bad d6dd50e07c5bec00db2005969b1a01f8ca3d25ef
>> > # bad: [bf65dea87e87c53ba4f97c6432761498bc977efd] Merge tag 'edac/v3.18-rc1' of git://git.kernel.org/pub/scm/linux/kernel/git/mchehab/linux-edac
>> > git bisect bad bf65dea87e87c53ba4f97c6432761498bc977efd
>> > # bad: [b211e9d7c861bdb37b86d6384da9edfb80949ceb] Merge branch 'for-3.18' of git://git.kernel.org/pub/scm/linux/kernel/git/tj/cgroup
>> > git bisect bad b211e9d7c861bdb37b86d6384da9edfb80949ceb
>> > # good: [80213c03c4151d900cf293ef0fc51f8d88495e14] Merge tag 'pci-v3.18-changes' of git://git.kernel.org/pub/scm/linux/kernel/git/helgaas/pci
>> > git bisect good 80213c03c4151d900cf293ef0fc51f8d88495e14
>> > # good: [7f8998c7aef3ac9c5f3f2943e083dfa6302e90d0] nosave: consolidate __nosave_{begin,end} in <asm/sections.h>
>> > git bisect good 7f8998c7aef3ac9c5f3f2943e083dfa6302e90d0
>> > # bad: [49a09c9ab012017c4673b86dbb28c616cf8f2381] Merge branch 'pm-domains'
>> > git bisect bad 49a09c9ab012017c4673b86dbb28c616cf8f2381
>> > # bad: [88b42a4883a7783972c8fc607e60bd3f027e24de] Merge branch 'pm-genirq'
>> > git bisect bad 88b42a4883a7783972c8fc607e60bd3f027e24de
>> > # bad: [a13f453140d542f9d5a0ee15601531c72e5401d7] Merge branch 'acpi-lpss'
>> > git bisect bad a13f453140d542f9d5a0ee15601531c72e5401d7
>> > # bad: [939558f2a4b7851c11ce8d08387730914a1e1f5f] Merge branch 'acpi-apple'
>> > git bisect bad 939558f2a4b7851c11ce8d08387730914a1e1f5f
>> > # good: [5a0b8deeeb19906b24a48d0078aa6b64dc0b4dab] ACPICA: Clear all non-wakeup GPEs in acpi_hw_enable_wakeup_gpe_block()
>> > git bisect good 5a0b8deeeb19906b24a48d0078aa6b64dc0b4dab
>> > # bad: [7bc5a2bad0b8d9d1ac9f7b8b33150e4ddf197334] ACPI: Support _OSI("Darwin") correctly
>> > git bisect bad 7bc5a2bad0b8d9d1ac9f7b8b33150e4ddf197334
>> > # good: [9faf6136ff4647452580b019f4b16f8c5082e589] ACPI / SBS: Disable smart battery manager on Apple
>> > git bisect good 9faf6136ff4647452580b019f4b16f8c5082e589
>> > # first bad commit: [7bc5a2bad0b8d9d1ac9f7b8b33150e4ddf197334] ACPI: Support _OSI("Darwin") correctly
>> > [mafra@linux-g29b:linux-2.6]$
>> > --
>> > To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
>> > the body of a message to [email protected]
>> > More majordomo info at http://vger.kernel.org/majordomo-info.html
>> > Please read the FAQ at http://www.tux.org/lkml/
>>
>> --
>> I speak only for myself.
>> Rafael J. Wysocki, Intel Open Source Technology Center.

2014-12-29 22:48:43

by Carlos R. Mafra

[permalink] [raw]
Subject: Re: Bisected regression from 3.17 still present in 3.19-rc1

On Mon, 29 Dec 2014 at 23:20:13 +0100, Andreas Noever wrote:
> On Mon, Dec 29, 2014 at 10:57 PM, Carlos R. Mafra <[email protected]> wrote:
> > On Mon, 29 Dec 2014 at 22:49:37 +0100, Rafael J. Wysocki wrote:
> >> On Sunday, December 28, 2014 06:54:09 PM Carlos R. Mafra wrote:
> >> >
> >> > The laptop is a 2-year-old macbook Pro with retina display.
> >> >
> >> > I use a dockapp called wmlaptop2 (http://repo.or.cz/w/wmlaptop2.git)
> >> > which, among other things, displays battery information (% of charge
> >> > remaining etc).
> >> >
> >> > With the kernel v3.17 the battery information stopped working and it
> >> > still does not work in the latest v3.19-rc1.
> >
> > Correction: v3.18 is when it stopped working, not v3.17.
> >
> >> > Looking at what wmlaptop2 does I noticed that one entry inside
> >> > /sys/class/power_supply is not present in a bad kernel.
> >> >
> >> > In a good kernel:
> >> >
> >> > [mafra@linux-g29b:~]$ ls /sys/class/power_supply/
> >> > ADP1@ BAT0@
> >> >
> >> > but in v3.17 onwards there's no BAT0@ entry:
> >> >
> >> > [mafra@linux-g29b:~]$ ls /sys/class/power_supply/
> >> > ADP1@
> >> >
> >> > I bisected the problem to this commit:
> >>
> >> Does it help if you revert this commit from 3.19-rc1 (or -rc2)?
> >
> > Yes, I just did that now (it reverts cleanly) and reverting the
> > commit makes it work again.
> Could you try to revert 9faf6136ff4647452580b019f4b16f8c5082e589 (ACPI
> / SBS: Disable smart battery manager on Apple) and possibly also
> 3031cddea633ea5328162d3d712a582e4205dbed (ACPI / SBS: Don't assume the
> existence of an SBS charger) without reverting "ACPI: Support
> _OSI("Darwin") correctly"?
>
> As far as I understand these two were added because the third patch
> breaks battery support on some Apple systems. Maybe the fix is only
> needed for some models (and makes your model fail)?

Of the following table:

revert 9faf613 | revert 3031cdd
yes | no
yes | yes
no | no
no | yes

I tested the first row and it does not fix the problem. Do
I need to test the other 3 possibilities too?


>
> >
> >> > commit 7bc5a2bad0b8d9d1ac9f7b8b33150e4ddf197334
> >> > Author: Matthew Garrett <[email protected]>
> >> > Date: Sat Sep 20 13:19:47 2014 +0200
> >> >
> >> > ACPI: Support _OSI("Darwin") correctly
> >> >
> >> >
> >> > Is there anything else I can do to help fixing this?
> >> >
> >> > PS: the bisect log:
> >> >
> >> > git bisect start
> >> > # bad: [f114040e3ea6e07372334ade75d1ee0775c355e1] Linux 3.18-rc1
> >> > git bisect bad f114040e3ea6e07372334ade75d1ee0775c355e1
> >> > # good: [bfe01a5ba2490f299e1d2d5508cbbbadd897bbe9] Linux 3.17
> >> > git bisect good bfe01a5ba2490f299e1d2d5508cbbbadd897bbe9
> >> > # good: [35a9ad8af0bb0fa3525e6d0d20e32551d226f38e] Merge git://git.kernel.org/pub/scm/linux/kernel/git/davem/net-next
> >> > git bisect good 35a9ad8af0bb0fa3525e6d0d20e32551d226f38e
> >> > # bad: [d6dd50e07c5bec00db2005969b1a01f8ca3d25ef] Merge branch 'core-rcu-for-linus' of git://git.kernel.org/pub/scm/linux/kernel/git/tip/tip
> >> > git bisect bad d6dd50e07c5bec00db2005969b1a01f8ca3d25ef
> >> > # bad: [bf65dea87e87c53ba4f97c6432761498bc977efd] Merge tag 'edac/v3.18-rc1' of git://git.kernel.org/pub/scm/linux/kernel/git/mchehab/linux-edac
> >> > git bisect bad bf65dea87e87c53ba4f97c6432761498bc977efd
> >> > # bad: [b211e9d7c861bdb37b86d6384da9edfb80949ceb] Merge branch 'for-3.18' of git://git.kernel.org/pub/scm/linux/kernel/git/tj/cgroup
> >> > git bisect bad b211e9d7c861bdb37b86d6384da9edfb80949ceb
> >> > # good: [80213c03c4151d900cf293ef0fc51f8d88495e14] Merge tag 'pci-v3.18-changes' of git://git.kernel.org/pub/scm/linux/kernel/git/helgaas/pci
> >> > git bisect good 80213c03c4151d900cf293ef0fc51f8d88495e14
> >> > # good: [7f8998c7aef3ac9c5f3f2943e083dfa6302e90d0] nosave: consolidate __nosave_{begin,end} in <asm/sections.h>
> >> > git bisect good 7f8998c7aef3ac9c5f3f2943e083dfa6302e90d0
> >> > # bad: [49a09c9ab012017c4673b86dbb28c616cf8f2381] Merge branch 'pm-domains'
> >> > git bisect bad 49a09c9ab012017c4673b86dbb28c616cf8f2381
> >> > # bad: [88b42a4883a7783972c8fc607e60bd3f027e24de] Merge branch 'pm-genirq'
> >> > git bisect bad 88b42a4883a7783972c8fc607e60bd3f027e24de
> >> > # bad: [a13f453140d542f9d5a0ee15601531c72e5401d7] Merge branch 'acpi-lpss'
> >> > git bisect bad a13f453140d542f9d5a0ee15601531c72e5401d7
> >> > # bad: [939558f2a4b7851c11ce8d08387730914a1e1f5f] Merge branch 'acpi-apple'
> >> > git bisect bad 939558f2a4b7851c11ce8d08387730914a1e1f5f
> >> > # good: [5a0b8deeeb19906b24a48d0078aa6b64dc0b4dab] ACPICA: Clear all non-wakeup GPEs in acpi_hw_enable_wakeup_gpe_block()
> >> > git bisect good 5a0b8deeeb19906b24a48d0078aa6b64dc0b4dab
> >> > # bad: [7bc5a2bad0b8d9d1ac9f7b8b33150e4ddf197334] ACPI: Support _OSI("Darwin") correctly
> >> > git bisect bad 7bc5a2bad0b8d9d1ac9f7b8b33150e4ddf197334
> >> > # good: [9faf6136ff4647452580b019f4b16f8c5082e589] ACPI / SBS: Disable smart battery manager on Apple
> >> > git bisect good 9faf6136ff4647452580b019f4b16f8c5082e589
> >> > # first bad commit: [7bc5a2bad0b8d9d1ac9f7b8b33150e4ddf197334] ACPI: Support _OSI("Darwin") correctly
> >> > [mafra@linux-g29b:linux-2.6]$
> >> > --
> >> > To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
> >> > the body of a message to [email protected]
> >> > More majordomo info at http://vger.kernel.org/majordomo-info.html
> >> > Please read the FAQ at http://www.tux.org/lkml/
> >>
> >> --
> >> I speak only for myself.
> >> Rafael J. Wysocki, Intel Open Source Technology Center.

2014-12-30 03:03:57

by Matthew Garrett

[permalink] [raw]
Subject: Re: Bisected regression from 3.17 still present in 3.19-rc1

On Sun, 2014-12-28 at 18:54 +0000, Carlos R. Mafra wrote:
> The laptop is a 2-year-old macbook Pro with retina display.
>
> I use a dockapp called wmlaptop2 (http://repo.or.cz/w/wmlaptop2.git)
> which, among other things, displays battery information (% of charge
> remaining etc).
>
> With the kernel v3.17 the battery information stopped working and it
> still does not work in the latest v3.19-rc1.

What is CONFIG_ACPI_SBS set to?

--
Matthew Garrett <[email protected]>
????{.n?+???????+%?????ݶ??w??{.n?+????{??G?????{ay?ʇڙ?,j??f???h?????????z_??(?階?ݢj"???m??????G????????????&???~???iO???z??v?^?m???? ????????I?

2014-12-30 10:11:09

by Carlos R. Mafra

[permalink] [raw]
Subject: Re: Bisected regression from 3.17 still present in 3.19-rc1

On Tue, 30 Dec 2014 at 1:28:21 +0000, Matthew Garrett wrote:
> On Sun, 2014-12-28 at 18:54 +0000, Carlos R. Mafra wrote:
> > The laptop is a 2-year-old macbook Pro with retina display.
> >
> > I use a dockapp called wmlaptop2 (http://repo.or.cz/w/wmlaptop2.git)
> > which, among other things, displays battery information (% of charge
> > remaining etc).
> >
> > With the kernel v3.17 the battery information stopped working and it
> > still does not work in the latest v3.19-rc1.
>
> What is CONFIG_ACPI_SBS set to?

CONFIG_ACPI_SBS is not set

2014-12-30 11:36:57

by Matthew Garrett

[permalink] [raw]
Subject: Re: Bisected regression from 3.17 still present in 3.19-rc1

On Tue, 2014-12-30 at 10:16 +0000, Carlos R. Mafra wrote:
> On Tue, 30 Dec 2014 at 1:28:21 +0000, Matthew Garrett wrote:
> > What is CONFIG_ACPI_SBS set to?
>
> CONFIG_ACPI_SBS is not set

There's your problem. The Darwin call results in the firmware exposing
the battery via SBS, not as an ACPI control method battery. Enable
CONFIG_ACPI_SBS and you should be good.

--
Matthew Garrett <[email protected]>
????{.n?+???????+%?????ݶ??w??{.n?+????{??G?????{ay?ʇڙ?,j??f???h?????????z_??(?階?ݢj"???m??????G????????????&???~???iO???z??v?^?m???? ????????I?

2014-12-30 15:19:08

by Carlos R. Mafra

[permalink] [raw]
Subject: Re: Bisected regression from 3.17 still present in 3.19-rc1

On Tue, 30 Dec 2014 at 11:36:53 +0000, Matthew Garrett wrote:
> On Tue, 2014-12-30 at 10:16 +0000, Carlos R. Mafra wrote:
> > On Tue, 30 Dec 2014 at 1:28:21 +0000, Matthew Garrett wrote:
> > > What is CONFIG_ACPI_SBS set to?
> >
> > CONFIG_ACPI_SBS is not set
>
> There's your problem. The Darwin call results in the firmware exposing
> the battery via SBS, not as an ACPI control method battery. Enable
> CONFIG_ACPI_SBS and you should be good.

Right, enabling CONFIG_ACPI_SBS makes it work again, thanks!