Skip site navigation (1)Skip section navigation (2)
Date:      Fri, 18 May 2012 10:23:48 +0000 (UTC)
From:      Konstantin Belousov <kib@FreeBSD.org>
To:        src-committers@freebsd.org, svn-src-all@freebsd.org, svn-src-head@freebsd.org
Subject:   svn commit: r235603 - head/sys/vm
Message-ID:  <201205181023.q4IANmak047642@svn.freebsd.org>

next in thread | raw e-mail | index | archive | help
Author: kib
Date: Fri May 18 10:23:47 2012
New Revision: 235603
URL: http://svn.freebsd.org/changeset/base/235603

Log:
  Do not double-reference the found vm object in cdev_pager_lookup().
  vm_pager_object_lookup() already referenced the object.
  
  Note that there is no in-tree consumers of cdev_pager_lookup(). The
  only known user of the function is i915 gem driver, which is not yet
  imported. This should make the KPI change minor.
  
  Submitted by:	avg
  MFC after:	1 week

Modified:
  head/sys/vm/device_pager.c

Modified: head/sys/vm/device_pager.c
==============================================================================
--- head/sys/vm/device_pager.c	Fri May 18 10:15:46 2012	(r235602)
+++ head/sys/vm/device_pager.c	Fri May 18 10:23:47 2012	(r235603)
@@ -111,7 +111,6 @@ cdev_pager_lookup(void *handle)
 
 	mtx_lock(&dev_pager_mtx);
 	object = vm_pager_object_lookup(&dev_pager_object_list, handle);
-	vm_object_reference(object);
 	mtx_unlock(&dev_pager_mtx);
 	return (object);
 }



Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?201205181023.q4IANmak047642>