Change in coreboot[master]: [WIP]Docs/project_ideas.md: Add a memtest libpayload based payload
Arthur Heymans has uploaded this change for review. ( https://review.coreboot.org/c/coreboot/+/36747 )
Change subject: [WIP]Docs/project_ideas.md: Add a memtest libpayload based payload ......................................................................
[WIP]Docs/project_ideas.md: Add a memtest libpayload based payload
Change-Id: Iebdb75b99e18fe92aa4c801769532781edf44d9a Signed-off-by: Arthur Heymans arthur@aheymans.xyz --- M Documentation/contributing/project_ideas.md 1 file changed, 18 insertions(+), 0 deletions(-)
git pull ssh://review.coreboot.org:29418/coreboot refs/changes/47/36747/1
diff --git a/Documentation/contributing/project_ideas.md b/Documentation/contributing/project_ideas.md index 21a756d..d2134ce 100644 --- a/Documentation/contributing/project_ideas.md +++ b/Documentation/contributing/project_ideas.md @@ -201,3 +201,21 @@
### Mentors * Ron Minnich rminnich@google.com + +## Libpayload based memtest payload +[Memtest86+](https://www.memtest.org/) has some limitations: first and +foremost it only works on x86, while it can print to serial console the +GUI only works in legacy VGA mode. + +This project would involve porting the memtest suite to libpayload and +build a payload around it. + +### Requirements +* coreboot knowledge: Should know how to build coreboot images and + include payloads. +* other knowledge: Knowledge on how dram works is a plus. +* hardware requirements: Initial work can happen on qemu targets, + being able to test on coreboot supported hardware is a plus. + +### Mentors +* TODO
Patrick Georgi has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/36747 )
Change subject: [WIP]Docs/project_ideas.md: Add a memtest libpayload based payload ......................................................................
Patch Set 1: Code-Review+1
drop the [WIP] and it's good to go?
Paul Menzel has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/36747 )
Change subject: [WIP]Docs/project_ideas.md: Add a memtest libpayload based payload ......................................................................
Patch Set 1: Code-Review+1
Hello Patrick Rudolph, Paul Menzel, build bot (Jenkins), Nico Huber, Patrick Georgi,
I'd like you to reexamine a change. Please visit
https://review.coreboot.org/c/coreboot/+/36747
to look at the new patch set (#2).
Change subject: Docs/project_ideas.md: Add a memtest libpayload based payload ......................................................................
Docs/project_ideas.md: Add a memtest libpayload based payload
Change-Id: Iebdb75b99e18fe92aa4c801769532781edf44d9a Signed-off-by: Arthur Heymans arthur@aheymans.xyz --- M Documentation/contributing/project_ideas.md 1 file changed, 18 insertions(+), 0 deletions(-)
git pull ssh://review.coreboot.org:29418/coreboot refs/changes/47/36747/2
Paul Menzel has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/36747 )
Change subject: Docs/project_ideas.md: Add a memtest libpayload based payload ......................................................................
Patch Set 2: Code-Review+1
Patrick Georgi has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/36747 )
Change subject: Docs/project_ideas.md: Add a memtest libpayload based payload ......................................................................
Patch Set 2: Code-Review+2
Patrick Georgi has submitted this change. ( https://review.coreboot.org/c/coreboot/+/36747 )
Change subject: Docs/project_ideas.md: Add a memtest libpayload based payload ......................................................................
Docs/project_ideas.md: Add a memtest libpayload based payload
Change-Id: Iebdb75b99e18fe92aa4c801769532781edf44d9a Signed-off-by: Arthur Heymans arthur@aheymans.xyz Reviewed-on: https://review.coreboot.org/c/coreboot/+/36747 Tested-by: build bot (Jenkins) no-reply@coreboot.org Reviewed-by: Paul Menzel paulepanter@users.sourceforge.net Reviewed-by: Patrick Georgi pgeorgi@google.com --- M Documentation/contributing/project_ideas.md 1 file changed, 18 insertions(+), 0 deletions(-)
Approvals: build bot (Jenkins): Verified Patrick Georgi: Looks good to me, approved Paul Menzel: Looks good to me, but someone else must approve
diff --git a/Documentation/contributing/project_ideas.md b/Documentation/contributing/project_ideas.md index 8271ea9..2c621f4 100644 --- a/Documentation/contributing/project_ideas.md +++ b/Documentation/contributing/project_ideas.md @@ -166,3 +166,21 @@
### Mentors * Ron Minnich rminnich@google.com + +## Libpayload based memtest payload +[Memtest86+](https://www.memtest.org/) has some limitations: first and +foremost it only works on x86, while it can print to serial console the +GUI only works in legacy VGA mode. + +This project would involve porting the memtest suite to libpayload and +build a payload around it. + +### Requirements +* coreboot knowledge: Should know how to build coreboot images and + include payloads. +* other knowledge: Knowledge on how dram works is a plus. +* hardware requirements: Initial work can happen on qemu targets, + being able to test on coreboot supported hardware is a plus. + +### Mentors +* TODO
-
Arthur Heymans (Code Review) -
Patrick Georgi (Code Review)
-
Paul Menzel (Code Review)