From owner-cvs-all Mon Oct 11 2:52:22 1999 Delivered-To: cvs-all@freebsd.org Received: from critter.freebsd.dk (critter.freebsd.dk [212.242.40.131]) by hub.freebsd.org (Postfix) with ESMTP id 2DC3414E81; Mon, 11 Oct 1999 02:52:16 -0700 (PDT) (envelope-from phk@critter.freebsd.dk) Received: from critter.freebsd.dk (localhost [127.0.0.1]) by critter.freebsd.dk (8.9.3/8.9.2) with ESMTP id LAA19434; Mon, 11 Oct 1999 11:48:23 +0200 (CEST) (envelope-from phk@critter.freebsd.dk) To: Matthew Dillon Cc: Peter Wemm , "Daniel C. Sobral" , cvs-committers@FreeBSD.ORG, cvs-all@FreeBSD.ORG Subject: Re: cvs commit: src/sys/vm vm_swap.c In-reply-to: Your message of "Mon, 11 Oct 1999 00:09:58 PDT." <199910110709.AAA20043@apollo.backplane.com> Date: Mon, 11 Oct 1999 11:48:22 +0200 Message-ID: <19432.939635302@critter.freebsd.dk> From: Poul-Henning Kamp Sender: owner-cvs-all@FreeBSD.ORG Precedence: bulk In message <199910110709.AAA20043@apollo.backplane.com>, Matthew Dillon writes: >:> >:> We're not removing the device interface. It provides a reasonable >:> abstraction and a nice demark between swap_pager.c and vm_swap.c, >:> as well as potential flexibility that could be useful in the future >:> Removing it will not save time or much in the way of code space. >: >:The swap *device* is non-functional and doesn't *do* anything except >:provide a subroutine call in a very roundabout way. I think it would be >:far better to avoid an indirection via the VOP_* system for no useful gain >:and do something like the aooended patch (which works perfectly here BTW, >:even under heavy swap load on multiple disks). Further (micro) >:optimizations are possible, for example pbgetvp() is used to get a p-buffer >:that's associated with swapdev_vp and the device. The device isn't used >:and presently vn_todev(vp) ends up returning NODEV. swapdev_vp is kinda >:orphaned with this change but still works ok. p-buffers are created being Peter, I have reviewed and tested your patch and it works fine. > Gentlemen. You might as well stop. I've said it before and I > will say it again: Nobody is screwing around with the swap > subsytem. I think we can safely say that this is not "screwing around with the swap subsytem", so obviously your possive position doesn't apply to Peters patch. -- Poul-Henning Kamp FreeBSD coreteam member phk@FreeBSD.ORG "Real hackers run -current on their laptop." FreeBSD -- It will take a long time before progress goes too far! To Unsubscribe: send mail to majordomo@FreeBSD.org with "unsubscribe cvs-all" in the body of the message