From owner-cvs-src@FreeBSD.ORG Fri Apr 25 02:11:52 2003 Return-Path: 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 5DA2437B405 for ; Fri, 25 Apr 2003 02:11:52 -0700 (PDT) Received: from rootlabs.com (root.org [67.118.192.226]) by mx1.FreeBSD.org (Postfix) with SMTP id 6454843F93 for ; Fri, 25 Apr 2003 02:11:51 -0700 (PDT) (envelope-from nate@rootlabs.com) Received: (qmail 64459 invoked by uid 1000); 25 Apr 2003 09:11:54 -0000 Date: Fri, 25 Apr 2003 02:11:54 -0700 (PDT) From: Nate Lawson To: cvs-all@FreeBSD.org, cvs-src@FreeBSD.org, src-committers@FreeBSD.org In-Reply-To: <20030425090201.5740737B48C@hub.freebsd.org> Message-ID: MIME-Version: 1.0 Content-Type: TEXT/PLAIN; charset=US-ASCII Subject: Re: cvs commit: src/sys/dev/fxp if_fxp.c if_fxpvar.h X-BeenThere: cvs-src@freebsd.org X-Mailman-Version: 2.1.1 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: Fri, 25 Apr 2003 09:11:52 -0000 On Fri, 25 Apr 2003, Nate Lawson wrote: > njl 2003/04/25 02:01:54 PDT > > Modified files: > sys/dev/fxp if_fxp.c if_fxpvar.h > Log: > Make fxp(4) INTR_MPSAFE (but do not enable MPSAFE just yet): As the comment in the code says, MPSAFE is not enabled because ifnet is not locked yet. I was informed of this by hsu@. However, the driver was very stable for me (no crashes or increased packet loss) so I am committing this code so others can test it. If you do enable MPSAFE, be aware of potential data loss or crashes. For developers, note that the locking in the code path only protects the various fxp routines (fxp_start, fxp_intr, fxp_tick, ...) and is not intended to serialize access to ANY external structures. This is how it should be. Please do not copy the exact approach taken here for a little while until ifnet locking is finished as there may need to be some changes made to this model. -Nate