Change in coreboot[master]: tmb/emulation/qemu-aarch64: Add MMU support

List overview All Threads
Download

newer

older

Change in coreboot[master]:...

Patrick Rudolph (Code Review)

23 Jan 2020 23 Jan '20

2:12 p.m.

Patrick Rudolph has uploaded this change for review. ( https://review.coreboot.org/c/coreboot/+/38534 )

Change subject: tmb/emulation/qemu-aarch64: Add MMU support ......................................................................

tmb/emulation/qemu-aarch64: Add MMU support

Enable MMU in bootblock.

Change-Id: Ifaed9d3cc11520f180a732d51adce634621b5844 Signed-off-by: Patrick Rudolph patrick.rudolph@9elements.com --- M src/mainboard/emulation/qemu-aarch64/Makefile.inc A src/mainboard/emulation/qemu-aarch64/bootblock.c M src/mainboard/emulation/qemu-aarch64/memlayout.ld 3 files changed, 35 insertions(+), 2 deletions(-)

git pull ssh://review.coreboot.org:29418/coreboot refs/changes/34/38534/1

diff --git a/src/mainboard/emulation/qemu-aarch64/Makefile.inc b/src/mainboard/emulation/qemu-aarch64/Makefile.inc index dc0e9f4..4d5f2bd 100644 --- a/src/mainboard/emulation/qemu-aarch64/Makefile.inc +++ b/src/mainboard/emulation/qemu-aarch64/Makefile.inc @@ -5,6 +5,8 @@ # # SPDX-License-Identifier: GPL-2.0-or-later

+bootblock-y += bootblock.c + romstage-y += cbmem.c

bootblock-y += media.c diff --git a/src/mainboard/emulation/qemu-aarch64/bootblock.c b/src/mainboard/emulation/qemu-aarch64/bootblock.c new file mode 100644 index 0000000..2c878b6 --- /dev/null +++ b/src/mainboard/emulation/qemu-aarch64/bootblock.c @@ -0,0 +1,31 @@ +/* + * This file is part of the coreboot project. + * + * This program is free software; you can redistribute it and/or modify + * it under the terms of the GNU General Public License as published by + * the Free Software Foundation; version 2 of the License. + * + * This program is distributed in the hope that it will be useful, + * but WITHOUT ANY WARRANTY; without even the implied warranty of + * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the + * GNU General Public License for more details. + * + */ + +#include <arch/mmu.h> +#include <bootblock_common.h> +#include <symbols.h> + +void bootblock_mainboard_init(void) +{ + mmu_init(); + + mmu_config_range((void *)0, (uintptr_t)4 * GiB, MA_DEV | MA_RW); + + mmu_config_range(_ttb, REGION_SIZE(ttb), MA_MEM | MA_S | MA_RW); + mmu_config_range(_bootblock, REGION_SIZE(bootblock), MA_MEM | MA_S | MA_RW); + mmu_config_range(_romstage, REGION_SIZE(romstage), MA_MEM | MA_S | MA_RW); + mmu_config_range(_ramstage, REGION_SIZE(ramstage), MA_MEM | MA_S | MA_RW); + + mmu_enable(); +} diff --git a/src/mainboard/emulation/qemu-aarch64/memlayout.ld b/src/mainboard/emulation/qemu-aarch64/memlayout.ld index aba4205..cd86073 100644 --- a/src/mainboard/emulation/qemu-aarch64/memlayout.ld +++ b/src/mainboard/emulation/qemu-aarch64/memlayout.ld @@ -29,6 +29,6 @@ ROMSTAGE(0x60030000, 128K) RAMSTAGE(0x60070000, 16M)

- TTB(0x61100000, 16K) - POSTRAM_CBFS_CACHE(0x61110000, 1M) + TTB(0x61100000, 1M) + POSTRAM_CBFS_CACHE(0x61200000, 1M) }

Attachments:

attachment.htm (text/html — 6.6 KB)

