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

Hamish Guthrie hamish at prodigi.ch
Sat Sep 3 18:41:55 CEST 2005


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
> 




More information about the coreboot mailing list