From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([2001:4830:134:3::10]:37618) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1YzWNN-0004Eo-Qy for qemu-devel@nongnu.org; Mon, 01 Jun 2015 16:32:02 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1YzWNK-0007lj-3L for qemu-devel@nongnu.org; Mon, 01 Jun 2015 16:32:01 -0400 Received: from mx1.redhat.com ([209.132.183.28]:44439) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1YzWNJ-0007lf-Ug for qemu-devel@nongnu.org; Mon, 01 Jun 2015 16:31:58 -0400 Message-ID: <556CC13C.3090109@redhat.com> Date: Mon, 01 Jun 2015 16:31:56 -0400 From: John Snow MIME-Version: 1.0 References: <1432881807-18164-1-git-send-email-edgar.iglesias@gmail.com> <1432881807-18164-2-git-send-email-edgar.iglesias@gmail.com> <556CBC38.8030603@redhat.com> In-Reply-To: Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: 7bit Subject: Re: [Qemu-devel] [PATCH v3 01/15] target-arm: Correct check for non-EL3 List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: Peter Maydell Cc: Edgar Iglesias , Alexander Graf , QEMU Developers , Sergey Fedorov , "Edgar E. Iglesias" , =?UTF-8?B?QWxleCBCZW5uw6ll?= On 06/01/2015 04:26 PM, Peter Maydell wrote: > On 1 June 2015 at 21:10, John Snow wrote: >> >> >> On 05/29/2015 02:43 AM, Edgar E. Iglesias wrote: >>> From: "Edgar E. Iglesias" >>> >>> Signed-off-by: Edgar E. Iglesias >>> --- >>> target-arm/op_helper.c | 2 +- >>> 1 file changed, 1 insertion(+), 1 deletion(-) >>> >>> diff --git a/target-arm/op_helper.c b/target-arm/op_helper.c >>> index 3f5b9ab..7583ae7 100644 >>> --- a/target-arm/op_helper.c >>> +++ b/target-arm/op_helper.c >>> @@ -421,7 +421,7 @@ void HELPER(access_check_cp_reg)(CPUARMState *env, void *rip, uint32_t syndrome) >>> /* Requesting a trap to EL2 when we're in EL3 or S-EL0/1 is >>> * a bug in the access function. >>> */ >>> - assert(!arm_is_secure(env) && !arm_current_el(env) == 3); >>> + assert(!arm_is_secure(env) && arm_current_el(env) != 3); >>> target_el = 2; >>> break; >>> case CP_ACCESS_TRAP_EL3: >>> >> >> This patch could even be liberated from this series if it hasn't been >> already, since the clang build is currently broken with -Werror without >> this fix. > > Only clang 3.5, I suspect -- my clang build uses Werror but it's > an older clang. I have queued the first half of this series into > target-arm.next anyway (might directly apply this one as a buildfix). > > -- PMM > Oh, yes, this is with Clang 3.5.0 as seen in F21. I don't have a particularly convenient way to test older versions at the moment, so I do all of my clang regression testing on this version. Wouldn't mind the buildfix for this and the other issue spotted by MST to help make build testing sane again this week :) Thanks, --js