Felix Held has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/32889 )
Change subject: Documentation: Warn about ME cleaner on Sandy Bridge ......................................................................
Patch Set 2:
(1 comment)
https://review.coreboot.org/#/c/32889/2/Documentation/northbridge/intel/sand... File Documentation/northbridge/intel/sandybridge/me_cleaner.md:
https://review.coreboot.org/#/c/32889/2/Documentation/northbridge/intel/sand... PS2, Line 7: Using a 'cleaned' ME partition is *not* recommended.
Maybe I'm just biased towards my script, but isn't this line a bit too definitive? I mean, on my mac […]
maybe also add not to file bug reports about weird issues when not having tested with the unmodified ifd and me section? if it works well enough for people, i don't see much reason to strongly advise against that, but since it might cause hard to debug problems, i wouldn't advise to use it by default for all users. i like that the users have the option to use me_cleaner, but i definitely wouldn't advertise it as "just check that box and everything will be great"; it's more like "only check this box if you really want that and you're aware that it might cause weird problems"