From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org Received: from lists.xenproject.org (lists.xenproject.org [192.237.175.120]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.lore.kernel.org (Postfix) with ESMTPS id B5193C25B78 for ; Wed, 22 May 2024 18:05:42 +0000 (UTC) Received: from list by lists.xenproject.org with outflank-mailman.727831.1132508 (Exim 4.92) (envelope-from ) id 1s9qKz-0006hO-SW; Wed, 22 May 2024 18:05:17 +0000 X-Outflank-Mailman: Message body and most headers restored to incoming version Received: by outflank-mailman (output) from mailman id 727831.1132508; Wed, 22 May 2024 18:05:17 +0000 Received: from localhost ([127.0.0.1] helo=lists.xenproject.org) by lists.xenproject.org with esmtp (Exim 4.92) (envelope-from ) id 1s9qKz-0006hH-Ov; Wed, 22 May 2024 18:05:17 +0000 Received: by outflank-mailman (input) for mailman id 727831; Wed, 22 May 2024 18:05:16 +0000 Received: from se1-gles-sth1-in.inumbo.com ([159.253.27.254] helo=se1-gles-sth1.inumbo.com) by lists.xenproject.org with esmtp (Exim 4.92) (envelope-from ) id 1s9qKy-0006hB-Iy for xen-devel@lists.xenproject.org; Wed, 22 May 2024 18:05:16 +0000 Received: from support.bugseng.com (mail.bugseng.com [162.55.131.47]) by se1-gles-sth1.inumbo.com (Halon) with ESMTPS id d6468b06-1865-11ef-90a0-e314d9c70b13; Wed, 22 May 2024 20:05:14 +0200 (CEST) Received: from support.bugseng.com (support.bugseng.com [162.55.131.47]) by support.bugseng.com (Postfix) with ESMTPA id 484994EE0738; Wed, 22 May 2024 20:05:13 +0200 (CEST) X-BeenThere: xen-devel@lists.xenproject.org List-Id: Xen developer discussion List-Unsubscribe: , List-Post: List-Help: List-Subscribe: , Errors-To: xen-devel-bounces@lists.xenproject.org Precedence: list Sender: "Xen-devel" X-Inumbo-ID: d6468b06-1865-11ef-90a0-e314d9c70b13 MIME-Version: 1.0 Date: Wed, 22 May 2024 20:05:13 +0200 From: Nicola Vetrini To: Julien Grall Cc: Alessandro Zucchelli , xen-devel@lists.xenproject.org, consulting@bugseng.com, Stefano Stabellini , Bertrand Marquis , Michal Orzel , Volodymyr Babchuk , Tamas K Lengyel , Alexandru Isaila , Petre Pircalabu Subject: Re: [XEN PATCH v3] arm/mem_access: add conditional build of mem_access.c In-Reply-To: References: Message-ID: <6b7fee28e8f54799fe8ec7fdc6d4001d@bugseng.com> X-Sender: nicola.vetrini@bugseng.com Organization: BUGSENG s.r.l. Content-Type: text/plain; charset=US-ASCII; format=flowed Content-Transfer-Encoding: 7bit On 2024-05-10 22:59, Julien Grall wrote: > Hi, Hi, > > On 10/05/2024 13:32, Alessandro Zucchelli wrote: >> In order to comply to MISRA C:2012 Rule 8.4 for ARM the following >> changes are done: >> revert preprocessor conditional changes to xen/mem_access.h which >> had it build unconditionally, add conditional build for >> xen/mem_access.c > > I am afraid, I don't understand this one as you don't seem to modify > xen/mem_access.h. Is this meant to be part of the changelog? > > You also don't seem to mention the change in Makefile. This is the one > I was asking for in the previous version. So what about: > > "xen/arm: mem_access: Conditionally compile mem_access.c > > Commit 634cfc8beb ("Make MEM_ACCESS configurable") intended to make > MEM_ACCESS configurable on Arm to reduce the code size when the user > doesn't need it. > > However, this didn't cover the arch specific code. None of the code in > arm/mem_access.c is necessary when MEM_ACCESS=n, so it can be compiled > out. This will require to provide some stub for functions called by the > common code. > > This is also fixing violation of the MISRA C:2012 Rule 8.4 reported by > ECLAIR. > " > > The patch itself loks good so once we agree on the commit message, then > I am happy to update it on commit. > > Cheers, since Julien is ok with the patch, with the commit message he proposed, I think this needs an R-by or an A-by in order to commit for 4.19. -- Nicola Vetrini, BSc Software Engineer, BUGSENG srl (https://bugseng.com)