[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH RFC 1/1] accel/tcg: Clear PAGE_WRITE before translation
From: |
Ilya Leoshkevich |
Subject: |
Re: [PATCH RFC 1/1] accel/tcg: Clear PAGE_WRITE before translation |
Date: |
Thu, 05 Aug 2021 22:36:13 +0200 |
User-agent: |
Evolution 3.38.4 (3.38.4-1.fc33) |
On Thu, 2021-08-05 at 06:59 -1000, Richard Henderson wrote:
> On 8/5/21 12:56 AM, Ilya Leoshkevich wrote:
> > On Wed, 2021-08-04 at 14:30 -1000, Richard Henderson wrote:
> > > I think that the translator_ld*_swap functions should be moved
> > > out of
> > > include/exec/translator.h into accel/tcg/translator.c.
> >
> > Do we really need this? In the end, the added code is not that
> > large.
>
> I suppose it's not required, but they're already larger than they
> need to be.
>
> In my opinion, if you're going to swap one out-of-line function call
> for two out-of-line
> function calls (cpu_ld*_code + plugin_insn_append), you've probably
> made a bad size trade-off.
>
> With your added code, it'll be 3 out-of-line calls.
>
>
> r~
Fair enough. Let me send a v3 then.
Best regards,
Ilya