Unsigned long is less than pointer, the reference passed in to the 'dst' variable.
Sincerely -- Mark Salyzyn -----Original Message----- From: [EMAIL PROTECTED] [mailto:[EMAIL PROTECTED] On Behalf Of Matthew Wilcox Sent: Wednesday, March 16, 2005 3:18 PM To: Salyzyn, Mark Cc: [email protected] Subject: Re: memcpy_toio/memcpy_fromio 64 bit safe? On Wed, Mar 16, 2005 at 03:08:13PM -0500, Salyzyn, Mark wrote: > The dpt_i2o driver uses the function memcpy_toio to copy the message > frame to the adapter. Under arch/x86_64/lib/io.c, it's helper function > is defined as: > > void __memcpy_toio(unsigned long dst,const void*src,unsigned len) > { > __inline_memcpy((void *) dst,src,len); > } > > The first argument should be 'volatile void __iomem * dst' I would have > expected. As such, the 64 bit virtual address representing the adapter > address passed in gets truncated (please correct me if I am wrong). Why would you think that 'void *' is smaller than 'volatile void __iomem *'? -- "Next the statesmen will invent cheap lies, putting the blame upon the nation that is attacked, and every man will be glad of those conscience-soothing falsities, and will diligently study them, and refuse to examine any refutations of them; and thus he will by and by convince himself that the war is just, and will thank God for the better sleep he enjoys after this process of grotesque self-deception." -- Mark Twain - To unsubscribe from this list: send the line "unsubscribe linux-scsi" in the body of a message to [EMAIL PROTECTED] More majordomo info at http://vger.kernel.org/majordomo-info.html
