[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-ppc] [PATCH 1/2] Fix confusing argument names of do_unaligned_
From: |
Sergey Sorokin |
Subject: |
Re: [Qemu-ppc] [PATCH 1/2] Fix confusing argument names of do_unaligned_access() functions |
Date: |
Fri, 10 Jun 2016 19:36:39 +0300 |
I agree
10.06.2016, 19:33, "Peter Maydell" <address@hidden>:
> On 10 June 2016 at 17:26, Sergey Sorokin <address@hidden> wrote:
>> There are functions cpu_unaligned_access() and do_unaligned_access() that
>> are called with access type and mmu index arguments. But these arguments
>> are named 'is_write' and 'is_user' in their declarations.
>> The patch fixes the names to avoid a confusion.
>>
>> Signed-off-by: Sergey Sorokin <address@hidden>
>
> If we're going to touch all of these then we have an enum type
> we should be using instead of just 'int' for the old
> is_write argument: MMUAccessType (defined in cpu-common.h).
>
> thanks
> -- PMM
Re: [Qemu-ppc] [PATCH 1/2] Fix confusing argument names of do_unaligned_access() functions, Aurelien Jarno, 2016/06/13
[Qemu-ppc] [PATCH 2/2] Fix a confusing argument name in tlb_fill() function, Sergey Sorokin, 2016/06/10