Date: Mon, 15 Sep 2008 06:47:52 +0000 (UTC) From: Joseph Koshy <jkoshy@FreeBSD.org> To: src-committers@FreeBSD.org, cvs-src@FreeBSD.org, cvs-all@FreeBSD.org Subject: cvs commit: src/sys/amd64/include pmc_mdep.h src/sys/dev/hwpmc hwpmc_x86.c src/sys/i386/include pmc_mdep.h Message-ID: <200809150648.m8F6meYe051665@repoman.freebsd.org>
next in thread | raw e-mail | index | archive | help
jkoshy 2008-09-15 06:47:52 UTC
FreeBSD src repository
Modified files:
sys/amd64/include pmc_mdep.h
sys/dev/hwpmc hwpmc_x86.c
sys/i386/include pmc_mdep.h
Log:
SVN rev 183033 on 2008-09-15 06:47:52Z by jkoshy
Correct a callchain capture bug on the i386.
On the i386 architecture, the processor only saves the current value
of `%esp' on stack if a privilege switch is necessary when entering
the interrupt handler. Thus, `frame->tf_esp' is only valid for
an entry from user mode. For interrupts taken in kernel mode, we
need to determine the top-of-stack for the interrupted kernel
procedure by adding the appropriate offset to the current frame
pointer.
Reported by: kris, Fabien Thomas
Tested by: Fabien Thomas <fabien.thomas at netasq dot com>
Revision Changes Path
1.6 +3 -2 src/sys/amd64/include/pmc_mdep.h
1.8 +3 -3 src/sys/dev/hwpmc/hwpmc_x86.c
1.8 +20 -2 src/sys/i386/include/pmc_mdep.h
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?200809150648.m8F6meYe051665>