From owner-freebsd-bugs Mon Dec 1 11:00:06 1997 Return-Path: Received: (from root@localhost) by hub.freebsd.org (8.8.7/8.8.7) id LAA02751 for bugs-outgoing; Mon, 1 Dec 1997 11:00:06 -0800 (PST) (envelope-from owner-freebsd-bugs) Received: (from gnats@localhost) by hub.freebsd.org (8.8.7/8.8.7) id LAA02738; Mon, 1 Dec 1997 11:00:02 -0800 (PST) (envelope-from gnats) Date: Mon, 1 Dec 1997 11:00:02 -0800 (PST) Message-Id: <199712011900.LAA02738@hub.freebsd.org> To: freebsd-bugs Cc: From: "Jin Guojun [ITG staff]" Subject: Re: i386/5183: ATAPI driver bug Reply-To: "Jin Guojun [ITG staff]" Sender: owner-freebsd-bugs@FreeBSD.ORG X-Loop: FreeBSD.org Precedence: bulk The following reply was made to PR i386/5183; it has been noted by GNATS. From: "Jin Guojun [ITG staff]" To: FreeBSD-gnats-submit@FreeBSD.ORG, junichi@astec.co.jp Cc: Subject: Re: i386/5183: ATAPI driver bug Date: Mon, 1 Dec 1997 10:56:03 -0800 >>Fix: > >Because of the ATAPI driver, 'atapi.c', works as AT_DRQT_INTR. >The ATAPI driver should work as AT_DRQT_MPROC. > >Apply the following patch to /usr/src/sys/i386/isa/atapi.c > > >*** atapi.c.org Sun Jul 20 23:09:50 1997 >--- atapi.c Sun Nov 30 16:49:29 1997 >*************** >*** 217,222 **** >--- 217,225 ---- > case AT_DRQT_ACCEL: printf (", accel"); break; > default: printf (", drq%d", ap->drqtype); > } >+ /* When 'slow' is set, 'intrcmd' is not acceptable. */ >+ if (ata->slow && ata->intrcmd) >+ ata->intrcmd = 0; > > /* overlap operation supported */ > if (ap->ovlapflag) Comment for syntax: No need to check ata->intrcmd if 'slow' is set. Simply clear it. + if (ata->slow) /* When 'slow' is set, 'intrcmd' is not acceptable. */ + ata->intrcmd = 0; -Jin