svn commit: r243960 - in head/sys: amd64/include i386/include x86/include

Carl Delsey carl.r.delsey at intel.com
Sat Dec 8 00:00:46 UTC 2012


On 12/07/12 10:08, John Baldwin wrote:
> On Thursday, December 06, 2012 5:33:32 pm Jim Harris wrote:
>> Author: jimharris
>> Date: Thu Dec  6 22:33:31 2012
>> New Revision: 243960
>> URL: http://svnweb.freebsd.org/changeset/base/243960
>>
>> Log:
>>    Add amd64 implementations for 8-byte bus_space routines.
>>    
>>    Submitted by:	Carl Delsey <carl.r.delsey at intel.com>
>>    Discussed with:	jhb, rwatson
>>    Reviewed by:	jimharris
>>    MFC after:	1 week
>>
>> Modified:
>>    head/sys/amd64/include/bus.h
>>    head/sys/i386/include/bus.h
>>    head/sys/x86/include/bus.h
> Do we still want the link errors on i386?  Do you get a compile error now?
>
You get a compile error now, since the function prototypes don't exist 
for i386. That was the case in the old code too. The comments about 
forcing a link error were incorrect or obsolete.


More information about the svn-src-all mailing list