From mboxrd@z Thu Jan 1 00:00:00 1970 From: Julien Grall Subject: Re: [PATCH v10 10/10] xen/arm: make accesses to desc->status flags atomic Date: Tue, 05 Aug 2014 13:35:52 +0100 Message-ID: <53E0CFA8.2010104@linaro.org> References: <1407237989-27654-10-git-send-email-stefano.stabellini@eu.citrix.com> Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Return-path: In-Reply-To: <1407237989-27654-10-git-send-email-stefano.stabellini@eu.citrix.com> List-Unsubscribe: , List-Post: List-Help: List-Subscribe: , Sender: xen-devel-bounces@lists.xen.org Errors-To: xen-devel-bounces@lists.xen.org To: Stefano Stabellini , xen-devel@lists.xensource.com Cc: julien.grall@citrix.com, Ian.Campbell@citrix.com List-Id: xen-devel@lists.xenproject.org Hi Stefano, On 08/05/2014 12:26 PM, Stefano Stabellini wrote: > This way we don't need to take the desc->lock in order to access > desc->status in many of the gic and vgic functions. > > Using *_bit manipulation functions on desc->status is safe on arm64: > status is an unsigned int but is the first field of a struct that > contains pointers, therefore the alignement of the struct is at least 8 > bytes. > > Signed-off-by: Stefano Stabellini > Acked-by: Ian Campbell Acked-by: Julien Grall Regards, -- Julien Grall