Bug 246116

Summary: sysutils/memtest86+: update to 5.31b
Product: Ports & Packages Reporter: Alessandro Sagratini <ale_sagra>
Component: Individual Port(s)Assignee: freebsd-ports-bugs (Nobody) <ports-bugs>
Status: Open ---    
Severity: Affects Only Me CC: avg, fernape
Priority: --- Flags: fernape: maintainer-feedback? (avg)
Version: Latest   
Hardware: Any   
OS: Any   
URL: https://www.memtest.org/#change
Description Flags
update of memtest86+ to 5.31b
additional improvements on top of submitter's patch fernape: maintainer-approval? (avg)

Description Alessandro Sagratini 2020-05-02 07:59:43 UTC
Created attachment 214020 [details]
update of memtest86+ to 5.31b

update of memtest86+ to 5.31b version, release a couple of months back. updated gcc dependency from 4.8 to 9

please let me know if you have any doubts or concerns
Comment 1 Nuno Teixeira freebsd_committer 2020-05-03 15:58:56 UTC
PR sent to wrong port.

This is memtest86, not memtest86+


Comment 2 Alessandro Sagratini 2020-05-03 16:42:55 UTC
I'm not sure about your "no" feedback: isn't this patch referring to latest version from www.memtest.org, which is what *existing* memtest86+ port uses? 

Let me know if there's anything else you need from me.
Comment 3 Nuno Teixeira freebsd_committer 2020-05-03 17:47:27 UTC
(In reply to Alessandro Sagratini from comment #2)

You are assigning PRs to sysutils/memtest86 and it should be sysutils/memtest86+

memtest86 its just a ISO
Comment 4 Alessandro Sagratini 2020-05-03 18:03:54 UTC
Oh, you are correct, I am not sure why maintainer feedback was assigned to you, I am going to see if I can change that on my own, otherwise, some developer can do that, hopefully :)
Comment 5 Fernando Apesteguía freebsd_committer 2020-05-04 06:59:48 UTC
Assigning proper maintainer address in maintainer-feedback.
Comment 6 Andriy Gapon freebsd_committer 2020-05-05 11:27:04 UTC
If the port builds and works then I do not have any objections.
I am a bit out of time to test it myself, so could anyone please confirm this?
It might be a good idea to advertise this update on ports@ and maybe hackers@.
Comment 7 Fernando Apesteguía freebsd_committer 2020-05-06 05:38:11 UTC
Created attachment 214186 [details]
additional improvements on top of submitter's patch

Thanks for the original patch. I made a few changes on top of it:

* USE_GCC= 9+ since it builds and this way we could benefit of future compiler improvementes (or go back to just 9 if something breaks)
* Remake patches to comply with the ports standard.

Additionally, when I tried to test this from boot loader, this is what I got:

OK unload
OK load /boot/opt/memtest86+
don't know how to load module '/boot/opt/memtest86+'

Same thing happens with current version (5.01). Any ideas?
Comment 8 Fernando Apesteguía freebsd_committer 2020-05-06 05:41:20 UTC
The port builds fine in {11.3,12.1}{amd64,i386} and 13-current amd64.
Comment 9 Andriy Gapon freebsd_committer 2020-05-06 06:43:46 UTC
In reply to Fernando Apesteguía from comment #8)
Thank you!
I see this on the memtest86+ page:
> Here is Memtest86+ 5.31b, with many bug fixes in the core functions.
> This build is not ready for production yet, because it lacks some feedbacks from beta-testers.
Maybe we should wait until the new version is out of beta?
Or, perhaps, create a new port for the devel version?
Comment 10 Andriy Gapon freebsd_committer 2020-05-06 06:44:45 UTC
(In reply to Fernando Apesteguía from comment #7)
It's been a while since I last started memtest86+ from loader.
Maybe something in loader changed.
Let me look into it.
Comment 11 Alessandro Sagratini 2020-10-12 16:02:48 UTC
Andriy, since it's been a few months, I'm wondering if any progress has been made? Or do you feel it's better to wait for next release? Thank you
Comment 12 Fernando Apesteguía freebsd_committer 2020-10-14 05:28:35 UTC
Alessandro, could you make it run?
Comment 13 Alessandro Sagratini 2020-10-22 06:54:15 UTC
mmmh, no, not yet, I am also getting "don't know how to load module '/boot/opt/memtest86+'" error message that was shown in previous messages
Comment 14 Fernando Apesteguía freebsd_committer 2020-10-22 08:33:54 UTC
(In reply to Alessandro Sagratini from comment #13)

OK, neither did I :-)

I think we need to investigate a bit more before committing a new version. I will try to have a look this week.
Comment 15 Fernando Apesteguía freebsd_committer 2020-11-12 18:40:20 UTC
I gave it another try but could not make it work.

12.1 amd64 --> don't know how to load module
13-current amd64 --> reboots the machine
Comment 16 Fernando Apesteguía freebsd_committer 2021-05-13 09:53:16 UTC
Hi Alessandro.

Does this mean that it works for you? In which arch/FreeBSD version?

Comment 17 Fernando Apesteguía freebsd_committer 2021-05-14 11:07:17 UTC
Reopening since it still reboots the machine in 14-current.