Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Other patches #4

Open
weedy opened this issue Sep 28, 2018 · 4 comments
Open

Other patches #4

weedy opened this issue Sep 28, 2018 · 4 comments

Comments

@weedy
Copy link

weedy commented Sep 28, 2018

A ipxe (coreboot related?) dev also made a whole bunch of changes. Anything useful here?

https://github.com/hadrava/memtest86plus/commits/develop

@anphsw
Copy link
Owner

anphsw commented Sep 28, 2018

That patches is merged near ad6dec4 commit, just from another source (canardpc forum, not github).
Or you mean any particular patch?

@weedy
Copy link
Author

weedy commented Sep 29, 2018

I know you grabbed the last three. But I mean stuff by mcb30.

I don't think I noticed hadrava/memtest86plus@2deedd2 or hadrava/memtest86plus@29edf9c

But then I also don't know how much any of his PXE stuff matters for you. I mainly brought it up because I was looking for a version of memtest+ that wasn't so god damn broken, and found a couple repos with patches on top of 5.01 release.

Yours works fine for me, thank you for the repo.

@anphsw
Copy link
Owner

anphsw commented Sep 29, 2018

Almost all mcb30 (not directly regarding PXE) patches are present too.
hadrava/memtest86plus@2deedd2 is ad6dec4

hadrava/memtest86plus@29edf9c I think that code not "unused", I tested it before (when was adding mcb30 patches) and decided not to include it.
But maybe I'm wrong.

I'm dont have much equipment to test PXE cases and patches, but you can fork and add patches as you wish, so I'll try to check them and merge.

@mirh
Copy link

mirh commented Dec 19, 2019

There's also https://review.coreboot.org/cgit/memtest86plus.git/

p.s. PXE can simply be tested with a VM

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

No branches or pull requests

3 participants