Re: [PATCH for-5.0? v2] target/xtensa: Statically allocate xtensa_insnbufs in DisasContext

[Richard Henderson <richard.henderson@linaro.org>]

On 4/7/20 12:17 PM, Max Filippov wrote:
> On Tue, Apr 7, 2020 at 12:07 PM Richard Henderson
> <richard.henderson@linaro.org> wrote:
>>
>> From: Max Filippov <jcmvbkbc@gmail.com>
>>
>> Rather than dynamically allocate, and risk failing to free
>> when we longjmp out of the translator, allocate the maximum
>> buffer size from any of the supported cpus.
>>
>> Signed-off-by: Max Filippov <jcmvbkbc@gmail.com>
>> [rth: Merged the fixup in Max's reply to his original]
>> Signed-off-by: Richard Henderson <richard.henderson@linaro.org>
>> ---
>>  target/xtensa/cpu.h       |  3 +++
>>  target/xtensa/helper.c    |  1 +
>>  target/xtensa/translate.c | 18 ++----------------
>>  3 files changed, 6 insertions(+), 16 deletions(-)
> 
> I've re-posted it almost at the same time (:
> Should I take it, or do you have another plan for it?

I'm happy for you to take it.


r~