diff options
author | kmacy <kmacy@FreeBSD.org> | 2006-12-17 06:48:40 +0000 |
---|---|---|
committer | kmacy <kmacy@FreeBSD.org> | 2006-12-17 06:48:40 +0000 |
commit | 20611107850a6c1de42880a35271d3a18a4b8811 (patch) | |
tree | 4a10bc4aba837d9f1be450b197838da90973dd1b /sys/amd64/amd64/apic_vector.S | |
parent | ee539b99dcacb73703a09f64d8ada227bb88a165 (diff) | |
download | FreeBSD-src-20611107850a6c1de42880a35271d3a18a4b8811.zip FreeBSD-src-20611107850a6c1de42880a35271d3a18a4b8811.tar.gz |
Newer versions of gcc don't support treating structures passed by value
as if they were really passed by reference. Specifically, the dead stores
elimination pass in the GCC 4.1 optimiser breaks the non-compliant behavior
on which FreeBSD relied. This change brings FreeBSD up to date by switching
trap frames to being explicitly passed by reference.
Reviewed by: kan
Tested by: kan
Diffstat (limited to 'sys/amd64/amd64/apic_vector.S')
-rw-r--r-- | sys/amd64/amd64/apic_vector.S | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/sys/amd64/amd64/apic_vector.S b/sys/amd64/amd64/apic_vector.S index ab781ca..14a6f87 100644 --- a/sys/amd64/amd64/apic_vector.S +++ b/sys/amd64/amd64/apic_vector.S @@ -60,6 +60,7 @@ IDTVEC(vec_name) ; \ jz 2f ; \ addl $(32 * index),%eax ; \ 1: ; \ + movq %rsp, %rsi ; \ movl %eax, %edi ; /* pass the IRQ */ \ call lapic_handle_intr ; \ MEXITCOUNT ; \ @@ -98,7 +99,7 @@ IDTVEC(spuriousint) IDTVEC(timerint) PUSH_FRAME FAKE_MCOUNT(TF_RIP(%rsp)) - + movq %rsp, %rdi call lapic_handle_timer MEXITCOUNT jmp doreti |