Re: [PATCH 1/4] xen/arm: io: handle_read: Use a local variable to store dabt

[Ian Campbell <ian.campbell@citrix.com>]

On Fri, 2015-12-11 at 15:28 +0000, Julien Grall wrote:
> Rather than getting dabt every time through info->dabt, introduce a
> local variable and use it.
> 
> Also fix a coding style error in the if condition.
> 
> Signed-off-by: Julien Grall <julien.grall@citrix.com>

Acked-by: Ian Campbell <ian.campbell@citrix.com>

> ---
>  xen/arch/arm/io.c | 5 +++--
>  1 file changed, 3 insertions(+), 2 deletions(-)
> 
> diff --git a/xen/arch/arm/io.c b/xen/arch/arm/io.c
> index de5765a..7e29943 100644
> --- a/xen/arch/arm/io.c
> +++ b/xen/arch/arm/io.c
> @@ -26,7 +26,8 @@
>  static int handle_read(const struct mmio_handler *handler, struct vcpu
> *v,
>                         mmio_info_t *info, register_t *r)
>  {
> -    uint8_t size = (1 << info->dabt.size) * 8;
> +    const struct hsr_dabt dabt = info->dabt;
> +    uint8_t size = (1 << dabt.size) * 8;
>  
>      if ( !handler->ops->read(v, info, r, handler->priv) )
>          return 0;
> @@ -36,7 +37,7 @@ static int handle_read(const struct mmio_handler
> *handler, struct vcpu *v,
>       * Note that we expect the read handler to have zeroed the bits
>       * outside the requested access size.
>       */
> -    if ( info->dabt.sign && (*r & (1UL << (size - 1)) ))
> +    if ( dabt.sign && (*r & (1UL << (size - 1))) )
>      {
>          /*
>           * We are relying on register_t using the same as

_______________________________________________
Xen-devel mailing list
Xen-devel@lists.xen.org
http://lists.xen.org/xen-devel