Re: [Qemu-devel] [RFC 35/38] cputlb: use cpu_tcg_sched_work for tlb_flush_all

2015-09-01 Thread Peter Maydell
On 1 September 2015 at 20:38, Emilio G. Cota wrote: > I can replicate it; what's happening is that tlb_flush_all calls > cpu_loop_exit(), then re-enters the cpu loop, performs the > job while other CPUs are asleep(i.e. __tlb_flush_all in this case), > but then when it continues execution it loads

Re: [Qemu-devel] [RFC 35/38] cputlb: use cpu_tcg_sched_work for tlb_flush_all

2015-09-01 Thread Emilio G. Cota
On Tue, Sep 01, 2015 at 17:10:30 +0100, Alex Bennée wrote: > > Emilio G. Cota writes: > > > Signed-off-by: Emilio G. Cota > > --- > > cputlb.c | 41 +++-- > > 1 file changed, 11 insertions(+), 30 deletions(-) > > I bisected my Jessie boot failure to this co

Re: [Qemu-devel] [RFC 35/38] cputlb: use cpu_tcg_sched_work for tlb_flush_all

2015-09-01 Thread Alex Bennée
Emilio G. Cota writes: > Signed-off-by: Emilio G. Cota > --- > cputlb.c | 41 +++-- > 1 file changed, 11 insertions(+), 30 deletions(-) I bisected my Jessie boot failure to this commit. Before it boots up fine, here it just hangs before the kernel starts in

Re: [Qemu-devel] [RFC 35/38] cputlb: use cpu_tcg_sched_work for tlb_flush_all

2015-08-25 Thread Paolo Bonzini
- Original Message - > From: "Emilio G. Cota" > To: "Paolo Bonzini" > Cc: qemu-devel@nongnu.org, mt...@greensocs.com, "guillaume delbergue" > , "alex > bennee" , "mark burton" , > "a rigo" > , "Frederic Konrad" > Sent: Wednesday, August 26, 2015 12:31:22 AM > Subject: Re: [RFC 35/38]

Re: [Qemu-devel] [RFC 35/38] cputlb: use cpu_tcg_sched_work for tlb_flush_all

2015-08-25 Thread Emilio G. Cota
On Sun, Aug 23, 2015 at 18:29:33 -0700, Paolo Bonzini wrote: > > > On 23/08/2015 17:24, Emilio G. Cota wrote: > > Signed-off-by: Emilio G. Cota > > --- > > cputlb.c | 41 +++-- > > 1 file changed, 11 insertions(+), 30 deletions(-) > > As suggested by me and

Re: [Qemu-devel] [RFC 35/38] cputlb: use cpu_tcg_sched_work for tlb_flush_all

2015-08-24 Thread Paolo Bonzini
On 23/08/2015 17:24, Emilio G. Cota wrote: > Signed-off-by: Emilio G. Cota > --- > cputlb.c | 41 +++-- > 1 file changed, 11 insertions(+), 30 deletions(-) As suggested by me and Peter, synchronization on TLB flushes should be arch-specific. CPUs can halt o

[Qemu-devel] [RFC 35/38] cputlb: use cpu_tcg_sched_work for tlb_flush_all

2015-08-24 Thread Emilio G. Cota
Signed-off-by: Emilio G. Cota --- cputlb.c | 41 +++-- 1 file changed, 11 insertions(+), 30 deletions(-) diff --git a/cputlb.c b/cputlb.c index 1b3673e..d81a4eb 100644 --- a/cputlb.c +++ b/cputlb.c @@ -73,43 +73,24 @@ void tlb_flush(CPUState *cpu, int flush_gl