Re: [Xen-devel] [PATCH v4 2/6] xen: arm: consolidate body of flush_xen_data_tlb_range_va_local

[Julien Grall <julien.grall@xxxxxxxxxx>]

On 04/03/2014 09:59 AM, Ian Campbell wrote:
> +/*
> + * Flush a range of VA's hypervisor mappings from the data TLB of the
> + * local processor. This is not sufficient when changing code mappings
> + * or for self modifying code.
> + */
> +static inline void flush_xen_data_tlb_range_va_local(unsigned long va,
> +                                                     unsigned long size)
> +{
> +    unsigned long end = va + size;
> +    dsb(sy); /* Ensure preceding are visible */
> +    while ( va < end ) {

while ( va < end )
{

Except this minor coding style error (I've noticed it was there before):

Acked-by: Julien Grall <julien.grall@xxxxxxxxxx>

Regards,

-- 
Julien Grall

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