diff options
author | Laszlo Ersek <lersek@redhat.com> | 2018-03-01 15:10:04 +0100 |
---|---|---|
committer | Laszlo Ersek <lersek@redhat.com> | 2018-03-06 13:30:05 +0100 |
commit | cde8c56809176269419f02a7884d31eb177d99a1 (patch) | |
tree | 7cf1b11f7bc31eb40ee84eb42e5d9a3111bbfd78 /OvmfPkg | |
parent | 154dcd6c58a397a6d31a8fe0bc3dab70b916f33a (diff) | |
download | edk2-cde8c56809176269419f02a7884d31eb177d99a1.tar.gz edk2-cde8c56809176269419f02a7884d31eb177d99a1.tar.bz2 edk2-cde8c56809176269419f02a7884d31eb177d99a1.zip |
OvmfPkg/MemEncryptSevLib: clean up SetMemoryEncDec() comment block
Document the "Cr3BaseAddress" parameter, and correct several parameter
references.
Cc: Ard Biesheuvel <ard.biesheuvel@linaro.org>
Cc: Brijesh Singh <brijesh.singh@amd.com>
Cc: Jordan Justen <jordan.l.justen@intel.com>
Contributed-under: TianoCore Contribution Agreement 1.1
Signed-off-by: Laszlo Ersek <lersek@redhat.com>
Tested-by: Brijesh Singh <brijesh.singh@amd.com>
Reviewed-by: Brijesh Singh <brijesh.singh@amd.com>
Diffstat (limited to 'OvmfPkg')
-rw-r--r-- | OvmfPkg/Library/BaseMemEncryptSevLib/X64/VirtualMemory.c | 8 |
1 files changed, 5 insertions, 3 deletions
diff --git a/OvmfPkg/Library/BaseMemEncryptSevLib/X64/VirtualMemory.c b/OvmfPkg/Library/BaseMemEncryptSevLib/X64/VirtualMemory.c index 65b8babaac..aed9212762 100644 --- a/OvmfPkg/Library/BaseMemEncryptSevLib/X64/VirtualMemory.c +++ b/OvmfPkg/Library/BaseMemEncryptSevLib/X64/VirtualMemory.c @@ -548,21 +548,23 @@ EnableReadOnlyPageWriteProtect ( /**
This function either sets or clears memory encryption bit for the memory
- region specified by PhysicalAddress and length from the current page table
+ region specified by PhysicalAddress and Length from the current page table
context.
- The function iterates through the physicalAddress one page at a time, and set
+ The function iterates through the PhysicalAddress one page at a time, and set
or clears the memory encryption mask in the page table. If it encounters
that a given physical address range is part of large page then it attempts to
change the attribute at one go (based on size), otherwise it splits the
large pages into smaller (e.g 2M page into 4K pages) and then try to set or
clear the encryption bit on the smallest page size.
+ @param[in] Cr3BaseAddress Cr3 Base Address (if zero then use
+ current CR3)
@param[in] PhysicalAddress The physical address that is the start
address of a memory region.
@param[in] Length The length of memory region
@param[in] Mode Set or Clear mode
- @param[in] Flush Flush the caches before applying the
+ @param[in] CacheFlush Flush the caches before applying the
encryption mask
@retval RETURN_SUCCESS The attributes were cleared for the
|