[src/trunk]: src/libexec/getty getty: Remove 2 second sleep before opening tty

[jmcneill <jmcneill%NetBSD.org@localhost>]

details:   https://anonhg.NetBSD.org/src/rev/56ff89c50dab
branches:  trunk
changeset: 1024231:56ff89c50dab
user:      jmcneill <jmcneill%NetBSD.org@localhost>
date:      Tue Oct 12 23:40:38 2021 +0000

description:
getty: Remove 2 second sleep before opening tty

This (apparently very old!) workaround to ensure that DTR stays down long
enough to be detected is now handled by serial drivers themselves. For
console devices (framebuffer, serial console, etc) this was an unnecessary
2 second penalty that we have all been paying since the beginning of time.

diffstat:

 libexec/getty/main.c |  9 ++-------
 1 files changed, 2 insertions(+), 7 deletions(-)

diffs (30 lines):

diff -r 79cc5aff745d -r 56ff89c50dab libexec/getty/main.c
--- a/libexec/getty/main.c      Tue Oct 12 22:51:28 2021 +0000
+++ b/libexec/getty/main.c      Tue Oct 12 23:40:38 2021 +0000
@@ -1,4 +1,4 @@
-/*     $NetBSD: main.c,v 1.67 2020/02/26 15:44:57 riastradh Exp $      */
+/*     $NetBSD: main.c,v 1.68 2021/10/12 23:40:38 jmcneill Exp $       */
 
 /*-
  * Copyright (c) 1980, 1993
@@ -40,7 +40,7 @@
 #if 0
 static char sccsid[] = "from: @(#)main.c       8.1 (Berkeley) 6/20/93";
 #else
-__RCSID("$NetBSD: main.c,v 1.67 2020/02/26 15:44:57 riastradh Exp $");
+__RCSID("$NetBSD: main.c,v 1.68 2021/10/12 23:40:38 jmcneill Exp $");
 #endif
 #endif /* not lint */
 
@@ -229,11 +229,6 @@
                        if (ttyaction(ttyn, "getty", "root"))
                                syslog(LOG_WARNING, "%s: ttyaction failed",
                                        ttyn);
-                       /*
-                        * Delay the open so DTR stays down long enough
-                        * to be detected.
-                        */
-                       (void)sleep(2);
                        while ((i = open(ttyn, O_RDWR)) == -1) {
                                if ((repcnt % 10 == 0) &&
                                    (errno != ENXIO || !failopenlogged)) {