Fix ExecuteFinalHltInstruction function in f15h family code
Current ExecuteFinalHltInstruction function doesn't work well. (at least in configuration Supermicro board with Orochi AMD Opteron processors (model OS6234WKTCGGU)) System reboots when trying to halt core 2,4,6,8 or 10 (OS6234WKTCGGU is 12 core processor) Based on this information, i think that code doesn't really work with f15 compute unit (CU) system. Replacing ExecuteFinalHltInstruction function with analogous function from f15tn family code fix this problem. Both functions written from the same cahalt.asm file, but f15tn version seems more completed Change-Id: I3942abcdf21f1b86a44c01cc477714e44a40b9cf Signed-off-by: Kostr <aladyshev@nicevt.ru> Reviewed-on: http://review.coreboot.org/1569 Tested-by: build bot (Jenkins) Reviewed-by: Siyuan Wang <wangsiyuanbuaa@gmail.com> Reviewed-by: Marc Jones <marcj303@gmail.com>
This commit is contained in:
parent
80adfdf8a9
commit
335450d0a1
|
@ -88,6 +88,7 @@ agesa_lib_src += ./Proc/Common/CommonReturns.c
|
|||
agesa_lib_src += ./Proc/Common/CreateStruct.c
|
||||
agesa_lib_src += ./Proc/Common/S3RestoreState.c
|
||||
agesa_lib_src += ./Proc/Common/S3SaveState.c
|
||||
agesa_lib_src += ./Proc/CPU/cahaltasm.S
|
||||
agesa_lib_src += ./Proc/CPU/cahalt.c
|
||||
agesa_lib_src += ./Proc/CPU/cpuApicUtilities.c
|
||||
agesa_lib_src += ./Proc/CPU/cpuBist.c
|
||||
|
|
|
@ -102,152 +102,6 @@ ExecuteWbinvdInstruction (
|
|||
*/
|
||||
|
||||
|
||||
//----------------------------------------------------------------------------
|
||||
|
||||
STATIC
|
||||
VOID
|
||||
PrimaryCoreFunctions (AP_MTRR_SETTINGS *ApMtrrSettingsList)
|
||||
{
|
||||
UINT64 data;
|
||||
UINT32 msrno;
|
||||
// Configure the MTRRs on the AP so
|
||||
// when it runs remote code it will execute
|
||||
// out of RAM instead of ROM.
|
||||
// Disable MTRRs and turn on modification enable bit
|
||||
|
||||
data = __readmsr (0xC0010010); // MTRR_SYS_CFG
|
||||
data &= ~(1 << 18); // MtrrFixDramEn
|
||||
data &= ~(1 << 20); // MtrrVarDramEn
|
||||
data |= (1 << 19); // MtrrFixDramModEn
|
||||
data |= (1 << 17); // SysUcLockEn
|
||||
|
||||
|
||||
__writemsr (0xC0010010, data);
|
||||
|
||||
// Set 7FFFh-00000h and 9FFFFh-80000h as WB DRAM
|
||||
__writemsr (0x250, 0x1E1E1E1E1E1E1E1E); // AMD_MTRR_FIX64k_00000
|
||||
__writemsr (0x258, 0x1E1E1E1E1E1E1E1E); // AMD_MTRR_FIX16k_80000
|
||||
|
||||
// Set BFFFFh-A0000h, DFFFFh-C0000h as Uncacheable Memory-mapped IO
|
||||
__writemsr (0x259, 0); // AMD_AP_MTRR_FIX16k_A0000
|
||||
__writemsr (0x268, 0); // AMD_MTRR_FIX4k_C0000
|
||||
__writemsr (0x269, 0); // AMD_MTRR_FIX4k_C8000
|
||||
__writemsr (0x26A, 0); // AMD_MTRR_FIX4k_D0000
|
||||
__writemsr (0x26B, 0); // AMD_MTRR_FIX4k_D8000
|
||||
|
||||
// Set FFFFFh-E0000h as Uncacheable Memory
|
||||
for (msrno = 0x26C; msrno <= 0x26F; msrno++)
|
||||
__writemsr (msrno, 0x1818181818181818);
|
||||
|
||||
// If IBV provided settings for Fixed-Sized MTRRs,
|
||||
// overwrite the default settings.
|
||||
if ((uintptr_t) ApMtrrSettingsList != 0 && (uintptr_t) ApMtrrSettingsList != 0xFFFFFFFF)
|
||||
{
|
||||
int index;
|
||||
for (index = 0; ApMtrrSettingsList [index].MsrAddr != CPU_LIST_TERMINAL; index++)
|
||||
__writemsr (ApMtrrSettingsList [index].MsrAddr, ApMtrrSettingsList [index].MsrData);
|
||||
}
|
||||
|
||||
// restore variable MTTR6 and MTTR7 to default states
|
||||
for (msrno = 0x20F; msrno <= 0x20C; msrno--) // decrement so that the pair is disable before the base is cleared
|
||||
__writemsr (msrno, 0);
|
||||
|
||||
// Enable fixed-range and variable-range MTRRs
|
||||
// Set Fixed-Range Enable (FE) and MTRR Enable (E) bits
|
||||
__writemsr (0x2FF, __readmsr (0x2FF) | 0xC00);
|
||||
|
||||
// Enable Top-of-Memory setting
|
||||
// Enable use of RdMem/WrMem bits attributes
|
||||
data = __readmsr (0xC0010010); // MTRR_SYS_CFG
|
||||
data |= (1 << 18); // MtrrFixDramEn
|
||||
data |= (1 << 20); // MtrrVarDramEn
|
||||
data &= ~(1 << 19); // MtrrFixDramModEn
|
||||
__writemsr (0xC0010010, data);
|
||||
}
|
||||
|
||||
//----------------------------------------------------------------------------
|
||||
|
||||
VOID
|
||||
ExecuteFinalHltInstruction (
|
||||
IN UINT32 SharedCore,
|
||||
IN AP_MTRR_SETTINGS *ApMtrrSettingsList,
|
||||
IN AMD_CONFIG_PARAMS *StdHeader
|
||||
)
|
||||
{
|
||||
int abcdRegs [4];
|
||||
UINT32 cr0val;
|
||||
UINT64 data;
|
||||
|
||||
cr0val = __readcr0 ();
|
||||
if (SharedCore & 2)
|
||||
{
|
||||
// set CombineCr0Cd and enable cache in CR0
|
||||
__writemsr (MSR_CU_CFG3, __readmsr (MSR_CU_CFG3) | 1ULL << 49);
|
||||
__writecr0 (cr0val & ~0x60000000);
|
||||
}
|
||||
else
|
||||
__writecr0 (cr0val | 0x60000000);
|
||||
|
||||
if (SharedCore & 1) PrimaryCoreFunctions (ApMtrrSettingsList);
|
||||
|
||||
// Make sure not to touch any Shared MSR from this point on
|
||||
|
||||
// Restore settings that were temporarily overridden for the cache as ram phase
|
||||
data = __readmsr (0xC0011022); // MSR_DC_CFG
|
||||
data &= ~(1 << 4); // DC_DIS_SPEC_TLB_RLD
|
||||
data &= ~(1 << 8); // DIS_CLR_WBTOL2_SMC_HIT
|
||||
data &= ~(1 << 13); // DIS_HW_PF
|
||||
__writemsr (0xC0011022, data);
|
||||
|
||||
data = __readmsr (0xC0011021); // MSR_IC_CFG - C001_1021
|
||||
data &= ~(1 << 9); // IC_DIS_SPEC_TLB_RLD
|
||||
__writemsr (0xC0011021, data);
|
||||
|
||||
// AMD_DISABLE_STACK_FAMILY_HOOK
|
||||
__cpuid (abcdRegs, 1);
|
||||
if ((abcdRegs [0] >> 20) == 1) //-----family 10h (Hydra) only-----
|
||||
{
|
||||
data = __readmsr (0xC0011022);
|
||||
data &= ~(1 << 4);
|
||||
data &= ~(1 << 8);
|
||||
data &= ~(1 << 13);
|
||||
__writemsr (0xC0011022, data);
|
||||
|
||||
data = __readmsr (0xC0011021);
|
||||
data &= ~(1 << 14);
|
||||
data &= ~(1 << 9);
|
||||
__writemsr (0xC0011021, data);
|
||||
|
||||
data = __readmsr (0xC001102A);
|
||||
data &= ~(1 << 15);
|
||||
data &= ~(1ull << 35);
|
||||
__writemsr (0xC001102A, data);
|
||||
}
|
||||
else if ((abcdRegs [0] >> 20) == 6) //-----family 15h (Orochi) only-----
|
||||
{
|
||||
data = __readmsr (0xC0011020);
|
||||
data &= ~(1 << 28);
|
||||
__writemsr (0xC0011020, data);
|
||||
|
||||
data = __readmsr (0xC0011021);
|
||||
data &= ~(1 << 9);
|
||||
__writemsr (0xC0011021, data);
|
||||
|
||||
data = __readmsr (0xC0011022);
|
||||
data &= ~(1 << 4);
|
||||
data &= ~(1l << 13);
|
||||
__writemsr (0xC0011022, data);
|
||||
}
|
||||
|
||||
for (;;)
|
||||
{
|
||||
_disable ();
|
||||
__halt ();
|
||||
}
|
||||
}
|
||||
|
||||
//----------------------------------------------------------------------------
|
||||
|
||||
/// Structure needed to load the IDTR using the lidt instruction
|
||||
|
||||
VOID
|
||||
|
|
|
@ -0,0 +1,209 @@
|
|||
/*
|
||||
* Copyright (c) 2011, Advanced Micro Devices, Inc.
|
||||
* All rights reserved.
|
||||
*
|
||||
* Redistribution and use in source and binary forms, with or without
|
||||
* modification, are permitted provided that the following conditions are met:
|
||||
* * Redistributions of source code must retain the above copyright
|
||||
* notice, this list of conditions and the following disclaimer.
|
||||
* * Redistributions in binary form must reproduce the above copyright
|
||||
* notice, this list of conditions and the following disclaimer in the
|
||||
* documentation and/or other materials provided with the distribution.
|
||||
* * Neither the name of Advanced Micro Devices, Inc. nor the names of
|
||||
* its contributors may be used to endorse or promote products derived
|
||||
* from this software without specific prior written permission.
|
||||
*
|
||||
* THIS SOFTWARE IS PROVIDED BY THE COPYRIGHT HOLDERS AND CONTRIBUTORS "AS IS" AND
|
||||
* ANY EXPRESS OR IMPLIED WARRANTIES, INCLUDING, BUT NOT LIMITED TO, THE IMPLIED
|
||||
* WARRANTIES OF MERCHANTABILITY AND FITNESS FOR A PARTICULAR PURPOSE ARE
|
||||
* DISCLAIMED. IN NO EVENT SHALL ADVANCED MICRO DEVICES, INC. BE LIABLE FOR ANY
|
||||
* DIRECT, INDIRECT, INCIDENTAL, SPECIAL, EXEMPLARY, OR CONSEQUENTIAL DAMAGES
|
||||
* (INCLUDING, BUT NOT LIMITED TO, PROCUREMENT OF SUBSTITUTE GOODS OR SERVICES;
|
||||
* LOSS OF USE, DATA, OR PROFITS; OR BUSINESS INTERRUPTION) HOWEVER CAUSED AND
|
||||
* ON ANY THEORY OF LIABILITY, WHETHER IN CONTRACT, STRICT LIABILITY, OR TORT
|
||||
* (INCLUDING NEGLIGENCE OR OTHERWISE) ARISING IN ANY WAY OUT OF THE USE OF THIS
|
||||
* SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE.
|
||||
*
|
||||
*/
|
||||
|
||||
.include "src/vendorcode/amd/agesa/f15/gcccar.inc"
|
||||
|
||||
.code32
|
||||
.align 4
|
||||
.globl ExecuteFinalHltInstruction
|
||||
.type ExecuteFinalHltInstruction, @function
|
||||
/* ExecuteFinalHltInstruction (
|
||||
IN UINT32 HaltFlags,
|
||||
IN AP_MTRR_SETTINGS *ApMtrrSettingsList,
|
||||
IN AMD_CONFIG_PARAMS *StdHeader
|
||||
)
|
||||
*/
|
||||
|
||||
/* This function disables CAR. We don't care about the stack on this CPU */
|
||||
ExecuteFinalHltInstruction:
|
||||
movl 4(%esp), %esi /* HaltFlags*/
|
||||
movl 8(%esp), %edi /* ApMtrrSettingList */
|
||||
//1: jmp 1b //good
|
||||
/* Do these special steps in case if the core is part of a compute unit
|
||||
* Note: The following bits are family specific flags, that gets set during build time,
|
||||
* and indicates things like "family cache control methodology", etc.
|
||||
* esi bit0 = 0 -> not a Primary core
|
||||
* esi bit0 = 1 -> Primary core
|
||||
* esi bit1 = 0 -> Cache disable
|
||||
* esi bit1 = 1 -> Cache enable
|
||||
*/
|
||||
|
||||
bt $1, %esi /* .if (esi & 2h) */
|
||||
jz 0f
|
||||
/* Set CombineCr0Cd bit */
|
||||
movl $CU_CFG3, %ecx
|
||||
rdmsr
|
||||
bts $(COMBINE_CR0_CD - 32), %edx
|
||||
wrmsr
|
||||
/* Clear the CR0.CD bit */
|
||||
movl %cr0, %eax /* Make sure cache is enabled for all APs */
|
||||
btr $CR0_CD, %eax
|
||||
btr $CR0_NW, %eax
|
||||
mov %eax, %cr0 /* Write back to CR0 */
|
||||
jmp 1f /* .else */
|
||||
0:
|
||||
movl %cr0, %eax /* Make sure cache is disabled for all APs */
|
||||
bts $CR0_CD, %eax /* Disable cache */
|
||||
bts $CR0_NW, %eax
|
||||
movl %eax, %cr0 /* Write back to CR0 */
|
||||
1: /* .endif */
|
||||
|
||||
// jmp 1b
|
||||
bt $0, %esi /* .if (esi & 1h) */
|
||||
jz 2f
|
||||
/* This core is a primary core and needs to do all the MTRRs, including shared MTRRs. */
|
||||
movl %edi, %esi /* Get ApMtrrSettingList */
|
||||
|
||||
/* Configure the MTRRs on the AP so
|
||||
* when it runs remote code it will execute
|
||||
* out of RAM instead of ROM.
|
||||
*/
|
||||
|
||||
/* Disable MTRRs and turn on modification enable bit */
|
||||
movl $MTRR_SYS_CFG, %ecx
|
||||
rdmsr
|
||||
/* TODO: why comment this? */
|
||||
//btr $MTRR_VAR_DRAM_EN, %eax /* Disable */
|
||||
bts $MTRR_FIX_DRAM_MOD_EN, %eax /* Enable */
|
||||
btr $MTRR_FIX_DRAM_EN, %eax /* Disable */
|
||||
//bts $SYS_UC_LOCK_EN, %eax
|
||||
wrmsr
|
||||
|
||||
/* Setup default values for Fixed-Sized MTRRs */
|
||||
/* Set 7FFFh-00000h as WB */
|
||||
movl $AMD_AP_MTRR_FIX64k_00000, %ecx
|
||||
movl $0x1E1E1E1E, %eax
|
||||
movl %eax, %edx
|
||||
wrmsr
|
||||
|
||||
/* Set 9FFFFh-80000h also as WB */
|
||||
movl $AMD_AP_MTRR_FIX16k_80000, %ecx
|
||||
wrmsr
|
||||
|
||||
/* Set BFFFFh-A0000h as Uncacheable Memory-mapped IO */
|
||||
movl $AMD_AP_MTRR_FIX16k_A0000, %ecx
|
||||
xorl %eax, %eax
|
||||
xorl %edx, %edx
|
||||
wrmsr
|
||||
|
||||
/* Set DFFFFh-C0000h as Uncacheable Memory-mapped IO */
|
||||
xorl %eax, %eax
|
||||
xorl %edx, %edx
|
||||
movl $AMD_AP_MTRR_FIX4k_C0000, %ecx
|
||||
|
||||
CDLoop:
|
||||
wrmsr
|
||||
inc %ecx
|
||||
cmp $AMD_AP_MTRR_FIX4k_D8000, %ecx
|
||||
jbe CDLoop
|
||||
|
||||
/* Set FFFFFh-E0000h as Uncacheable Memory */
|
||||
movl $0x18181818, %eax
|
||||
movl %eax, %edx
|
||||
|
||||
mov $AMD_AP_MTRR_FIX4k_E0000, %ecx
|
||||
|
||||
EFLoop:
|
||||
wrmsr
|
||||
inc %ecx
|
||||
cmp $AMD_AP_MTRR_FIX4k_F8000, %ecx
|
||||
jbe EFLoop
|
||||
|
||||
/* If IBV provided settings for Fixed-Sized MTRRs,
|
||||
* overwrite the default settings. */
|
||||
cmp $0, %esi /*.if ((esi != 0) && (esi != 0FFFFFFFFh)) */
|
||||
jz 4f
|
||||
cmp $0xFFFFFFFF, %esi
|
||||
jz 4f
|
||||
5:
|
||||
mov (%esi), %ecx /* (AP_MTRR_SETTINGS ptr [esi]).MsrAddr */
|
||||
/* While we are not at the end of the list */
|
||||
cmp $CPU_LIST_TERMINAL, %ecx /* .while (ecx != CPU_LIST_TERMINAL)*/
|
||||
je 4f
|
||||
/* TODO - coreboot isn't checking for valid data.
|
||||
* Ensure that the MSR address is valid for Fixed-Sized MTRRs */
|
||||
/*.if ( ((ecx >= AMD_AP_MTRR_FIX4k_C0000) && (ecx <= AMD_AP_MTRR_FIX4k_F8000)) || \
|
||||
(ecx == AMD_AP_MTRR_FIX64k_00000) || (ecx == AMD_AP_MTRR_FIX16k_80000 ) || \
|
||||
(ecx == AMD_AP_MTRR_FIX16k_A0000))
|
||||
*/
|
||||
mov 4(%esi), %eax /* MsrData */
|
||||
mov 8(%esi), %edx /* MsrData */
|
||||
wrmsr
|
||||
/* .endif */
|
||||
add $12, %esi /* sizeof (AP_MTRR_SETTINGS) */
|
||||
jmp 5b /* .endw */
|
||||
4: /* .endif */
|
||||
|
||||
/* Enable fixed-range and variable-range MTRRs */
|
||||
mov $AMD_MTRR_DEFTYPE, %ecx
|
||||
rdmsr
|
||||
bts $MTRR_DEF_TYPE_EN, %eax /* MtrrDefTypeEn */
|
||||
bts $MTRR_DEF_TYPE_FIX_EN, %eax /* MtrrDefTypeFixEn */
|
||||
wrmsr
|
||||
|
||||
/* Enable Top-of-Memory setting */
|
||||
/* Enable use of RdMem/WrMem bits attributes */
|
||||
mov $MTRR_SYS_CFG, %ecx
|
||||
rdmsr
|
||||
/* TODO: */
|
||||
//bts $MTRR_VAR_DRAM_EN, %eax /* Enable */
|
||||
btr $MTRR_FIX_DRAM_MOD_EN, %eax /* Disable */
|
||||
bts $MTRR_FIX_DRAM_EN, %eax /* Enable */
|
||||
wrmsr
|
||||
|
||||
bts $FLAG_IS_PRIMARY, %esi
|
||||
jmp 3f /* .else ; end if primary core */
|
||||
2:
|
||||
xor %esi, %esi
|
||||
3: /* .endif*/
|
||||
|
||||
//8: jmp 8b //bad
|
||||
/* Make sure not to touch any Shared MSR from this point on */
|
||||
|
||||
AMD_DISABLE_STACK_FAMILY_HOOK
|
||||
|
||||
/* restore variable MTTR6 and MTTR7 to default states */
|
||||
bt $FLAG_IS_PRIMARY, %esi /* .if (esi & 1h) */
|
||||
jz 6f
|
||||
movl $AMD_MTRR_VARIABLE_MASK7, %ecx /* clear MTRRPhysBase6 MTRRPhysMask6 */
|
||||
xor %eax, %eax /* and MTRRPhysBase7 MTRRPhysMask7 */
|
||||
xor %edx, %edx
|
||||
cmp $AMD_MTRR_VARIABLE_BASE6, %ecx /* .while (cl < 010h) */
|
||||
jl 6f
|
||||
wrmsr
|
||||
dec %ecx
|
||||
6: /* .endw */
|
||||
|
||||
xor %eax, %eax
|
||||
|
||||
7:
|
||||
cli
|
||||
hlt
|
||||
jmp 7b /* ExecuteHltInstruction */
|
||||
|
||||
.size ExecuteFinalHltInstruction, .-ExecuteFinalHltInstruction
|
|
@ -49,6 +49,7 @@ APIC_BASE_ADDRESS = 0x0000001B
|
|||
|
||||
AMD_MTRR_VARIABLE_BASE0 = 0x0200
|
||||
AMD_MTRR_VARIABLE_BASE6 = 0x020C
|
||||
AMD_MTRR_VARIABLE_MASK7 = 0x020F
|
||||
AMD_MTRR_FIX64k_00000 = 0x0250
|
||||
AMD_MTRR_FIX16k_80000 = 0x0258
|
||||
AMD_MTRR_FIX16k_A0000 = 0x0259
|
||||
|
@ -61,6 +62,20 @@ AMD_MTRR_FIX4k_E8000 = 0x026D
|
|||
AMD_MTRR_FIX4k_F0000 = 0x026E
|
||||
AMD_MTRR_FIX4k_F8000 = 0x026F
|
||||
|
||||
/* Reproduced from AGESA.h */
|
||||
AMD_AP_MTRR_FIX64k_00000 = 0x00000250
|
||||
AMD_AP_MTRR_FIX16k_80000 = 0x00000258
|
||||
AMD_AP_MTRR_FIX16k_A0000 = 0x00000259
|
||||
AMD_AP_MTRR_FIX4k_C0000 = 0x00000268
|
||||
AMD_AP_MTRR_FIX4k_C8000 = 0x00000269
|
||||
AMD_AP_MTRR_FIX4k_D0000 = 0x0000026A
|
||||
AMD_AP_MTRR_FIX4k_D8000 = 0x0000026B
|
||||
AMD_AP_MTRR_FIX4k_E0000 = 0x0000026C
|
||||
AMD_AP_MTRR_FIX4k_E8000 = 0x0000026D
|
||||
AMD_AP_MTRR_FIX4k_F0000 = 0x0000026E
|
||||
AMD_AP_MTRR_FIX4k_F8000 = 0x0000026F
|
||||
CPU_LIST_TERMINAL = 0xFFFFFFFF
|
||||
|
||||
AMD_MTRR_DEFTYPE = 0x02FF
|
||||
WB_DRAM_TYPE = 0x1E /* MemType - memory type */
|
||||
MTRR_DEF_TYPE_EN = 11 /* MtrrDefTypeEn - variable and fixed MTRRs default enabled */
|
||||
|
|
Loading…
Reference in New Issue