Aaron Durbin has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/42459 )
Change subject: [RFC] device: Add method for configuring bus master based on config option ......................................................................
Patch Set 6:
(1 comment)
https://review.coreboot.org/c/coreboot/+/42459/6//COMMIT_MSG Commit Message:
https://review.coreboot.org/c/coreboot/+/42459/6//COMMIT_MSG@16 PS6, Line 16: for internal PCI devices. Is a global option granular enough to set the desired policy? There are lots of internal devices that may stop decoding transactions properly. Were we wanting to enable such a thing for every access? And, presumably, we would then punt new behavior to the payloads that may have been taking advantage of all that behavior?