From owner-cvs-all Thu Oct 15 13:11:42 1998 Return-Path: Received: (from majordom@localhost) by hub.freebsd.org (8.8.8/8.8.8) id NAA22023 for cvs-all-outgoing; Thu, 15 Oct 1998 13:11:42 -0700 (PDT) (envelope-from owner-cvs-all@FreeBSD.ORG) Received: from implode.root.com (implode.root.com [198.145.90.17]) by hub.freebsd.org (8.8.8/8.8.8) with ESMTP id NAA22004; Thu, 15 Oct 1998 13:11:40 -0700 (PDT) (envelope-from root@implode.root.com) Received: from implode.root.com (localhost [127.0.0.1]) by implode.root.com (8.8.5/8.8.5) with ESMTP id NAA18162; Thu, 15 Oct 1998 13:09:27 -0700 (PDT) Message-Id: <199810152009.NAA18162@implode.root.com> To: Doug Rabson cc: cvs-committers@FreeBSD.ORG, cvs-all@FreeBSD.ORG Subject: Re: cvs commit: src/sys/cam/scsi scsi_da.c src/sys/i386/i386 busdma_machdep.c vm_machdep.c src/sys/i386/ibcs2 imgact_coff.c src/sys/i386/include param.h src/sys/i386/isa isa.c wd.c src/sys/kern imgact_elf.c sys_pipe.c sysv_shm.c vfs_bio.c ... In-reply-to: Your message of "Wed, 14 Oct 1998 21:29:36 BST." From: David Greenman Reply-To: dg@root.com Date: Thu, 15 Oct 1998 13:09:26 -0700 Sender: owner-cvs-all@FreeBSD.ORG Precedence: bulk >> 2) Fixed a bogus type cast in round_page() and trunc_page() that >> caused some 64bit offsets and sizes to be scrambled. Removing >> the cast required adding casts at a few dozen callers. >> There may be problems with other bogus casts in close-by >> macros. A quick check seemed to indicate that those were okay, >> however. > >Do I need to track this change in the alpha port? Can you give me a rough >idea of the changes to busdma_machdep.c, vm_machdep.c and param.h so that >I can make the appropriate change (if needed). Basically, round_page and trunc_page and the calls to them need to be 64bit safe. Since a long is 64bits on alpha, I don't think there is a bug there, except for perhaps the cast being wrong: param.h:#define round_page(x) ((((unsigned long)(x)) + PAGE_MASK) & ~(PAGE_MASK)) The cast should be removed, and then any compile errors that this creates with the callers need to be fixed by casting to vm_offset_t. -DG David Greenman Co-founder/Principal Architect, The FreeBSD Project To Unsubscribe: send mail to majordomo@FreeBSD.org with "unsubscribe cvs-all" in the body of the message