From owner-cvs-all Wed Dec 6 13:52:27 2000 From owner-cvs-all@FreeBSD.ORG Wed Dec 6 13:52:24 2000 Return-Path: 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 E05F237B400; Wed, 6 Dec 2000 13:52:23 -0800 (PST) Received: (from alfred@localhost) by freefall.freebsd.org (8.11.1/8.11.1) id eB6LqNU01059; Wed, 6 Dec 2000 13:52:23 -0800 (PST) (envelope-from alfred) Message-Id: <200012062152.eB6LqNU01059@freefall.freebsd.org> From: Alfred Perlstein Date: Wed, 6 Dec 2000 13:52:23 -0800 (PST) To: cvs-committers@FreeBSD.org, cvs-all@FreeBSD.org Subject: cvs commit: src/sys/vm phys_pager.c X-FreeBSD-CVS-Branch: HEAD Sender: owner-cvs-all@FreeBSD.ORG Precedence: bulk X-Loop: FreeBSD.ORG alfred 2000/12/06 13:52:23 PST Modified files: sys/vm phys_pager.c Log: Really fix phys_pager: Backout the previous delta (rev 1.4), it didn't make any difference. If the requested handle is NULL then don't add it to the list of objects, to be found by handle. The problem is that when asking for a NULL handle you are implying you want a new object. Because objects with NULL handles were being added to the list, any further requests for phys backed objects with NULL handles would return a reference to the initial NULL handle object after finding it on the list. Basically one couldn't have more than one phys backed object without a handle in the entire system without this fix. If you did more than one shared memory allocation using the phys pager it would give you your initial allocation again. Revision Changes Path 1.5 +37 -32 src/sys/vm/phys_pager.c To Unsubscribe: send mail to majordomo@FreeBSD.org with "unsubscribe cvs-all" in the body of the message