Re: MacBookPro2,2 unable too boot with the latest HEAD

Previous thread: [PATCH 0/2] Synaptics Clickpad support by Takashi Iwai on Wednesday, April 14, 2010 - 8:10 am. (15 messages)

Next thread: [PATCH] Documentation: SubmittingDrivers: Resources by Arce, Abraham on Wednesday, April 14, 2010 - 8:32 am. (2 messages)
From: Justin Mattock
Date: Wednesday, April 14, 2010 - 8:12 am

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
[  169.207332] RSP: 0018:ffff88003eced8d0  EFLAGS: 00010246
[  169.207338] RAX: 00000000000000ff RBX: ffff88003ec61090 RCX: 0000000000000000
[  169.207343] RDX: 0000000000000000 RSI: 00000000000000ff RDI: ffff88003ec61090
[  169.207349] RBP: ffff88003eced940 R08: 0000000000000080 R09: ffffffff815fde18
[  169.207355] R10: ffffffff00000008 R11: ffffffff8165a740 R12: ffff88003ed445a0
[  169.207361] R13: 0000000000000020 R14: ffff88003a102c30 R15: ffff88003eced908
[  169.207368] FS:  0000000000000000(0000) GS:ffff880001a00000(0000)
knlGS:0000000000000000
[  169.207374] CS:  0010 DS: 0000 ES: 0000 CR0: 000000008005003b
[  169.207380] CR2: 00000000009b42c0 CR3: 000000000165c000 CR4: 00000000000006f0
[  169.207386] DR0: 0000000000000000 DR1: 0000000000000000 DR2: 0000000000000000
[  169.207392] DR3: 0000000000000000 DR6: 00000000ffff0ff0 DR7: 0000000000000400
[  169.207398] Process kacpi_notify (pid: 182, threadinfo
ffff88003ecec000, task ffff88003eda0fe0)
[  169.207403] Stack:
[  169.207407]  ffff88003a102c30 ...
From: Andrew Morton
Date: Monday, April 19, 2010 - 2:28 pm

On Wed, 14 Apr 2010 08:12:57 -0700

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.

--

From: Justin P. Mattock
Date: Monday, April 19, 2010 - 2:36 pm

the patch here fixes it for me:ACPI: EC: Limit burst to 64 bit
https://bugzilla.kernel.org/attachment.cgi?id=25962


Justin P. Mattock
--

From: Andrew Morton
Date: Monday, April 19, 2010 - 2:57 pm

On Mon, 19 Apr 2010 14:36:31 -0700

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?

Thanks.
--

From: Justin P. Mattock
Date: Monday, April 19, 2010 - 3:09 pm

I don't know.. maybe backwards thinking(me lefthanded people


cool..

Justin P. Mattock
--

From: Thomas Backlund
Date: Monday, April 19, 2010 - 3:02 pm

It's already in Linus tree, commit 2060c44576c79086ff24718878d7edaa7384a985

--
Thomas
--

From: Justin P. Mattock
Date: Monday, April 19, 2010 - 3:56 pm

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

cheers,

Justin P. Mattock
--

From: Justin P. Mattock
Date: Thursday, April 29, 2010 - 7:38 am

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
--

From: Justin P. Mattock
Date: Thursday, April 29, 2010 - 9:03 am

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
--

From: Matthew Garrett
Date: Friday, April 30, 2010 - 1:49 pm

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@srcf.ucam.org
--

From: Len Brown
Date: Tuesday, April 20, 2010 - 7:17 am

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

it shipped in 2.6.34-rc5 in commit 
2060c44576c79086ff24718878d7edaa7384a985

thanks,
Len Brown, Intel Open Source Technology Center

--

Previous thread: [PATCH 0/2] Synaptics Clickpad support by Takashi Iwai on Wednesday, April 14, 2010 - 8:10 am. (15 messages)

Next thread: [PATCH] Documentation: SubmittingDrivers: Resources by Arce, Abraham on Wednesday, April 14, 2010 - 8:32 am. (2 messages)