Date: Sun, 7 Oct 2012 16:10:31 -0700 From: Devin Teske <devin.teske@fisglobal.com> To: Garrett Cooper <yanegomi@gmail.com> Cc: Devin Teske <dteske@freebsd.org>, freebsd-arch@freebsd.org Subject: Re: Tell the user we're booting Message-ID: <6DBE2255-40A3-452B-A503-04AF6A3FBE42@fisglobal.com> In-Reply-To: <EF63E60A-CF60-433C-9D91-D25C2B7E2645@gmail.com> References: <DFD7A734-6876-4232-AA9E-521B1041C3D9@fisglobal.com> <EF63E60A-CF60-433C-9D91-D25C2B7E2645@gmail.com>
next in thread | previous in thread | raw e-mail | index | archive | help
--Apple-Mail=_293A43D5-87B6-4CBC-A47B-F6BDE3C45D69
Content-Transfer-Encoding: quoted-printable
Content-Type: text/plain; charset="windows-1252"
On Oct 7, 2012, at 3:53 PM, Garrett Cooper wrote:
> On Oct 7, 2012, at 3:39 PM, Devin Teske wrote:
>=20
>> Hi,
>>=20
>> I'd like to propose a small (tiny even) patch to sys/boot/forth/loader.4=
th
>>=20
>> Said patch will echo "Booting=85" before invoking the kernel.
>>=20
>> This patch would effect every architecture (even those that don't use th=
e beastie menu).
>>=20
>> The reason for proposing this tiny change is that as of 9.0-R I've notic=
ed a significant lag between executing "boot" and getting something on-scre=
en to show that something is happening.
>=20
> This is probably the memory test (set "hw.memtest.tests" to "0" in loade=
r.conf and see if that speeds things up). Regardless, I think it'd be a goo=
d thing to have.
> Thanks!
> -Garrett
>=20
> PS Got patch?
Here's patch:
_____________
The information contained in this message is proprietary and/or confidentia=
l. If you are not the intended recipient, please: (i) delete the message an=
d all copies; (ii) do not disclose, distribute or use the message in any ma=
nner; and (iii) notify the sender immediately. In addition, please be aware=
that any message addressed to our domain is subject to archiving and revie=
w by persons other than the intended recipient. Thank you.
--Apple-Mail=_293A43D5-87B6-4CBC-A47B-F6BDE3C45D69
Content-Disposition: attachment; filename="patch.txt"
Content-Type: text/plain; name="patch.txt"
Content-Transfer-Encoding: 7bit
Index: color.4th.8
===================================================================
--- color.4th.8 (revision 241310)
+++ color.4th.8 (working copy)
@@ -50,7 +50,7 @@
.Dl include color.4th
.Pp
This line is present in
-.Pa /boot/beastie.4th
+.Pa /boot/loader.4th
file, so it is not needed (and should not be re-issued) in a normal setup.
.Pp
The commands provided by it are:
@@ -102,7 +102,6 @@
.Sh SEE ALSO
.Xr loader.conf 5 ,
.Xr loader 8 ,
-.Xr beastie.4th 8 ,
.Xr loader.4th 8
.Sh HISTORY
The
Index: loader.4th
===================================================================
--- loader.4th (revision 241310)
+++ loader.4th (working copy)
@@ -40,6 +40,7 @@
2048 dictincrease ! \ 2048 additional cells each time
include /boot/support.4th
+include /boot/color.4th
only forth also support-functions also builtins definitions
@@ -64,6 +65,12 @@
: boot
0= if ( interpreted ) get_arguments then
+ loader_color? if
+ ." [37;44mBooting...[0m" cr
+ else
+ ." Booting..." cr
+ then
+
\ Unload only if a path was passed
dup if
>r over r> swap
Index: beastie.4th
===================================================================
--- beastie.4th (revision 241310)
+++ beastie.4th (working copy)
@@ -28,7 +28,6 @@
marker task-beastie.4th
-include /boot/color.4th
include /boot/delay.4th
variable logoX
--Apple-Mail=_293A43D5-87B6-4CBC-A47B-F6BDE3C45D69--
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?6DBE2255-40A3-452B-A503-04AF6A3FBE42>