Kyösti Mälkki has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/35161 )
Change subject: arch/x86: Remove WB attribute from 0..CACHE_TMP_RAMTOP ......................................................................
Patch Set 1:
Patch Set 1: Code-Review-1
kindly help to write test on what all board you have seen this is fixing regression ?
Personally I have not witnessed this to fix a regressionn on any board. Going by the reports and feedback in CB:34893. To remove this WB CACHE_TMP_RAMPTOP and to generally take a different approach with this common_mtrrs() was discussed with Aaron even before the regressions were discovered.
As for test coverage; toolchain, architecture and subsystem maintenance (and maintainers) often take the liberty of not targeting for 100% test coverage on hardware before code merges. That's the only way we ever get anything to evolve around here. Take it to the mailing list or coreboot leadership if you want to see that change, but as an organisation coreboot does not currently run any automated testrigs that I know of.
That said: Subrata, you had been added as reviewer to CB:34893 Aug 18th. I only see your feedback on it Aug 27th, the day after it was submitted to master. In the future, to avoid see regressions on master branch, I advice you take the time to checkout any changes you see touch your codebase and run them on your rigs.