Re: [PATCH 3/8] SFI: core support

From: Len Brown
Date: Tue Jun 23 2009 - 12:58:21 EST


On Tue, 23 Jun 2009, Andi Kleen wrote:

> Len Brown <lenb@xxxxxxxxxx> writes:
>
> > +static ulong __init sfi_early_find_syst(void)
> > +{
> > + unsigned long i;
> > + char *pchar = (char *)SFI_SYST_SEARCH_BEGIN;
> > +
> > + for (i = 0; SFI_SYST_SEARCH_BEGIN + i < SFI_SYST_SEARCH_END; i += 16, pchar += 16) {
> > + if (!strncmp(SFI_SIG_SYST, pchar, SFI_SIGNATURE_SIZE))
> > + return SFI_SYST_SEARCH_BEGIN + i;
>
>
> Such additional memory scans are always a bit risky, e.g. if there's
> stray hardware there. Has it been verified that existing kernels
> already scan this area?

Yes, SFI is the same as ACPI here (actually, a proper sub-set).

Note that the UEFI folks have suggested that on UEFI systems,
that they hand us the address like they can do for ACPI,
and we'll probably add that at some point.

thanks,
-Len Brown, Intel Open Source Technology Center
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/