From: Thomas Backlund on
On 20.4.2010 00:57, Andrew Morton wrote:
> On Mon, 19 Apr 2010 14:36:31 -0700
> "Justin P. Mattock"<justinmattock(a)gmail.com> wrote:
>
>> On 04/19/2010 02:28 PM, Andrew Morton wrote:
>>> On Wed, 14 Apr 2010 08:12:57 -0700
>>> Justin Mattock<justinmattock(a)gmail.com> wrote:
>>>
>>>> Just noticed this this morning,
>>>> system can only boot up when I take the battery
>>>> out.(when the battery is plugged in this(below)
>>>> shows up, and if booting no go)
>>>>
>>>> [ 169.207193] divide error: 0000 [#1] SMP
>>>> [ 169.207203] last sysfs file: /sys/devices/platform/applesmc.768/light
>>>> [ 169.207208] CPU 0
>>>> [ 169.207212] Modules linked in: rfcomm sco radeon bnep l2cap ttm
>>>> drm_kms_helper drm ipt_LOG iptable_nat nf_nat xt_state
>>>> nf_conntrack_ftp nf_conntrack_ipv4 nf_conntrack nf_defrag_ipv4
>>>> iptable_filter ip_tables x_tables btusb bluetooth firewire_ohci ath9k
>>>> firewire_core ath9k_common video battery ac ath9k_hw evdev joydev
>>>> ohci1394 ath button thermal sky2 i2c_i801 raw1394 ieee1394 uhci_hcd
>>>> ehci_hcd coretemp acpi_cpufreq processor appletouch applesmc uvcvideo
>>>> [ 169.207303]
>>>> [ 169.207310] Pid: 182, comm: kacpi_notify Not tainted
>>>> 2.6.34-rc3-00502-gd3b0936 #42 Mac-F42187C8/MacBookPro2,2
>>>> [ 169.207316] RIP: 0010:[<ffffffff81237e43>] [<ffffffff81237e43>]
>>>> acpi_ex_extract_from_field+0xb1/0x24a
>>>
>>> Presumably acpi_ex_extract_from_field() fed zero into (the misnamed)
>>> ACPI_ROUND_UP_TO().
>>>
>>> I'll ask Rafael and Maciej to track this as a post-26.33 regression,
>>> thanks.
>>>
>>>
>>
>> the patch here fixes it for me:ACPI: EC: Limit burst to 64 bit
>> https://bugzilla.kernel.org/attachment.cgi?id=25962
>
> hm. I wonder how you go from a bugzilla attachment back up tot he bug
> to which it is attached?
>
> Oh well. Alexey, I trust that patch is in the 2.6.34 queue?
>

It's already in Linus tree, commit 2060c44576c79086ff24718878d7edaa7384a985

--
Thomas
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo(a)vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
From: Justin P. Mattock on
On 04/19/2010 03:02 PM, Thomas Backlund wrote:
> On 20.4.2010 00:57, Andrew Morton wrote:
>> On Mon, 19 Apr 2010 14:36:31 -0700
>> "Justin P. Mattock"<justinmattock(a)gmail.com> wrote:
>>
>>> On 04/19/2010 02:28 PM, Andrew Morton wrote:
>>>> On Wed, 14 Apr 2010 08:12:57 -0700
>>>> Justin Mattock<justinmattock(a)gmail.com> wrote:
>>>>
>>>>> Just noticed this this morning,
>>>>> system can only boot up when I take the battery
>>>>> out.(when the battery is plugged in this(below)
>>>>> shows up, and if booting no go)
>>>>>
>>>>> [ 169.207193] divide error: 0000 [#1] SMP
>>>>> [ 169.207203] last sysfs file:
>>>>> /sys/devices/platform/applesmc.768/light
>>>>> [ 169.207208] CPU 0
>>>>> [ 169.207212] Modules linked in: rfcomm sco radeon bnep l2cap ttm
>>>>> drm_kms_helper drm ipt_LOG iptable_nat nf_nat xt_state
>>>>> nf_conntrack_ftp nf_conntrack_ipv4 nf_conntrack nf_defrag_ipv4
>>>>> iptable_filter ip_tables x_tables btusb bluetooth firewire_ohci ath9k
>>>>> firewire_core ath9k_common video battery ac ath9k_hw evdev joydev
>>>>> ohci1394 ath button thermal sky2 i2c_i801 raw1394 ieee1394 uhci_hcd
>>>>> ehci_hcd coretemp acpi_cpufreq processor appletouch applesmc uvcvideo
>>>>> [ 169.207303]
>>>>> [ 169.207310] Pid: 182, comm: kacpi_notify Not tainted
>>>>> 2.6.34-rc3-00502-gd3b0936 #42 Mac-F42187C8/MacBookPro2,2
>>>>> [ 169.207316] RIP: 0010:[<ffffffff81237e43>] [<ffffffff81237e43>]
>>>>> acpi_ex_extract_from_field+0xb1/0x24a
>>>>
>>>> Presumably acpi_ex_extract_from_field() fed zero into (the misnamed)
>>>> ACPI_ROUND_UP_TO().
>>>>
>>>> I'll ask Rafael and Maciej to track this as a post-26.33 regression,
>>>> thanks.
>>>>
>>>>
>>>
>>> the patch here fixes it for me:ACPI: EC: Limit burst to 64 bit
>>> https://bugzilla.kernel.org/attachment.cgi?id=25962
>>
>> hm. I wonder how you go from a bugzilla attachment back up tot he bug
>> to which it is attached?
>>
>> Oh well. Alexey, I trust that patch is in the 2.6.34 queue?
>>
>
> It's already in Linus tree, commit 2060c44576c79086ff24718878d7edaa7384a985
>
> --
> Thomas
>


cool thanks.. just pulled,
rebooted everything looks good.

cheers,

Justin P. Mattock
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo(a)vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
From: Justin P. Mattock on
On 04/20/2010 07:17 AM, Len Brown wrote:
>
>>> the patch here fixes it for me:ACPI: EC: Limit burst to 64 bit
>>> https://bugzilla.kernel.org/attachment.cgi?id=25962
>>
>> hm. I wonder how you go from a bugzilla attachment back up tot he bug
>> to which it is attached?
>
> Use "A comment contains this string" in bugzilla's advanced search
> and plug in "attachment.cgi?id=25962" for this case.
>
> you'll find the patch in this comment:
> https://bugzilla.kernel.org/show_bug.cgi?id=15749#c7
>
> It is a bit more user friendly to supply the
> URL of the commment containing the patch rather than
> the direct URL of the attachment itself.
>
>> Oh well. Alexey, I trust that patch is in the 2.6.34 queue?
>
> it shipped in 2.6.34-rc5 in commit
> 2060c44576c79086ff24718878d7edaa7384a985
>
> thanks,
> Len Brown, Intel Open Source Technology Center
>
>


ahh... o.k. so I just have to add
#(then a comment number) i.g.
#7 at the end of the bug URL.

make sense...

off topic of this bug, I've another
issue over here with the iMac9,1
(that I've been slowly working on)

basically long story short there is
no entry for the AC adapter in it's
dsdt(ACPI0003)
ac.c:
static const struct acpi_device_id ac_device_ids[] = {
{"ACPI0003", 0},
{"", 0},
};

causing no entries in /proc/acpi/*
and /sys/class/power_supply

This doesn't seem to be a big issue
it's just one machine defaults to ac
and then another machine defaults to battery
(keep in mind both machines are iMac9,1's
just different gpu's).

is there some boot param to tell the kernel
to simply go to ac?

Justin P. Mattock
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo(a)vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
From: Justin P. Mattock on
On 04/29/2010 08:10 AM, Alexey Starikovskiy wrote:
> 29.04.2010 18:38, Justin P. Mattock пишет:
>> On 04/20/2010 07:17 AM, Len Brown wrote:
>>>
>>>>> the patch here fixes it for me:ACPI: EC: Limit burst to 64 bit
>>>>> https://bugzilla.kernel.org/attachment.cgi?id=25962
>>>>
>>>> hm. I wonder how you go from a bugzilla attachment back up tot he bug
>>>> to which it is attached?
>>>
>>> Use "A comment contains this string" in bugzilla's advanced search
>>> and plug in "attachment.cgi?id=25962" for this case.
>>>
>>> you'll find the patch in this comment:
>>> https://bugzilla.kernel.org/show_bug.cgi?id=15749#c7
>>>
>>> It is a bit more user friendly to supply the
>>> URL of the commment containing the patch rather than
>>> the direct URL of the attachment itself.
>>>
>>>> Oh well. Alexey, I trust that patch is in the 2.6.34 queue?
>>>
>>> it shipped in 2.6.34-rc5 in commit
>>> 2060c44576c79086ff24718878d7edaa7384a985
>>>
>>> thanks,
>>> Len Brown, Intel Open Source Technology Center
>>>
>>>
>>
>>
>> ahh... o.k. so I just have to add
>> #(then a comment number) i.g.
>> #7 at the end of the bug URL.
>>
>> make sense...
>>
>> off topic of this bug, I've another
>> issue over here with the iMac9,1
>> (that I've been slowly working on)
>>
>> basically long story short there is
>> no entry for the AC adapter in it's
>> dsdt(ACPI0003)
>> ac.c:
>> static const struct acpi_device_id ac_device_ids[] = {
>> {"ACPI0003", 0},
>> {"", 0},
>> };
>>
>> causing no entries in /proc/acpi/*
>> and /sys/class/power_supply
>>
>> This doesn't seem to be a big issue
>> it's just one machine defaults to ac
>> and then another machine defaults to battery
>> (keep in mind both machines are iMac9,1's
>> just different gpu's).
>>
>> is there some boot param to tell the kernel
>> to simply go to ac?
>>
>> Justin P. Mattock
> There is no external AC switch. You may try to blacklist ac driver, so
> kernel might be fulled into being run on a desktop...
> You might also open a bug report for this issue, so we could give a more
> meaningful answer...
>
> Regards,
> Alex.
>

your right!!
(I've been putting off filling a bug
only to really see if I can do this,but the more
I look into it the more daunting it becomes).

as for the issue, I was thinking in the area of a dmi blacklist
or something to where proc_mkdir etc..
will just make the file "online" to trick
the kernel and any other mechanism reading
those files.

but first things first, I'll file a bug.

Justin P. Mattock
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo(a)vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
From: Matthew Garrett on
On Thu, Apr 29, 2010 at 07:38:00AM -0700, Justin P. Mattock wrote:

> This doesn't seem to be a big issue
> it's just one machine defaults to ac
> and then another machine defaults to battery
> (keep in mind both machines are iMac9,1's
> just different gpu's).

What do you mean, "Defaults to battery"? The iMac is a desktop machine,
isn't it? There typically isn't an AC adapter present in the ACPI
description.

--
Matthew Garrett | mjg59(a)srcf.ucam.org
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo(a)vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/