Re: [PATCH] x86_64: update e820 instead of updating end_pfn v2

From: Andi Kleen
Date: Mon Jan 21 2008 - 00:58:25 EST



> {
> u32 l, h;
>
> @@ -661,8 +661,9 @@ static __init int amd_special_default_mt
> * Memory between 4GB and top of mem is forced WB by this magic bit.
> * Reserved before K8RevF, but should be zero there.
> */
> - if (l & Tom2ForceMemTypeWB)
> - return 1;
> + if (l & Tom2Enabled)
> + if (l & Tom2ForceMemTypeWB)
> + return 1;

That change is good agreed, but I would suggest to put it into a separate
patch with a description

-Andi
--
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/