Module Name: src Committed By: riastradh Date: Sun Apr 9 08:18:03 UTC 2023
Modified Files: src/sys/arch/i386/include: cpu.h Log Message: i386: Make curlwp and curcpu() flushable. The only effect of the `volatile' qualifier on an asm block with outputs is to force the instructions to appear in the generated code, even if the outputs end up being unused. Since these instructions have no (architectural) side effects -- provided %fs is set correctly, which must be the case here -- there's no need for the volatile qualifier, so nix it. To generate a diff of this commit: cvs rdiff -u -r1.183 -r1.184 src/sys/arch/i386/include/cpu.h Please note that diffs are not public domain; they are subject to the copyright notices on the relevant files.
Modified files: Index: src/sys/arch/i386/include/cpu.h diff -u src/sys/arch/i386/include/cpu.h:1.183 src/sys/arch/i386/include/cpu.h:1.184 --- src/sys/arch/i386/include/cpu.h:1.183 Tue Nov 2 11:26:04 2021 +++ src/sys/arch/i386/include/cpu.h Sun Apr 9 08:18:03 2023 @@ -1,4 +1,4 @@ -/* $NetBSD: cpu.h,v 1.183 2021/11/02 11:26:04 ryo Exp $ */ +/* $NetBSD: cpu.h,v 1.184 2023/04/09 08:18:03 riastradh Exp $ */ /*- * Copyright (c) 1990 The Regents of the University of California. @@ -50,7 +50,7 @@ x86_curcpu(void) { struct cpu_info *ci; - __asm volatile("movl %%fs:%1, %0" : + __asm("movl %%fs:%1, %0" : "=r" (ci) : "m" (*(struct cpu_info * const *)offsetof(struct cpu_info, ci_self))); @@ -62,7 +62,7 @@ x86_curlwp(void) { lwp_t *l; - __asm volatile("movl %%fs:%1, %0" : + __asm("movl %%fs:%1, %0" : "=r" (l) : "m" (*(struct cpu_info * const *)offsetof(struct cpu_info, ci_curlwp)));