From mboxrd@z Thu Jan  1 00:00:00 1970
From: Tony Jones <tonyj@suse.de>
Subject: Re: [PATCH] audit:  add ppc64 mach support
Date: Mon, 01 Dec 2014 15:58:09 -0800
Message-ID: <547D0091.909@suse.de>
References: <5400DF80.1080700@suse.de> <1622774.lL0rysIyod@x2>
Mime-Version: 1.0
Content-Type: text/plain; charset="us-ascii"
Content-Transfer-Encoding: 7bit
Return-path: <linux-audit-bounces@redhat.com>
In-Reply-To: <1622774.lL0rysIyod@x2>
List-Unsubscribe: <https://www.redhat.com/mailman/options/linux-audit>,
	<mailto:linux-audit-request@redhat.com?subject=unsubscribe>
List-Archive: <https://www.redhat.com/archives/linux-audit>
List-Post: <mailto:linux-audit@redhat.com>
List-Help: <mailto:linux-audit-request@redhat.com?subject=help>
List-Subscribe: <https://www.redhat.com/mailman/listinfo/linux-audit>,
	<mailto:linux-audit-request@redhat.com?subject=subscribe>
Sender: linux-audit-bounces@redhat.com
Errors-To: linux-audit-bounces@redhat.com
To: Steve Grubb <sgrubb@redhat.com>
Cc: linux-audit@redhat.com
List-Id: linux-audit@redhat.com

On 12/01/2014 02:09 PM, Steve Grubb wrote:
> Hi Tony,
> 
> On Friday, August 29, 2014 01:16:00 PM Tony Jones wrote:
>> Add support for ppc64le.
>>
>> $ uname -a
>> Linux cabernet 3.12.26-3-default #1 SMP Mon Aug 18 15:07:30 UTC 2014
>> (d318f3a) ppc64le ppc64le ppc64le GNU/Linux
>>
>> Without this,  perf trace and auditctl fail.   There is no 32 bit (ppcle).
>>
>> Signed-off-by: Tony Jones <tonyj@suse.de>
>> ---
>>
>> Index: trunk/lib/libaudit.c
>> ===================================================================
>> --- trunk/lib/libaudit.c	(revision 1011)
>> +++ trunk/lib/libaudit.c	(working copy)
>> @@ -1195,6 +1195,11 @@
>>  				return -6;
>>  			break;
>>  #endif
>> +		case MACH_PPC64LE:
>> +			if (bits != __AUDIT_ARCH_64BIT)
>> +				return -6;
>> +			break;
>> +
>>  		case MACH_86_64: /* fallthrough */
>>  		case MACH_PPC64: /* fallthrough */
>>  		case MACH_S390X: /* fallthrough */
>> Index: trunk/lib/libaudit.h
>> ===================================================================
>> --- trunk/lib/libaudit.h	(revision 1011)
>> +++ trunk/lib/libaudit.h	(working copy)
>> @@ -356,6 +356,9 @@
>>  #define AUDIT_ARCH_AARCH64	(EM_AARCH64|__AUDIT_ARCH_64BIT|
> __AUDIT_ARCH_LE)
>>  #endif
>>
>> +#ifndef AUDIT_ARCH_PPC64LE
>> +#define AUDIT_ARCH_PPC64LE	(EM_PPC64|__AUDIT_ARCH_64BIT|__AUDIT_ARCH_LE)
>> +#endif
> 
> Is this ^^^ define upstream or in some public branch?
> 
> -Steve


Mainline kernel reports ppc64le (per a0588015deab1844261b27a67ae6f5b910fe2830) but there is no matching AUDIT_ARCH_PPC64LE.  Eric just pinged me on irc about it.   Fix to add it to include/uapi/linux/audit.h shoudld probably go upstream.