From owner-cvs-all Sun Jul 8 13:50:28 2001 Delivered-To: cvs-all@freebsd.org Received: from freefall.freebsd.org (freefall.freebsd.org [216.136.204.21]) by hub.freebsd.org (Postfix) with ESMTP id 986EF37B401; Sun, 8 Jul 2001 13:50:21 -0700 (PDT) (envelope-from joerg@FreeBSD.org) Received: (from joerg@localhost) by freefall.freebsd.org (8.11.3/8.11.3) id f68KoLH89971; Sun, 8 Jul 2001 13:50:21 -0700 (PDT) (envelope-from joerg) Message-Id: <200107082050.f68KoLH89971@freefall.freebsd.org> From: Joerg Wunsch Date: Sun, 8 Jul 2001 13:50:21 -0700 (PDT) To: cvs-committers@FreeBSD.org, cvs-all@FreeBSD.org Subject: cvs commit: src/sys/isa fd.c X-FreeBSD-CVS-Branch: HEAD Sender: owner-cvs-all@FreeBSD.ORG Precedence: bulk List-ID: List-Archive: (Web Archive) List-Help: (List Instructions) List-Subscribe: List-Unsubscribe: X-Loop: FreeBSD.ORG joerg 2001/07/08 13:50:21 PDT Modified files: sys/isa fd.c Log: Another large patchset from Bruce. Despite of a few cosmetic things like adding ``irritating silly parentheses'' around all return values, this mainly improves FDC reset handling by no longer gratuitously resetting the FDC all the time (which causes it to lose the notion of the current track) but only in case of errors, and it sanitizes the block and offset calculations in fdstrategy() and fdstate(). Some additional cleanup added by me, in particular the large switch in fdstate() now always uses return to break out, and no branch falls off the end of the switch statement anymore. Per Bruce's suggestion, removed M_NOWAIT from the malloc()s to simplify things. Submitted by: bde (mostly) Revision Changes Path 1.212 +114 -159 src/sys/isa/fd.c To Unsubscribe: send mail to majordomo@FreeBSD.org with "unsubscribe cvs-all" in the body of the message