Re: [PATCH v5 23/55] translator: add translator_ld{ub,sw,uw,l,q}

2019-10-15 Thread Alex Bennée
Alex Bennée writes: > Peter Maydell writes: > >> On Mon, 14 Oct 2019 at 12:38, Alex Bennée wrote: >>> >>> From: "Emilio G. Cota" >>> >>> We don't bother with replicating the fast path (tlb_hit) of the old >>> cpu_ldst helpers as it has no measurable effect on performance. This >>> probably i

Re: [PATCH v5 23/55] translator: add translator_ld{ub,sw,uw,l,q}

2019-10-15 Thread Alex Bennée
Peter Maydell writes: > On Mon, 14 Oct 2019 at 12:38, Alex Bennée wrote: >> >> From: "Emilio G. Cota" >> >> We don't bother with replicating the fast path (tlb_hit) of the old >> cpu_ldst helpers as it has no measurable effect on performance. This >> probably indicates we should consider flat

Re: [PATCH v5 23/55] translator: add translator_ld{ub,sw,uw,l,q}

2019-10-15 Thread Alex Bennée
Peter Maydell writes: > On Mon, 14 Oct 2019 at 12:38, Alex Bennée wrote: >> >> From: "Emilio G. Cota" >> >> We don't bother with replicating the fast path (tlb_hit) of the old >> cpu_ldst helpers as it has no measurable effect on performance. This >> probably indicates we should consider flat

Re: [PATCH v5 23/55] translator: add translator_ld{ub,sw,uw,l,q}

2019-10-14 Thread Peter Maydell
On Mon, 14 Oct 2019 at 12:38, Alex Bennée wrote: > > From: "Emilio G. Cota" > > We don't bother with replicating the fast path (tlb_hit) of the old > cpu_ldst helpers as it has no measurable effect on performance. This > probably indicates we should consider flattening the whole set of > helpers

Re: [PATCH v5 23/55] translator: add translator_ld{ub,sw,uw,l,q}

2019-10-14 Thread Richard Henderson
On 10/14/19 3:49 AM, Alex Bennée wrote: > From: "Emilio G. Cota" > > We don't bother with replicating the fast path (tlb_hit) of the old > cpu_ldst helpers as it has no measurable effect on performance. This > probably indicates we should consider flattening the whole set of > helpers but that is

[PATCH v5 23/55] translator: add translator_ld{ub,sw,uw,l,q}

2019-10-14 Thread Alex Bennée
From: "Emilio G. Cota" We don't bother with replicating the fast path (tlb_hit) of the old cpu_ldst helpers as it has no measurable effect on performance. This probably indicates we should consider flattening the whole set of helpers but that is out of scope for this change. Suggested-by: Richar