qemu-devel.nongnu.org archive mirror
 help / color / mirror / Atom feed
* [Qemu-devel] [PATCH] target-moxie: set do_interrupt to a target-specific helper function
@ 2013-03-31  1:35 Dunrong Huang
  2013-03-31 12:33 ` Anthony Green
                   ` (2 more replies)
  0 siblings, 3 replies; 6+ messages in thread
From: Dunrong Huang @ 2013-03-31  1:35 UTC (permalink / raw)
  To: qemu-devel; +Cc: Blue Swirl, Anthony Green, Andreas Färber

The value of "do_interrupt" member of CPUClass shoule be set to a
target-specific function, or it will lead to a segfault like below:

$ moxie-softmmu/qemu-system-moxie -M moxiesim
Segmentation fault

Cc: Anthony Green <green@moxielogic.com>
Cc: Blue Swirl <blauwirbel@gmail.com>
Cc: Andreas Färber <afaerber@suse.de>
Signed-off-by: Dunrong Huang <huangdr@cloud-times.com>
---
 target-moxie/cpu.c    | 1 +
 target-moxie/cpu.h    | 2 +-
 target-moxie/helper.c | 7 +++++--
 3 files changed, 7 insertions(+), 3 deletions(-)

diff --git a/target-moxie/cpu.c b/target-moxie/cpu.c
index c17d3f0..c0855f0 100644
--- a/target-moxie/cpu.c
+++ b/target-moxie/cpu.c
@@ -98,6 +98,7 @@ static void moxie_cpu_class_init(ObjectClass *oc, void *data)
     cc->class_by_name = moxie_cpu_class_by_name;
 
     dc->vmsd = &vmstate_moxie_cpu;
+    cc->do_interrupt = moxie_cpu_do_interrupt;
 }
 
 static void moxielite_initfn(Object *obj)
diff --git a/target-moxie/cpu.h b/target-moxie/cpu.h
index b96236f..988729a 100644
--- a/target-moxie/cpu.h
+++ b/target-moxie/cpu.h
@@ -117,7 +117,7 @@ static inline MoxieCPU *moxie_env_get_cpu(CPUMoxieState *env)
 
 MoxieCPU *cpu_moxie_init(const char *cpu_model);
 int cpu_moxie_exec(CPUMoxieState *s);
-void do_interrupt(CPUMoxieState *env);
+void moxie_cpu_do_interrupt(CPUState *cs);
 void moxie_translate_init(void);
 int cpu_moxie_signal_handler(int host_signum, void *pinfo,
                              void *puc);
diff --git a/target-moxie/helper.c b/target-moxie/helper.c
index 8604ce8..6e0ac2a 100644
--- a/target-moxie/helper.c
+++ b/target-moxie/helper.c
@@ -102,7 +102,7 @@ void helper_debug(CPUMoxieState *env)
 
 #if defined(CONFIG_USER_ONLY)
 
-void do_interrupt(CPUState *env)
+void moxie_cpu_do_interrupt(CPUState *env)
 {
     env->exception_index = -1;
 }
@@ -147,8 +147,11 @@ int cpu_moxie_handle_mmu_fault(CPUMoxieState *env, target_ulong address,
 }
 
 
