lkml.org 
[lkml]   [2006]   [Mar]   [4]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
Date
From
SubjectRe: [PATCH 1/1] EFI: Fix gdt load
Zwane Mwaikambo <zwane@arm.linux.org.uk> wrote:
>
> On Thu, 2 Mar 2006, Edgar Hucek wrote:
>
> > This patch makes the kernel bootable again on ia32 EFI systems.
> >
> > Signed-off-by: Edgar Hucek <hostmaster@ed-soft.at>
> >
>
> spin_lock(&efi_rt_lock);
> local_irq_save(efi_rt_eflags);
>
> That looks like a race, not to mention that efi_rt_eflags is a global
> variable. The same strange ordering occurs on unlock, i presume this code
> 'works' because it's done early during boot?

No, the code's OK. This is the prologue function. The epilogue function
does the opposite. In between, this CPU will dink around with EFI stuff.

And the order is correct too - the lock protects efi_rt_eflags.
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/

\
 
 \ /
  Last update: 2006-03-05 03:49    [W:0.957 / U:0.096 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site