From owner-cvs-all Sat Sep 18 22:33:23 1999 Delivered-To: cvs-all@freebsd.org Received: from proxy4.ba.best.com (proxy4.ba.best.com [206.184.139.15]) by hub.freebsd.org (Postfix) with ESMTP id 2A33515049; Sat, 18 Sep 1999 22:33:19 -0700 (PDT) (envelope-from dillon@apollo.backplane.com) Received: from apollo.backplane.com ([209.157.86.2]) by proxy4.ba.best.com (8.9.3/8.9.2/best.out) with ESMTP id WAA10237; Sat, 18 Sep 1999 22:31:37 -0700 (PDT) Received: (from dillon@localhost) by apollo.backplane.com (8.9.3/8.9.1) id WAA68523; Sat, 18 Sep 1999 22:31:36 -0700 (PDT) (envelope-from dillon) Date: Sat, 18 Sep 1999 22:31:36 -0700 (PDT) From: Matthew Dillon Message-Id: <199909190531.WAA68523@apollo.backplane.com> To: Poul-Henning Kamp Cc: Julian Elischer , cvs-committers@FreeBSD.ORG, cvs-all@FreeBSD.ORG Subject: Re: cvs commit: src/sys/dev/vn vn.c References: <13144.937692636@critter.freebsd.dk> Sender: owner-cvs-all@FreeBSD.ORG Precedence: bulk :In message , Julian : Elischer writes: :>Poul-Henning, since Matt (who is working on this driver) STATED that he :>didn't want you to do this, why did you do it anyway? : :Uhm, I read your last message as your approval for this... : :>If you want to do this right.. :> :>Move the calculation of blocksize in spec_read and spec_write :>to spec_open, and store the value in to si_bsize_best if it's still unset, :>and then in read/write just blindly use _best, instead of doing the :>calculation on every read and write. : :I think si_bsize_best should not even be used in spec_read/_write. : :I think they should blindly use max(PAGE_SIZE, si_bsize_phys). : :-- :Poul-Henning Kamp FreeBSD coreteam member Considering that you knew there was a dispute, and especially considering that you knew that I was considering making the default larger, not removing it, you should not have made that commit. However, I am willing to give you the benefit of the doubt this time around on the assumption that it will not happen again. Since I've pretty much been the person who has made the VN device useful again over the last 6 months, I should probably be considered to be the current maintainer on it. Please make no further commits to the VN device without running them past me first. -Matt Matthew Dillon To Unsubscribe: send mail to majordomo@FreeBSD.org with "unsubscribe cvs-all" in the body of the message