[LinuxBIOS] [PATCH][v3] Spinlock fixes

Stefan Reinauer stepan at coresystems.de
Fri Jul 6 08:22:59 CEST 2007


* Uwe Hermann <uwe at hermann-uwe.de> [070706 02:05]:
> Various spinlock-related cleanups:
> 
>  - Revert back to spinlock_t (instead of struct spinlock). This is fine
>    in this special case, as the contents of spinlock_t are not meant to
>    ever be accessed directly (only by "accessor" functions).
 
NACK.

Why would that be required? The code in svn needs no fixing. Let's not
invent typedefs without any need for them

>  - Drop the spin_lock_string and spin_unlock_string macros, they're pretty
>    useless as they're only used in a single place and a macro doesn't make
>    this code any more readable, IMO.

NACK.

They are not useless at all as they are _the_ (one and only) spinlock
implementation on x86. The "only single place" is the declaration of the
x86 specific spin_lock and spin_unlock functions. 

-- 
coresystems GmbH • Brahmsstr. 16 • D-79104 Freiburg i. Br.
      Tel.: +49 761 7668825 • Fax: +49 761 7664613
Email: info at coresystems.dehttp://www.coresystems.de/
Registergericht: Amtsgericht Freiburg • HRB 7656
Geschäftsführer: Stefan Reinauer • Ust-IdNr.: DE245674866




More information about the coreboot mailing list