xen-project/xen
1
0 Fork 1
mirror of https://github.com/xen-project/xen.git synced 2026-10-29 18:50:05 +01:00
Code Issues Activity

x86/desc: Shorten boot_{,compat_}gdt[] variable names

Browse Source 
The current names, boot_cpu_{,compat_}gdt_table, have a table suffix which is
redundant with the T of GDT, and the cpu infix doesn't provide any meaningful
context.  Drop them both.

Likewise, shorten the {,compat_}gdt{,_l1e} variables.

Finally, rename gdt_descr to boot_gdtr to more clearly identify its purpose.

Signed-off-by: Andrew Cooper <andrew.cooper3@citrix.com>
Reviewed-by: Roger Pau Monné <roger.pau@citrix.com>
Acked-by: Jan Beulich <jbeulich@suse.com>
This commit is contained in:
Andrew Cooper committed 2019-08-07 13:34:56 +01:00
1 parent af292b41e9
commit 50feecdc09
11 files changed
+44 -48
Download Patch File Download Diff File Expand all files Collapse all files

No files matched your search

xen/arch/x86/boot/x86_64.S
+5 -5
View File
@@ -3,7 +3,7 @@
ENTRY(__high_start)
/* Install relocated data selectors. */
lgdt gdt_descr(%rip)
lgdt boot_gdtr(%rip)
mov $(__HYPERVISOR_DS64),%ecx
mov %ecx,%ds
mov %ecx,%es
@@ -44,16 +44,16 @@ multiboot_ptr:
.long 0
.word 0
GLOBAL(gdt_descr)
GLOBAL(boot_gdtr)
.word LAST_RESERVED_GDT_BYTE
.quad boot_cpu_gdt_table - FIRST_RESERVED_GDT_BYTE
.quad boot_gdt - FIRST_RESERVED_GDT_BYTE
GLOBAL(stack_start)
.quad cpu0_stack
.section .data.page_aligned, "aw", @progbits
.align PAGE_SIZE, 0
GLOBAL(boot_cpu_gdt_table)
GLOBAL(boot_gdt)
.quad 0x0000000000000000 /* unused */
.quad 0x00af9b000000ffff /* 0xe008 ring 0 code, 64-bit mode */
.quad 0x00cf93000000ffff /* 0xe010 ring 0 data */
@@ -68,7 +68,7 @@ GLOBAL(boot_cpu_gdt_table)
.align PAGE_SIZE, 0
/* NB. Even rings != 0 get access to the full 4Gb, as only the */
/* (compatibility) machine->physical mapping table lives there. */
GLOBAL(boot_cpu_compat_gdt_table)
GLOBAL(boot_compat_gdt)
.quad 0x0000000000000000 /* unused */
.quad 0x00af9b000000ffff /* 0xe008 ring 0 code, 64-bit mode */
.quad 0x00cf93000000ffff /* 0xe010 ring 0 data */
xen/arch/x86/cpu/common.c
+2 -2
View File
@@ -709,9 +709,9 @@ void load_system_tables(void)
struct tss_struct *tss = &this_cpu(init_tss);
seg_desc_t *gdt =
this_cpu(gdt_table) - FIRST_RESERVED_GDT_ENTRY;
this_cpu(gdt) - FIRST_RESERVED_GDT_ENTRY;
seg_desc_t *compat_gdt =
this_cpu(compat_gdt_table) - FIRST_RESERVED_GDT_ENTRY;
this_cpu(compat_gdt) - FIRST_RESERVED_GDT_ENTRY;
const struct desc_ptr gdtr = {
.base = (unsigned long)gdt,
xen/arch/x86/domain.c
+3 -4
View File
@@ -1666,8 +1666,8 @@ static inline bool need_full_gdt(const struct domain *d)
static void update_xen_slot_in_full_gdt(const struct vcpu *v, unsigned int cpu)
{
l1e_write(pv_gdt_ptes(v) + FIRST_RESERVED_GDT_PAGE,
!is_pv_32bit_vcpu(v) ? per_cpu(gdt_table_l1e, cpu)
: per_cpu(compat_gdt_table_l1e, cpu));
!is_pv_32bit_vcpu(v) ? per_cpu(gdt_l1e, cpu)
: per_cpu(compat_gdt_l1e, cpu));
}
static void load_full_gdt(const struct vcpu *v, unsigned int cpu)
@@ -1686,8 +1686,7 @@ static void load_default_gdt(unsigned int cpu)
{
struct desc_ptr gdt_desc = {
.limit = LAST_RESERVED_GDT_BYTE,
.base = (unsigned long)(per_cpu(gdt_table, cpu) -
FIRST_RESERVED_GDT_ENTRY),
.base = (unsigned long)(per_cpu(gdt, cpu) - FIRST_RESERVED_GDT_ENTRY),
};
lgdt(&gdt_desc);
xen/arch/x86/efi/efi-boot.h
+1 -1
View File
@@ -250,7 +250,7 @@ static void __init noreturn efi_arch_post_exit_boot(void)
"mov %[cr4], %%cr4\n\t"
#endif
"movabs $__start_xen, %[rip]\n\t"
"lgdt gdt_descr(%%rip)\n\t"
"lgdt boot_gdtr(%%rip)\n\t"
"mov stack_start(%%rip), %%rsp\n\t"
"mov %[ds], %%ss\n\t"
"mov %[ds], %%ds\n\t"
xen/arch/x86/hvm/svm/svm.c
+1 -1
View File
@@ -1568,7 +1568,7 @@ bool svm_load_segs(unsigned int ldt_ents, unsigned long ldt_base,
{
/* Keep GDT in sync. */
seg_desc_t *desc =
this_cpu(gdt_table) + LDT_ENTRY - FIRST_RESERVED_GDT_ENTRY;
this_cpu(gdt) + LDT_ENTRY - FIRST_RESERVED_GDT_ENTRY;
_set_tssldt_desc(desc, ldt_base, ldt_ents * 8 - 1, SYS_DESC_ldt);
xen/arch/x86/hvm/vmx/vmcs.c
+1 -1
View File
@@ -793,7 +793,7 @@ static void vmx_set_host_env(struct vcpu *v)
unsigned int cpu = smp_processor_id();
__vmwrite(HOST_GDTR_BASE,
(unsigned long)(this_cpu(gdt_table) - FIRST_RESERVED_GDT_ENTRY));
(unsigned long)(this_cpu(gdt) - FIRST_RESERVED_GDT_ENTRY));
__vmwrite(HOST_IDTR_BASE, (unsigned long)idt_tables[cpu]);
__vmwrite(HOST_TR_BASE, (unsigned long)&per_cpu(init_tss, cpu));
xen/arch/x86/smpboot.c
+9 -9
View File
@@ -944,11 +944,11 @@ static void cpu_smpboot_free(unsigned int cpu, bool remove)
free_domheap_page(mfn_to_page(mfn));
}
FREE_XENHEAP_PAGE(per_cpu(compat_gdt_table, cpu));
FREE_XENHEAP_PAGE(per_cpu(compat_gdt, cpu));
if ( remove )
{
FREE_XENHEAP_PAGE(per_cpu(gdt_table, cpu));
FREE_XENHEAP_PAGE(per_cpu(gdt, cpu));
FREE_XENHEAP_PAGE(idt_tables[cpu]);
if ( stack_base[cpu] )
@@ -976,22 +976,22 @@ static int cpu_smpboot_alloc(unsigned int cpu)
goto out;
memguard_guard_stack(stack_base[cpu]);
gdt = per_cpu(gdt_table, cpu) ?: alloc_xenheap_pages(0, memflags);
gdt = per_cpu(gdt, cpu) ?: alloc_xenheap_pages(0, memflags);
if ( gdt == NULL )
goto out;
per_cpu(gdt_table, cpu) = gdt;
per_cpu(gdt_table_l1e, cpu) =
per_cpu(gdt, cpu) = gdt;
per_cpu(gdt_l1e, cpu) =
l1e_from_pfn(virt_to_mfn(gdt), __PAGE_HYPERVISOR_RW);
memcpy(gdt, boot_cpu_gdt_table, NR_RESERVED_GDT_PAGES * PAGE_SIZE);
memcpy(gdt, boot_gdt, NR_RESERVED_GDT_PAGES * PAGE_SIZE);
BUILD_BUG_ON(NR_CPUS > 0x10000);
gdt[PER_CPU_GDT_ENTRY - FIRST_RESERVED_GDT_ENTRY].a = cpu;
per_cpu(compat_gdt_table, cpu) = gdt = alloc_xenheap_pages(0, memflags);
per_cpu(compat_gdt, cpu) = gdt = alloc_xenheap_pages(0, memflags);
if ( gdt == NULL )
goto out;
per_cpu(compat_gdt_table_l1e, cpu) =
per_cpu(compat_gdt_l1e, cpu) =
l1e_from_pfn(virt_to_mfn(gdt), __PAGE_HYPERVISOR_RW);
memcpy(gdt, boot_cpu_compat_gdt_table, NR_RESERVED_GDT_PAGES * PAGE_SIZE);
memcpy(gdt, boot_compat_gdt, NR_RESERVED_GDT_PAGES * PAGE_SIZE);
gdt[PER_CPU_GDT_ENTRY - FIRST_RESERVED_GDT_ENTRY].a = cpu;
if ( idt_tables[cpu] == NULL )
xen/arch/x86/traps.c
+14 -16
View File
@@ -96,10 +96,10 @@ string_param("nmi", opt_nmi);
DEFINE_PER_CPU(uint64_t, efer);
static DEFINE_PER_CPU(unsigned long, last_extable_addr);
DEFINE_PER_CPU_READ_MOSTLY(seg_desc_t *, gdt_table);
DEFINE_PER_CPU_READ_MOSTLY(l1_pgentry_t, gdt_table_l1e);
DEFINE_PER_CPU_READ_MOSTLY(seg_desc_t *, compat_gdt_table);
DEFINE_PER_CPU_READ_MOSTLY(l1_pgentry_t, compat_gdt_table_l1e);
DEFINE_PER_CPU_READ_MOSTLY(seg_desc_t *, gdt);
DEFINE_PER_CPU_READ_MOSTLY(l1_pgentry_t, gdt_l1e);
DEFINE_PER_CPU_READ_MOSTLY(seg_desc_t *, compat_gdt);
DEFINE_PER_CPU_READ_MOSTLY(l1_pgentry_t, compat_gdt_l1e);
/* Master table, used by CPU0. */
idt_entry_t __section(".bss.page_aligned") __aligned(PAGE_SIZE)
@@ -1899,17 +1899,17 @@ void load_TR(void)
{
struct tss_struct *tss = &this_cpu(init_tss);
struct desc_ptr old_gdt, tss_gdt = {
.base = (long)(this_cpu(gdt_table) - FIRST_RESERVED_GDT_ENTRY),
.base = (long)(this_cpu(gdt) - FIRST_RESERVED_GDT_ENTRY),
.limit = LAST_RESERVED_GDT_BYTE
};
_set_tssldt_desc(
this_cpu(gdt_table) + TSS_ENTRY - FIRST_RESERVED_GDT_ENTRY,
this_cpu(gdt) + TSS_ENTRY - FIRST_RESERVED_GDT_ENTRY,
(unsigned long)tss,
offsetof(struct tss_struct, __cacheline_filler) - 1,
SYS_DESC_tss_avail);
_set_tssldt_desc(
this_cpu(compat_gdt_table) + TSS_ENTRY - FIRST_RESERVED_GDT_ENTRY,
this_cpu(compat_gdt) + TSS_ENTRY - FIRST_RESERVED_GDT_ENTRY,
(unsigned long)tss,
offsetof(struct tss_struct, __cacheline_filler) - 1,
SYS_DESC_tss_busy);
@@ -2000,8 +2000,8 @@ void __init init_idt_traps(void)
/* CPU0 uses the master IDT. */
idt_tables[0] = idt_table;
this_cpu(gdt_table) = boot_cpu_gdt_table;
this_cpu(compat_gdt_table) = boot_cpu_compat_gdt_table;
this_cpu(gdt) = boot_gdt;
this_cpu(compat_gdt) = boot_compat_gdt;
}
extern void (*const autogen_entrypoints[NR_VECTORS])(void);
@@ -2029,13 +2029,11 @@ void __init trap_init(void)
}
}
/* Cache {,compat_}gdt_table_l1e now that physically relocation is done. */
this_cpu(gdt_table_l1e) =
l1e_from_pfn(virt_to_mfn(boot_cpu_gdt_table),
__PAGE_HYPERVISOR_RW);
this_cpu(compat_gdt_table_l1e) =
l1e_from_pfn(virt_to_mfn(boot_cpu_compat_gdt_table),
__PAGE_HYPERVISOR_RW);
/* Cache {,compat_}gdt_l1e now that physically relocation is done. */
this_cpu(gdt_l1e) =
l1e_from_pfn(virt_to_mfn(boot_gdt), __PAGE_HYPERVISOR_RW);
this_cpu(compat_gdt_l1e) =
l1e_from_pfn(virt_to_mfn(boot_compat_gdt), __PAGE_HYPERVISOR_RW);
percpu_traps_init();
xen/common/efi/runtime.c
+1 -1
View File
@@ -104,7 +104,7 @@ struct efi_rs_state efi_rs_enter(void)
{
struct desc_ptr gdt_desc = {
.limit = LAST_RESERVED_GDT_BYTE,
.base = (unsigned long)(per_cpu(gdt_table, smp_processor_id()) -
.base = (unsigned long)(per_cpu(gdt, smp_processor_id()) -
FIRST_RESERVED_GDT_ENTRY)
};
xen/include/asm-x86/desc.h
+6 -6
View File
@@ -204,12 +204,12 @@ struct __packed desc_ptr {
unsigned long base;
};
extern seg_desc_t boot_cpu_gdt_table[];
DECLARE_PER_CPU(seg_desc_t *, gdt_table);
DECLARE_PER_CPU(l1_pgentry_t, gdt_table_l1e);
extern seg_desc_t boot_cpu_compat_gdt_table[];
DECLARE_PER_CPU(seg_desc_t *, compat_gdt_table);
DECLARE_PER_CPU(l1_pgentry_t, compat_gdt_table_l1e);
extern seg_desc_t boot_gdt[];
DECLARE_PER_CPU(seg_desc_t *, gdt);
DECLARE_PER_CPU(l1_pgentry_t, gdt_l1e);
extern seg_desc_t boot_compat_gdt[];
DECLARE_PER_CPU(seg_desc_t *, compat_gdt);
DECLARE_PER_CPU(l1_pgentry_t, compat_gdt_l1e);
DECLARE_PER_CPU(bool, full_gdt_loaded);
extern void load_TR(void);
xen/include/asm-x86/ldt.h
+1 -2
View File
@@ -13,8 +13,7 @@ static inline void load_LDT(struct vcpu *v)
lldt(0);
else
{
desc = (!is_pv_32bit_vcpu(v)
? this_cpu(gdt_table) : this_cpu(compat_gdt_table))
desc = (!is_pv_32bit_vcpu(v) ? this_cpu(gdt) : this_cpu(compat_gdt))
+ LDT_ENTRY - FIRST_RESERVED_GDT_ENTRY;
_set_tssldt_desc(desc, LDT_VIRT_START(v), ents*8-1, SYS_DESC_ldt);
lldt(LDT_ENTRY << 3);
Reference in new issue
xen-project/xen
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.

The note is not visible to the blocked user.