From patchwork Wed Apr 3 03:43:43 2019 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 8bit X-Patchwork-Submitter: Richard Henderson X-Patchwork-Id: 161668 Delivered-To: patch@linaro.org Received: by 2002:a02:c6d8:0:0:0:0:0 with SMTP id r24csp2519737jan; Tue, 2 Apr 2019 20:59:33 -0700 (PDT) X-Google-Smtp-Source: APXvYqwXxzsylSlqwWQn7K+XKvvRU96P/B12v5qGeE1IzYgovUna3X2/RFMsSs8ptOuq/EThKUaF X-Received: by 2002:a25:56d4:: with SMTP id k203mr59689530ybb.475.1554263973506; Tue, 02 Apr 2019 20:59:33 -0700 (PDT) ARC-Seal: i=1; a=rsa-sha256; t=1554263973; cv=none; d=google.com; s=arc-20160816; b=dJ+z4tEOZWgHqZqvNI5703MQatTPqYn2ik6LE2TBmZn79Po4HqkvXkxL/XFyu/q5QM YASLYyw4lLrNVLpYhyBu2fU5KSiZnysxzuW7Bu+9E1nzYulQykhrk3ZzFd8BOuY2G0Pm 3sMQhiidOXWA4sG8RHkAJNakMp8Wwx5aDqEBgtIo5raMEYQHVAwCaZQbhfjT9Vm7UNoe i+BQ6vaJQsWf72qEkv4DT9ueMVpizk3YGUD1nUf5sqfstjpttm1VJbnK+MfztXcIU97H I8Q/Wo7zuT7thaYvfku7318muULgfSmgyhQloSRpHoRVOdYhDoTef6tAa2aW/2DQ0pVK nYmA== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=sender:errors-to:cc:list-subscribe:list-help:list-post:list-archive :list-unsubscribe:list-id:precedence:subject:references:in-reply-to :message-id:date:to:from:dkim-signature; bh=hsT/4r55/BmYXvm+N7+ZWx+INVnPxar+m3RehQ8v76k=; b=yqpz8PcUnVNDHfsry6MCulRqvpf4CYwWy9IhzZ4TJdleJJB6r6TS9/L0L5831/lNJd 01UI6NCmAZ4r5L3ynILZgeRzxbotpZS2RSm9N5jGt+b0Wj4E/C7gjmKsqyflAngs+O5v rjxdYLw5S/kEaFhfuHwHZn10SPmTzkRXV+OwUNLJxzsUyrON9l3/3mDTuuOaYLZo2Yry PK0l5hF5mHvsYx+EgZaO4Ctnnt8NMXQDTVL24lBCsvWRS6Q6AMxTVEY9Myb0S9HKjIhN wTDjUlUGeOZJjySM3Vrtr7xCeftQVOTZFv80WLwtjL1l5GeTtXjA9jV4E7MzYoCp8wxa EVyA== ARC-Authentication-Results: i=1; mx.google.com; dkim=fail header.i=@linaro.org header.s=google header.b=QWQGkT3A; spf=pass (google.com: domain of qemu-devel-bounces+patch=linaro.org@nongnu.org designates 209.51.188.17 as permitted sender) smtp.mailfrom="qemu-devel-bounces+patch=linaro.org@nongnu.org"; dmarc=fail (p=NONE sp=NONE dis=NONE) header.from=linaro.org Return-Path: Received: from lists.gnu.org (lists.gnu.org. [209.51.188.17]) by mx.google.com with ESMTPS id l82si8828234ybb.346.2019.04.02.20.59.33 for (version=TLS1 cipher=AES128-SHA bits=128/128); Tue, 02 Apr 2019 20:59:33 -0700 (PDT) Received-SPF: pass (google.com: domain of qemu-devel-bounces+patch=linaro.org@nongnu.org designates 209.51.188.17 as permitted sender) client-ip=209.51.188.17; Authentication-Results: mx.google.com; dkim=fail header.i=@linaro.org header.s=google header.b=QWQGkT3A; spf=pass (google.com: domain of qemu-devel-bounces+patch=linaro.org@nongnu.org designates 209.51.188.17 as permitted sender) smtp.mailfrom="qemu-devel-bounces+patch=linaro.org@nongnu.org"; dmarc=fail (p=NONE sp=NONE dis=NONE) header.from=linaro.org Received: from localhost ([127.0.0.1]:55082 helo=lists.gnu.org) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1hBX3l-0007YU-1b for patch@linaro.org; Tue, 02 Apr 2019 23:59:33 -0400 Received: from eggs.gnu.org ([209.51.188.92]:48634) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1hBWp9-0002sd-8k for qemu-devel@nongnu.org; Tue, 02 Apr 2019 23:44:28 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1hBWp7-0008MK-MX for qemu-devel@nongnu.org; Tue, 02 Apr 2019 23:44:27 -0400 Received: from mail-pg1-x543.google.com ([2607:f8b0:4864:20::543]:43566) by eggs.gnu.org with esmtps (TLS1.0:RSA_AES_128_CBC_SHA1:16) (Exim 4.71) (envelope-from ) id 1hBWp7-0008LM-At for qemu-devel@nongnu.org; Tue, 02 Apr 2019 23:44:25 -0400 Received: by mail-pg1-x543.google.com with SMTP id z9so7577662pgu.10 for ; Tue, 02 Apr 2019 20:44:25 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=linaro.org; s=google; h=from:to:cc:subject:date:message-id:in-reply-to:references; bh=hsT/4r55/BmYXvm+N7+ZWx+INVnPxar+m3RehQ8v76k=; b=QWQGkT3AtW2SLec4EL7fTVJg/0l4CDTeQoyActooDWWFhrTd3sngWxWB6gWdn5BL9y E6T53uBDdbVFxZl0Pr6Ei2aK47CaGDvH5lfyrb81hfALtqH2nYxpjnrcF1HmiWLdpDrR w+JKlXGQ92oBr8uNL/1d79RhQdDtEhWl7a3c00wgiSX6ssJRy2/bt6Na4DzLTI1maXp0 eLRYinlUPINAOQu/XMsnsa5y/TFeFflL+Zja9TNPWQ3phSavn/Z8yP7AHa7ZoBYWaaHc tj3ov5Y0LeAsX8hXK7qBxvjGx1drSVwSt6TxDVqY8upoF6Bvum46AUZYH/xFiA4RrdeT pYCA== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:from:to:cc:subject:date:message-id:in-reply-to :references; bh=hsT/4r55/BmYXvm+N7+ZWx+INVnPxar+m3RehQ8v76k=; b=AT6eSAlHjySB3eBf1hLH4PipjG2vpr95fqk7PVurAguCTnt9+3Pcs6cxiJusW1FBDo yb4srQI668cyZuvmYsg9Q8Stt/rWn4jmGKnRsLBzDKcZA4ELzKrMrrh0aX/D65xM3X7N lpkKliPjfJUhFXe5ss83m4li1VwLwZybpeW1907mM8slvJ7jwU04lqWENnlq+A4oEYFh Gs7bmTk00n4ojRMOUaEQyXYeOvJ4P0RKUX9jf/sdN1uCdGe7qaQQ/OzpuqtsPq9zNP3c 2F2dPnEuq8WIW5sxj2S0pRIEIW08hir160Qa7+rOv2F7jkisMdQGM/f79oejLfMhZwJF 3Sig== X-Gm-Message-State: APjAAAWJxyibwYuXeD3wHFmxYswfYIaK8NzICxdstOeuPEtxPifWh08E wcQAygNmizyaaKWXsjSPn8e9Y7wstHVRmg== X-Received: by 2002:a63:c84c:: with SMTP id l12mr70606042pgi.287.1554263063941; Tue, 02 Apr 2019 20:44:23 -0700 (PDT) Received: from cloudburst.imgcgcw.net ([147.50.13.10]) by smtp.gmail.com with ESMTPSA id z6sm26753214pgo.31.2019.04.02.20.44.21 (version=TLS1_2 cipher=ECDHE-RSA-AES128-GCM-SHA256 bits=128/128); Tue, 02 Apr 2019 20:44:23 -0700 (PDT) From: Richard Henderson To: qemu-devel@nongnu.org Date: Wed, 3 Apr 2019 10:43:43 +0700 Message-Id: <20190403034358.21999-12-richard.henderson@linaro.org> X-Mailer: git-send-email 2.17.1 In-Reply-To: <20190403034358.21999-1-richard.henderson@linaro.org> References: <20190403034358.21999-1-richard.henderson@linaro.org> X-detected-operating-system: by eggs.gnu.org: Genre and OS details not recognized. X-Received-From: 2607:f8b0:4864:20::543 Subject: [Qemu-devel] [PATCH 11/26] target/mips: Convert to CPUClass::tlb_fill X-BeenThere: qemu-devel@nongnu.org X-Mailman-Version: 2.1.21 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Cc: Aleksandar Rikalo , Aleksandar Markovic Errors-To: qemu-devel-bounces+patch=linaro.org@nongnu.org Sender: "Qemu-devel" Note that env->active_tc.PC is removed from the qemu_log as that value is garbage. The PC isn't recovered until cpu_restore_state, called from cpu_loop_exit_restore, called from do_raise_exception_err. Cc: Aleksandar Markovic Cc: Aleksandar Rikalo Signed-off-by: Richard Henderson --- target/mips/internal.h | 5 +- target/mips/cpu.c | 5 +- target/mips/helper.c | 115 +++++++++++++++++++--------------------- target/mips/op_helper.c | 15 ------ 4 files changed, 61 insertions(+), 79 deletions(-) -- 2.17.1 Reviewed-by: Philippe Mathieu-Daudé diff --git a/target/mips/internal.h b/target/mips/internal.h index 8f6fc919d5..5ec9d0bd65 100644 --- a/target/mips/internal.h +++ b/target/mips/internal.h @@ -203,8 +203,9 @@ void cpu_mips_start_count(CPUMIPSState *env); void cpu_mips_stop_count(CPUMIPSState *env); /* helper.c */ -int mips_cpu_handle_mmu_fault(CPUState *cpu, vaddr address, int size, int rw, - int mmu_idx); +bool mips_cpu_tlb_fill(CPUState *cs, vaddr address, int size, + MMUAccessType access_type, int mmu_idx, + bool probe, uintptr_t retaddr); /* op_helper.c */ uint32_t float_class_s(uint32_t arg, float_status *fst); diff --git a/target/mips/cpu.c b/target/mips/cpu.c index e217fb3e36..ebdb834b97 100644 --- a/target/mips/cpu.c +++ b/target/mips/cpu.c @@ -197,9 +197,8 @@ static void mips_cpu_class_init(ObjectClass *c, void *data) cc->synchronize_from_tb = mips_cpu_synchronize_from_tb; cc->gdb_read_register = mips_cpu_gdb_read_register; cc->gdb_write_register = mips_cpu_gdb_write_register; -#ifdef CONFIG_USER_ONLY - cc->handle_mmu_fault = mips_cpu_handle_mmu_fault; -#else + cc->tlb_fill = mips_cpu_tlb_fill; +#ifndef CONFIG_USER_ONLY cc->do_unassigned_access = mips_cpu_unassigned_access; cc->do_unaligned_access = mips_cpu_do_unaligned_access; cc->get_phys_page_debug = mips_cpu_get_phys_page_debug; diff --git a/target/mips/helper.c b/target/mips/helper.c index c44cdca3b5..7fe0ba4754 100644 --- a/target/mips/helper.c +++ b/target/mips/helper.c @@ -874,85 +874,82 @@ refill: #endif #endif -int mips_cpu_handle_mmu_fault(CPUState *cs, vaddr address, int size, int rw, - int mmu_idx) +bool mips_cpu_tlb_fill(CPUState *cs, vaddr address, int size, + MMUAccessType access_type, int mmu_idx, + bool probe, uintptr_t retaddr) { MIPSCPU *cpu = MIPS_CPU(cs); CPUMIPSState *env = &cpu->env; -#if !defined(CONFIG_USER_ONLY) + int ret = TLBRET_NOMATCH; + +#ifndef CONFIG_USER_ONLY hwaddr physical; int prot; - int access_type; -#endif - int ret = 0; + int mips_access_type = ACCESS_INT; -#if 0 - log_cpu_state(cs, 0); -#endif qemu_log_mask(CPU_LOG_MMU, - "%s pc " TARGET_FMT_lx " ad %" VADDR_PRIx " rw %d mmu_idx %d\n", - __func__, env->active_tc.PC, address, rw, mmu_idx); + "%s ad %" VADDR_PRIx " rw %d mmu_idx %d\n", + __func__, address, access_type, mmu_idx); /* data access */ -#if !defined(CONFIG_USER_ONLY) /* XXX: put correct access by using cpu_restore_state() correctly */ - access_type = ACCESS_INT; - ret = get_physical_address(env, &physical, &prot, - address, rw, access_type, mmu_idx); - switch (ret) { - case TLBRET_MATCH: + ret = get_physical_address(env, &physical, &prot, address, + access_type, mips_access_type, mmu_idx); + if (ret == TLBRET_MATCH) { qemu_log_mask(CPU_LOG_MMU, "%s address=%" VADDR_PRIx " physical " TARGET_FMT_plx " prot %d\n", __func__, address, physical, prot); - break; - default: - qemu_log_mask(CPU_LOG_MMU, - "%s address=%" VADDR_PRIx " ret %d\n", __func__, address, - ret); - break; - } - if (ret == TLBRET_MATCH) { tlb_set_page(cs, address & TARGET_PAGE_MASK, physical & TARGET_PAGE_MASK, prot | PAGE_EXEC, mmu_idx, TARGET_PAGE_SIZE); - ret = 0; - } else if (ret < 0) -#endif - { -#if !defined(CONFIG_USER_ONLY) -#if !defined(TARGET_MIPS64) - if ((ret == TLBRET_NOMATCH) && (env->tlb->nb_tlb > 1)) { - /* - * Memory reads during hardware page table walking are performed - * as if they were kernel-mode load instructions. - */ - int mode = (env->hflags & MIPS_HFLAG_KSU); - bool ret_walker; - env->hflags &= ~MIPS_HFLAG_KSU; - ret_walker = page_table_walk_refill(env, address, rw, mmu_idx); - env->hflags |= mode; - if (ret_walker) { - ret = get_physical_address(env, &physical, &prot, - address, rw, access_type, mmu_idx); - if (ret == TLBRET_MATCH) { - tlb_set_page(cs, address & TARGET_PAGE_MASK, - physical & TARGET_PAGE_MASK, prot | PAGE_EXEC, - mmu_idx, TARGET_PAGE_SIZE); - ret = 0; - return ret; - } - } - } -#endif -#endif - raise_mmu_exception(env, address, rw, ret); - ret = 1; + return true; } - return ret; + qemu_log_mask(CPU_LOG_MMU, "%s address=%" VADDR_PRIx " ret %d\n", + __func__, address, ret); + +#ifndef TARGET_MIPS64 + if ((ret == TLBRET_NOMATCH) && (env->tlb->nb_tlb > 1)) { + /* + * Memory reads during hardware page table walking are performed + * as if they were kernel-mode load instructions. + */ + int mode = (env->hflags & MIPS_HFLAG_KSU); + bool ret_walker; + + env->hflags &= ~MIPS_HFLAG_KSU; + ret_walker = page_table_walk_refill(env, address, access_type, mmu_idx); + env->hflags |= mode; + + if (ret_walker) { + ret = get_physical_address(env, &physical, &prot, address, + access_type, mips_access_type, mmu_idx); + if (ret == TLBRET_MATCH) { + tlb_set_page(cs, address & TARGET_PAGE_MASK, + physical & TARGET_PAGE_MASK, prot | PAGE_EXEC, + mmu_idx, TARGET_PAGE_SIZE); + return true; + } + } + } +#endif + + if (probe) { + return false; + } +#endif /* !CONFIG_USER_ONLY */ + + raise_mmu_exception(env, address, access_type, ret); + do_raise_exception_err(env, cs->exception_index, env->error_code, retaddr); +} + +#ifndef CONFIG_USER_ONLY +void tlb_fill(CPUState *cs, target_ulong addr, int size, + MMUAccessType access_type, int mmu_idx, uintptr_t retaddr) +{ + mips_cpu_tlb_fill(cs, addr, size, access_type, mmu_idx, false, retaddr); } -#if !defined(CONFIG_USER_ONLY) hwaddr cpu_mips_translate_address(CPUMIPSState *env, target_ulong address, int rw) { hwaddr physical; diff --git a/target/mips/op_helper.c b/target/mips/op_helper.c index 0f272a5b93..6d86912958 100644 --- a/target/mips/op_helper.c +++ b/target/mips/op_helper.c @@ -2669,21 +2669,6 @@ void mips_cpu_do_unaligned_access(CPUState *cs, vaddr addr, do_raise_exception_err(env, excp, error_code, retaddr); } -void tlb_fill(CPUState *cs, target_ulong addr, int size, - MMUAccessType access_type, int mmu_idx, uintptr_t retaddr) -{ - int ret; - - ret = mips_cpu_handle_mmu_fault(cs, addr, size, access_type, mmu_idx); - if (ret) { - MIPSCPU *cpu = MIPS_CPU(cs); - CPUMIPSState *env = &cpu->env; - - do_raise_exception_err(env, cs->exception_index, - env->error_code, retaddr); - } -} - void mips_cpu_unassigned_access(CPUState *cs, hwaddr addr, bool is_write, bool is_exec, int unused, unsigned size)