From owner-cvs-src@FreeBSD.ORG Tue Sep 27 16:12:50 2005 Return-Path: X-Original-To: cvs-src@FreeBSD.org Delivered-To: cvs-src@FreeBSD.org Received: from mx1.FreeBSD.org (mx1.freebsd.org [216.136.204.125]) by hub.freebsd.org (Postfix) with ESMTP id 3F5C216A41F; Tue, 27 Sep 2005 16:12:50 +0000 (GMT) (envelope-from rik@FreeBSD.org) Received: from repoman.freebsd.org (repoman.freebsd.org [216.136.204.115]) by mx1.FreeBSD.org (Postfix) with ESMTP id 0F33043D48; Tue, 27 Sep 2005 16:12:50 +0000 (GMT) (envelope-from rik@FreeBSD.org) Received: from repoman.freebsd.org (localhost [127.0.0.1]) by repoman.freebsd.org (8.13.1/8.13.1) with ESMTP id j8RGCncO006161; Tue, 27 Sep 2005 16:12:49 GMT (envelope-from rik@repoman.freebsd.org) Received: (from rik@localhost) by repoman.freebsd.org (8.13.1/8.13.1/Submit) id j8RGCnqL006160; Tue, 27 Sep 2005 16:12:49 GMT (envelope-from rik) Message-Id: <200509271612.j8RGCnqL006160@repoman.freebsd.org> From: Roman Kurakin Date: Tue, 27 Sep 2005 16:12:49 +0000 (UTC) To: src-committers@FreeBSD.org, cvs-src@FreeBSD.org, cvs-all@FreeBSD.org X-FreeBSD-CVS-Branch: HEAD Cc: Subject: cvs commit: src/sys/dev/cp if_cp.c src/sys/dev/ctau if_ct.c src/sys/dev/cx if_cx.c X-BeenThere: cvs-src@freebsd.org X-Mailman-Version: 2.1.5 Precedence: list List-Id: CVS commit messages for the src tree List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Tue, 27 Sep 2005 16:12:50 -0000 rik 2005-09-27 16:12:49 UTC FreeBSD src repository Modified files: sys/dev/cp if_cp.c sys/dev/ctau if_ct.c sys/dev/cx if_cx.c Log: Backout if_cp 1.26, if_ct 1.27, if_cx 1.47 by obrien: ---------------------------- revision 1.26 date: 2005/09/07 09:53:35; author: obrien; state: Exp; lines: +1452 -1453 Reorder code to not depend on an ISO-C illegal forward extern declaration. ---------------------------- Reason: do not move large functions location without serious reason. The same could be done by forward function declaration. Please do not enlarge diff without a reason any more. Backout if_cp 1.27 ---------------------------- revision 1.27 date: 2005/09/19 03:10:16; author: imp; state: Exp; lines: +3 -2 Make sure that we call if_free(ifp) after bus_teardown_intr. Since we could get an interrupt after we free the ifp, and the interrupt handler depended on the ifp being still alive, this could, in theory, cause a crash. Eliminate this possibility by moving the if_free to after the bus_teardown_intr() call. Reason: bad previous commit. Would be restored by next commit. Revision Changes Path 1.28 +1451 -1450 src/sys/dev/cp/if_cp.c 1.28 +1368 -1366 src/sys/dev/ctau/if_ct.c 1.48 +1542 -1539 src/sys/dev/cx/if_cx.c