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

[Xen-changelog] [xen master] x86: drop dead asm-offsets definitions



commit 3243d69213f2367fdea01f7d4dcf219fdd287bc5
Author:     Jan Beulich <jbeulich@xxxxxxxx>
AuthorDate: Mon Jul 9 10:53:32 2018 +0200
Commit:     Jan Beulich <jbeulich@xxxxxxxx>
CommitDate: Mon Jul 9 10:53:32 2018 +0200

    x86: drop dead asm-offsets definitions
    
    UREGS_user_sizeof had a use long, long ago on x86, and apparently was
    blindly cloned into ARM code.
    
    The uses of UREGS_saved_upcall_mask and UREGS_{ds,es,fs,gs} disappeared
    in 71eb3a65e3 ("VMX: streamline entry.S code") and 110b2d6e8a ("SVM:
    streamline entry.S code").
    
    There are other ARM side UREGS_* which look to be unused, but I'd prefer
    to leave deciding on their disposition to ARM folks.
    
    irq_caps_offset and next_in_list_offset were introduced in 57b0566668
    ("Re-enable MSI support") without any user.
    
    VCPU_failsafe_{addr,sel} and _VGCF_failsafe_disables_events uses went
    away in d9b7ef209a ("x86: drop failsafe callback invocation from
    assembly").
    
    CPUINFO_processor_id looks to have been introduced without user by
    917335d8b3 ("x86: Avoid assumptions about C struct layouts from asm
    code").
    
    Signed-off-by: Jan Beulich <jbeulich@xxxxxxxx>
    Acked-by: Andrew Cooper <andrew.cooper3@xxxxxxxxxx>
    Acked-by: Julien Grall <julien.grall@xxxxxxx>
---
 xen/arch/arm/arm32/asm-offsets.c  |  1 -
 xen/arch/arm/arm64/asm-offsets.c  |  1 -
 xen/arch/x86/x86_64/asm-offsets.c | 14 --------------
 3 files changed, 16 deletions(-)

diff --git a/xen/arch/arm/arm32/asm-offsets.c b/xen/arch/arm/arm32/asm-offsets.c
index 5b543ab894..2116ba5b95 100644
--- a/xen/arch/arm/arm32/asm-offsets.c
+++ b/xen/arch/arm/arm32/asm-offsets.c
@@ -58,7 +58,6 @@ void __dummy__(void)
    OFFSET(UREGS_R12_fiq, struct cpu_user_regs, r12_fiq);
 
    OFFSET(UREGS_kernel_sizeof, struct cpu_user_regs, cpsr);
-   DEFINE(UREGS_user_sizeof, sizeof(struct cpu_user_regs));
    BLANK();
 
    DEFINE(CPUINFO_sizeof, sizeof(struct cpu_info));
diff --git a/xen/arch/arm/arm64/asm-offsets.c b/xen/arch/arm/arm64/asm-offsets.c
index f5c696d092..62833d8c8b 100644
--- a/xen/arch/arm/arm64/asm-offsets.c
+++ b/xen/arch/arm/arm64/asm-offsets.c
@@ -42,7 +42,6 @@ void __dummy__(void)
    OFFSET(UREGS_ELR_el1, struct cpu_user_regs, elr_el1);
 
    OFFSET(UREGS_kernel_sizeof, struct cpu_user_regs, spsr_el1);
-   DEFINE(UREGS_user_sizeof, sizeof(struct cpu_user_regs));
    BLANK();
 
    DEFINE(CPUINFO_sizeof, sizeof(struct cpu_info));
diff --git a/xen/arch/x86/x86_64/asm-offsets.c 
b/xen/arch/x86/x86_64/asm-offsets.c
index 5957c764f1..934a8aa044 100644
--- a/xen/arch/x86/x86_64/asm-offsets.c
+++ b/xen/arch/x86/x86_64/asm-offsets.c
@@ -41,22 +41,14 @@ void __dummy__(void)
     OFFSET(UREGS_rdi, struct cpu_user_regs, rdi);
     OFFSET(UREGS_error_code, struct cpu_user_regs, error_code);
     OFFSET(UREGS_entry_vector, struct cpu_user_regs, entry_vector);
-    OFFSET(UREGS_saved_upcall_mask, struct cpu_user_regs, saved_upcall_mask);
     OFFSET(UREGS_rip, struct cpu_user_regs, rip);
     OFFSET(UREGS_cs, struct cpu_user_regs, cs);
     OFFSET(UREGS_eflags, struct cpu_user_regs, rflags);
     OFFSET(UREGS_rsp, struct cpu_user_regs, rsp);
     OFFSET(UREGS_ss, struct cpu_user_regs, ss);
-    OFFSET(UREGS_ds, struct cpu_user_regs, ds);
-    OFFSET(UREGS_es, struct cpu_user_regs, es);
-    OFFSET(UREGS_fs, struct cpu_user_regs, fs);
-    OFFSET(UREGS_gs, struct cpu_user_regs, gs);
     OFFSET(UREGS_kernel_sizeof, struct cpu_user_regs, es);
-    DEFINE(UREGS_user_sizeof, sizeof(struct cpu_user_regs));
     BLANK();
 
-    OFFSET(irq_caps_offset, struct domain, irq_caps);
-    OFFSET(next_in_list_offset, struct domain, next_in_list);
     OFFSET(VCPU_processor, struct vcpu, processor);
     OFFSET(VCPU_domain, struct vcpu, domain);
     OFFSET(VCPU_vcpu_info, struct vcpu, vcpu_info);
@@ -64,10 +56,6 @@ void __dummy__(void)
     OFFSET(VCPU_thread_flags, struct vcpu, arch.flags);
     OFFSET(VCPU_event_addr, struct vcpu, arch.pv_vcpu.event_callback_eip);
     OFFSET(VCPU_event_sel, struct vcpu, arch.pv_vcpu.event_callback_cs);
-    OFFSET(VCPU_failsafe_addr, struct vcpu,
-           arch.pv_vcpu.failsafe_callback_eip);
-    OFFSET(VCPU_failsafe_sel, struct vcpu,
-           arch.pv_vcpu.failsafe_callback_cs);
     OFFSET(VCPU_syscall_addr, struct vcpu,
            arch.pv_vcpu.syscall_callback_eip);
     OFFSET(VCPU_syscall32_addr, struct vcpu,
@@ -96,7 +84,6 @@ void __dummy__(void)
     OFFSET(VCPU_async_exception_mask, struct vcpu, async_exception_mask);
     DEFINE(VCPU_TRAP_NMI, VCPU_TRAP_NMI);
     DEFINE(VCPU_TRAP_MCE, VCPU_TRAP_MCE);
-    DEFINE(_VGCF_failsafe_disables_events, _VGCF_failsafe_disables_events);
     DEFINE(_VGCF_syscall_disables_events,  _VGCF_syscall_disables_events);
     BLANK();
 
@@ -128,7 +115,6 @@ void __dummy__(void)
     BLANK();
 
     OFFSET(CPUINFO_guest_cpu_user_regs, struct cpu_info, guest_cpu_user_regs);
-    OFFSET(CPUINFO_processor_id, struct cpu_info, processor_id);
     OFFSET(CPUINFO_current_vcpu, struct cpu_info, current_vcpu);
     OFFSET(CPUINFO_cr4, struct cpu_info, cr4);
     OFFSET(CPUINFO_xen_cr3, struct cpu_info, xen_cr3);
--
generated by git-patchbot for /home/xen/git/xen.git#master

_______________________________________________
Xen-changelog mailing list
Xen-changelog@xxxxxxxxxxxxxxxxxxxx
https://lists.xenproject.org/xen-changelog

 


Rackspace

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