xen/arch/x86/acpi/cpufreq/cpufreq.c | 9 ++------- xen/arch/x86/hvm/mtrr.c | 6 ++---- xen/arch/x86/hvm/vpic.c | 5 ++--- xen/arch/x86/oprofile/op_model_athlon.c | 5 +---- xen/arch/x86/time.c | 5 +---- 5 files changed, 8 insertions(+), 22 deletions(-)
Compress two lines into a single line if immediate return statement is found.
It also remove variables retval, freq, effective, vector, ovf and now
as they are no longer needed.
Signed-off-by: Simran Singhal <singhalsimran0@gmail.com>
---
Changes in v2:
- Remove variable ovf as it is also not needed.
xen/arch/x86/acpi/cpufreq/cpufreq.c | 9 ++-------
xen/arch/x86/hvm/mtrr.c | 6 ++----
xen/arch/x86/hvm/vpic.c | 5 ++---
xen/arch/x86/oprofile/op_model_athlon.c | 5 +----
xen/arch/x86/time.c | 5 +----
5 files changed, 8 insertions(+), 22 deletions(-)
diff --git a/xen/arch/x86/acpi/cpufreq/cpufreq.c b/xen/arch/x86/acpi/cpufreq/cpufreq.c
index 281be131a3..f1f3c6923f 100644
--- a/xen/arch/x86/acpi/cpufreq/cpufreq.c
+++ b/xen/arch/x86/acpi/cpufreq/cpufreq.c
@@ -270,7 +270,6 @@ unsigned int get_measured_perf(unsigned int cpu, unsigned int flag)
struct cpufreq_policy *policy;
struct perf_pair readin, cur, *saved;
unsigned int perf_percent;
- unsigned int retval;
if (!cpu_online(cpu))
return 0;
@@ -318,16 +317,13 @@ unsigned int get_measured_perf(unsigned int cpu, unsigned int flag)
else
perf_percent = 0;
- retval = policy->cpuinfo.max_freq * perf_percent / 100;
-
- return retval;
+ return policy->cpuinfo.max_freq * perf_percent / 100;
}
static unsigned int get_cur_freq_on_cpu(unsigned int cpu)
{
struct cpufreq_policy *policy;
struct acpi_cpufreq_data *data;
- unsigned int freq;
if (!cpu_online(cpu))
return 0;
@@ -341,8 +337,7 @@ static unsigned int get_cur_freq_on_cpu(unsigned int cpu)
data->acpi_data == NULL || data->freq_table == NULL))
return 0;
- freq = extract_freq(get_cur_val(cpumask_of(cpu)), data);
- return freq;
+ return extract_freq(get_cur_val(cpumask_of(cpu)), data);
}
static void feature_detect(void *info)
diff --git a/xen/arch/x86/hvm/mtrr.c b/xen/arch/x86/hvm/mtrr.c
index 8356e8de3d..511c3be1c8 100644
--- a/xen/arch/x86/hvm/mtrr.c
+++ b/xen/arch/x86/hvm/mtrr.c
@@ -317,7 +317,7 @@ static uint8_t effective_mm_type(struct mtrr_state *m,
uint32_t pte_flags,
uint8_t gmtrr_mtype)
{
- uint8_t mtrr_mtype, pat_value, effective;
+ uint8_t mtrr_mtype, pat_value;
/* if get_pat_flags() gives a dedicated MTRR type,
* just use it
@@ -329,9 +329,7 @@ static uint8_t effective_mm_type(struct mtrr_state *m,
pat_value = page_pat_type(pat, pte_flags);
- effective = mm_type_tbl[mtrr_mtype][pat_value];
-
- return effective;
+ return mm_type_tbl[mtrr_mtype][pat_value];
}
uint32_t get_pat_flags(struct vcpu *v,
diff --git a/xen/arch/x86/hvm/vpic.c b/xen/arch/x86/hvm/vpic.c
index 4897a0e05b..61f4b6784c 100644
--- a/xen/arch/x86/hvm/vpic.c
+++ b/xen/arch/x86/hvm/vpic.c
@@ -484,7 +484,7 @@ void vpic_irq_negative_edge(struct domain *d, int irq)
int vpic_ack_pending_irq(struct vcpu *v)
{
- int irq, vector;
+ int irq;
struct hvm_hw_vpic *vpic = &v->domain->arch.hvm.vpic[0];
ASSERT(has_vpic(v->domain));
@@ -498,8 +498,7 @@ int vpic_ack_pending_irq(struct vcpu *v)
if ( irq == -1 )
return -1;
- vector = vpic[irq >> 3].irq_base + (irq & 7);
- return vector;
+ return vpic[irq >> 3].irq_base + (irq & 7);
}
/*
diff --git a/xen/arch/x86/oprofile/op_model_athlon.c b/xen/arch/x86/oprofile/op_model_athlon.c
index 5c48f868ae..ee6eb0ecae 100644
--- a/xen/arch/x86/oprofile/op_model_athlon.c
+++ b/xen/arch/x86/oprofile/op_model_athlon.c
@@ -315,7 +315,6 @@ static int athlon_check_ctrs(unsigned int const cpu,
{
uint64_t msr_content;
int i;
- int ovf = 0;
unsigned long eip = regs->rip;
int mode = 0;
struct vcpu *v = current;
@@ -339,13 +338,11 @@ static int athlon_check_ctrs(unsigned int const cpu,
if (CTR_OVERFLOWED(msr_content)) {
xenoprof_log_event(current, regs, eip, mode, i);
CTR_WRITE(reset_value[i], msrs, i);
- ovf = 1;
}
}
- ovf = handle_ibs(mode, regs);
/* See op_model_ppro.c */
- return ovf;
+ return handle_ibs(mode, regs);
}
static inline void start_ibs(void)
diff --git a/xen/arch/x86/time.c b/xen/arch/x86/time.c
index 2d4430b283..bbaea3aa65 100644
--- a/xen/arch/x86/time.c
+++ b/xen/arch/x86/time.c
@@ -1142,16 +1142,13 @@ s_time_t get_s_time_fixed(u64 at_tsc)
{
const struct cpu_time *t = &this_cpu(cpu_time);
u64 tsc, delta;
- s_time_t now;
if ( at_tsc )
tsc = at_tsc;
else
tsc = rdtsc_ordered();
delta = tsc - t->stamp.local_tsc;
- now = t->stamp.local_stime + scale_delta(delta, &t->tsc_scale);
-
- return now;
+ return t->stamp.local_stime + scale_delta(delta, &t->tsc_scale);
}
s_time_t get_s_time()
--
2.17.1
On Mon, Mar 30, 2020 at 05:53:08PM +0530, Simran Singhal wrote: > Compress two lines into a single line if immediate return statement is found. > It also remove variables retval, freq, effective, vector, ovf and now > as they are no longer needed. > > Signed-off-by: Simran Singhal <singhalsimran0@gmail.com> Reviewed-by: Wei Liu <wl@xen.org>
On 30.03.2020 15:18, Wei Liu wrote: > On Mon, Mar 30, 2020 at 05:53:08PM +0530, Simran Singhal wrote: >> Compress two lines into a single line if immediate return statement is found. >> It also remove variables retval, freq, effective, vector, ovf and now >> as they are no longer needed. >> >> Signed-off-by: Simran Singhal <singhalsimran0@gmail.com> > > Reviewed-by: Wei Liu <wl@xen.org> Acked-by: Jan Beulich <jbeulich@suse.com>
© 2016 - 2024 Red Hat, Inc.