-void do_interrupt(CPUMoxieState *env)
+void moxie_cpu_do_interrupt(CPUState *cs)
 {
+    MoxieCPU *cpu = MOXIE_CPU(cs);
+    CPUMoxieState *env = &cpu->env;
+
     switch (env->exception_index) {
     case MOXIE_EX_BREAK:
         break;
-- 
1.8.1.5

^ permalink raw reply related	[flat|nested] 6+ messages in thread

* Re: [Qemu-devel] [PATCH] target-moxie: set do_interrupt to a target-specific helper function
  2013-03-31  1:35 [Qemu-devel] [PATCH] target-moxie: set do_interrupt to a target-specific helper function Dunrong Huang
@ 2013-03-31 12:33 ` Anthony Green
  2013-03-31 13:43   ` 黄敦荣
  2013-03-31 17:01 ` Andreas Färber
  2013-03-31 18:54 ` Blue Swirl
  2 siblings, 1 reply; 6+ messages in thread
From: Anthony Green @ 2013-03-31 12:33 UTC (permalink / raw)
  To: Dunrong Huang; +Cc: Blue Swirl, qemu-devel, Andreas Färber

Hi Dunrong,

  I can't reproduce the segfault, but your patch still looks right to
me.  Thanks!

Signed-of-by: Anthony Green <green@moxielogic.com>

AG


On Sat, Mar 30, 2013 at 9:35 PM, Dunrong Huang <huangdr@cloud-times.com> wrote:
> The value of "do_interrupt" member of CPUClass shoule be set to a
> target-specific function, or it will lead to a segfault like below:
>
> $ moxie-softmmu/qemu-system-moxie -M moxiesim
> Segmentation fault
>
> Cc: Anthony Green <green@moxielogic.com>
> Cc: Blue Swirl <blauwirbel@gmail.com>
> Cc: Andreas Färber <afaerber@suse.de>
> Signed-off-by: Dunrong Huang <huangdr@cloud-times.com>
> ---
>  target-moxie/cpu.c    | 1 +
>  target-moxie/cpu.h    | 2 +-
>  target-moxie/helper.c | 7 +++++--
>  3 files changed, 7 insertions(+), 3 deletions(-)
>
> diff --git a/target-moxie/cpu.c b/target-moxie/cpu.c
> index c17d3f0..c0855f0 100644
> --- a/target-moxie/cpu.c
> +++ b/target-moxie/cpu.c
> @@ -98,6 +98,7 @@ static void moxie_cpu_class_init(ObjectClass *oc, void *data)
>      cc->class_by_name = moxie_cpu_class_by_name;
>
>      dc->vmsd = &vmstate_moxie_cpu;
> +    cc->do_interrupt = moxie_cpu_do_interrupt;
>  }
>
>  static void moxielite_initfn(Object *obj)
> diff --git a/target-moxie/cpu.h b/target-moxie/cpu.h
> index b96236f..988729a 100644
> --- a/target-moxie/cpu.h
> +++ b/target-moxie/cpu.h
> @@ -117,7 +117,7 @@ static inline MoxieCPU *moxie_env_get_cpu(CPUMoxieState *env)
>
>  MoxieCPU *cpu_moxie_init(const char *cpu_model);
>  int cpu_moxie_exec(CPUMoxieState *s);
> -void do_interrupt(CPUMoxieState *env);
> +void moxie_cpu_do_interrupt(CPUState *cs);
>  void moxie_translate_init(void);
>  int cpu_moxie_signal_handler(int host_signum, void *pinfo,
>                               void *puc);
> diff --git a/target-moxie/helper.c b/target-moxie/helper.c
> index 8604ce8..6e0ac2a 100644
> --- a/target-moxie/helper.c
> +++ b/target-moxie/helper.c
> @@ -102,7 +102,7 @@ void helper_debug(CPUMoxieState *env)
>
>  #if defined(CONFIG_USER_ONLY)
>
> -void do_interrupt(CPUState *env)
> +void moxie_cpu_do_interrupt(CPUState *env)
>  {
>      env->exception_index = -1;
>  }
> @@ -147,8 +147,11 @@ int cpu_moxie_handle_mmu_fault(CPUMoxieState *env, target_ulong address,
>  }
>
>
> -void do_interrupt(CPUMoxieState *env)
> +void moxie_cpu_do_interrupt(CPUState *cs)
>  {
> +    MoxieCPU *cpu = MOXIE_CPU(cs);
> +    CPUMoxieState *env = &cpu->env;
> +
>      switch (env->exception_index) {
>      case MOXIE_EX_BREAK:
>          break;
> --
> 1.8.1.5
>

^ permalink raw reply	[flat|nested] 6+ messages in thread

* Re: [Qemu-devel] [PATCH] target-moxie: set do_interrupt to a target-specific helper function
  2013-03-31 12:33 ` Anthony Green
@ 2013-03-31 13:43   ` 黄敦荣
  0 siblings, 0 replies; 6+ messages in thread
From: 黄敦荣 @ 2013-03-31 13:43 UTC (permalink / raw)
  To: Anthony Green; +Cc: Blue Swirl, qemu-devel, Andreas Färber


[-- Attachment #1.1: Type: text/plain, Size: 6004 bytes --]

Hi Anthony, thanks for your reply.

Below is the backtrace from core dump file, it may help.

$ moxie-softmmu/qemu-system-moxie  -M moxiesim
Segmentation fault (core dumped)
$ gdb moxie-softmmu/qemu-system-moxie core
GNU gdb (Gentoo) 7.4.1
Copyright (C) 2012 Free Software Foundation, Inc.
License GPLv3+: GNU GPL version 3 or later <http://gnu.org/licenses/gpl.html
>
This is free software: you are free to change and redistribute it.
There is NO WARRANTY, to the extent permitted by law.  Type "show copying"
and "show warranty" for details.
This GDB was configured as "x86_64-pc-linux-gnu".
For bug reporting instructions, please see:
<http://bugs.gentoo.org/>...
Reading symbols from
/home/mathslinux/Develop/QEMU/qemu/moxie-softmmu/qemu-system-moxie...done.

warning: core file may not match specified executable file.
[New LWP 1016]
[New LWP 1015]

warning: Could not load shared library symbols for linux-vdso.so.1.
Do you need "set solib-search-path" or "set sysroot"?
[Thread debugging using libthread_db enabled]
Using host libthread_db library "/lib64/libthread_db.so.1".
Core was generated by `moxie-softmmu/qemu-system-moxie -M moxiesim'.
Program terminated with signal 11, Segmentation fault.
#0  0x0000000000000000 in ?? ()
(gdb) bt
#0  0x0000000000000000 in ?? ()
#1  0x00007fc256f31691 in cpu_moxie_exec (env=0x7fc2589c5ee0) at
/home/mathslinux/Develop/QEMU/qemu/cpu-exec.c:278
#2  0x00007fc256f34833 in tcg_cpu_exec (env=0x7fc2589c5ee0) at
/home/mathslinux/Develop/QEMU/qemu/cpus.c:1117
#3  0x00007fc256f3496a in tcg_exec_all () at
/home/mathslinux/Develop/QEMU/qemu/cpus.c:1150
#4  0x00007fc256f33c60 in qemu_tcg_cpu_thread_fn (arg=0x7fc2589c5dd0) at
/home/mathslinux/Develop/QEMU/qemu/cpus.c:843
#5  0x00007fc2533f1ea7 in start_thread () from /lib64/libpthread.so.0
#6  0x00007fc24f6fa06d in clone () from /lib64/libc.so.6
(gdb) l cpu-exec.c:278
273                    cc->do_interrupt(cpu);
274 #endif
275                    ret = env->exception_index;
276                    break;
277 #else
278                    cc->do_interrupt(cpu);
279                    env->exception_index = -1;
280 #endif
281                }
282            }


On Sun, Mar 31, 2013 at 8:33 PM, Anthony Green <green@moxielogic.com> wrote:

> Hi Dunrong,
>
>   I can't reproduce the segfault, but your patch still looks right to
> me.  Thanks!
>
> Signed-of-by: Anthony Green <green@moxielogic.com>
>
> AG
>
>
> On Sat, Mar 30, 2013 at 9:35 PM, Dunrong Huang <huangdr@cloud-times.com>
> wrote:
> > The value of "do_interrupt" member of CPUClass shoule be set to a
> > target-specific function, or it will lead to a segfault like below:
> >
> > $ moxie-softmmu/qemu-system-moxie -M moxiesim
> > Segmentation fault
> >
> > Cc: Anthony Green <green@moxielogic.com>
> > Cc: Blue Swirl <blauwirbel@gmail.com>
> > Cc: Andreas Färber <afaerber@suse.de>
> > Signed-off-by: Dunrong Huang <huangdr@cloud-times.com>
> > ---
> >  target-moxie/cpu.c    | 1 +
> >  target-moxie/cpu.h    | 2 +-
> >  target-moxie/helper.c | 7 +++++--
> >  3 files changed, 7 insertions(+), 3 deletions(-)
> >
> > diff --git a/target-moxie/cpu.c b/target-moxie/cpu.c
> > index c17d3f0..c0855f0 100644
> > --- a/target-moxie/cpu.c
> > +++ b/target-moxie/cpu.c
> > @@ -98,6 +98,7 @@ static void moxie_cpu_class_init(ObjectClass *oc, void
> *data)
> >      cc->class_by_name = moxie_cpu_class_by_name;
> >
> >      dc->vmsd = &vmstate_moxie_cpu;
> > +    cc->do_interrupt = moxie_cpu_do_interrupt;
> >  }
> >
> >  static void moxielite_initfn(Object *obj)
> > diff --git a/target-moxie/cpu.h b/target-moxie/cpu.h
> > index b96236f..988729a 100644
> > --- a/target-moxie/cpu.h
> > +++ b/target-moxie/cpu.h
> > @@ -117,7 +117,7 @@ static inline MoxieCPU
> *moxie_env_get_cpu(CPUMoxieState *env)
> >
> >  MoxieCPU *cpu_moxie_init(const char *cpu_model);
> >  int cpu_moxie_exec(CPUMoxieState *s);
> > -void do_interrupt(CPUMoxieState *env);
> > +void moxie_cpu_do_interrupt(CPUState *cs);
> >  void moxie_translate_init(void);
> >  int cpu_moxie_signal_handler(int host_signum, void *pinfo,
> >                               void *puc);
> > diff --git a/target-moxie/helper.c b/target-moxie/helper.c
> > index 8604ce8..6e0ac2a 100644
> > --- a/target-moxie/helper.c
> > +++ b/target-moxie/helper.c
> > @@ -102,7 +102,7 @@ void helper_debug(CPUMoxieState *env)
> >
> >  #if defined(CONFIG_USER_ONLY)
> >
> > -void do_interrupt(CPUState *env)
> > +void moxie_cpu_do_interrupt(CPUState *env)
> >  {
> >      env->exception_index = -1;
> >  }
> > @@ -147,8 +147,11 @@ int cpu_moxie_handle_mmu_fault(CPUMoxieState *env,
> target_ulong address,
> >  }
> >
> >
> > -void do_interrupt(CPUMoxieState *env)
> > +void moxie_cpu_do_interrupt(CPUState *cs)
> >  {
> > +    MoxieCPU *cpu = MOXIE_CPU(cs);
> > +    CPUMoxieState *env = &cpu->env;
> > +
> >      switch (env->exception_index) {
> >      case MOXIE_EX_BREAK:
> >          break;
> > --
> > 1.8.1.5
> >
>
>


-- 

*
Best Regards,
------------------------------
*
*黄敦荣
Linux **Developer
**北京云端时代科技有限公司 **
  *
*电 话:+86 10 62415355-620*
*传 真:+86 10 62415355-609*
 *手 机:+86 18618450559*
 *邮 箱: huangdr@cbcos.com*
*地址: 北京市海淀区中关村南4街18号紫金数码园3号楼1层(100190)*
*
云基地: 北京亦庄经济技术开发区北工大软件园(100176)
 ------------------------------

北京云端时代科技有限公司邮件法律声明:

本邮件所有信息及附件内容版权、知识产权归我公司所有,未经事先书面许可,禁止复制、转载、传播、储存。

CLOUD TIMES EMAIL NOTICE:

The information and any attachments contained in this email is all
copyright and intellectual property of our company reserved. Without prior
written permission of any copy, reproduce, propagation or storage will be
prohibited.
*

[-- Attachment #1.2: Type: text/html, Size: 11345 bytes --]

[-- Attachment #2: "云端0301-light.png" --]
[-- Type: application/octet-stream, Size: 11877 bytes --]

^ permalink raw reply	[flat|nested] 6+ messages in thread

* Re: [Qemu-devel] [PATCH] target-moxie: set do_interrupt to a target-specific helper function
  2013-03-31  1:35 [Qemu-devel] [PATCH] target-moxie: set do_interrupt to a target-specific helper function Dunrong Huang
  2013-03-31 12:33 ` Anthony Green
@ 2013-03-31 17:01 ` Andreas Färber
  2013-03-31 23:18   ` Anthony Green
  2013-03-31 18:54 ` Blue Swirl
  2 siblings, 1 reply; 6+ messages in thread
From: Andreas Färber @ 2013-03-31 17:01 UTC (permalink / raw)
  To: Dunrong Huang, Anthony Green; +Cc: Blue Swirl, qemu-devel

Am 31.03.2013 03:35, schrieb Dunrong Huang:
> The value of "do_interrupt" member of CPUClass shoule be set to a
> target-specific function, or it will lead to a segfault like below:
> 
> $ moxie-softmmu/qemu-system-moxie -M moxiesim
> Segmentation fault
> 
> Cc: Anthony Green <green@moxielogic.com>
> Cc: Blue Swirl <blauwirbel@gmail.com>
> Cc: Andreas Färber <afaerber@suse.de>
> Signed-off-by: Dunrong Huang <huangdr@cloud-times.com>
> ---
>  target-moxie/cpu.c    | 1 +
>  target-moxie/cpu.h    | 2 +-
>  target-moxie/helper.c | 7 +++++--
>  3 files changed, 7 insertions(+), 3 deletions(-)
> 
> diff --git a/target-moxie/cpu.c b/target-moxie/cpu.c
> index c17d3f0..c0855f0 100644
> --- a/target-moxie/cpu.c
> +++ b/target-moxie/cpu.c
> @@ -98,6 +98,7 @@ static void moxie_cpu_class_init(ObjectClass *oc, void *data)
>      cc->class_by_name = moxie_cpu_class_by_name;
>  
>      dc->vmsd = &vmstate_moxie_cpu;
> +    cc->do_interrupt = moxie_cpu_do_interrupt;
>  }
>  
>  static void moxielite_initfn(Object *obj)
> diff --git a/target-moxie/cpu.h b/target-moxie/cpu.h
> index b96236f..988729a 100644
> --- a/target-moxie/cpu.h
> +++ b/target-moxie/cpu.h
> @@ -117,7 +117,7 @@ static inline MoxieCPU *moxie_env_get_cpu(CPUMoxieState *env)
>  
>  MoxieCPU *cpu_moxie_init(const char *cpu_model);
>  int cpu_moxie_exec(CPUMoxieState *s);
> -void do_interrupt(CPUMoxieState *env);
> +void moxie_cpu_do_interrupt(CPUState *cs);
>  void moxie_translate_init(void);
>  int cpu_moxie_signal_handler(int host_signum, void *pinfo,
>                               void *puc);
> diff --git a/target-moxie/helper.c b/target-moxie/helper.c
> index 8604ce8..6e0ac2a 100644
> --- a/target-moxie/helper.c
> +++ b/target-moxie/helper.c
> @@ -102,7 +102,7 @@ void helper_debug(CPUMoxieState *env)
>  
>  #if defined(CONFIG_USER_ONLY)
>  
> -void do_interrupt(CPUState *env)
> +void moxie_cpu_do_interrupt(CPUState *env)
>  {
>      env->exception_index = -1;
>  }

Anthony, CPUState should not be named "env" but rather "cs" (to reserve
"cpu" for MoxieCPU). That's unrelated to this patch though.

> @@ -147,8 +147,11 @@ int cpu_moxie_handle_mmu_fault(CPUMoxieState *env, target_ulong address,
>  }
>  
>  
> -void do_interrupt(CPUMoxieState *env)
> +void moxie_cpu_do_interrupt(CPUState *cs)
>  {
> +    MoxieCPU *cpu = MOXIE_CPU(cs);
> +    CPUMoxieState *env = &cpu->env;
> +
>      switch (env->exception_index) {
>      case MOXIE_EX_BREAK:
>          break;

That exception_index is used once from CPUMoxieState and once from
CPUState is telling me something is fishy here...

Are any test images available?

Hooking up cc->do_interrupt is the correct thing to do though, so that
could be sorted out later,

Reviewed-by: Andreas Färber <afaerber@suse.de>

Andreas

-- 
SUSE LINUX Products GmbH, Maxfeldstr. 5, 90409 Nürnberg, Germany
GF: Jeff Hawn, Jennifer Guild, Felix Imendörffer; HRB 16746 AG Nürnberg

^ permalink raw reply	[flat|nested] 6+ messages in thread

* Re: [Qemu-devel] [PATCH] target-moxie: set do_interrupt to a target-specific helper function
  2013-03-31  1:35 [Qemu-devel] [PATCH] target-moxie: set do_interrupt to a target-specific helper function Dunrong Huang
  2013-03-31 12:33 ` Anthony Green
  2013-03-31 17:01 ` Andreas Färber
@ 2013-03-31 18:54 ` Blue Swirl
  2 siblings, 0 replies; 6+ messages in thread
From: Blue Swirl @ 2013-03-31 18:54 UTC (permalink / raw)
  To: Dunrong Huang; +Cc: Anthony Green, qemu-devel, Andreas Färber

Thanks, applied.

On Sun, Mar 31, 2013 at 1:35 AM, Dunrong Huang <huangdr@cloud-times.com> wrote:
> The value of "do_interrupt" member of CPUClass shoule be set to a
> target-specific function, or it will lead to a segfault like below:
>
> $ moxie-softmmu/qemu-system-moxie -M moxiesim
> Segmentation fault
>
> Cc: Anthony Green <green@moxielogic.com>
> Cc: Blue Swirl <blauwirbel@gmail.com>
> Cc: Andreas Färber <afaerber@suse.de>
> Signed-off-by: Dunrong Huang <huangdr@cloud-times.com>
> ---
>  target-moxie/cpu.c    | 1 +
>  target-moxie/cpu.h    | 2 +-
>  target-moxie/helper.c | 7 +++++--
>  3 files changed, 7 insertions(+), 3 deletions(-)
>
> diff --git a/target-moxie/cpu.c b/target-moxie/cpu.c
> index c17d3f0..c0855f0 100644
> --- a/target-moxie/cpu.c
> +++ b/target-moxie/cpu.c
> @@ -98,6 +98,7 @@ static void moxie_cpu_class_init(ObjectClass *oc, void *data)
>      cc->class_by_name = moxie_cpu_class_by_name;
>
>      dc->vmsd = &vmstate_moxie_cpu;
> +    cc->do_interrupt = moxie_cpu_do_interrupt;
>  }
>
>  static void moxielite_initfn(Object *obj)
> diff --git a/target-moxie/cpu.h b/target-moxie/cpu.h
> index b96236f..988729a 100644
> --- a/target-moxie/cpu.h
> +++ b/target-moxie/cpu.h
> @@ -117,7 +117,7 @@ static inline MoxieCPU *moxie_env_get_cpu(CPUMoxieState *env)
>
>  MoxieCPU *cpu_moxie_init(const char *cpu_model);
>  int cpu_moxie_exec(CPUMoxieState *s);
> -void do_interrupt(CPUMoxieState *env);
> +void moxie_cpu_do_interrupt(CPUState *cs);
>  void moxie_translate_init(void);
>  int cpu_moxie_signal_handler(int host_signum, void *pinfo,
>                               void *puc);
> diff --git a/target-moxie/helper.c b/target-moxie/helper.c
> index 8604ce8..6e0ac2a 100644
> --- a/target-moxie/helper.c
> +++ b/target-moxie/helper.c
> @@ -102,7 +102,7 @@ void helper_debug(CPUMoxieState *env)
>
>  #if defined(CONFIG_USER_ONLY)
>
> -void do_interrupt(CPUState *env)
> +void moxie_cpu_do_interrupt(CPUState *env)
>  {
>      env->exception_index = -1;
>  }
> @@ -147,8 +147,11 @@ int cpu_moxie_handle_mmu_fault(CPUMoxieState *env, target_ulong address,
>  }
>
>
> -void do_interrupt(CPUMoxieState *env)
> +void moxie_cpu_do_interrupt(CPUState *cs)
>  {
> +    MoxieCPU *cpu = MOXIE_CPU(cs);
> +    CPUMoxieState *env = &cpu->env;
> +
>      switch (env->exception_index) {
>      case MOXIE_EX_BREAK:
>          break;
> --
> 1.8.1.5
>

^ permalink raw reply	[flat|nested] 6+ messages in thread

* Re: [Qemu-devel] [PATCH] target-moxie: set do_interrupt to a target-specific helper function
  2013-03-31 17:01 ` Andreas Färber
@ 2013-03-31 23:18   ` Anthony Green
  0 siblings, 0 replies; 6+ messages in thread
From: Anthony Green @ 2013-03-31 23:18 UTC (permalink / raw)
  To: Andreas Färber; +Cc: Blue Swirl, Dunrong Huang, qemu-devel

Hi Andreas,

On Sun, Mar 31, 2013 at 1:01 PM, Andreas Färber <afaerber@suse.de> wrote:
> That exception_index is used once from CPUMoxieState and once from
> CPUState is telling me something is fishy here...
>
> Are any test images available?

I have some basic RTEMS based test apps, but nothing that generates an
interrupt (just exceptions).  That's because....

>
> Hooking up cc->do_interrupt is the correct thing to do though, so that
> could be sorted out later,

...I just implemented a basic interrupt controller and timer interrupt
device in an FPGA-based SoC last week (
http://moxielogic.org/blog/?p=734 ).  Let me implement this platform
support in QEMU, and maybe these issues will sort themselves out as I
do the work.

Thanks,

AG

>
> Reviewed-by: Andreas Färber <afaerber@suse.de>
>
> Andreas
>
> --
> SUSE LINUX Products GmbH, Maxfeldstr. 5, 90409 Nürnberg, Germany
> GF: Jeff Hawn, Jennifer Guild, Felix Imendörffer; HRB 16746 AG Nürnberg

^ permalink raw reply	[flat|nested] 6+ messages in thread

end of thread, other threads:[~2013-03-31 23:18 UTC | newest]

Thread overview: 6+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2013-03-31  1:35 [Qemu-devel] [PATCH] target-moxie: set do_interrupt to a target-specific helper function Dunrong Huang
2013-03-31 12:33 ` Anthony Green
2013-03-31 13:43   ` 黄敦荣
2013-03-31 17:01 ` Andreas Färber
2013-03-31 23:18   ` Anthony Green
2013-03-31 18:54 ` Blue Swirl

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).