* [Qemu-devel] [PATCH] riscv: remove define cpu_init()
@ 2018-05-15 11:16 Igor Mammedov
2018-05-15 14:39 ` Philippe Mathieu-Daudé
` (2 more replies)
0 siblings, 3 replies; 8+ messages in thread
From: Igor Mammedov @ 2018-05-15 11:16 UTC (permalink / raw
To: qemu-devel; +Cc: mjc, palmer
cpu_init() was removed since 2.12, so drop define that now isn't used
nad point to nowhere.
Signed-off-by: Igor Mammedov <imammedo@redhat.com>
---
target/riscv/cpu.h | 1 -
1 file changed, 1 deletion(-)
diff --git a/target/riscv/cpu.h b/target/riscv/cpu.h
index 34abc38..0562a02 100644
--- a/target/riscv/cpu.h
+++ b/target/riscv/cpu.h
@@ -251,7 +251,6 @@ int riscv_cpu_handle_mmu_fault(CPUState *cpu, vaddr address, int size,
char *riscv_isa_string(RISCVCPU *cpu);
void riscv_cpu_list(FILE *f, fprintf_function cpu_fprintf);
-#define cpu_init(cpu_model) cpu_generic_init(TYPE_RISCV_CPU, cpu_model)
#define cpu_signal_handler cpu_riscv_signal_handler
#define cpu_list riscv_cpu_list
#define cpu_mmu_index riscv_cpu_mmu_index
--
2.7.4
^ permalink raw reply related [flat|nested] 8+ messages in thread
* Re: [Qemu-devel] [PATCH] riscv: remove define cpu_init()
2018-05-15 11:16 [Qemu-devel] [PATCH] riscv: remove define cpu_init() Igor Mammedov
@ 2018-05-15 14:39 ` Philippe Mathieu-Daudé
2018-05-15 15:44 ` Eric Blake
2018-05-15 16:00 ` [Qemu-devel] [PATCH v2] " Igor Mammedov
2 siblings, 0 replies; 8+ messages in thread
From: Philippe Mathieu-Daudé @ 2018-05-15 14:39 UTC (permalink / raw
To: Igor Mammedov, qemu-devel; +Cc: mjc, palmer
On 05/15/2018 08:16 AM, Igor Mammedov wrote:
> cpu_init() was removed since 2.12, so drop define that now isn't used
> nad point to nowhere.
>
> Signed-off-by: Igor Mammedov <imammedo@redhat.com>
Reviewed-by: Philippe Mathieu-Daudé <f4bug@amsat.org>
> ---
> target/riscv/cpu.h | 1 -
> 1 file changed, 1 deletion(-)
>
> diff --git a/target/riscv/cpu.h b/target/riscv/cpu.h
> index 34abc38..0562a02 100644
> --- a/target/riscv/cpu.h
> +++ b/target/riscv/cpu.h
> @@ -251,7 +251,6 @@ int riscv_cpu_handle_mmu_fault(CPUState *cpu, vaddr address, int size,
> char *riscv_isa_string(RISCVCPU *cpu);
> void riscv_cpu_list(FILE *f, fprintf_function cpu_fprintf);
>
> -#define cpu_init(cpu_model) cpu_generic_init(TYPE_RISCV_CPU, cpu_model)
> #define cpu_signal_handler cpu_riscv_signal_handler
> #define cpu_list riscv_cpu_list
> #define cpu_mmu_index riscv_cpu_mmu_index
>
^ permalink raw reply [flat|nested] 8+ messages in thread
* Re: [Qemu-devel] [PATCH] riscv: remove define cpu_init()
2018-05-15 11:16 [Qemu-devel] [PATCH] riscv: remove define cpu_init() Igor Mammedov
2018-05-15 14:39 ` Philippe Mathieu-Daudé
@ 2018-05-15 15:44 ` Eric Blake
2018-05-15 16:00 ` [Qemu-devel] [PATCH v2] " Igor Mammedov
2 siblings, 0 replies; 8+ messages in thread
From: Eric Blake @ 2018-05-15 15:44 UTC (permalink / raw
To: Igor Mammedov, qemu-devel; +Cc: mjc, palmer
On 05/15/2018 06:16 AM, Igor Mammedov wrote:
> cpu_init() was removed since 2.12, so drop define that now isn't used
> nad point to nowhere.
Reads better as:
so drop the define that is now unused
or maybe
so drop the unused define rather than letting it point to nowhere
--
Eric Blake, Principal Software Engineer
Red Hat, Inc. +1-919-301-3266
Virtualization: qemu.org | libvirt.org
^ permalink raw reply [flat|nested] 8+ messages in thread
* [Qemu-devel] [PATCH v2] riscv: remove define cpu_init()
2018-05-15 11:16 [Qemu-devel] [PATCH] riscv: remove define cpu_init() Igor Mammedov
2018-05-15 14:39 ` Philippe Mathieu-Daudé
2018-05-15 15:44 ` Eric Blake
@ 2018-05-15 16:00 ` Igor Mammedov
2018-05-18 2:10 ` Michael Clark
2 siblings, 1 reply; 8+ messages in thread
From: Igor Mammedov @ 2018-05-15 16:00 UTC (permalink / raw
To: qemu-devel; +Cc: eblake, mjc, palmer, f4bug
cpu_init() was removed since 2.12, so drop the define that is now unused.
Signed-off-by: Igor Mammedov <imammedo@redhat.com>
Reviewed-by: Philippe Mathieu-Daudé <f4bug@amsat.org>
---
v2:
* refine commit message (Eric Blake <eblake@redhat.com>)
---
target/riscv/cpu.h | 1 -
1 file changed, 1 deletion(-)
diff --git a/target/riscv/cpu.h b/target/riscv/cpu.h
index 34abc38..0562a02 100644
--- a/target/riscv/cpu.h
+++ b/target/riscv/cpu.h
@@ -251,7 +251,6 @@ int riscv_cpu_handle_mmu_fault(CPUState *cpu, vaddr address, int size,
char *riscv_isa_string(RISCVCPU *cpu);
void riscv_cpu_list(FILE *f, fprintf_function cpu_fprintf);
-#define cpu_init(cpu_model) cpu_generic_init(TYPE_RISCV_CPU, cpu_model)
#define cpu_signal_handler cpu_riscv_signal_handler
#define cpu_list riscv_cpu_list
#define cpu_mmu_index riscv_cpu_mmu_index
--
2.7.4
^ permalink raw reply related [flat|nested] 8+ messages in thread
* Re: [Qemu-devel] [PATCH v2] riscv: remove define cpu_init()
2018-05-15 16:00 ` [Qemu-devel] [PATCH v2] " Igor Mammedov
@ 2018-05-18 2:10 ` Michael Clark
2018-05-18 8:02 ` Igor Mammedov
0 siblings, 1 reply; 8+ messages in thread
From: Michael Clark @ 2018-05-18 2:10 UTC (permalink / raw
To: Igor Mammedov
Cc: QEMU Developers, Eric Blake, Palmer Dabbelt,
Philippe Mathieu-Daudé
On Wed, May 16, 2018 at 4:00 AM, Igor Mammedov <imammedo@redhat.com> wrote:
> cpu_init() was removed since 2.12, so drop the define that is now unused.
>
> Signed-off-by: Igor Mammedov <imammedo@redhat.com>
> Reviewed-by: Philippe Mathieu-Daudé <f4bug@amsat.org>
>
Reviewed-by: Michael Clark <mjc@sifive.com>
---
> v2:
> * refine commit message (Eric Blake <eblake@redhat.com>)
> ---
> target/riscv/cpu.h | 1 -
> 1 file changed, 1 deletion(-)
>
> diff --git a/target/riscv/cpu.h b/target/riscv/cpu.h
> index 34abc38..0562a02 100644
> --- a/target/riscv/cpu.h
> +++ b/target/riscv/cpu.h
> @@ -251,7 +251,6 @@ int riscv_cpu_handle_mmu_fault(CPUState *cpu, vaddr
> address, int size,
> char *riscv_isa_string(RISCVCPU *cpu);
> void riscv_cpu_list(FILE *f, fprintf_function cpu_fprintf);
>
> -#define cpu_init(cpu_model) cpu_generic_init(TYPE_RISCV_CPU, cpu_model)
> #define cpu_signal_handler cpu_riscv_signal_handler
> #define cpu_list riscv_cpu_list
> #define cpu_mmu_index riscv_cpu_mmu_index
> --
> 2.7.4
>
>
^ permalink raw reply [flat|nested] 8+ messages in thread
* Re: [Qemu-devel] [PATCH v2] riscv: remove define cpu_init()
2018-05-18 2:10 ` Michael Clark
@ 2018-05-18 8:02 ` Igor Mammedov
2018-07-25 22:27 ` Michael Clark
0 siblings, 1 reply; 8+ messages in thread
From: Igor Mammedov @ 2018-05-18 8:02 UTC (permalink / raw
To: Michael Clark
Cc: QEMU Developers, Eric Blake, Palmer Dabbelt,
Philippe Mathieu-Daudé
On Fri, 18 May 2018 14:10:24 +1200
Michael Clark <mjc@sifive.com> wrote:
> On Wed, May 16, 2018 at 4:00 AM, Igor Mammedov <imammedo@redhat.com> wrote:
>
> > cpu_init() was removed since 2.12, so drop the define that is now unused.
> >
> > Signed-off-by: Igor Mammedov <imammedo@redhat.com>
> > Reviewed-by: Philippe Mathieu-Daudé <f4bug@amsat.org>
> >
>
> Reviewed-by: Michael Clark <mjc@sifive.com>
Michael,
could you merge it through RISCV queue?
>
> ---
> > v2:
> > * refine commit message (Eric Blake <eblake@redhat.com>)
> > ---
> > target/riscv/cpu.h | 1 -
> > 1 file changed, 1 deletion(-)
> >
> > diff --git a/target/riscv/cpu.h b/target/riscv/cpu.h
> > index 34abc38..0562a02 100644
> > --- a/target/riscv/cpu.h
> > +++ b/target/riscv/cpu.h
> > @@ -251,7 +251,6 @@ int riscv_cpu_handle_mmu_fault(CPUState *cpu, vaddr
> > address, int size,
> > char *riscv_isa_string(RISCVCPU *cpu);
> > void riscv_cpu_list(FILE *f, fprintf_function cpu_fprintf);
> >
> > -#define cpu_init(cpu_model) cpu_generic_init(TYPE_RISCV_CPU, cpu_model)
> > #define cpu_signal_handler cpu_riscv_signal_handler
> > #define cpu_list riscv_cpu_list
> > #define cpu_mmu_index riscv_cpu_mmu_index
> > --
> > 2.7.4
> >
> >
^ permalink raw reply [flat|nested] 8+ messages in thread
* Re: [Qemu-devel] [PATCH v2] riscv: remove define cpu_init()
2018-05-18 8:02 ` Igor Mammedov
@ 2018-07-25 22:27 ` Michael Clark
2018-07-26 8:46 ` Igor Mammedov
0 siblings, 1 reply; 8+ messages in thread
From: Michael Clark @ 2018-07-25 22:27 UTC (permalink / raw
To: Igor Mammedov
Cc: QEMU Developers, Eric Blake, Palmer Dabbelt,
Philippe Mathieu-Daudé
On Fri, May 18, 2018 at 8:02 PM, Igor Mammedov <imammedo@redhat.com> wrote:
> On Fri, 18 May 2018 14:10:24 +1200
> Michael Clark <mjc@sifive.com> wrote:
>
> > On Wed, May 16, 2018 at 4:00 AM, Igor Mammedov <imammedo@redhat.com>
> wrote:
> >
> > > cpu_init() was removed since 2.12, so drop the define that is now
> unused.
> > >
> > > Signed-off-by: Igor Mammedov <imammedo@redhat.com>
> > > Reviewed-by: Philippe Mathieu-Daudé <f4bug@amsat.org>
> > >
> >
> > Reviewed-by: Michael Clark <mjc@sifive.com>
> Michael,
>
> could you merge it through RISCV queue?
Yes indeed. Sorry I missed this email.
I'm going to bring this into the riscv-qemu queue right away and move it to
the head of the queue, now we have Alastair's stuff in master. Although I
won't be making any PRs until after 3.0 is released.
I'm currently re-basing and testing the RISC-V tree against Alastair's
changes. There were quite a few conflicts so it is going to take a bit of
testing. I also need to spend some time re-testing master.
I've been AWOL for the last month and a half due to some work related to
SiFive's new E20 and E21 cores which were recently announced. In the QEMU
3.1 or 3.2 time-frame we should have the new CLIC interrupt controller
which supports multiple levels of interrupt pre-emption, interrupt tail
chaining and new vectored interrupt modes. In the proposed CLIC spec (Core
Level Interrupt Controller), pre-emption is mostly handled in software but
there is a tiny bit of hardware support. We don't have any magic hardware
state machines that save multiple registers at a time. The proposed CLIC
sticks pretty closely to a set of RISC-V ideals with respect to how much is
done in a micro-op in the ISA and avoidance of complex hardware state
machines (e.g. no wiring of the ABI caller save registers into hardware).
I'm currently working on test cases for the CLIC and hopefully SiFive will
have tests for other changes we have in the RISC-V tree, so we have
"proofs" accompanying each patch that makes any sufficiantly complex
change. It of course will take a bit of time for us to accompany each of
our changes with a test. SiFive do actually have resources devoted to
formal verification but I have not yet been able to convince them to devote
resources to formally verifying the RISC-V QEMU implementation. Anyway we
do at least hope to have some tests for the RISC-V QEMU target in the near
future...
Phillipe, I noticed the recent cross complier infrastructure support so we
could potentially bring in riscv-tests as a submodule in QEMU... this will
help...
Michael.
> > ---
> > > v2:
> > > * refine commit message (Eric Blake <eblake@redhat.com>)
> > > ---
> > > target/riscv/cpu.h | 1 -
> > > 1 file changed, 1 deletion(-)
> > >
> > > diff --git a/target/riscv/cpu.h b/target/riscv/cpu.h
> > > index 34abc38..0562a02 100644
> > > --- a/target/riscv/cpu.h
> > > +++ b/target/riscv/cpu.h
> > > @@ -251,7 +251,6 @@ int riscv_cpu_handle_mmu_fault(CPUState *cpu,
> vaddr
> > > address, int size,
> > > char *riscv_isa_string(RISCVCPU *cpu);
> > > void riscv_cpu_list(FILE *f, fprintf_function cpu_fprintf);
> > >
> > > -#define cpu_init(cpu_model) cpu_generic_init(TYPE_RISCV_CPU,
> cpu_model)
> > > #define cpu_signal_handler cpu_riscv_signal_handler
> > > #define cpu_list riscv_cpu_list
> > > #define cpu_mmu_index riscv_cpu_mmu_index
> > > --
> > > 2.7.4
> > >
> > >
>
>
^ permalink raw reply [flat|nested] 8+ messages in thread
* Re: [Qemu-devel] [PATCH v2] riscv: remove define cpu_init()
2018-07-25 22:27 ` Michael Clark
@ 2018-07-26 8:46 ` Igor Mammedov
0 siblings, 0 replies; 8+ messages in thread
From: Igor Mammedov @ 2018-07-26 8:46 UTC (permalink / raw
To: Michael Clark
Cc: QEMU Developers, Eric Blake, Palmer Dabbelt,
Philippe Mathieu-Daudé
On Thu, 26 Jul 2018 10:27:38 +1200
Michael Clark <mjc@sifive.com> wrote:
> On Fri, May 18, 2018 at 8:02 PM, Igor Mammedov <imammedo@redhat.com> wrote:
>
> > On Fri, 18 May 2018 14:10:24 +1200
> > Michael Clark <mjc@sifive.com> wrote:
> >
> > > On Wed, May 16, 2018 at 4:00 AM, Igor Mammedov <imammedo@redhat.com>
> > wrote:
> > >
> > > > cpu_init() was removed since 2.12, so drop the define that is now
> > unused.
> > > >
> > > > Signed-off-by: Igor Mammedov <imammedo@redhat.com>
> > > > Reviewed-by: Philippe Mathieu-Daudé <f4bug@amsat.org>
> > > >
> > >
> > > Reviewed-by: Michael Clark <mjc@sifive.com>
> > Michael,
> >
> > could you merge it through RISCV queue?
>
>
> Yes indeed. Sorry I missed this email.
>
> I'm going to bring this into the riscv-qemu queue right away and move it to
> the head of the queue, now we have Alastair's stuff in master. Although I
> won't be making any PRs until after 3.0 is released.
Thanks, it's cleanup so it's ok if it goes to 3.1
[...]
^ permalink raw reply [flat|nested] 8+ messages in thread
end of thread, other threads:[~2018-07-26 8:46 UTC | newest]
Thread overview: 8+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2018-05-15 11:16 [Qemu-devel] [PATCH] riscv: remove define cpu_init() Igor Mammedov
2018-05-15 14:39 ` Philippe Mathieu-Daudé
2018-05-15 15:44 ` Eric Blake
2018-05-15 16:00 ` [Qemu-devel] [PATCH v2] " Igor Mammedov
2018-05-18 2:10 ` Michael Clark
2018-05-18 8:02 ` Igor Mammedov
2018-07-25 22:27 ` Michael Clark
2018-07-26 8:46 ` Igor Mammedov
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.