From owner-freebsd-current Mon Dec 27 18:11:48 1999 Delivered-To: freebsd-current@freebsd.org Received: from critter.freebsd.dk (critter.freebsd.dk [212.242.40.131]) by hub.freebsd.org (Postfix) with ESMTP id 99B8114DFB for ; Mon, 27 Dec 1999 18:11:45 -0800 (PST) (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 DAA13071; Tue, 28 Dec 1999 03:11:38 +0100 (CET) (envelope-from phk@critter.freebsd.dk) To: Matthew Dillon Cc: freebsd-current@FreeBSD.ORG Subject: Re: Proposed patch to fix VN device (again) In-reply-to: Your message of "Mon, 27 Dec 1999 17:53:57 PST." <199912280153.RAA35369@apollo.backplane.com> Date: Tue, 28 Dec 1999 03:11:38 +0100 Message-ID: <13069.946347098@critter.freebsd.dk> From: Poul-Henning Kamp Sender: owner-freebsd-current@FreeBSD.ORG Precedence: bulk X-Loop: FreeBSD.ORG In message <199912280153.RAA35369@apollo.backplane.com>, Matthew Dillon writes: > Huh? Look Poul, I don't know what your problem is but I'm through > playing around with you. You aren't making any sense and, frankly my > dear sir, the buffer cache, vm_pager interface, VN device, and > vm_pager_strategy subsystem is my ball of wax, not yours... Sorry, not so (and you've been told plenty many times already). You should fix the swap-pager to remove this last smoldering assumption about the existence of /dev/drum, rather than obfuscate the rest of the system by adding it back. I know it means more work for you, but we're not here to cut corners, we're here to get a better system. We don't get a better system if you keep pushing the dirty bits into obscure corners of the kernel rather than fix them where they need fixed. > I don't need > your permission and I certainly don't need your help if this is all you > can come up with. I objected to your patch on the grounds that it would be a step backwards, that has nothing to do with getting my permission. > can come up with. I'm not going to twist up all that wonderful clean code > just to avoid making the swapper a real device. You can call that "wonderful clean code" all you like, it is most certainly not the term I would use for it, and either way the matter of the fact is that struct buf is being abused to do something which it isn't able to do in its current design, and the way to fix that is not to add a hacked up dev_t but to give it special handling where it needs that. And adding a bogus dev_t and a equally bogus devsw does by no means make the swapper "a real device". I still object to your patch. It is a step backwards and it would further obfuscate the swap_pagers code. -- 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 freebsd-current" in the body of the message