From owner-freebsd-arch Sat Mar 16 4: 1:46 2002 Delivered-To: freebsd-arch@freebsd.org Received: from mailman.zeta.org.au (mailman.zeta.org.au [203.26.10.16]) by hub.freebsd.org (Postfix) with ESMTP id 8A1D537B402 for ; Sat, 16 Mar 2002 04:01:43 -0800 (PST) Received: from bde.zeta.org.au (bde.zeta.org.au [203.2.228.102]) by mailman.zeta.org.au (8.9.3/8.8.7) with ESMTP id WAA29159; Sat, 16 Mar 2002 22:59:02 +1100 Date: Sat, 16 Mar 2002 23:00:38 +1100 (EST) From: Bruce Evans X-X-Sender: To: "M. Warner Losh" Cc: , Subject: Re: HEADS UP: caddr_t sweep In-Reply-To: <20020315.213757.117909745.imp@village.org> Message-ID: <20020316225716.I28305-100000@gamplex.bde.org> MIME-Version: 1.0 Content-Type: TEXT/PLAIN; charset=US-ASCII 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 On Fri, 15 Mar 2002, M. Warner Losh wrote: > 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. > ... > > 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... Maybe limit it to copy* and other userland interfaces (mmap?) then. There are also some bcopies which should be struct assignments. Bruce To Unsubscribe: send mail to majordomo@FreeBSD.org with "unsubscribe freebsd-arch" in the body of the message