Commit 0959f20863b119c5ca7d1e663bc22e6329814386 introduced new x64 functions
for saving and restoring the FPU control registers. The patch was correct,
but used a very misleading variable name "addr", while in fact these functions
pass values - the actual bit mask saved in those register - not addresses.

This patch only changes variable names - it should have no functional
significance.

Signed-off-by: Nadav Har'El <n...@scylladb.com>
---
 arch/x64/processor.hh | 20 ++++++++++----------
 1 file changed, 10 insertions(+), 10 deletions(-)

diff --git a/arch/x64/processor.hh b/arch/x64/processor.hh
index 250153be..6b1fbf66 100644
--- a/arch/x64/processor.hh
+++ b/arch/x64/processor.hh
@@ -413,26 +413,26 @@ inline bool rdrand(u64* dest)
 
 inline uint32_t stmxcsr()
 {
-    uint32_t addr;
-    asm volatile ("stmxcsr %0" : "=m" (addr));
-    return addr;
+    uint32_t bits;
+    asm volatile ("stmxcsr %0" : "=m" (bits));
+    return bits;
 }
 
-inline void ldmxcsr(uint32_t addr)
+inline void ldmxcsr(uint32_t bits)
 {
-    asm volatile ("ldmxcsr %0" : : "m" (addr));
+    asm volatile ("ldmxcsr %0" : : "m" (bits));
 }
 
 inline uint16_t fnstcw()
 {
-    uint16_t addr;
-    asm volatile ("fnstcw %0" : "=m" (addr));
-    return addr;
+    uint16_t bits;
+    asm volatile ("fnstcw %0" : "=m" (bits));
+    return bits;
 }
 
-inline void fldcw(uint16_t addr)
+inline void fldcw(uint16_t bits)
 {
-    asm volatile ("fldcw %0" : : "m" (addr));
+    asm volatile ("fldcw %0" : : "m" (bits));
 }
 
 struct task_state_segment {
-- 
2.14.3

-- 
You received this message because you are subscribed to the Google Groups "OSv 
Development" group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to osv-dev+unsubscr...@googlegroups.com.
For more options, visit https://groups.google.com/d/optout.

Reply via email to