From: Chuck Ebbert <76306.1226@compuserve.com>
To: Ondrej Zary <linux@rainbow-software.org>
Cc: linux-kernel <linux-kernel@vger.kernel.org>,
Stephen Rothwell <sfr@canb.auug.org.au>
Subject: Re: Debugging APM - cat /proc/apm produces oops
Date: Tue, 25 Jul 2006 03:46:38 -0400 [thread overview]
Message-ID: <200607250348_MC3-1-C5FB-CC7F@compuserve.com> (raw)
In-Reply-To: <200607242351.37578.linux@rainbow-software.org>
On Mon, 24 Jul 2006 23:51:37 +0200, Ondrej Zary wrote:
>
> > > printing eip:
> > > 00002f9d
> > > *pre = 00000000
> > > Oops: 0002 [#4]
> > > Modules linked in:
> > > CPU: 0
> > > EIP: 00c0:[<00002f9d>] Not tainted VLI
> >
> > ^^^^
> > This is the APM BIOS 16 bit code segment.
>
> Looking at BIOS disassembly:
> 2F97: push bp
> 2F98: mov bp,sp
> 2F9A: add sp,-2
> 2F9D: mov [bp][-2],bx <-- it oopses here
That's expected. You can push/pop/call/ret using the kernel stack
because its 32-bit stack-size attribute controls how the stack is
addressed, but using it like that makes it use 16 bits (the CS
address size.)
This could probably be fixed in the kernel but it doesn't look
worth the trouble since the fix could be really ugly.
> I realized that I can modify the BIOS easily as it's stored in shadow RAM. So
> I replaced the offending MOV with three NOPs and tested again. This time it
> oopsed at 0x2FAD:
> 2FAD: cmp w,[bp][-2],1
> 2FB1: je 2FCB
>
> that jump was taken during my single stepping, so I NOPped out the CMP and
> replaced JE with JMPS. Then booted Linux and APM seems to work fine - battery
> percentage and remaining time is there as well as AC power status.
> There seems to be 4 these operations:
> mov [bp][-2],bx
> cmp w,[bp][-2],1
> cmp w,[bp][-2],8002
> cmp w,[bp][-2],8001
> but I've hit only the first two of them. I wonder what's that for (especially
> when it works without that).
Something is calling this after pushing the arg to the function onto the
stack. I guess it's always calling it with a 1 if that's all you are seeing.
--
Chuck
next reply other threads:[~2006-07-25 7:53 UTC|newest]
Thread overview: 9+ messages / expand[flat|nested] mbox.gz Atom feed top
2006-07-25 7:46 Chuck Ebbert [this message]
-- strict thread matches above, loose matches on Subject: below --
2006-07-23 14:30 Ondrej Zary
2006-07-23 14:41 ` Ondrej Zary
2006-07-23 15:06 ` Stephen Rothwell
2006-07-23 16:35 ` Ondrej Zary
2006-07-24 21:51 ` Ondrej Zary
2006-07-25 20:04 ` Ondrej Zary
2006-07-25 19:15 ` Alan Cox
2006-07-25 20:11 ` Ondrej Zary
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=200607250348_MC3-1-C5FB-CC7F@compuserve.com \
--to=76306.1226@compuserve.com \
--cc=linux-kernel@vger.kernel.org \
--cc=linux@rainbow-software.org \
--cc=sfr@canb.auug.org.au \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox
Powered by JetHome