x86/clear_page: Update clear_page_sse2() after dropping 32bit Xen

This code was never updated when the 32bit build of Xen was dropped.

 * Expand the now-redundant ptr_reg macro.
 * The number of iterations in the loop can be halfed by using 64bit writes,
   without consuming any extra execution resource in the pipeline.  Adjust all
   numbers/offsets appropriately.
 * Replace dec with sub to avoid a eflags stall, and position it to be
   macro-fused with the related jmp.
 * With no need to preserve eflags across the body of the loop, replace lea
   with add which has 1/3'rd the latency on basically all 64bit hardware.

A quick userspace perf test on my Haswell dev box indicates that the old
version takes ~1385 cycles on average (ignoring outliers), and the new version
takes ~1060 cyles, or about 77% of the time.

Reported-by: Edwin Török <edvin.torok@citrix.com>
Signed-off-by: Andrew Cooper <andrew.cooper3@citrix.com>
Reviewed-by: Jan Beulich <jbeulich@suse.com>
This commit is contained in:
Andrew Cooper committed 2019-06-18 15:47:52 +01:00
1 parent 6ff560f7f1
commit 260acc521d
1 file changed
+7 -9
+7 -9
View File
@@ -2,18 +2,16 @@
#include <asm/page.h>
#define ptr_reg %rdi
ENTRY(clear_page_sse2)
mov $PAGE_SIZE/16, %ecx
mov $PAGE_SIZE/32, %ecx
xor %eax,%eax
0: dec %ecx
movnti %eax, (ptr_reg)
movnti %eax, 4(ptr_reg)
movnti %eax, 8(ptr_reg)
movnti %eax, 12(ptr_reg)
lea 16(ptr_reg), ptr_reg
0: movnti %rax, (%rdi)
movnti %rax, 8(%rdi)
movnti %rax, 16(%rdi)
movnti %rax, 24(%rdi)
add $32, %rdi
sub $1, %ecx
jnz 0b
sfence
Reference in new issue
Block a user
Blocking a user prevents them from interacting with repositories, such as opening or commenting on pull requests or issues. Learn more about blocking a user.