ACK: [PATCH][V2] acpica: ppc64el needs to be built as a 64 bit architecture

Keng-Yu Lin keng-yu.lin at canonical.com
Fri Mar 14 05:46:16 UTC 2014


On Fri, Mar 14, 2014 at 12:26 PM, IvanHu <ivan.hu at canonical.com> wrote:
> On 03/13/2014 12:34 AM, Colin King wrote:
>>
>> From: Colin Ian King <colin.king at canonical.com>
>>
>> ppc64el was segfaulting because the build was defaulting to
>> a 32 bit build.  This is a temporary fix for the current
>> release of fwts, I hope to get this fixed upstream in ACPICA
>> some time soon.  With this fix the syntaxcheck and disassemble
>> regression tests were failing because iasl was segfaulting.
>>
>> Signed-off-by: Colin Ian King <colin.king at canonical.com>
>> ---
>>   src/acpica/source/include/platform/aclinux.h | 2 +-
>>   1 file changed, 1 insertion(+), 1 deletion(-)
>>
>> diff --git a/src/acpica/source/include/platform/aclinux.h
>> b/src/acpica/source/include/platform/aclinux.h
>> index b8da18b..e85cc79 100644
>> --- a/src/acpica/source/include/platform/aclinux.h
>> +++ b/src/acpica/source/include/platform/aclinux.h
>> @@ -161,7 +161,7 @@
>>   #define ACPI_FLUSH_CPU_CACHE()
>>   #define ACPI_CAST_PTHREAD_T(Pthread) ((ACPI_THREAD_ID) (Pthread))
>>
>> -#if defined(__ia64__) || defined(__x86_64__) || defined(__aarch64__)
>> +#if defined(__ia64__) || defined(__x86_64__) || defined(__aarch64__) ||
>> defined(__PPC64__)
>>   #define ACPI_MACHINE_WIDTH          64
>>   #define COMPILER_DEPENDENT_INT64    long
>>   #define COMPILER_DEPENDENT_UINT64   unsigned long
>>
>
> Acked-by: Ivan Hu <ivan.hu at canonical.com>
>

Acked-by: Keng-Yu Lin <kengyu at canonical.com>



More information about the fwts-devel mailing list