Angel Pons has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/46491 )
Change subject: sec/intel/txt: Move DPR size to Kconfig ......................................................................
Patch Set 1:
(1 comment)
https://review.coreboot.org/c/coreboot/+/46491/1/src/security/intel/txt/Kcon... File src/security/intel/txt/Kconfig:
https://review.coreboot.org/c/coreboot/+/46491/1/src/security/intel/txt/Kcon... PS1, Line 35: "DMA Protected Region size in MiB"
I'm not sure if this should be user configurable? […]
DPR can be used for things other than TXT, which is why I decided to make this user-configurable. I'm not sure where the original value of 3 MiB comes from. I think SKL would work with just 2 MiB. For Haswell, I hardcoded 4 MiB when I patched the MRC, as this is what reference code uses. 255 MiB is the maximum supported value.