[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[Qemu-devel] Re: sparc32 don't mark page dirty when failing
From: |
Blue Swirl |
Subject: |
[Qemu-devel] Re: sparc32 don't mark page dirty when failing |
Date: |
Sun, 31 Jan 2010 11:49:17 +0200 |
Thanks, applied.
On Sun, Jan 31, 2010 at 4:27 AM, Artyom Tarasenko
<address@hidden> wrote:
> if the access check fails, the page can not be modified
> and shouldn't be marked dirty.
> The patch fixes the "hsfs_putpage: dirty HSFS page"
> error in Solaris guests.
>
> Signed-off-by: Artyom Tarasenko <address@hidden>
> ---
> diff --git a/target-sparc/helper.c b/target-sparc/helper.c
> index b5b4e7c..ffe93e3 100644
> --- a/target-sparc/helper.c
> +++ b/target-sparc/helper.c
> @@ -185,6 +185,12 @@ static int get_physical_address(CPUState *env,
> target_phys_addr_t *physical,
> }
> }
>
> + /* check access */
> + access_perms = (pde & PTE_ACCESS_MASK) >> PTE_ACCESS_SHIFT;
> + error_code = access_table[*access_index][access_perms];
> + if (error_code && !((env->mmuregs[0] & MMU_NF) && is_user))
> + return error_code;
> +
> /* update page modified and dirty bits */
> is_dirty = (rw & 1) && !(pde & PG_MODIFIED_MASK);
> if (!(pde & PG_ACCESSED_MASK) || is_dirty) {
> @@ -193,11 +199,6 @@ static int get_physical_address(CPUState *env,
> target_phys_addr_t *physical,
> pde |= PG_MODIFIED_MASK;
> stl_phys_notdirty(pde_ptr, pde);
> }
> - /* check access */
> - access_perms = (pde & PTE_ACCESS_MASK) >> PTE_ACCESS_SHIFT;
> - error_code = access_table[*access_index][access_perms];
> - if (error_code && !((env->mmuregs[0] & MMU_NF) && is_user))
> - return error_code;
>
> /* the page can be put in the TLB */
> *prot = perm_table[is_user][access_perms];
>