[coreboot-gerrit] [M] Change in coreboot[main]: device/pciexp_device.c: Fix setting Max Payload Size

[Krystian Hebel (Code Review)]

Attention is currently required from: Angel Pons, Arthur Heymans, Felix Held, Kyösti Mälkki, Michał Żygowski, Nico Huber, Patrick Rudolph.

Krystian Hebel has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/77338?usp=email )

Change subject: device/pciexp_device.c: Fix setting Max Payload Size
......................................................................

Patch Set 11:

(1 comment)

File src/device/pciexp_device.c:

https://review.coreboot.org/c/coreboot/+/77338/comment/d2756151_5cb1c09c :
PS11, Line 630: 	root->max_payload_set = 1;
> So basically, at the end of root port scanning, we need to set the root port's Max Payload Size to a […]

This would also work, and is much easier than what I suggested.

The problem with testing is that this code only limits _max_ payload size. If higher layers only use smaller packages, a potential misconfiguration won't show problems anyway.