Re: Remove unused variable "cpu" from arch/x86/xen/setup.c

From: Ingo Molnar
Date: Tue May 13 2008 - 08:07:49 EST



* Manish Katiyar <mkatiyar@xxxxxxxxx> wrote:

> Remove unused variable "cpu" from arch/x86/xen/setup.c

it's not unused:

> void xen_enable_sysenter(void)
> {
> - int cpu = smp_processor_id();
> extern void xen_sysenter_target(void);
> /* Mask events on entry, even though they get enabled immediately */
> static struct callback_register sysenter = {

if (!boot_cpu_has(X86_FEATURE_SEP) ||
HYPERVISOR_callback_op(CALLBACKOP_register, &sysenter) != 0) {
clear_cpu_cap(&cpu_data(cpu), X86_FEATURE_SEP);
^^^^^^^^^^^^^^
clear_cpu_cap(&boot_cpu_data, X86_FEATURE_SEP);
}

if it generates an unused variable warning on UP then per_cpu() needs to
be fixed to do a (void)(cpu) to indicate use of that variable.

Ingo
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/