Show replies by date

Paul Menzel (Code Review)

23 Jan 23 Jan

4:47 p.m.

Paul Menzel has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/38534 )

Change subject: tmb/emulation/qemu-aarch64: Add MMU support ......................................................................

Patch Set 1:

(1 comment)

https://review.coreboot.org/c/coreboot/+/38534/1//COMMIT_MSG Commit Message:

https://review.coreboot.org/c/coreboot/+/38534/1//COMMIT_MSG@6 PS1, Line 6: : tmb/emulation/qemu-aarch64 Remove t in the beginning.

-- To view, visit https://review.coreboot.org/c/coreboot/+/38534 To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings Gerrit-Project: coreboot Gerrit-Branch: master Gerrit-Change-Id: Ifaed9d3cc11520f180a732d51adce634621b5844 Gerrit-Change-Number: 38534 Gerrit-PatchSet: 1 Gerrit-Owner: Patrick Rudolph patrick.rudolph@9elements.com Gerrit-Reviewer: Julius Werner jwerner@chromium.org Gerrit-Reviewer: Martin Roth martinroth@google.com Gerrit-Reviewer: Patrick Georgi pgeorgi@google.com Gerrit-Reviewer: Patrick Rudolph patrick.rudolph@9elements.com Gerrit-Reviewer: build bot (Jenkins) no-reply@coreboot.org Gerrit-CC: Aaron Durbin adurbin@chromium.org Gerrit-CC: Paul Menzel paulepanter@users.sourceforge.net Gerrit-Comment-Date: Thu, 23 Jan 2020 15:47:39 +0000 Gerrit-HasComments: Yes Gerrit-Has-Labels: No Gerrit-MessageType: comment

Patrick Rudolph (Code Review)

24 Jan 24 Jan

8:04 a.m.

New subject: Change in coreboot[master]: mb/emulation/qemu-aarch64: Add MMU support

Hello Julius Werner, build bot (Jenkins), Patrick Georgi, Martin Roth,

I'd like you to reexamine a change. Please visit

https://review.coreboot.org/c/coreboot/+/38534

to look at the new patch set (#2).

Change subject: mb/emulation/qemu-aarch64: Add MMU support ......................................................................

mb/emulation/qemu-aarch64: Add MMU support

Enable MMU in bootblock. Makes qemu look more similar to real hardware. There's no real need to active the MMU.

Tested on qemu-system-aarch64.

git pull ssh://review.coreboot.org:29418/coreboot refs/changes/34/38534/2

Patrick Rudolph (Code Review)

8:12 a.m.

New subject: Change in coreboot[master]: mb/emulation/qemu-aarch64: Add MMU support

Patrick Rudolph has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/38534 )

Change subject: mb/emulation/qemu-aarch64: Add MMU support ......................................................................

Patch Set 2:

(1 comment)

https://review.coreboot.org/c/coreboot/+/38534/1//COMMIT_MSG Commit Message:

https://review.coreboot.org/c/coreboot/+/38534/1//COMMIT_MSG@6 PS1, Line 6: : tmb/emulation/qemu-aarch64

...

Remove t in the beginning.

Done

-- To view, visit https://review.coreboot.org/c/coreboot/+/38534 To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings Gerrit-Project: coreboot Gerrit-Branch: master Gerrit-Change-Id: Ifaed9d3cc11520f180a732d51adce634621b5844 Gerrit-Change-Number: 38534 Gerrit-PatchSet: 2 Gerrit-Owner: Patrick Rudolph patrick.rudolph@9elements.com Gerrit-Reviewer: Julius Werner jwerner@chromium.org Gerrit-Reviewer: Martin Roth martinroth@google.com Gerrit-Reviewer: Patrick Georgi pgeorgi@google.com Gerrit-Reviewer: Patrick Rudolph patrick.rudolph@9elements.com Gerrit-Reviewer: build bot (Jenkins) no-reply@coreboot.org Gerrit-CC: Aaron Durbin adurbin@chromium.org Gerrit-CC: Paul Menzel paulepanter@users.sourceforge.net Gerrit-Comment-Date: Fri, 24 Jan 2020 07:12:20 +0000 Gerrit-HasComments: Yes Gerrit-Has-Labels: No Comment-In-Reply-To: Paul Menzel paulepanter@users.sourceforge.net Gerrit-MessageType: comment

