On Wed, Nov 29, 2017 at 11:33:05AM +0100, Peter Zijlstra wrote: > @@ -220,7 +215,27 @@ For 32-bit we have the following convent > .macro SWITCH_TO_USER_CR3 scratch_reg:req > STATIC_JUMP_IF_FALSE .Lend_\@, kaiser_enabled_key, def=1 > mov %cr3, \scratch_reg > - ADJUST_USER_CR3 \scratch_reg > + > + /* > + * Test if the ASID needs a flush. > + */ > + push \scratch_reg /* preserve CR3 */ So I was just staring at disasm of a few functions and I noticed this one reads like push, while others read like pushq. So does the stupid assembler thing really do a 32bit push if you provide it with a 64bit register? > + andq $(0x7FF), \scratch_reg /* mask ASID */ > + bt \scratch_reg, PER_CPU_VAR(user_asid_flush_mask) > + jnc .Lnoflush_\@ > + > + /* Flush needed, clear the bit */ > + btr \scratch_reg, PER_CPU_VAR(user_asid_flush_mask) > + pop \scratch_reg /* original CR3 */ > + jmp .Ldo_\@ > + > +.Lnoflush_\@: > + pop \scratch_reg /* original CR3 */ > + ALTERNATIVE "", "bts $63, \scratch_reg", X86_FEATURE_PCID > + > +.Ldo_\@: > + /* Flip the PGD and ASID to the user version */ > + orq $(KAISER_SWITCH_MASK), \scratch_reg > mov \scratch_reg, %cr3 > .Lend_\@: > .endm -- To unsubscribe, send a message with 'unsubscribe linux-mm' in the body to majordomo@xxxxxxxxx. For more info on Linux MM, see: http://www.linux-mm.org/ . Don't email: <a href=mailto:"dont@xxxxxxxxx";> email@xxxxxxxxx </a>
