[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH 04/11] microblaze: avoid "naked" qemu_log
From: |
Edgar E. Iglesias |
Subject: |
Re: [Qemu-devel] [PATCH 04/11] microblaze: avoid "naked" qemu_log |
Date: |
Thu, 17 Dec 2015 14:34:03 +0100 |
User-agent: |
Mutt/1.5.21 (2010-09-15) |
On Sat, Dec 12, 2015 at 11:36:20AM +0100, Paolo Bonzini wrote:
> Cc: Edgar E. Iglesias <address@hidden>
> Signed-off-by: Paolo Bonzini <address@hidden>
Reviewed-by: Edgar E. Iglesias <address@hidden>
Thanks,
Edgar
> ---
> target-microblaze/helper.c | 2 +-
> target-microblaze/mmu.c | 20 ++++++++++----------
> target-microblaze/op_helper.c | 8 ++++----
> target-microblaze/translate.c | 2 +-
> 4 files changed, 16 insertions(+), 16 deletions(-)
>
> diff --git a/target-microblaze/helper.c b/target-microblaze/helper.c
> index 8257b0e..a482e47 100644
> --- a/target-microblaze/helper.c
> +++ b/target-microblaze/helper.c
> @@ -128,7 +128,7 @@ void mb_cpu_do_interrupt(CPUState *cs)
> switch (cs->exception_index) {
> case EXCP_HW_EXCP:
> if (!(env->pvr.regs[0] & PVR0_USE_EXC_MASK)) {
> - qemu_log("Exception raised on system without exceptions!\n");
> + qemu_log_mask(LOG_GUEST_ERROR, "Exception raised on system
> without exceptions!\n");
> return;
> }
>
> diff --git a/target-microblaze/mmu.c b/target-microblaze/mmu.c
> index 2ef1dc2..ee95a04 100644
> --- a/target-microblaze/mmu.c
> +++ b/target-microblaze/mmu.c
> @@ -60,7 +60,7 @@ static void mmu_change_pid(CPUMBState *env, unsigned int
> newpid)
> uint32_t t;
>
> if (newpid & ~0xff)
> - qemu_log("Illegal rpid=%x\n", newpid);
> + qemu_log_mask(LOG_GUEST_ERROR, "Illegal rpid=%x\n", newpid);
>
> for (i = 0; i < ARRAY_SIZE(mmu->rams[RAM_TAG]); i++) {
> /* Lookup and decode. */
> @@ -121,7 +121,7 @@ unsigned int mmu_translate(struct microblaze_mmu *mmu,
> t0 &= 0x3;
>
> if (tlb_zsel > mmu->c_mmu_zones) {
> - qemu_log("tlb zone select out of range! %d\n", tlb_zsel);
> + qemu_log_mask(LOG_GUEST_ERROR, "tlb zone select out of
> range! %d\n", tlb_zsel);
> t0 = 1; /* Ignore. */
> }
>
> @@ -183,7 +183,7 @@ uint32_t mmu_read(CPUMBState *env, uint32_t rn)
> uint32_t r;
>
> if (env->mmu.c_mmu < 2 || !env->mmu.c_mmu_tlb_access) {
> - qemu_log("MMU access on MMU-less system\n");
> + qemu_log_mask(LOG_GUEST_ERROR, "MMU access on MMU-less system\n");
> return 0;
> }
>
> @@ -192,7 +192,7 @@ uint32_t mmu_read(CPUMBState *env, uint32_t rn)
> case MMU_R_TLBLO:
> case MMU_R_TLBHI:
> if (!(env->mmu.c_mmu_tlb_access & 1)) {
> - qemu_log("Invalid access to MMU reg %d\n", rn);
> + qemu_log_mask(LOG_GUEST_ERROR, "Invalid access to MMU reg
> %d\n", rn);
> return 0;
> }
>
> @@ -204,7 +204,7 @@ uint32_t mmu_read(CPUMBState *env, uint32_t rn)
> case MMU_R_PID:
> case MMU_R_ZPR:
> if (!(env->mmu.c_mmu_tlb_access & 1)) {
> - qemu_log("Invalid access to MMU reg %d\n", rn);
> + qemu_log_mask(LOG_GUEST_ERROR, "Invalid access to MMU reg
> %d\n", rn);
> return 0;
> }
> r = env->mmu.regs[rn];
> @@ -224,7 +224,7 @@ void mmu_write(CPUMBState *env, uint32_t rn, uint32_t v)
> D(qemu_log("%s rn=%d=%x old=%x\n", __func__, rn, v, env->mmu.regs[rn]));
>
> if (env->mmu.c_mmu < 2 || !env->mmu.c_mmu_tlb_access) {
> - qemu_log("MMU access on MMU-less system\n");
> + qemu_log_mask(LOG_GUEST_ERROR, "MMU access on MMU-less system\n");
> return;
> }
>
> @@ -235,7 +235,7 @@ void mmu_write(CPUMBState *env, uint32_t rn, uint32_t v)
> i = env->mmu.regs[MMU_R_TLBX] & 0xff;
> if (rn == MMU_R_TLBHI) {
> if (i < 3 && !(v & TLB_VALID) && qemu_loglevel_mask(~0))
> - qemu_log("invalidating index %x at pc=%x\n",
> + qemu_log_mask(LOG_GUEST_ERROR, "invalidating index %x at
> pc=%x\n",
> i, env->sregs[SR_PC]);
> env->mmu.tids[i] = env->mmu.regs[MMU_R_PID] & 0xff;
> mmu_flush_idx(env, i);
> @@ -246,7 +246,7 @@ void mmu_write(CPUMBState *env, uint32_t rn, uint32_t v)
> break;
> case MMU_R_ZPR:
> if (env->mmu.c_mmu_tlb_access <= 1) {
> - qemu_log("Invalid access to MMU reg %d\n", rn);
> + qemu_log_mask(LOG_GUEST_ERROR, "Invalid access to MMU reg
> %d\n", rn);
> return;
> }
>
> @@ -259,7 +259,7 @@ void mmu_write(CPUMBState *env, uint32_t rn, uint32_t v)
> break;
> case MMU_R_PID:
> if (env->mmu.c_mmu_tlb_access <= 1) {
> - qemu_log("Invalid access to MMU reg %d\n", rn);
> + qemu_log_mask(LOG_GUEST_ERROR, "Invalid access to MMU reg
> %d\n", rn);
> return;
> }
>
> @@ -274,7 +274,7 @@ void mmu_write(CPUMBState *env, uint32_t rn, uint32_t v)
> int hit;
>
> if (env->mmu.c_mmu_tlb_access <= 1) {
> - qemu_log("Invalid access to MMU reg %d\n", rn);
> + qemu_log_mask(LOG_GUEST_ERROR, "Invalid access to MMU reg
> %d\n", rn);
> return;
> }
>
> diff --git a/target-microblaze/op_helper.c b/target-microblaze/op_helper.c
> index d324347..5637462 100644
> --- a/target-microblaze/op_helper.c
> +++ b/target-microblaze/op_helper.c
> @@ -55,7 +55,7 @@ void helper_put(uint32_t id, uint32_t ctrl, uint32_t data)
> int nonblock = ctrl & STREAM_NONBLOCK;
> int exception = ctrl & STREAM_EXCEPTION;
>
> - qemu_log("Unhandled stream put to stream-id=%d data=%x %s%s%s%s%s\n",
> + qemu_log_mask(LOG_UNIMP, "Unhandled stream put to stream-id=%d data=%x
> %s%s%s%s%s\n",
> id, data,
> test ? "t" : "",
> nonblock ? "n" : "",
> @@ -72,7 +72,7 @@ uint32_t helper_get(uint32_t id, uint32_t ctrl)
> int nonblock = ctrl & STREAM_NONBLOCK;
> int exception = ctrl & STREAM_EXCEPTION;
>
> - qemu_log("Unhandled stream get from stream-id=%d %s%s%s%s%s\n",
> + qemu_log_mask(LOG_UNIMP, "Unhandled stream get from stream-id=%d
> %s%s%s%s%s\n",
> id,
> test ? "t" : "",
> nonblock ? "n" : "",
> @@ -465,8 +465,8 @@ void helper_memalign(CPUMBState *env, uint32_t addr,
> uint32_t dr, uint32_t wr,
> void helper_stackprot(CPUMBState *env, uint32_t addr)
> {
> if (addr < env->slr || addr > env->shr) {
> - qemu_log("Stack protector violation at %x %x %x\n",
> - addr, env->slr, env->shr);
> + qemu_log_mask(CPU_LOG_INT, "Stack protector violation at %x %x %x\n",
> + addr, env->slr, env->shr);
> env->sregs[SR_EAR] = addr;
> env->sregs[SR_ESR] = ESR_EC_STACKPROT;
> helper_raise_exception(env, EXCP_HW_EXCP);
> diff --git a/target-microblaze/translate.c b/target-microblaze/translate.c
> index 154b9d6..9e52011 100644
> --- a/target-microblaze/translate.c
> +++ b/target-microblaze/translate.c
> @@ -1516,7 +1516,7 @@ static void dec_null(DisasContext *dc)
> t_gen_raise_exception(dc, EXCP_HW_EXCP);
> return;
> }
> - qemu_log ("unknown insn pc=%x opc=%x\n", dc->pc, dc->opcode);
> + qemu_log_mask(LOG_GUEST_ERROR, "unknown insn pc=%x opc=%x\n", dc->pc,
> dc->opcode);
> dc->abort_at_next_insn = 1;
> }
>
> --
> 2.5.0
>
>
>
- [Qemu-devel] [PATCH 00/11] Avoid always-active qemu_log calls, Paolo Bonzini, 2015/12/12
- [Qemu-devel] [PATCH 02/11] alpha: convert "naked" qemu_log to tracepoint, Paolo Bonzini, 2015/12/12
- [Qemu-devel] [PATCH 01/11] qemu-log: introduce qemu_log_separate, Paolo Bonzini, 2015/12/12
- [Qemu-devel] [PATCH 03/11] cris: avoid "naked" qemu_log, Paolo Bonzini, 2015/12/12
- [Qemu-devel] [PATCH 05/11] s390x: avoid "naked" qemu_log, Paolo Bonzini, 2015/12/12
- [Qemu-devel] [PATCH 04/11] microblaze: avoid "naked" qemu_log, Paolo Bonzini, 2015/12/12
- Re: [Qemu-devel] [PATCH 04/11] microblaze: avoid "naked" qemu_log,
Edgar E. Iglesias <=
- [Qemu-devel] [PATCH 06/11] ppc: cleanup logging, Paolo Bonzini, 2015/12/12
- [Qemu-devel] [PATCH 07/11] tricore: avoid "naked" qemu_log, Paolo Bonzini, 2015/12/12
- [Qemu-devel] [PATCH 09/11] user: introduce "-d page", Paolo Bonzini, 2015/12/12
- [Qemu-devel] [PATCH 10/11] linux-user: avoid "naked" qemu_log, Paolo Bonzini, 2015/12/12
- [Qemu-devel] [PATCH 08/11] xtensa: avoid "naked" qemu_log, Paolo Bonzini, 2015/12/12
- [Qemu-devel] [PATCH 11/11] linux-user: convert DEBUG_SIGNAL logging to tracepoints, Paolo Bonzini, 2015/12/12