Re: [LinuxBIOS] LNXI Merge: lnxi-patch-06/16

[Hamish Guthrie]

I would like to apologise for my previous tirade, but I have fixed this 
problem a few times in the past and the SMP guys just ignore 
UNDERSTANDING THE CODE!!!

In any event, the patch is crap!

Hamish

jason schildt wrote:
> DESCRIPTION:
>
> ## lnxi-patch-6 ##
> src/cpu/x86/tsc/delay_tsc.c
>     cpu_relax() gets called unconditionally.
> DIFFSTAT:
>
> delay_tsc.c |    4 ----
>  1 files changed, 4 deletions(-)
> PATCH:
> Index: delay_tsc.c
>
> --- delay_tsc.c (revision 1105)
> +++ delay_tsc.c (working copy)
> @@ -159,11 +159,7 @@
>          count = rdtscll();
>          stop = clocks + count;
>          while(stop > count) {
> -#ifdef CONFIG_SMP
> -#if CONFIG_SMP == 1
>                 cpu_relax();
> -#endif
> -#endif
>                 count = rdtscll();
>          }
>  }
>
> Jason W. Schildt
> LinuxBIOS Software Engineer
> Linux Networx