On Mon, 2008-01-21 at 06:58 +0100, Andi Kleen wrote: > > { > > 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 >
Perhaps like this: if (l & (Tom2Enabled|Tom2ForceMemTypeWB)) return 1; Harvey -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to [EMAIL PROTECTED] More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/