From owner-cvs-src@FreeBSD.ORG  Sun Aug 24 15:20:48 2008
Return-Path: <owner-cvs-src@FreeBSD.ORG>
Delivered-To: cvs-src@FreeBSD.org
Received: from mx1.freebsd.org (mx1.freebsd.org [IPv6:2001:4f8:fff6::34])
	by hub.freebsd.org (Postfix) with ESMTP id D2E731065689;
	Sun, 24 Aug 2008 15:20:48 +0000 (UTC) (envelope-from ed@FreeBSD.org)
Received: from repoman.freebsd.org (repoman.freebsd.org
	[IPv6:2001:4f8:fff6::29])
	by mx1.freebsd.org (Postfix) with ESMTP id B0E9B8FC23;
	Sun, 24 Aug 2008 15:20:48 +0000 (UTC) (envelope-from ed@FreeBSD.org)
Received: from repoman.freebsd.org (localhost [127.0.0.1])
	by repoman.freebsd.org (8.14.2/8.14.2) with ESMTP id m7OFKmWR013025;
	Sun, 24 Aug 2008 15:20:48 GMT (envelope-from ed@repoman.freebsd.org)
Received: (from svn2cvs@localhost)
	by repoman.freebsd.org (8.14.2/8.14.1/Submit) id m7OFKma1013024;
	Sun, 24 Aug 2008 15:20:48 GMT (envelope-from ed@repoman.freebsd.org)
Message-Id: <200808241520.m7OFKma1013024@repoman.freebsd.org>
X-Authentication-Warning: repoman.freebsd.org: svn2cvs set sender to
	ed@repoman.freebsd.org using -f
From: Ed Schouten <ed@FreeBSD.org>
Date: Sun, 24 Aug 2008 15:20:44 +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/syscons sysmouse.c
X-BeenThere: cvs-src@freebsd.org
X-Mailman-Version: 2.1.5
Precedence: list
List-Id: CVS commit messages for the src tree <cvs-src.freebsd.org>
List-Unsubscribe: <http://lists.freebsd.org/mailman/listinfo/cvs-src>,
	<mailto:cvs-src-request@freebsd.org?subject=unsubscribe>
List-Archive: <http://lists.freebsd.org/pipermail/cvs-src>
List-Post: <mailto:cvs-src@freebsd.org>
List-Help: <mailto:cvs-src-request@freebsd.org?subject=help>
List-Subscribe: <http://lists.freebsd.org/mailman/listinfo/cvs-src>,
	<mailto:cvs-src-request@freebsd.org?subject=subscribe>
X-List-Received-Date: Sun, 24 Aug 2008 15:20:48 -0000

ed          2008-08-24 15:20:44 UTC

  FreeBSD src repository

  Modified files:
    sys/dev/syscons      sysmouse.c 
  Log:
  SVN rev 182109 on 2008-08-24 15:20:44Z by ed
  
  Make sysmouse(4) use its own locks, instead of using Giant.
  
  When I changed syscons(4) to work with the MPSAFE TTY code, I just
  locked all device nodes down using the compatibility feature that allows
  you to override the TTY's lock (Giant in this case). Upon closer
  inspection, it seems sysmouse(4) only has two internal variables that
  need locking: mouse_level and mouse_status.
  
  I haven't done any performance benchmarks on this, though I think it
  won't have any dramatic improvements on the system. It is good to get
  rid of Giant here, because the third argument of tty_alloc() has only
  been added to ease migration to MPSAFE TTY. It should not be used when
  not needed.
  
  While there, remove SC_MOUSE, which is a leftover from the MPSAFE TTY
  import.
  
  Revision  Changes    Path
  1.32      +10 -14    src/sys/dev/syscons/sysmouse.c