Signed-off-by: Jan Beulich --- a/xen/arch/x86/cpu/intel_cacheinfo.c +++ b/xen/arch/x86/cpu/intel_cacheinfo.c @@ -170,7 +170,6 @@ unsigned int __cpuinit init_intel_cachei unsigned int trace = 0, l1i = 0, l1d = 0, l2 = 0, l3 = 0; /* Cache sizes */ unsigned int new_l1d = 0, new_l1i = 0; /* Cache sizes from cpuid(4) */ unsigned int new_l2 = 0, new_l3 = 0, i; /* Cache sizes from cpuid(4) */ - unsigned int l2_id = 0, l3_id = 0, num_threads_sharing, index_msb; if (c->cpuid_level > 3) { static int is_initialized; @@ -203,15 +202,9 @@ unsigned int __cpuinit init_intel_cachei break; case 2: new_l2 = this_leaf.size/1024; - num_threads_sharing = 1 + this_leaf.eax.split.num_threads_sharing; - index_msb = get_count_order(num_threads_sharing); - l2_id = c->apicid >> index_msb; break; case 3: new_l3 = this_leaf.size/1024; - num_threads_sharing = 1 + this_leaf.eax.split.num_threads_sharing; - index_msb = get_count_order(num_threads_sharing); - l3_id = c->apicid >> index_msb; break; default: break; --- a/xen/arch/x86/hvm/emulate.c +++ b/xen/arch/x86/hvm/emulate.c @@ -58,11 +58,10 @@ static int hvmemul_do_io( ioreq_t *p = get_ioreq(curr); unsigned long ram_gfn = paddr_to_pfn(ram_gpa); p2m_type_t p2mt; - mfn_t ram_mfn; int rc; /* Check for paged out page */ - ram_mfn = gfn_to_mfn_unshare(curr->domain, ram_gfn, &p2mt); + gfn_to_mfn_unshare(curr->domain, ram_gfn, &p2mt); if ( p2m_is_paging(p2mt) ) { p2m_mem_paging_populate(curr->domain, ram_gfn); --- a/xen/arch/x86/hvm/hvm.c +++ b/xen/arch/x86/hvm/hvm.c @@ -3782,8 +3782,8 @@ long do_hvm_op(unsigned long op, XEN_GUE { p2m_type_t t; p2m_type_t nt; - mfn_t mfn; - mfn = gfn_to_mfn_unshare(d, pfn, &t); + + gfn_to_mfn_unshare(d, pfn, &t); if ( p2m_is_paging(t) ) { p2m_mem_paging_populate(d, pfn); --- a/xen/arch/x86/hvm/viridian.c +++ b/xen/arch/x86/hvm/viridian.c @@ -323,6 +323,9 @@ int viridian_hypercall(struct cpu_user_r status = HV_STATUS_SUCCESS; break; default: + /* Silence compiler warnings. */ + (void)input_params_gpa; + (void)output_params_gpa; status = HV_STATUS_INVALID_HYPERCALL_CODE; break; } --- a/xen/arch/x86/mm/p2m-pod.c +++ b/xen/arch/x86/mm/p2m-pod.c @@ -1078,7 +1078,6 @@ guest_physmap_mark_populate_on_demand(st struct p2m_domain *p2m = p2m_get_hostp2m(d); unsigned long i; p2m_type_t ot; - mfn_t omfn; int pod_count = 0; int rc = 0; @@ -1096,7 +1095,7 @@ guest_physmap_mark_populate_on_demand(st /* Make sure all gpfns are unused */ for ( i = 0; i < (1UL << order); i++ ) { - omfn = gfn_to_mfn_query(d, gfn + i, &ot); + gfn_to_mfn_query(d, gfn + i, &ot); if ( p2m_is_ram(ot) ) { printk("%s: gfn_to_mfn returned type %d!\n", --- a/xen/arch/x86/time.c +++ b/xen/arch/x86/time.c @@ -1012,8 +1012,7 @@ static void local_time_calibration(void) * System timestamps, extrapolated from local and master oscillators, * taken during this calibration and the previous calibration. */ - s_time_t prev_local_stime, curr_local_stime; - s_time_t prev_master_stime, curr_master_stime; + s_time_t curr_local_stime, prev_master_stime, curr_master_stime; /* TSC timestamps taken during this calibration and prev calibration. */ u64 prev_tsc, curr_tsc; @@ -1050,7 +1049,6 @@ static void local_time_calibration(void) } prev_tsc = t->local_tsc_stamp; - prev_local_stime = t->stime_local_stamp; prev_master_stime = t->stime_master_stamp; /* Disabling IRQs ensures we atomically read cpu_calibration struct. */ @@ -1062,7 +1060,7 @@ static void local_time_calibration(void) #if 0 printk("PRE%d: tsc=%"PRIu64" stime=%"PRIu64" master=%"PRIu64"\n", - smp_processor_id(), prev_tsc, prev_local_stime, prev_master_stime); + smp_processor_id(), prev_tsc, t->stime_local_stamp, prev_master_stime); printk("CUR%d: tsc=%"PRIu64" stime=%"PRIu64" master=%"PRIu64 " -> %"PRId64"\n", smp_processor_id(), curr_tsc, curr_local_stime, curr_master_stime, --- a/xen/common/cpupool.c +++ b/xen/common/cpupool.c @@ -35,7 +35,7 @@ static DEFINE_SPINLOCK(cpupool_lock); DEFINE_PER_CPU(struct cpupool *, cpupool); -#define cpupool_dprintk(x...) ((void)0) +static inline void cpupool_dprintk(const char *fmt, ...) { } static struct cpupool *alloc_cpupool_struct(void) { --- a/xen/common/kexec.c +++ b/xen/common/kexec.c @@ -594,7 +594,7 @@ static int kexec_exec(XEN_GUEST_HANDLE(v break; } - return -EINVAL; /* never reached */ + return ret; /* never reached */ } int do_kexec_op_internal(unsigned long op, XEN_GUEST_HANDLE(void) uarg, --- a/xen/common/unlzo.c +++ b/xen/common/unlzo.c @@ -68,7 +68,6 @@ static int INIT parse_header(u8 *input, { int l; u8 *parse = input; - u8 level = 0; u16 version; /* read magic: 9 first bits */ @@ -82,7 +81,7 @@ static int INIT parse_header(u8 *input, version = get_unaligned_be16(parse); parse += 7; if (version >= 0x0940) - level = *parse++; + parse++; if (get_unaligned_be32(parse) & HEADER_HAS_FILTER) parse += 8; /* flags + filter info */ else