Angel Pons (Code Review)

30 Jan 30 Jan

10:40 p.m.

New subject: Change in coreboot[master]: mb/emulation/qemu-aarch64: Add MMU support

Angel Pons has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/38534 )

Change subject: mb/emulation/qemu-aarch64: Add MMU support ......................................................................

Patch Set 2:

(2 comments)

https://review.coreboot.org/c/coreboot/+/38534/2//COMMIT_MSG Commit Message:

https://review.coreboot.org/c/coreboot/+/38534/2//COMMIT_MSG@10 PS2, Line 10: active activate ?

https://review.coreboot.org/c/coreboot/+/38534/2/src/mainboard/emulation/qem... File src/mainboard/emulation/qemu-aarch64/bootblock.c:

https://review.coreboot.org/c/coreboot/+/38534/2/src/mainboard/emulation/qem... PS2, Line 12: * please remove this empty line

-- To view, visit https://review.coreboot.org/c/coreboot/+/38534 To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings Gerrit-Project: coreboot Gerrit-Branch: master Gerrit-Change-Id: Ifaed9d3cc11520f180a732d51adce634621b5844 Gerrit-Change-Number: 38534 Gerrit-PatchSet: 2 Gerrit-Owner: Patrick Rudolph patrick.rudolph@9elements.com Gerrit-Reviewer: Asami Doi d0iasm.pub@gmail.com Gerrit-Reviewer: Julius Werner jwerner@chromium.org Gerrit-Reviewer: Martin Roth martinroth@google.com Gerrit-Reviewer: Patrick Georgi pgeorgi@google.com Gerrit-Reviewer: Patrick Rudolph patrick.rudolph@9elements.com Gerrit-Reviewer: build bot (Jenkins) no-reply@coreboot.org Gerrit-CC: Aaron Durbin adurbin@chromium.org Gerrit-CC: Angel Pons th3fanbus@gmail.com Gerrit-CC: Paul Menzel paulepanter@users.sourceforge.net Gerrit-Comment-Date: Thu, 30 Jan 2020 21:40:36 +0000 Gerrit-HasComments: Yes Gerrit-Has-Labels: No Gerrit-MessageType: comment

Arthur Heymans (Code Review)

8 Feb 8 Feb

11:57 a.m.

New subject: Change in coreboot[master]: mb/emulation/qemu-aarch64: Add MMU support

Arthur Heymans has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/38534 )

Change subject: mb/emulation/qemu-aarch64: Add MMU support ......................................................................

Patch Set 2: Code-Review+1

(1 comment)

https://review.coreboot.org/c/coreboot/+/38534/2/src/mainboard/emulation/qem... File src/mainboard/emulation/qemu-aarch64/memlayout.ld:

https://review.coreboot.org/c/coreboot/+/38534/2/src/mainboard/emulation/qem... PS2, Line 32: 1M) That's unusually large?

