svn commit: r235050 - head/sys/arm/arm

Warner Losh imp at bsdimp.com
Sat May 5 17:16:03 UTC 2012


On May 5, 2012, at 10:19 AM, Nathan Whitehorn wrote:

> On 05/05/12 02:15, Warner Losh wrote:
>> Author: imp
>> Date: Sat May  5 07:15:34 2012
>> New Revision: 235050
>> URL: http://svn.freebsd.org/changeset/base/235050
>> 
>> Log:
>>   Big endian arm boxes need to have a uname -m of armeb, not arm, so
>>   that the bootstrap from source works correctly.
>> 
>>   MFC after:	4 days
>> 
>> Modified:
>>   head/sys/arm/arm/identcpu.c
>> 
>> Modified: head/sys/arm/arm/identcpu.c
>> ==============================================================================
>> --- head/sys/arm/arm/identcpu.c	Sat May  5 02:53:19 2012	(r235049)
>> +++ head/sys/arm/arm/identcpu.c	Sat May  5 07:15:34 2012	(r235050)
>> @@ -52,11 +52,16 @@ __FBSDID("$FreeBSD$");
>>  #include<sys/kernel.h>
>>  #include<sys/sysctl.h>
>>  #include<machine/cpu.h>
>> +#include<machine/endian.h>
>> 
>>  #include<machine/cpuconf.h>
>>  #include<machine/md_var.h>
>> 
>> +#if _BYTE_ORDER == _LITTLE_ENDIAN
>>  char machine[] = "arm";
>> +#else
>> +char machine[] = "armeb";
>> +#endif
>> 
>>  SYSCTL_STRING(_hw, HW_MACHINE, machine, CTLFLAG_RD,
>>          machine, 0, "Machine class");
> 
> Didn't you mean to change MACHINE_ARCH in /sys/arm/include/param.h? Setting MACHINE=armeb will break lots of things, including self-hosting.

DOh!  You're right...  I'll fix that. :(

Warner




More information about the svn-src-all mailing list