[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH] target-mips: Use EXCP_SC rather than a magic nu
From: |
li guang |
Subject: |
Re: [Qemu-devel] [PATCH] target-mips: Use EXCP_SC rather than a magic number |
Date: |
Thu, 20 Dec 2012 09:59:43 +0800 |
在 2012-12-20四的 09:23 +0800,陳韋任 (Wei-Ren Chen)写道:
> ping? :-)
>
> On Tue, Dec 11, 2012 at 12:15:55AM +0800, 陳韋任 (Wei-Ren Chen) wrote:
> > From the discussion on the ML [1], the exception limit defined by
> > magic number 0x100 is actually EXCP_SC defined in cpu.h. Replace the
> > magic number with EXCP_SC. Remove "#if 1 .. #endif" as well.
> >
> > [1] http://lists.gnu.org/archive/html/qemu-devel/2012-11/msg03080.html
> >
> > Signed-off-by: Chen Wei-Ren <address@hidden>
> > ---
> > target-mips/op_helper.c | 6 +++---
> > 1 files changed, 3 insertions(+), 3 deletions(-)
> >
> > diff --git a/target-mips/op_helper.c b/target-mips/op_helper.c
> > index f45d494..98a445c 100644
> > --- a/target-mips/op_helper.c
> > +++ b/target-mips/op_helper.c
> > @@ -39,10 +39,10 @@ static inline void QEMU_NORETURN
> > do_raise_exception_err(CPUMIPSState *env,
> > uintptr_t pc)
> > {
> > TranslationBlock *tb;
> > -#if 1
> > - if (exception < 0x100)
> > + if (exception < EXCP_SC) {
> > qemu_log("%s: %d %d\n", __func__, exception, error_code);
> > -#endif
> > + }
> > +
seems original '#if 1, #endif' statement
is for temporary debug only,
so maybe can be concealed out entirely.
for log purpose, every exception code
will log when 'do_interrupt' for MIPS
if log is enabled.
> > env->exception_index = exception;
> > env->error_code = error_code;
> >
> > --
> > 1.7.3.4
>
--
regards!
li guang