Patch Set 18: Code-Review+2

(1 comment)

Did we abandon the idea of passing through to main() now? I'd consider this is okay as is, but I still think passing through to main() would be a bit cleaner.

I thought about doing that. It's a bit cleaner on platforms where the entry is just C code, but on platforms where the entry is written in assembly it's more involving as you have to deal with the ABI, which can change depending on compiler version, flags, ... Also on x86 the ramstage entry would be different than the postcar stage, where making the calling argument the argument of the main() function is not really an option.

It seems to me that the balance is slightly in favour of doing it in the stage entry than in the main(), although I might be biased towards making it easier on x86.

View Change

To view, visit change 36145. To unsubscribe, or for help writing mail filters, visit settings.

Gerrit-Project: coreboot
Gerrit-Branch: master
Gerrit-Change-Id: If31f0f1de17ffc92c9397f32b26db25aff4b7cab
Gerrit-Change-Number: 36145
Gerrit-PatchSet: 18
Gerrit-Owner: Arthur Heymans <arthur@aheymans.xyz>
Gerrit-Reviewer: Arthur Heymans <arthur@aheymans.xyz>
Gerrit-Reviewer: Julius Werner <jwerner@chromium.org>
Gerrit-Reviewer: Martin Roth <martinroth@google.com>
Gerrit-Reviewer: Patrick Georgi <pgeorgi@google.com>
Gerrit-Reviewer: build bot (Jenkins) <no-reply@coreboot.org>
Gerrit-CC: Aaron Durbin <adurbin@chromium.org>
Gerrit-Comment-Date: Wed, 30 Oct 2019 07:43:55 +0000
Gerrit-HasComments: No
Gerrit-Has-Labels: No
Gerrit-MessageType: comment