-- To view, visit https://review.coreboot.org/c/coreboot/+/38534 To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings Gerrit-Project: coreboot Gerrit-Branch: master Gerrit-Change-Id: Ifaed9d3cc11520f180a732d51adce634621b5844 Gerrit-Change-Number: 38534 Gerrit-PatchSet: 2 Gerrit-Owner: Patrick Rudolph patrick.rudolph@9elements.com Gerrit-Reviewer: Arthur Heymans arthur@aheymans.xyz Gerrit-Reviewer: Asami Doi d0iasm.pub@gmail.com Gerrit-Reviewer: Julius Werner jwerner@chromium.org Gerrit-Reviewer: Martin Roth martinroth@google.com Gerrit-Reviewer: Patrick Georgi pgeorgi@google.com Gerrit-Reviewer: Patrick Rudolph patrick.rudolph@9elements.com Gerrit-Reviewer: build bot (Jenkins) no-reply@coreboot.org Gerrit-CC: Aaron Durbin adurbin@chromium.org Gerrit-CC: Angel Pons th3fanbus@gmail.com Gerrit-CC: Paul Menzel paulepanter@users.sourceforge.net Gerrit-Comment-Date: Sat, 08 Feb 2020 10:57:40 +0000 Gerrit-HasComments: Yes Gerrit-Has-Labels: Yes Gerrit-MessageType: comment

Patrick Rudolph (Code Review)

20 Feb 20 Feb

12:08 p.m.

New subject: Change in coreboot[master]: mb/emulation/qemu-aarch64: Add MMU support

Hello Asami Doi, Julius Werner, Arthur Heymans, build bot (Jenkins), Patrick Georgi, Martin Roth,

I'd like you to reexamine a change. Please visit

https://review.coreboot.org/c/coreboot/+/38534

to look at the new patch set (#3).

Change subject: mb/emulation/qemu-aarch64: Add MMU support ......................................................................

mb/emulation/qemu-aarch64: Add MMU support

Enable MMU in bootblock. Makes qemu look more similar to real hardware. There's no real need to activate the MMU.

Tested on qemu-system-aarch64: 5 page entries are used out of 32.

git pull ssh://review.coreboot.org:29418/coreboot refs/changes/34/38534/3

-- To view, visit https://review.coreboot.org/c/coreboot/+/38534 To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings Gerrit-Project: coreboot Gerrit-Branch: master Gerrit-Change-Id: Ifaed9d3cc11520f180a732d51adce634621b5844 Gerrit-Change-Number: 38534 Gerrit-PatchSet: 3 Gerrit-Owner: Patrick Rudolph patrick.rudolph@9elements.com Gerrit-Reviewer: Arthur Heymans arthur@aheymans.xyz Gerrit-Reviewer: Asami Doi d0iasm.pub@gmail.com Gerrit-Reviewer: Julius Werner jwerner@chromium.org Gerrit-Reviewer: Martin Roth martinroth@google.com Gerrit-Reviewer: Patrick Georgi pgeorgi@google.com Gerrit-Reviewer: Patrick Rudolph patrick.rudolph@9elements.com Gerrit-Reviewer: build bot (Jenkins) no-reply@coreboot.org Gerrit-CC: Aaron Durbin adurbin@chromium.org Gerrit-CC: Angel Pons th3fanbus@gmail.com Gerrit-CC: Paul Menzel paulepanter@users.sourceforge.net Gerrit-MessageType: newpatchset

Patrick Rudolph (Code Review)

12:08 p.m.

New subject: Change in coreboot[master]: mb/emulation/qemu-aarch64: Add MMU support

Patrick Rudolph has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/38534 )

Change subject: mb/emulation/qemu-aarch64: Add MMU support ......................................................................

Patch Set 3:

(3 comments)

https://review.coreboot.org/c/coreboot/+/38534/2//COMMIT_MSG Commit Message:

https://review.coreboot.org/c/coreboot/+/38534/2//COMMIT_MSG@10 PS2, Line 10: active

...

activate ?

Done

https://review.coreboot.org/c/coreboot/+/38534/2/src/mainboard/emulation/qem... File src/mainboard/emulation/qemu-aarch64/bootblock.c:

https://review.coreboot.org/c/coreboot/+/38534/2/src/mainboard/emulation/qem... PS2, Line 12: *

...

please remove this empty line

Done

https://review.coreboot.org/c/coreboot/+/38534/2/src/mainboard/emulation/qem... File src/mainboard/emulation/qemu-aarch64/memlayout.ld:

