|
Re: [PATCH v4] UefiCpuPkg/MtrrLib/UnitTest: Add host based unit test
Ray,
I worry that this style of testing will lead to inconsistant results. Generating random test cases means that the test cases on any given run could find a bug in this code without this code
Ray,
I worry that this style of testing will lead to inconsistant results. Generating random test cases means that the test cases on any given run could find a bug in this code without this code
|
By
Sean
·
#63409
·
|
|
Re: [PATCH v12 07/46] MdePkg/BaseLib: Add support for the VMGEXIT instruction
Yes -- either that, or even just open-code (copy) what we have in
MdePkg/Library/BaseLib/Ia32/CpuBreakpoint.nasm
now:
global ASM_PFX(AsmVmgExit)
ASM_PFX(AsmVmgExit):
;
; VMGEXIT makes no sense on
Yes -- either that, or even just open-code (copy) what we have in
MdePkg/Library/BaseLib/Ia32/CpuBreakpoint.nasm
now:
global ASM_PFX(AsmVmgExit)
ASM_PFX(AsmVmgExit):
;
; VMGEXIT makes no sense on
|
By
Laszlo Ersek
·
#63408
·
|
|
Re: TianoCore Bug Triage - APAC / NAMO - Tue, 07/28/2020 6:30pm-7:30pm
#cal-reminder
The following Bug will be reviewed this week. Welcome the reporter join the meeting to introduce the more background and detail.
The following Bug will be reviewed this week. Welcome the reporter join the meeting to introduce the more background and detail.
|
By
Liming Gao
·
#63407
·
|
|
Re: [PATCH v12 07/46] MdePkg/BaseLib: Add support for the VMGEXIT instruction
I have 2.13.02.
With version 2.13.02, I can successfully build with '-f elf32', so it
seems to be version related.
We are only enabling SEV-ES support for X64, so this is a valid point. It
would be
I have 2.13.02.
With version 2.13.02, I can successfully build with '-f elf32', so it
seems to be version related.
We are only enabling SEV-ES support for X64, so this is a valid point. It
would be
|
By
Lendacky, Thomas
·
#63406
·
|
|
Re: [PATCH v12 00/46] SEV-ES guest support
Thank you for explaining.
I've now filed a request with <support@groups.io> to raise the limit (on
this list only) to 200 messages per 30 minutes. That should allow people
to post large series *and*
Thank you for explaining.
I've now filed a request with <support@groups.io> to raise the limit (on
this list only) to 200 messages per 30 minutes. That should allow people
to post large series *and*
|
By
Laszlo Ersek
·
#63405
·
|
|
Re: [PATCH v12 07/46] MdePkg/BaseLib: Add support for the VMGEXIT instruction
Laszlo:
By
Liming Gao
·
#63404
·
|
|
Re: [PATCH v12 00/46] SEV-ES guest support
I've tried with and without batch-size and relogin-delay. My initial
submissions were without these parameters and I hit the 40 email limit. I
later tried a batch-size of 1 with a relogin-delay of 1
I've tried with and without batch-size and relogin-delay. My initial
submissions were without these parameters and I hit the 40 email limit. I
later tried a batch-size of 1 with a relogin-delay of 1
|
By
Lendacky, Thomas
·
#63403
·
|
|
Re: [PATCH v4] UefiCpuPkg/MtrrLib/UnitTest: Add host based unit test
Hi Ray,
I don't have comments on this patch; I've been giving my Acked-by to
confirm that I'm OK with this particular set of files being introduced /
modified under UefiCpuPkg. So I'll defer on
Hi Ray,
I don't have comments on this patch; I've been giving my Acked-by to
confirm that I'm OK with this particular set of files being introduced /
modified under UefiCpuPkg. So I'll defer on
|
By
Laszlo Ersek
·
#63402
·
|
|
Re: [PATCH v12 07/46] MdePkg/BaseLib: Add support for the VMGEXIT instruction
"tools_def.template" says:
NASM 2.10 or later for use with the GCC toolchain family
Bumping the NASM requirement from 2.10 to 2.12 will rule out:
- Debian "jessie" (oldoldstable),
- Ubuntu
"tools_def.template" says:
NASM 2.10 or later for use with the GCC toolchain family
Bumping the NASM requirement from 2.10 to 2.12 will rule out:
- Debian "jessie" (oldoldstable),
- Ubuntu
|
By
Laszlo Ersek
·
#63401
·
|
|
Re: [PATCH v5 4/5] BaseTools: Compile AML bytecode arrays into .obj file
Yes,
Do a git clean -fx in your edk2 directory, and delete your Build
directory completely.
Regards,
Leif
Yes,
Do a git clean -fx in your edk2 directory, and delete your Build
directory completely.
Regards,
Leif
|
By
Leif Lindholm
·
#63400
·
|
|
Re: [PATCH v12 00/46] SEV-ES guest support
Sigh.
Phil found one way to deal with this non-interactively: see the
"sendemail.smtpBatchSize" and "sendemail.smtpReloginDelay" git-config
options. From git-send-email(1):
Sigh.
Phil found one way to deal with this non-interactively: see the
"sendemail.smtpBatchSize" and "sendemail.smtpReloginDelay" git-config
options. From git-send-email(1):
|
By
Laszlo Ersek
·
#63399
·
|
|
Re: [PATCH v3] UefiCpuPkg/MtrrLib/UnitTest: Add host based unit test
Acked-by: Laszlo Ersek <lersek@...>
Acked-by: Laszlo Ersek <lersek@...>
|
By
Laszlo Ersek
·
#63398
·
|
|
Re: [PATCH 10/15] OvmfPkg/OvmfPkg.ci.yaml: Add configuration for LicenseCheck
Thanks!
So we should ignore "Bhyve/BhyveRfbDxe/VbeShim.h".
Thanks
Laszlo
Thanks!
So we should ignore "Bhyve/BhyveRfbDxe/VbeShim.h".
Thanks
Laszlo
|
By
Laszlo Ersek
·
#63397
·
|
|
Re: [PATCH v5 4/5] BaseTools: Compile AML bytecode arrays into .obj file
Hello Masahisa,
Maybe you will have to delete/re-generate the build_rule.txt and tools_def.txt files that you have. This patch makes the build system generate a .amli file for each .asl file. If
Hello Masahisa,
Maybe you will have to delete/re-generate the build_rule.txt and tools_def.txt files that you have. This patch makes the build system generate a .amli file for each .asl file. If
|
By
PierreGondois
·
#63396
·
|
|
Re: [PATCH v4] UefiCpuPkg/MtrrLib/UnitTest: Add host based unit test
Eric,
I just found another GCC build failure with my original C code.
Can you please review this patch again?
Thanks,
Ray
Eric,
I just found another GCC build failure with my original C code.
Can you please review this patch again?
Thanks,
Ray
|
By
Ni, Ray
·
#63395
·
|
|
[PATCH v4] UefiCpuPkg/MtrrLib/UnitTest: Add host based unit test
Add host based unit tests for the MtrrLib services.
The BaseLib services AsmCpuid(), AsmReadMsr64(), and
AsmWriteMsr64() are hooked and provide simple emulation
of the CPUID leafs and MSRs required by
Add host based unit tests for the MtrrLib services.
The BaseLib services AsmCpuid(), AsmReadMsr64(), and
AsmWriteMsr64() are hooked and provide simple emulation
of the CPUID leafs and MSRs required by
|
By
Ni, Ray
·
#63394
·
|
|
[PATCH v4 2/2] SecurityPkg/Tcg2Config: remove TPM2_ChangEPS if it is not supported.
REF: https://bugzilla.tianocore.org/show_bug.cgi?id=3D2793
In current implementation TPM2_ChangeEPS command is always available
in the TPM2 operation pull down list in TCG2 Configuration, which
is
REF: https://bugzilla.tianocore.org/show_bug.cgi?id=3D2793
In current implementation TPM2_ChangeEPS command is always available
in the TPM2 operation pull down list in TCG2 Configuration, which
is
|
By
Qi Zhang
·
#63393
·
|
|
[PATCH v4 1/2] SecurityPkg/Tpm2CommandLib: add a new function
From: "Zhang, Qi" <qi1.zhang@...>
Tpm2GetCapabilityIsCommandImplemented
REF: https://bugzilla.tianocore.org/show_bug.cgi?id=3D2793
check if the commad is supported by comparing the command
From: "Zhang, Qi" <qi1.zhang@...>
Tpm2GetCapabilityIsCommandImplemented
REF: https://bugzilla.tianocore.org/show_bug.cgi?id=3D2793
check if the commad is supported by comparing the command
|
By
Qi Zhang
·
#63392
·
|
|
[PATCH v4 0/2] refine TPM2 operation pull down list
v2 change: separate the change into two patches
v3 change: rename Tpm2GetCapabilityIsCmdImpl to Tpm2GetCapabilityIsCommandImplemented
v4 change: fix two build errors reported by CI
Cc: Jian J Wang
v2 change: separate the change into two patches
v3 change: rename Tpm2GetCapabilityIsCmdImpl to Tpm2GetCapabilityIsCommandImplemented
v4 change: fix two build errors reported by CI
Cc: Jian J Wang
|
By
Qi Zhang
·
#63391
·
|
|
Re: [PATCH v12 07/46] MdePkg/BaseLib: Add support for the VMGEXIT instruction
This error is reported from nasm compiler. My nasm compiler version is 2.11.08. It may be a little old. 2.12 should be fine.
This change also requires to update
This error is reported from nasm compiler. My nasm compiler version is 2.11.08. It may be a little old. 2.12 should be fine.
This change also requires to update
|
By
Liming Gao
·
#63390
·
|