From owner-freebsd-arch Fri Mar 15 20:38:11 2002 Delivered-To: freebsd-arch@freebsd.org Received: from rover.village.org (rover.bsdimp.com [204.144.255.66]) by hub.freebsd.org (Postfix) with ESMTP id B107E37B400 for ; Fri, 15 Mar 2002 20:38:08 -0800 (PST) Received: from harmony.village.org (harmony.village.org [10.0.0.6]) by rover.village.org (8.11.3/8.11.3) with ESMTP id g2G4c7i47113; Fri, 15 Mar 2002 21:38:07 -0700 (MST) (envelope-from imp@village.org) Received: from localhost (warner@rover2.village.org [10.0.0.1]) by harmony.village.org (8.11.6/8.11.6) with ESMTP id g2G4c6L62627; Fri, 15 Mar 2002 21:38:07 -0700 (MST) (envelope-from imp@village.org) Date: Fri, 15 Mar 2002 21:37:57 -0700 (MST) Message-Id: <20020315.213757.117909745.imp@village.org> To: des@ofug.org Cc: arch@FreeBSD.ORG Subject: Re: HEADS UP: caddr_t sweep From: "M. Warner Losh" In-Reply-To: References: X-Mailer: Mew version 2.1 on Emacs 21.1 / Mule 5.0 (SAKAKI) Mime-Version: 1.0 Content-Type: Text/Plain; charset=us-ascii Content-Transfer-Encoding: 7bit Sender: owner-freebsd-arch@FreeBSD.ORG Precedence: bulk List-ID: List-Archive: (Web Archive) List-Help: (List Instructions) List-Subscribe: List-Unsubscribe: X-Loop: FreeBSD.ORG In message: Dag-Erling Smorgrav writes: : I'm preparing a sweep that will remove incorrect caddr_t casts in : copy{in,out,str,instr}(), bcopy() and bzero() calls in the kernel. : This will affect 650 to 700 lines of code, leaving approximately 2,000 : lines of code containing caddr_t casts, most of which are probably : equally bogus but will take more work to eradicate. : : This will probably cause merge conflicts for some of you. I apologize : for the inconvenience. i'd like to take a look at this patch before you commit it please. There are many driver uses of bcopy that are non-obvious and likely should be bus_space_read instead... Warner To Unsubscribe: send mail to majordomo@FreeBSD.org with "unsubscribe freebsd-arch" in the body of the message