From owner-cvs-all Sat Jan 11 21: 4: 2 2003 Delivered-To: cvs-all@freebsd.org Received: from mx1.FreeBSD.org (mx1.freebsd.org [216.136.204.125]) by hub.freebsd.org (Postfix) with ESMTP id C767F37B401; Sat, 11 Jan 2003 21:04:00 -0800 (PST) Received: from apollo.backplane.com (apollo.backplane.com [216.240.41.2]) by mx1.FreeBSD.org (Postfix) with ESMTP id E7A8C43F1E; Sat, 11 Jan 2003 21:03:59 -0800 (PST) (envelope-from dillon@apollo.backplane.com) Received: from apollo.backplane.com (localhost [127.0.0.1]) by apollo.backplane.com (8.12.6/8.12.6) with ESMTP id h0C53xYp052105; Sat, 11 Jan 2003 21:03:59 -0800 (PST) (envelope-from dillon@apollo.backplane.com) Received: (from dillon@localhost) by apollo.backplane.com (8.12.6/8.12.6/Submit) id h0C53xTg052104; Sat, 11 Jan 2003 21:03:59 -0800 (PST) Date: Sat, 11 Jan 2003 21:03:59 -0800 (PST) From: Matthew Dillon Message-Id: <200301120503.h0C53xTg052104@apollo.backplane.com> To: Alfred Perlstein Cc: Jake Burkholder , cvs-committers@FreeBSD.ORG, cvs-all@FreeBSD.ORG Subject: Re: cvs commit: src/sys/kern kern_acl.c kern_descrip.c kern_event.c kern_mac.c sys_pipe.c sys_socket.c uipc_socket.c uipc_syscalls.c uipc_usrreq.c vfs_aio.c vfs_syscalls.c vfs_vnops.c src/sys/netgraph ng_socket.c src/sys/sys file.h src/sys/ufs/ffs ... References: <200301120137.h0C1bD0E098037@repoman.freebsd.org> <20030111213259.E212@locore.ca> <200301120254.h0C2srcS043241@apollo.backplane.com> <20030112040835.GA33821@elvis.mu.org> Sender: owner-cvs-all@FreeBSD.ORG Precedence: bulk List-ID: List-Archive: (Web Archive) List-Help: (List Instructions) List-Subscribe: List-Unsubscribe: X-Loop: FreeBSD.ORG : :* Matthew Dillon [030111 18:55] wrote: :> :> It's a cleanup. Ugliness I noticed as I was evaluating how easy it :> would be to add a memory object file type. I don't see why you believe :> a justification is required other then cleanup and code readability. :> void *'s are things you only use when you need to. We certainly didn't :> need to here. How exactly is this commit so bad that you are against it? : :What about third party/loadable DTYPEs? Isn't this more annoying :than helpful for that case? You've also broken the style of this :file by removing the f_ prefix. Please fix. : :-- :-Alfred Perlstein [alfred@freebsd.org] I will also point out that although the union is using named structures, the structures themselves are only used as pointers and therefore are opaque. Any third party struct file pointer structure that we incorporate in the main tree can be added to sys/file.h as 'struct blah;' without introducing any include file dependancies, whether the third party module is compiled in or not (hence it not being a layering violation). Any truely third party code would have to use un_data.generic, but that is nothing more then a #define in the third party module's header file. We could also #define f_data to un_data.generic in sys/file.h (for #ifdef KERNEL code only to avoid pollution) and while I didn't do it, I left that option open on purpose. -Matt Matthew Dillon To Unsubscribe: send mail to majordomo@FreeBSD.org with "unsubscribe cvs-all" in the body of the message