Skip to content

Commit d40159a

Browse files
committed
remove QEMU_CORE_NUM support for SBSA as it is hardcoded anyway...
1 parent c1a6ff0 commit d40159a

File tree

3 files changed

+5
-4
lines changed

3 files changed

+5
-4
lines changed

Platforms/QemuSbsaPkg/PlatformBuild.py

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -322,7 +322,6 @@ def SetPlatformEnv(self):
322322
self.env.SetValue("BUILDREPORT_TYPES", "PCD DEPEX FLASH BUILD_FLAGS LIBRARY FIXED_ADDRESS HASH", "Setting build report types")
323323
self.env.SetValue("ARM_TFA_PATH", str(Path(self.GetWorkspaceRoot()) / "Silicon" / "Arm" / "TFA"), "Platform hardcoded")
324324
self.env.SetValue("ARM_HAF_PATH", str(Path(self.GetWorkspaceRoot()) / "Silicon" / "Arm" / "HAF"), "Platform hardcoded")
325-
self.env.SetValue("BLD_*_QEMU_CORE_NUM", "4", "Default")
326325
self.env.SetValue("BLD_*_MEMORY_PROTECTION", "TRUE", "Default")
327326
# Include the MFCI test cert by default, override on the commandline with "BLD_*_SHIP_MODE=TRUE" if you want the retail MFCI cert
328327
self.env.SetValue("BLD_*_SHIP_MODE", "FALSE", "Default")

Platforms/QemuSbsaPkg/Plugins/QemuRunner/QemuRunner.py

Lines changed: 4 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -117,8 +117,10 @@ def Runner(env):
117117

118118
args += " -machine sbsa-ref" #,accel=(tcg|kvm)"
119119
args += " -cpu max,sve=off,sme=off"
120-
if env.GetBuildValue ("QEMU_CORE_NUM") is not None:
121-
args += " -smp " + env.GetBuildValue ("QEMU_CORE_NUM")
120+
# Hardcoded to 4 cores for now, changing this also needs to pair with the ARM_CORE_INFO data
121+
# in SbsaQemuLib.c, otherwise the MP services protocol will be initialized with the wrong number
122+
# of cores and cause issues with some tests that rely on it.
123+
args += " -smp 4"
122124
args += " -global driver=cfi.pflash01,property=secure,value=on"
123125
args += " -drive if=pflash,format=raw,unit=0,file=" + \
124126
os.path.join(OutputPath_FV, "SECURE_FLASH0.fd")

Platforms/QemuSbsaPkg/QemuSbsaPkg.dsc

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -653,7 +653,7 @@
653653

654654
# Size of the region used by UEFI in permanent memory (Reserved 64MB)
655655
gArmPlatformTokenSpaceGuid.PcdSystemMemoryUefiRegionSize|0x04000000
656-
gArmPlatformTokenSpaceGuid.PcdCoreCount|$(QEMU_CORE_NUM)
656+
gArmPlatformTokenSpaceGuid.PcdCoreCount|4
657657

658658
#
659659
# ARM PrimeCell

0 commit comments

Comments
 (0)