https://review.coreboot.org/c/coreboot/+/38534/2/src/mainboard/emulation/qem... PS2, Line 32: 1M)

...

That's unusually large?

Reduced to 128K

-- To view, visit https://review.coreboot.org/c/coreboot/+/38534 To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings Gerrit-Project: coreboot Gerrit-Branch: master Gerrit-Change-Id: Ifaed9d3cc11520f180a732d51adce634621b5844 Gerrit-Change-Number: 38534 Gerrit-PatchSet: 3 Gerrit-Owner: Patrick Rudolph patrick.rudolph@9elements.com Gerrit-Reviewer: Arthur Heymans arthur@aheymans.xyz Gerrit-Reviewer: Asami Doi d0iasm.pub@gmail.com Gerrit-Reviewer: Julius Werner jwerner@chromium.org Gerrit-Reviewer: Martin Roth martinroth@google.com Gerrit-Reviewer: Patrick Georgi pgeorgi@google.com Gerrit-Reviewer: Patrick Rudolph patrick.rudolph@9elements.com Gerrit-Reviewer: build bot (Jenkins) no-reply@coreboot.org Gerrit-CC: Aaron Durbin adurbin@chromium.org Gerrit-CC: Angel Pons th3fanbus@gmail.com Gerrit-CC: Paul Menzel paulepanter@users.sourceforge.net Gerrit-Comment-Date: Thu, 20 Feb 2020 11:08:55 +0000 Gerrit-HasComments: Yes Gerrit-Has-Labels: No Comment-In-Reply-To: Angel Pons th3fanbus@gmail.com Comment-In-Reply-To: Arthur Heymans arthur@aheymans.xyz Gerrit-MessageType: comment

Philipp Deppenwiese (Code Review)

2 Mar 2 Mar

11:33 a.m.

New subject: Change in coreboot[master]: mb/emulation/qemu-aarch64: Add MMU support

Philipp Deppenwiese has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/38534 )

Change subject: mb/emulation/qemu-aarch64: Add MMU support ......................................................................

Patch Set 5: Code-Review+2

-- To view, visit https://review.coreboot.org/c/coreboot/+/38534 To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings Gerrit-Project: coreboot Gerrit-Branch: master Gerrit-Change-Id: Ifaed9d3cc11520f180a732d51adce634621b5844 Gerrit-Change-Number: 38534 Gerrit-PatchSet: 5 Gerrit-Owner: Patrick Rudolph patrick.rudolph@9elements.com Gerrit-Reviewer: Arthur Heymans arthur@aheymans.xyz Gerrit-Reviewer: Asami Doi d0iasm.pub@gmail.com Gerrit-Reviewer: Julius Werner jwerner@chromium.org Gerrit-Reviewer: Martin Roth martinroth@google.com Gerrit-Reviewer: Patrick Georgi pgeorgi@google.com Gerrit-Reviewer: Patrick Rudolph patrick.rudolph@9elements.com Gerrit-Reviewer: Patrick Rudolph siro@das-labor.org Gerrit-Reviewer: Philipp Deppenwiese zaolin.daisuki@gmail.com Gerrit-Reviewer: build bot (Jenkins) no-reply@coreboot.org Gerrit-CC: Aaron Durbin adurbin@chromium.org Gerrit-CC: Angel Pons th3fanbus@gmail.com Gerrit-CC: Paul Menzel paulepanter@users.sourceforge.net Gerrit-Comment-Date: Mon, 02 Mar 2020 10:33:24 +0000 Gerrit-HasComments: No Gerrit-Has-Labels: Yes Gerrit-MessageType: comment

Philipp Deppenwiese (Code Review)

11:33 a.m.

New subject: Change in coreboot[master]: mb/emulation/qemu-aarch64: Add MMU support

