Bug#1000208: ITP: pcmemtest -- stand-alone memory tester
Hi Felix, I'm finished with my review. I've opened a PR on Salsa [1] with my suggestions. Regards, Stephan [1] https://salsa.debian.org/fzielcke/pcmemtest/-/merge_requests/1 On Mon, Nov 29, 2021 at 4:17 PM Stephan Lachnit wrote: > > Hi Felix, > > thanks for the ping, looking at it right now. > > Regards, > Stephan > > On Mon, Nov 29, 2021 at 2:14 PM Felix Zielcke wrote: > > > > Am Freitag, dem 26.11.2021 um 15:34 +0100 schrieb Stephan Lachnit: > > > Hi Felix, > > > > > > I was a bit more stressed this week than I hoped, I'll try to look at > > > it > > > saturday or sunday. If you don't get a reply by monday morning, feel > > > free > > > to ping me again. > > > > Hi Stephan, > > > > here's your friendly ping. > > Did you have time to look at pcmemtest? > > > > Regards, > > Felix > >
Bug#1000208: ITP: pcmemtest -- stand-alone memory tester
Hi Felix, thanks for the ping, looking at it right now. Regards, Stephan On Mon, Nov 29, 2021 at 2:14 PM Felix Zielcke wrote: > > Am Freitag, dem 26.11.2021 um 15:34 +0100 schrieb Stephan Lachnit: > > Hi Felix, > > > > I was a bit more stressed this week than I hoped, I'll try to look at > > it > > saturday or sunday. If you don't get a reply by monday morning, feel > > free > > to ping me again. > > Hi Stephan, > > here's your friendly ping. > Did you have time to look at pcmemtest? > > Regards, > Felix >
Bug#1000208: ITP: pcmemtest -- stand-alone memory tester
Am Freitag, dem 26.11.2021 um 15:34 +0100 schrieb Stephan Lachnit: > Hi Felix, > > I was a bit more stressed this week than I hoped, I'll try to look at > it > saturday or sunday. If you don't get a reply by monday morning, feel > free > to ping me again. Hi Stephan, here's your friendly ping. Did you have time to look at pcmemtest? Regards, Felix
Bug#1000208: ITP: pcmemtest -- stand-alone memory tester
Hi Felix, I was a bit more stressed this week than I hoped, I'll try to look at it saturday or sunday. If you don't get a reply by monday morning, feel free to ping me again. Regards, Stephan On Fri, 26 Nov 2021, 15:11 Felix Zielcke, wrote: > Am Samstag, dem 20.11.2021 um 18:47 +0100 schrieb Stephan Lachnit: > > Sounds interesting. > > > > On Fri, Nov 19, 2021 at 8:03 PM wrote: > > > > > > I'm happy to maintain it inside a team or with co-maintainer(s). > > > I'm only DM so if someone has interest in sponsoring this, feel > > > free to > > > contact me. > > > > If you need me as sponsor, ping me when the package is ready for > > upload. > > Hi Stephan, do you have time to review it? I think it's ready to > upload. > > > Regards > > Stephan > >
Bug#1000208: ITP: pcmemtest -- stand-alone memory tester
Am Samstag, dem 20.11.2021 um 18:47 +0100 schrieb Stephan Lachnit: > Sounds interesting. > > On Fri, Nov 19, 2021 at 8:03 PM wrote: > > > > I'm happy to maintain it inside a team or with co-maintainer(s). > > I'm only DM so if someone has interest in sponsoring this, feel > > free to > > contact me. > > If you need me as sponsor, ping me when the package is ready for > upload. Hi Stephan, do you have time to review it? I think it's ready to upload. > Regards > Stephan
Bug#1000208: ITP: pcmemtest -- stand-alone memory tester
On 2021-11-26 14:34:14, Felix Zielcke wrote: > Am Samstag, dem 20.11.2021 um 10:00 -0500 schrieb Antoine Beaupré: >> Do let me know if you need a sponsor. > > Hi Antoine, > > as talked on IRC: I would be happy if you do the review (audit) of > pcmemtest. > Since you didn't reply back if you actually do this now: Would be good > to have a clear answer here on the ITP so others know I have you as a > sponsor. I'm always kind of busy, but I always document what I do. If I would have done anything, it would have been documented here. If anyone else is available to sponsor, please go ahead. a. -- L'art n'est pas un bureau d'anthropométrie. - Léo Ferré, "Préface"
Bug#1000208: ITP: pcmemtest -- stand-alone memory tester
Am Samstag, dem 20.11.2021 um 10:00 -0500 schrieb Antoine Beaupré: > Do let me know if you need a sponsor. Hi Antoine, as talked on IRC: I would be happy if you do the review (audit) of pcmemtest. Since you didn't reply back if you actually do this now: Would be good to have a clear answer here on the ITP so others know I have you as a sponsor. TIA Felix
Bug#1000208: ITP: pcmemtest -- stand-alone memory tester
Hi Adam, thanks for all your comments. Am Montag, dem 22.11.2021 um 09:27 +0100 schrieb Adam Borowski: > On Sun, Nov 21, 2021 at 04:46:50PM +0100, Felix Zielcke wrote: > > Am Sonntag, dem 21.11.2021 um 10:15 -0500 schrieb Antoine Beaupré: > > > i would suggest not blocking on the grub deployment. you can > > > probably > > > just deploy whatever architecture the package was built from, in > > > any > > > case. > > > I just added now the GRUB 2 integration and a README.Debian. > > The integration isn't triggered on the package's install, but it is > picked > up the next time something else bumps grub. I forgot that grub2 isn't using triggers. Added now a postinst based on the one from memtest86+. Except that I removed the lilo parts. > > > > Note that I mention in README.Debian one nasty bug/issue: > > > > In EFI mode the keyboard only works if you have the CSM aka legacy > > boot > > also enabled: > > > > https://github.com/martinwhitaker/pcmemtest/issues/2 > > This is a nasty one. It seems most new machines lack CSM; no one wants > to > support and validate 16-bit stuff -- it's effectively expending > resources > to have two BIOSes instead of one, and the 8086 one has no practical > usage. > I opened now upstream a new issue about this: https://github.com/martinwhitaker/pcmemtest/issues/13 But in the closed one he said, it takes a while to write a usb keyboard driver for EFI. > > Upstream doestn't say anything there that this will change. Issue was > > closed with the hint it's documented. And it will just run with > > default > > settings. > > It does, but using just a single thread. There's not exactly many x86 > machines with only a single hardware thread that are still in use. > > What about defaulting to SMP when there's no user input? The UP mode > has little purpose for existing -- if concurrent accesses to memory > break, they'll also break when running the actual productive task the > machine is supposed to do. > I also made an issue for this: https://github.com/martinwhitaker/pcmemtest/issues/14 I don't know the reasons why it's not enabled by default. So I woudn't change that for the first upload to Debian on my own. > Meow! Cheers! Felix
Bug#1000208: ITP: pcmemtest -- stand-alone memory tester
On Sun, Nov 21, 2021 at 04:46:50PM +0100, Felix Zielcke wrote: > Am Sonntag, dem 21.11.2021 um 10:15 -0500 schrieb Antoine Beaupré: > > i would suggest not blocking on the grub deployment. you can probably > > just deploy whatever architecture the package was built from, in any > > case. > I just added now the GRUB 2 integration and a README.Debian. The integration isn't triggered on the package's install, but it is picked up the next time something else bumps grub. > You can now review it. And I think it would be better to first upload > this to experimental? So a bit more experienced Debian users can test > it first. That sounds like a good idea. And, pcmemtest works for me on new machines, while memtest86* either instantly reboots or locks up. > Note that I mention in README.Debian one nasty bug/issue: > > In EFI mode the keyboard only works if you have the CSM aka legacy boot > also enabled: > > https://github.com/martinwhitaker/pcmemtest/issues/2 This is a nasty one. It seems most new machines lack CSM; no one wants to support and validate 16-bit stuff -- it's effectively expending resources to have two BIOSes instead of one, and the 8086 one has no practical usage. > Upstream doestn't say anything there that this will change. Issue was > closed with the hint it's documented. And it will just run with default > settings. It does, but using just a single thread. There's not exactly many x86 machines with only a single hardware thread that are still in use. What about defaulting to SMP when there's no user input? The UP mode has little purpose for existing -- if concurrent accesses to memory break, they'll also break when running the actual productive task the machine is supposed to do. Meow! -- ⢀⣴⠾⠻⢶⣦⠀ ⣾⠁⢠⠒⠀⣿⡁ ⢿⡄⠘⠷⠚⠋⠀ At least spammers get it right: "Hello beautiful!". ⠈⠳⣄
Bug#1000208: ITP: pcmemtest -- stand-alone memory tester
Am Sonntag, dem 21.11.2021 um 10:15 -0500 schrieb Antoine Beaupré: > > i would suggest not blocking on the grub deployment. you can probably > just deploy whatever architecture the package was built from, in any > case. > > a. > Hi Antoine and Stephan, I just added now the GRUB 2 integration and a README.Debian. You can now review it. And I think it would be better to first upload this to experimental? So a bit more experienced Debian users can test it first. Note that I mention in README.Debian one nasty bug/issue: In EFI mode the keyboard only works if you have the CSM aka legacy boot also enabled: https://github.com/martinwhitaker/pcmemtest/issues/2 Upstream doestn't say anything there that this will change. Issue was closed with the hint it's documented. And it will just run with default settings. Is for you my salsa repo enough or just I also upload to mentors? Git repo is in git-buildpackage format: https://salsa.debian.org/fzielcke/pcmemtest TIA Felix
Bug#1000208: ITP: pcmemtest -- stand-alone memory tester
On 2021-11-20 17:56:22, Felix Zielcke wrote: > Am Samstag, dem 20.11.2021 um 10:00 -0500 schrieb Antoine Beaupré: >> On 2021-11-19 19:50:45, fziel...@z-51.de wrote: >> [...] >> >> > PCMemTest is a fork and rewrite of Memtest86+, which in turn was a >> > fork of >> > Memtest86. >> > >> > >> > I'm happy to maintain it inside a team or with co-maintainer(s). >> > I'm only DM so if someone has interest in sponsoring this, feel free >> > to >> > contact me. >> >> Great find! I was disappointed to find out that memtest86* is basically >> unusable these days in Debian, and find this ITP to be very >> interesting! >> Do let me know if you need a sponsor. >> >> Did you test the program at all? Does it behave better than the >> existing >> memtest86 packages currently in Debian? >> >> a. > > Thanks for your interest :) > > I pushed now my first work to my personal salsa profile: > > https://salsa.debian.org/fzielcke/pcmemtest > > The files are for now in /usr/lib/pcmemtest > There are 32bit and 64bit legacy BIOS + EFI files > Not sure if there's actually a different between the 2 legacy BIOS > versions. > > I haven't yet tried it myself but will do soon. > The suggestion came from #btrfs IRC channel. So it can't be that bad. > > > And I'm not sure how I want to do the GRUB integration. Due to the > differences with 32bit and 64bit EFI. > Is there actually a way to find out with what EFI version the system > booted? i would suggest not blocking on the grub deployment. you can probably just deploy whatever architecture the package was built from, in any case. a. -- C'est trop facile quand les guerres sont finies D'aller gueuler que c'était la dernière Amis bourgeois vous me faites envie Ne voyez vous pas donc point vos cimetières? - Jaques Brel
Bug#1000208: ITP: pcmemtest -- stand-alone memory tester
Sounds interesting. On Fri, Nov 19, 2021 at 8:03 PM wrote: > > I'm happy to maintain it inside a team or with co-maintainer(s). > I'm only DM so if someone has interest in sponsoring this, feel free to > contact me. If you need me as sponsor, ping me when the package is ready for upload. Regards Stephan
Bug#1000208: ITP: pcmemtest -- stand-alone memory tester
Am Samstag, dem 20.11.2021 um 10:00 -0500 schrieb Antoine Beaupré: > On 2021-11-19 19:50:45, fziel...@z-51.de wrote: > [...] > > > PCMemTest is a fork and rewrite of Memtest86+, which in turn was a > > fork of > > Memtest86. > > > > > > I'm happy to maintain it inside a team or with co-maintainer(s). > > I'm only DM so if someone has interest in sponsoring this, feel free > > to > > contact me. > > Great find! I was disappointed to find out that memtest86* is basically > unusable these days in Debian, and find this ITP to be very > interesting! > Do let me know if you need a sponsor. > > Did you test the program at all? Does it behave better than the > existing > memtest86 packages currently in Debian? > > a. Thanks for your interest :) I pushed now my first work to my personal salsa profile: https://salsa.debian.org/fzielcke/pcmemtest The files are for now in /usr/lib/pcmemtest There are 32bit and 64bit legacy BIOS + EFI files Not sure if there's actually a different between the 2 legacy BIOS versions. I haven't yet tried it myself but will do soon. The suggestion came from #btrfs IRC channel. So it can't be that bad. And I'm not sure how I want to do the GRUB integration. Due to the differences with 32bit and 64bit EFI. Is there actually a way to find out with what EFI version the system booted? Regards Felix
Bug#1000208: ITP: pcmemtest -- stand-alone memory tester
On 2021-11-19 19:50:45, fziel...@z-51.de wrote: [...] > PCMemTest is a fork and rewrite of Memtest86+, which in turn was a fork of > Memtest86. > > > I'm happy to maintain it inside a team or with co-maintainer(s). > I'm only DM so if someone has interest in sponsoring this, feel free to > contact me. Great find! I was disappointed to find out that memtest86* is basically unusable these days in Debian, and find this ITP to be very interesting! Do let me know if you need a sponsor. Did you test the program at all? Does it behave better than the existing memtest86 packages currently in Debian? a. -- We have to talk about liberating minds as well as liberating society. - Angela Davis
Bug#1000208: ITP: pcmemtest -- stand-alone memory tester
Package: wnpp Severity: wishlist Owner: Felix Zielcke X-Debbugs-Cc: debian-de...@lists.debian.org * Package name: pcmemtest Version : 1.5 Upstream Author : Martin Whitaker * URL : https://github.com/martinwhitaker/pcmemtest * License : GPL-2 Programming Lang: C, Assembly Description : stand-alone memory tester PCMemTest is a stand-alone memory tester for x86 and x86-64 architecture computers. It provides a more thorough memory check than that provided by BIOS memorytests. . PCMemTest can be loaded and run either directly by a PC BIOS (legacy or UEFI) or via an intermediate bootloader that supports the Linux 16-bit, 32-bit, 64-bit, or EFI handover boot protocol. It should work on any Pentium class or later 32-bit or 64-bit CPU. . PCMemTest is a fork and rewrite of Memtest86+, which in turn was a fork of Memtest86. I'm happy to maintain it inside a team or with co-maintainer(s). I'm only DM so if someone has interest in sponsoring this, feel free to contact me.
