[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [Qemu-ppc] [PATCH 8/8] target-ppc Disentangle ppc64 has
From: |
Alexander Graf |
Subject: |
Re: [Qemu-devel] [Qemu-ppc] [PATCH 8/8] target-ppc Disentangle ppc64 hash mmu path for cpu_ppc_handle_mmu_fault |
Date: |
Sat, 23 Feb 2013 14:47:23 +0100 |
On 23.02.2013, at 09:13, David Gibson wrote:
> On Fri, Feb 22, 2013 at 05:13:56PM +0100, Alexander Graf wrote:
>> On 12.02.2013, at 03:00, David Gibson wrote:
> [snip]
>>> + } else if (ret < 0) {
>>> + LOG_MMU_STATE(env);
>>> + if (access_type == ACCESS_CODE) {
>>> + switch (ret) {
>>> + case -1:
>>> + env->exception_index = POWERPC_EXCP_ISI;
>>> + env->error_code = 0x40000000;
>>> + break;
>>> + case -2:
>>> + /* Access rights violation */
>>
>> These really should become an enum or defines. Something where the
>> thing we check on tells us what we're checking for without a comment
>> :).
>
> So, I didn't want to introduce an emum or whatever at the same time as
> I was just moving the code around - makes for more chance of
> mistakes. And by the end of the series, I've gotten rid of these
> magic error values anyway, by reporting the exceptions in-line in mmu
> fault handle, so I think it's kind of moot.
I was reviewing the earlier 8 patch set which didn't convert the return values
to inline exception paths yet. The new set looks very nice :)
Alex
- Re: [Qemu-devel] [Qemu-ppc] [PATCH 7/8] target-ppc: Disentangle 64-bit hash MMU get_physical_address() paths, (continued)
[Qemu-devel] [PATCH 8/8] target-ppc Disentangle ppc64 hash mmu path for cpu_ppc_handle_mmu_fault, David Gibson, 2013/02/11
[Qemu-devel] [PATCH 4/8] target-ppc: Disentangle 64-bit version of find_pte(), David Gibson, 2013/02/11
[Qemu-devel] [PATCH 2/8] target-ppc: Move SLB handling into a mmu-hash64.c, David Gibson, 2013/02/11
[Qemu-devel] [PATCH 5/8] target-ppc: Disentangle 64-bit version of get_segment(), David Gibson, 2013/02/11
Re: [Qemu-devel] [0/8] RFC: target-ppc: Start disentangling different MMU types, Andreas Färber, 2013/02/12
Re: [Qemu-devel] [0/8] RFC: target-ppc: Start disentangling different MMU types, Alexander Graf, 2013/02/22