Philipp Deppenwiese has submitted this change. ( https://review.coreboot.org/c/coreboot/+/38534 )

Change subject: mb/emulation/qemu-aarch64: Add MMU support ......................................................................

mb/emulation/qemu-aarch64: Add MMU support

Enable MMU in bootblock. Makes qemu look more similar to real hardware. There's no real need to activate the MMU.

Tested on qemu-system-aarch64: 5 page entries are used out of 32.

Change-Id: Ifaed9d3cc11520f180a732d51adce634621b5844 Signed-off-by: Patrick Rudolph patrick.rudolph@9elements.com Reviewed-on: https://review.coreboot.org/c/coreboot/+/38534 Reviewed-by: Philipp Deppenwiese zaolin.daisuki@gmail.com Tested-by: build bot (Jenkins) no-reply@coreboot.org --- M src/mainboard/emulation/qemu-aarch64/Makefile.inc A src/mainboard/emulation/qemu-aarch64/bootblock.c M src/mainboard/emulation/qemu-aarch64/memlayout.ld 3 files changed, 38 insertions(+), 3 deletions(-)

Approvals: build bot (Jenkins): Verified Philipp Deppenwiese: Looks good to me, approved

+bootblock-y += bootblock.c + romstage-y += cbmem.c

bootblock-y += media.c diff --git a/src/mainboard/emulation/qemu-aarch64/bootblock.c b/src/mainboard/emulation/qemu-aarch64/bootblock.c new file mode 100644 index 0000000..280f77e --- /dev/null +++ b/src/mainboard/emulation/qemu-aarch64/bootblock.c @@ -0,0 +1,33 @@ +/* + * This file is part of the coreboot project. + * + * This program is free software; you can redistribute it and/or modify + * it under the terms of the GNU General Public License as published by + * the Free Software Foundation; version 2 of the License. + * + * This program is distributed in the hope that it will be useful, + * but WITHOUT ANY WARRANTY; without even the implied warranty of + * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the + * GNU General Public License for more details. + */ + +#include <arch/mmu.h> +#include <bootblock_common.h> +#include <symbols.h> + +void bootblock_mainboard_init(void) +{ + mmu_init(); + + /* Everything below DRAM is device memory */ + mmu_config_range((void *)0, (uintptr_t)_dram, MA_DEV | MA_RW); + /* Set a dummy value for DRAM. ramstage should update the mapping. */ + mmu_config_range(_dram, 1 * GiB, MA_MEM | MA_RW); + + mmu_config_range(_ttb, REGION_SIZE(ttb), MA_MEM | MA_S | MA_RW); + mmu_config_range(_bootblock, REGION_SIZE(bootblock), MA_MEM | MA_S | MA_RW); + mmu_config_range(_romstage, REGION_SIZE(romstage), MA_MEM | MA_S | MA_RW); + mmu_config_range(_ramstage, REGION_SIZE(ramstage), MA_MEM | MA_S | MA_RW); + + mmu_enable(); +} diff --git a/src/mainboard/emulation/qemu-aarch64/memlayout.ld b/src/mainboard/emulation/qemu-aarch64/memlayout.ld index aba4205..248d0ab 100644 --- a/src/mainboard/emulation/qemu-aarch64/memlayout.ld +++ b/src/mainboard/emulation/qemu-aarch64/memlayout.ld @@ -27,8 +27,8 @@ STACK(0x60020000, 62K) FMAP_CACHE(0x6002F800, 2K) ROMSTAGE(0x60030000, 128K) - RAMSTAGE(0x60070000, 16M) + TTB(0x60070000, 128K) + RAMSTAGE(0x600b0000, 16M)

- TTB(0x61100000, 16K) - POSTRAM_CBFS_CACHE(0x61110000, 1M) + POSTRAM_CBFS_CACHE(0x61200000, 1M) }

