diff options
author | Anton Blanchard <anton@samba.org> | 2007-01-09 02:43:02 +1100 |
---|---|---|
committer | Paul Mackerras <paulus@samba.org> | 2007-01-09 17:03:03 +1100 |
commit | dc40127ca5c6e1da48d2b5f9d0c65b5795faac12 (patch) | |
tree | 7d57b9a53adc8f4416380b126108d1e458a8e20c /include/asm-powerpc/hvcall.h | |
parent | ab87e8dc88a7cae341c403547cea6b022f5ac023 (diff) | |
download | blackbird-op-linux-dc40127ca5c6e1da48d2b5f9d0c65b5795faac12.tar.gz blackbird-op-linux-dc40127ca5c6e1da48d2b5f9d0c65b5795faac12.zip |
[POWERPC] Fix bugs in the hypervisor call stats code
There were a few issues with the HCALL_STATS code:
- PURR cpu feature checks were backwards
- We iterated one entry off the end of the hcall_stats array
- Remove dead update_hcall_stats() function prototype
I noticed one thing while debugging, and that is we call H_ENTER (to set
up the MMU hashtable in early init) before we have done the cpu fixups.
This means we will execute the PURR SPR reads even on a CPU that isnt
capable of it. I wonder if we can move the CPU feature fixups earlier.
Signed-off-by: Anton Blanchard <anton@samba.org>
Signed-off-by: Paul Mackerras <paulus@samba.org>
Diffstat (limited to 'include/asm-powerpc/hvcall.h')
-rw-r--r-- | include/asm-powerpc/hvcall.h | 2 |
1 files changed, 0 insertions, 2 deletions
diff --git a/include/asm-powerpc/hvcall.h b/include/asm-powerpc/hvcall.h index 257d1cecb8c9..7a500732b671 100644 --- a/include/asm-powerpc/hvcall.h +++ b/include/asm-powerpc/hvcall.h @@ -252,8 +252,6 @@ struct hcall_stats { unsigned long tb_total; /* total wall time (mftb) of calls. */ unsigned long purr_total; /* total cpu time (PURR) of calls. */ }; -void update_hcall_stats(unsigned long opcode, unsigned long tb_delta, - unsigned long purr_delta); #define HCALL_STAT_ARRAY_SIZE ((MAX_HCALL_OPCODE >> 2) + 1) #endif /* __ASSEMBLY__ */ |