Page Menu
Home
FreeBSD
Search
Configure Global Search
Log In
Files
F163615045
D45827.id140436.diff
No One
Temporary
Actions
View File
Edit File
Delete File
View Transforms
Subscribe
Mute Notifications
Flag For Later
Award Token
Size
3 KB
Referenced Files
None
Subscribers
None
D45827.id140436.diff
View Options
diff --git a/sys/amd64/include/pcpu.h b/sys/amd64/include/pcpu.h
--- a/sys/amd64/include/pcpu.h
+++ b/sys/amd64/include/pcpu.h
@@ -182,16 +182,8 @@
/*
* Evaluates to the address of the per-cpu variable name.
*/
-#define __PCPU_PTR(name) __extension__ ({ \
- __pcpu_type(name) *__p; \
- \
- __asm __volatile("movq %%gs:%1,%0; addq %2,%0" \
- : "=r" (__p) \
- : "m" (*(struct pcpu *)(__pcpu_offset(pc_prvspace))), \
- "i" (__pcpu_offset(name))); \
- \
- __p; \
-})
+#define __PCPU_PTR(name) \
+ (&get_pcpu()->name)
/*
* Evaluates to the value of the per-cpu variable name.
@@ -200,14 +192,13 @@
__pcpu_type(name) __res; \
struct __s { \
u_char __b[MIN(sizeof(__pcpu_type(name)), 8)]; \
- } __s; \
+ }; \
\
if (sizeof(__res) == 1 || sizeof(__res) == 2 || \
sizeof(__res) == 4 || sizeof(__res) == 8) { \
- __asm __volatile("mov %%gs:%1,%0" \
- : "=r" (__s) \
- : "m" (*(struct __s *)(__pcpu_offset(name)))); \
- *(struct __s *)(void *)&__res = __s; \
+ __asm __volatile("mov %%gs:%c1,%0" \
+ : "=r" (*(struct __s *)(void *)&__res) \
+ : "i" (__pcpu_offset(name))); \
} else { \
__res = *__PCPU_PTR(name); \
} \
@@ -222,15 +213,16 @@
__pcpu_type(name) __val; \
struct __s { \
u_char __b[MIN(sizeof(__pcpu_type(name)), 8)]; \
- } __s; \
+ }; \
\
__val = (val); \
if (sizeof(__val) == 1 || sizeof(__val) == 2 || \
sizeof(__val) == 4 || sizeof(__val) == 8) { \
- __s = *(struct __s *)(void *)&__val; \
- __asm __volatile("add %1,%%gs:%0" \
- : "=m" (*(struct __s *)(__pcpu_offset(name))) \
- : "r" (__s)); \
+ __asm __volatile("add %1,%%gs:%c0" \
+ : \
+ : "i" (__pcpu_offset(name)), \
+ "r" (*(struct __s *)(void *)&__val) \
+ : "cc", "memory"); \
} else \
*__PCPU_PTR(name) += __val; \
} while (0)
@@ -238,30 +230,31 @@
/*
* Sets the value of the per-cpu variable name to value val.
*/
-#define __PCPU_SET(name, val) { \
+#define __PCPU_SET(name, val) do { \
__pcpu_type(name) __val; \
struct __s { \
u_char __b[MIN(sizeof(__pcpu_type(name)), 8)]; \
- } __s; \
+ }; \
\
__val = (val); \
if (sizeof(__val) == 1 || sizeof(__val) == 2 || \
sizeof(__val) == 4 || sizeof(__val) == 8) { \
- __s = *(struct __s *)(void *)&__val; \
- __asm __volatile("mov %1,%%gs:%0" \
- : "=m" (*(struct __s *)(__pcpu_offset(name))) \
- : "r" (__s)); \
+ __asm __volatile("mov %1,%%gs:%c0" \
+ : \
+ : "i" (__pcpu_offset(name)), \
+ "r" (*(struct __s *)(void *)&__val) \
+ : "memory"); \
} else { \
*__PCPU_PTR(name) = __val; \
} \
-}
+} while (0)
#define get_pcpu() __extension__ ({ \
struct pcpu *__pc; \
\
- __asm __volatile("movq %%gs:%1,%0" \
+ __asm __volatile("movq %%gs:%c1,%0" \
: "=r" (__pc) \
- : "m" (*(struct pcpu *)(__pcpu_offset(pc_prvspace)))); \
+ : "i" (__pcpu_offset(pc_prvspace))); \
__pc; \
})
#endif /* !__SEG_GS */
diff --git a/sys/amd64/include/pcpu_aux.h b/sys/amd64/include/pcpu_aux.h
--- a/sys/amd64/include/pcpu_aux.h
+++ b/sys/amd64/include/pcpu_aux.h
@@ -54,8 +54,8 @@
{
struct thread *td;
- __asm("movq %%gs:%P1,%0" : "=r" (td) : "n" (offsetof(struct pcpu,
- pc_curthread)));
+ __asm("movq %%gs:%c1,%0" : "=r" (td)
+ : "i" (offsetof(struct pcpu, pc_curthread)));
return (td);
}
#define curthread (__curthread())
File Metadata
Details
Attached
Mime Type
text/plain
Expires
Sat, Jul 25, 9:31 PM (2 h, 46 m)
Storage Engine
blob
Storage Format
Raw Data
Storage Handle
35500003
Default Alt Text
D45827.id140436.diff (3 KB)
Attached To
Mode
D45827: amd64 pcpu: fix clobbers, suppress warnings, and clean up
Attached
Detach File
Event Timeline
Log In to Comment