[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

[PATCH] x86/boot: Reduce verbosity in reloc_trampoline??()



The name 'trampoline_ptr' is voluminous without merit.

Shorten it to just ptr.  No functional change.

Signed-off-by: Andrew Cooper <andrew.cooper3@xxxxxxxxxx>
---
CC: Jan Beulich <jbeulich@xxxxxxxx>
CC: Roger Pau Monné <roger@xxxxxxxxxxxxxx>
CC: Teddy Astie <teddy.astie@xxxxxxxxxx>
---
 xen/arch/x86/boot/reloc-trampoline.c | 14 +++++---------
 1 file changed, 5 insertions(+), 9 deletions(-)

diff --git a/xen/arch/x86/boot/reloc-trampoline.c 
b/xen/arch/x86/boot/reloc-trampoline.c
index e35e7c78aa86..f52b9b265b42 100644
--- a/xen/arch/x86/boot/reloc-trampoline.c
+++ b/xen/arch/x86/boot/reloc-trampoline.c
@@ -18,7 +18,7 @@ void reloc_trampoline64(void)
 #endif
 {
     uint32_t phys = trampoline_phys;
-    const int32_t *trampoline_ptr;
+    const int32_t *ptr;
 
     /*
      * Apply relocations to trampoline.
@@ -26,13 +26,9 @@ void reloc_trampoline64(void)
      * This modifies the trampoline in place within Xen, so that it will
      * operate correctly when copied into place.
      */
-    for ( trampoline_ptr = __trampoline_rel_start;
-          trampoline_ptr < __trampoline_rel_stop;
-          ++trampoline_ptr )
-        *(uint32_t *)(*trampoline_ptr + (long)trampoline_ptr) += phys;
+    for ( ptr = __trampoline_rel_start; ptr < __trampoline_rel_stop; ++ptr )
+        *(uint32_t *)(*ptr + (long)ptr) += phys;
 
-    for ( trampoline_ptr = __trampoline_seg_start;
-          trampoline_ptr < __trampoline_seg_stop;
-          ++trampoline_ptr )
-        *(uint16_t *)(*trampoline_ptr + (long)trampoline_ptr) = phys >> 4;
+    for ( ptr = __trampoline_seg_start; ptr < __trampoline_seg_stop; ++ptr )
+        *(uint16_t *)(*ptr + (long)ptr) = phys >> 4;
 }

base-commit: 97bf370375113caf92fccf04e2877fc6b5ff887f
-- 
2.39.5




 


Rackspace

Lists.xenproject.org is hosted with RackSpace, monitoring our
servers 24x7x365 and backed by RackSpace's Fanatical Support®.