-- To view, visit https://review.coreboot.org/c/coreboot/+/38534 To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings Gerrit-Project: coreboot Gerrit-Branch: master Gerrit-Change-Id: Ifaed9d3cc11520f180a732d51adce634621b5844 Gerrit-Change-Number: 38534 Gerrit-PatchSet: 6 Gerrit-Owner: Patrick Rudolph patrick.rudolph@9elements.com Gerrit-Reviewer: Arthur Heymans arthur@aheymans.xyz Gerrit-Reviewer: Asami Doi d0iasm.pub@gmail.com Gerrit-Reviewer: Julius Werner jwerner@chromium.org Gerrit-Reviewer: Martin Roth martinroth@google.com Gerrit-Reviewer: Patrick Georgi pgeorgi@google.com Gerrit-Reviewer: Patrick Rudolph patrick.rudolph@9elements.com Gerrit-Reviewer: Patrick Rudolph siro@das-labor.org Gerrit-Reviewer: Philipp Deppenwiese zaolin.daisuki@gmail.com Gerrit-Reviewer: build bot (Jenkins) no-reply@coreboot.org Gerrit-CC: Aaron Durbin adurbin@chromium.org Gerrit-CC: Angel Pons th3fanbus@gmail.com Gerrit-CC: Paul Menzel paulepanter@users.sourceforge.net Gerrit-MessageType: merged

9elements QA (Code Review)

12:09 p.m.

New subject: Change in coreboot[master]: mb/emulation/qemu-aarch64: Add MMU support

9elements QA has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/38534 )

Change subject: mb/emulation/qemu-aarch64: Add MMU support ......................................................................

Patch Set 6:

Automatic boot test returned (PASS/FAIL/TOTAL): 3/0/3 Emulation targets: EMULATION_QEMU_X86_Q35 using payload TianoCore : SUCCESS : https://lava.9esec.io/r/999 EMULATION_QEMU_X86_Q35 using payload SeaBIOS : SUCCESS : https://lava.9esec.io/r/998 EMULATION_QEMU_X86_I440FX using payload SeaBIOS : SUCCESS : https://lava.9esec.io/r/997

Please note: This test is under development and might not be accurate at all!

-- To view, visit https://review.coreboot.org/c/coreboot/+/38534 To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings Gerrit-Project: coreboot Gerrit-Branch: master Gerrit-Change-Id: Ifaed9d3cc11520f180a732d51adce634621b5844 Gerrit-Change-Number: 38534 Gerrit-PatchSet: 6 Gerrit-Owner: Patrick Rudolph patrick.rudolph@9elements.com Gerrit-Reviewer: Arthur Heymans arthur@aheymans.xyz Gerrit-Reviewer: Asami Doi d0iasm.pub@gmail.com Gerrit-Reviewer: Julius Werner jwerner@chromium.org Gerrit-Reviewer: Martin Roth martinroth@google.com Gerrit-Reviewer: Patrick Georgi pgeorgi@google.com Gerrit-Reviewer: Patrick Rudolph patrick.rudolph@9elements.com Gerrit-Reviewer: Patrick Rudolph siro@das-labor.org Gerrit-Reviewer: Philipp Deppenwiese zaolin.daisuki@gmail.com Gerrit-Reviewer: build bot (Jenkins) no-reply@coreboot.org Gerrit-CC: 9elements QA hardwaretestrobot@gmail.com Gerrit-CC: Aaron Durbin adurbin@chromium.org Gerrit-CC: Angel Pons th3fanbus@gmail.com Gerrit-CC: Paul Menzel paulepanter@users.sourceforge.net Gerrit-Comment-Date: Mon, 02 Mar 2020 11:09:49 +0000 Gerrit-HasComments: No Gerrit-Has-Labels: No Gerrit-MessageType: comment

1727

days inactive

1766

days old

coreboot-gerrit@coreboot.org

10 comments

6 participants

tags (0)

participants (6)

9elements QA (Code Review)
Angel Pons (Code Review)
Arthur Heymans (Code Review)
Patrick Rudolph (Code Review)
Paul Menzel (Code Review)
Philipp Deppenwiese (Code Review)