Improve man page documentation on memstest 'all' argument and add

a run time warning about it as well

Prompted by:	Denis N. Peplin <info@volginfo.ru>
Approved by:	portmgr (self & no objections)

2 files changed, 32 insertions, 7 deletions

diff --git a/sysutils/memtest/files/patch-aa b/sysutils/memtest/files/patch-aa
index 33aa4b187437..e7aabad83438 100644
--- a/sysutils/memtest/files/patch-aa
+++ b/sysutils/memtest/files/patch-aa
@@ -1,5 +1,5 @@
---- memtest.c.orig	Tue Jul 18 22:13:31 2000
-+++ memtest.c	Tue Jul 18 22:13:58 2000
+--- memtest.c.orig	Thu Dec 19 13:43:09 2002
++++ memtest.c	Thu Dec 19 13:42:49 2002
 @@ -20,6 +20,7 @@
  #include <stdio.h>
  #include <stdlib.h>
@@ -8,3 +8,14 @@
  #include <sys/mman.h>
  #include <signal.h>
  #include <unistd.h>
+@@ -249,6 +250,10 @@
+ 			if (!strcmp (argv[i], "all"))
+ 			{
+ 				membytes = 4095 << MB;
++				fprintf (stderr, "%s: ATTENTION: option '%s' might cause a segmentation"
++						 " fault crash on %s because it will try to test as much"
++						 " as 4G of memory even if your system does not have as much"
++						 " memory.\n", argv[0], argv[i], argv[0]);
+ 				continue;
+ 			}
+ 			t = (ui32) strtol (argv[i], &endptr, 0);
diff --git a/sysutils/memtest/files/patch-ab b/sysutils/memtest/files/patch-ab
index 7027a28e13ca..53e606475459 100644
--- a/sysutils/memtest/files/patch-ab
+++ b/sysutils/memtest/files/patch-ab
@@ -1,13 +1,27 @@
 --- memtest.1.orig	Wed Apr 26 19:58:23 2000
-+++ memtest.1	Tue Jul 18 22:37:20 2000
-@@ -196,7 +196,9 @@
++++ memtest.1	Thu Dec 19 13:45:32 2002
+@@ -196,7 +196,12 @@
  faulty memory modules in Linux, or any Unix-like environment.
  .Sh "Invoking memtest"
  \fBmemtest\fR takes the first supplied argument as the amount of memory to test.
 -The value \fB'all\*(R'\fR means to test all available memory.  A specific amount of
-+The value \fB'all\*(R'\fR means to test all available memory. However, the value \fB'all\*(R'\fR
-+may not work on all systems, so a specific amount of memory might have to
-+be specified. A specific amount of
++.PP
++The value \fB'all\*(R'\fR means testing all available memory.
++However, the value \fB'all\*(R'\fR may not work on all systems, so
++a specific amount of memory might have to be specified.
++.PP
++A specific amount of
  memory can be supplied as a number, followed by a letter to indicate units:
  .Ip "\(bu" 4
  \fBB\fR or \fBb\fR \- bytes
+@@ -332,6 +337,10 @@
+ .Sh "Known Bugs"
+ The output examples in this manpage have fallen out of sync with the software
+ in minor ways.
++.PP
++The memory size value \fB'all\*(R'\fR means testing all available
++memory.  However, the value \fB'all\*(R'\fR may not work on all
++systems, so a specific amount of memory might have to be specified.
+ .Sh "How To Report Bugs"
+ Bugs should be reported to <memtest@discworld.dyndns.org>.
+ .PP