locking/x86/xen: Use try_cmpxchg() in xen_alloc_p2m_entry()
authorUros Bizjak <ubizjak@gmail.com>
Fri, 5 Apr 2024 08:32:53 +0000 (10:32 +0200)
committerJuergen Gross <jgross@suse.com>
Fri, 17 May 2024 10:14:13 +0000 (12:14 +0200)
commitcdf9df020130237edf8860187aca3636d679aa48
tree54d570852c9cdad56963dc8ed2a331d6881aa385
parenta38297e3fb012ddfa7ce0321a7e5a8daeb1872b6
locking/x86/xen: Use try_cmpxchg() in xen_alloc_p2m_entry()

Use try_cmpxchg() instead of cmpxchg(*ptr, old, new) == old.

The x86 CMPXCHG instruction returns success in the ZF flag,
so this change saves a compare after CMPXCHG.

Also, try_cmpxchg() implicitly assigns old *ptr value to "old"
when CMPXCHG fails. There is no need to explicitly assign
old *ptr value to the temporary, which can simplify the
surrounding source code.

No functional change intended.

Signed-off-by: Uros Bizjak <ubizjak@gmail.com>
Reviewed-by: Juergen Gross <jgross@suse.com>
Cc: Juergen Gross <jgross@suse.com>
Cc: Boris Ostrovsky <boris.ostrovsky@oracle.com>
Cc: Thomas Gleixner <tglx@linutronix.de>
Cc: Ingo Molnar <mingo@kernel.org>
Cc: Borislav Petkov <bp@alien8.de>
Cc: Dave Hansen <dave.hansen@linux.intel.com>
Cc: "H. Peter Anvin" <hpa@zytor.com>
Link: https://lore.kernel.org/r/20240405083335.507471-1-ubizjak@gmail.com
Signed-off-by: Juergen Gross <jgross@suse.com>
arch/x86/xen/p2m.c