Skip to content

Conversation

@krystian-hebel
Copy link

Signed-off-by: Krystian Hebel [email protected]

@krystian-hebel krystian-hebel self-assigned this May 24, 2019
@pietrushnic pietrushnic requested a review from miczyg1 May 26, 2019 22:44
@pietrushnic
Copy link
Member

@miczyg1 I don't believe we can review that, but we should definitely test that. @krystian-hebel where we can find any readme?

@krystian-hebel
Copy link
Author

@pietrushnic I wouldn't call it readme, but some instructions are available at CM-620. I'm still not sure if VMM should be included as a part of this payload or as a separate file in CBFS. First option would make it harder to customize a hypervisor, but it could be built in one process.

krystian-hebel and others added 6 commits May 28, 2019 18:38
Signed-off-by: Michał Żygowski <[email protected]>
Forwarding table is no longer present in its normal regions, probably
overwritten by SeaBIOS or payload. This address is specific to platform
and coreboot version, so this (terrible) fix is useless anywhere else.

Signed-off-by: Krystian Hebel <[email protected]>
…ot tables"

This reverts commit fd7b089.

Signed-off-by: Krystian Hebel <[email protected]>
All include/* files were obtained from Bareflank commit ba613e2c687f

Signed-off-by: Krystian Hebel <[email protected]>
@krystian-hebel krystian-hebel changed the base branch from coreinfo_test64 to lp64 May 29, 2019 15:26

ifeq ($(filter %clean,$(MAKECMDGOALS)),)
export KERNELVERSION := 0.1.0
export KCONFIG_AUTOHEADER := $(coreinfo_obj)/config.h
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@krystian-hebel this coreinfo_obj stuff looks weird. Does it have any relation to coreinfo payload?

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@miczyg1 I've copied this MAkefile from coreinfo, there may be more leftovers, I guess I should clear it

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please do, This looks confusing

miczyg1 and others added 2 commits May 30, 2019 14:20
Tested with #DE, #GP and #DB

Signed-off-by: Krystian Hebel <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants