1 comment:
File src/soc/amd/common/block/pi/agesawrapper.c:
This section of code is needed, otherwise aip->AllocationMethod will always be ByHost and cause con […]
The point of this entire commit is that the lifetime of the parameter struct is the duration of the execution of the state, we release the struct before we return from agesa_execute_state().
As for agesa internals, ByHost means the artifact will not be allocated fro the heap at all. There is one argument that would justify allocating it from the heap; the size of the struct involved. It would be faster to have these on stack or .bss, I don't know if we can use latter due to SMP aspects.
To view, visit change 31489. To unsubscribe, or for help writing mail filters, visit settings.