On Tue, Mar 27, 2007 at 08:29:54AM -0500, Jack Steiner wrote:
> +     if (local_cpu_data->itc_freq != per_cpu(cpu_info, cpuid - 1).itc_freq ||
> +                     local_cpu_data->proc_freq != per_cpu(cpu_info, cpuid - 
> 1).proc_freq ||
> +                     local_cpu_data->features != per_cpu(cpu_info, cpuid - 
> 1).features ||
> +                     local_cpu_data->revision != per_cpu(cpu_info, cpuid - 
> 1).revision ||
> +                     local_cpu_data->family != per_cpu(cpu_info, cpuid - 
> 1).family ||
> +                     local_cpu_data->archrev != per_cpu(cpu_info, cpuid - 
> 1).archrev ||
> +                     local_cpu_data->model != per_cpu(cpu_info, cpuid - 
> 1).model)
> +             calibrate_delay();

If you use a temporary variable, you improve speed a little, and you
don't exceed 80 columns:

+       struct cpuinfo_ia64 *last_cpu_info = cpu_data(cpuid - 1);
+       struct cpuinfo_ia64 *this_cpu_info = local_cpu_data;
+       if (last_cpu_info->itc_freq != this_cpu_info->itc_freq ||
+           last_cpu_info->proc_freq != this_cpu_info->proc_freq ||
+           last_cpu_info->features != this_cpu_info->features ||
+           last_cpu_info->revision != this_cpu_info->revision ||
+           last_cpu_info->family != this_cpu_info->family ||
+           last_cpu_info->archrev != this_cpu_info->archrev ||
+           last_cpu_info->model != this_cpu_info->model)
+               calibrate_delay();

-
To unsubscribe from this list: send the line "unsubscribe linux-ia64" in
the body of a message to [EMAIL PROTECTED]
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Reply via email to