From owner-cvs-all  Wed Dec 15  9:15:49 1999
Delivered-To: cvs-all@freebsd.org
Received: from duke.cs.duke.edu (duke.cs.duke.edu [152.3.140.1])
	by hub.freebsd.org (Postfix) with ESMTP
	id 1FCBB150E4; Wed, 15 Dec 1999 09:15:44 -0800 (PST)
	(envelope-from gallatin@cs.duke.edu)
Received: from grasshopper.cs.duke.edu (grasshopper.cs.duke.edu [152.3.145.30])
	by duke.cs.duke.edu (8.9.1/8.9.1) with ESMTP id MAA28370;
	Wed, 15 Dec 1999 12:15:42 -0500 (EST)
Received: (from gallatin@localhost)
	by grasshopper.cs.duke.edu (8.9.3/8.9.1) id MAA18864;
	Wed, 15 Dec 1999 12:15:12 -0500 (EST)
	(envelope-from gallatin@cs.duke.edu)
From: Andrew Gallatin <gallatin@cs.duke.edu>
MIME-Version: 1.0
Content-Type: text/plain; charset=us-ascii
Content-Transfer-Encoding: 7bit
Date: Wed, 15 Dec 1999 12:15:11 -0500 (EST)
To: Matthew Dillon <dillon@apollo.backplane.com>
Cc: cvs-committers@FreeBSD.ORG, cvs-all@FreeBSD.ORG
Subject: Re: cvs commit: src/sys/nfs nfs_serv.c
In-Reply-To: <199912151651.IAA32041@apollo.backplane.com>
References: <199912131734.JAA02524@freefall.freebsd.org>
	<14423.40355.215768.533614@grasshopper.cs.duke.edu>
	<199912151651.IAA32041@apollo.backplane.com>
X-Mailer: VM 6.43 under 20.4 "Emerald" XEmacs  Lucid
Message-ID: <14423.52047.200482.909691@grasshopper.cs.duke.edu>
Sender: owner-cvs-all@FreeBSD.ORG
Precedence: bulk


Matthew Dillon writes:
 > : >   Revision  Changes    Path
 > : >   1.90      +78 -2     src/sys/nfs/nfs_serv.c
 > :
 > :This introduces a 32/64-bit problem on alphas:
 > :
 > :cc -c -O -Wall -Wredundant-decls -Wnested-externs -Wstrict-prototypes  -Wmissing-prototypes -Wpointer-arith -Winline -Wcast-qual  -fformat-extensions -ansi -g -nostdinc -I- -I. -I../.. -I/usr/include  -DKERNEL -include opt_global.h -D_KERNEL   -mno-fp-regs -Wa,-mev56  ../../nfs/nfs_serv.c
 > :../../nfs/nfs_serv.c: In function `nfsrv_read':
 > :../../nfs/nfs_serv.c:874: warning: cast from pointer to integer of different size
 > :
 > :
 > :How about casting to a vm_offset_t?
 > :
 > :Eg:
 > 
 >     Try (int)(vm_offset_t).  We converting the pointer to an int only
 >     to generate a hash index so we don't care if we lose bits.  If that
 >     removes the warning for alpha compilation I'll commit it.
 > 
 > 						-Matt

Yep, that shuts the compiler up.  Whenever I see 'cast from pointer to
integer of different size' I start frothing at the mouth ;-)

Thanks,

Drew




To Unsubscribe: send mail to majordomo@FreeBSD.org
with "unsubscribe cvs-all" in the body of the message