From owner-freebsd-bugs  Sun Apr 27 11:30:07 1997
Return-Path: <owner-bugs>
Received: (from root@localhost)
          by hub.freebsd.org (8.8.5/8.8.5) id LAA22255
          for bugs-outgoing; Sun, 27 Apr 1997 11:30:07 -0700 (PDT)
Received: (from gnats@localhost)
          by hub.freebsd.org (8.8.5/8.8.5) id LAA22236;
          Sun, 27 Apr 1997 11:30:02 -0700 (PDT)
Resent-Date: Sun, 27 Apr 1997 11:30:02 -0700 (PDT)
Resent-Message-Id: <199704271830.LAA22236@hub.freebsd.org>
Resent-From: gnats (GNATS Management)
Resent-To: freebsd-bugs
Resent-Reply-To: FreeBSD-gnats@freefall.FreeBSD.org,
        Received:(from nobody@localhost) by.hub.freebsd.org.id.LAA21901;Sun; (8.8.5/8.8.5);,
        27 Apr 1997 11:20:37.-0700 (PDT)
Message-Id: <199704271820.LAA21901@hub.freebsd.org>
Date: Sun, 27 Apr 1997 11:20:37 -0700 (PDT)
From: dmm125@bellatlantic.net
To: freebsd-gnats-submit@freebsd.org
X-Send-Pr-Version: www-1.0
Subject: kern/3395: sio2: not found, but there
Sender: owner-bugs@freebsd.org
X-Loop: FreeBSD.org
Precedence: bulk


>Number:         3395
>Category:       kern
>Synopsis:       sio2: not found, but there
>Confidential:   no
>Severity:       critical
>Priority:       medium
>Responsible:    freebsd-bugs
>State:          open
>Class:          sw-bug
>Submitter-Id:   current-users
>Arrival-Date:   Sun Apr 27 11:30:01 PDT 1997
>Last-Modified:
>Originator:     Donn Miller
>Organization:
>Release:        2.2.1-RELEASE
>Environment:
i386 16MHz
>Description:
sio2 was configured for correct port and irq, but was not being
probed.  Turned on flags 0x80 to find failing routine.  Said probe
test 3 failed.  Tried adjusting DELAY() values before failure[3]
to no avail.
>How-To-Repeat:
just reboot with internal modem with correct irq and port address set
(proper sio? enabled, of course), in 2.2.1 RELEASE.
>Fix:
in /sys/i386/isa/sio.c, apply this patch:

DELAY(10000);           /* Some internal modems need this time */
        if (idev->id_irq != 0)
-                failures[3] = isa_irq_pending(idev) ? 0 : 1;
+                failures[3] = isa_irq_pending(idev) ? 1 : 0;
        failures[4] = (inb(iobase + com_iir) & IIR_IMASK) - IIR_TXRDY;

Anyhow, this is what my sio.c said.  In the above at '-', it is implying
that probe test 3 will fail if there is no irq pending, which is
the opposite of what we want.       
>Audit-Trail:
>Unformatted: