Re: [PATCH v2 16/21] accel/tcg: actually cache our partial icount TB

qemu-devel

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [PATCH v2 16/21] accel/tcg: actually cache our partial icount TB

From:	Richard Henderson
Subject:	Re: [PATCH v2 16/21] accel/tcg: actually cache our partial icount TB
Date:	Thu, 11 Feb 2021 10:48:25 -0800
User-agent:	Mozilla/5.0 (X11; Linux x86_64; rv:68.0) Gecko/20100101 Thunderbird/68.10.0

On 2/11/21 2:21 AM, Philippe Mathieu-Daudé wrote:
>> -    insns_left = MIN(0xffff, cpu->icount_budget);
>> +    insns_left = MIN(CF_COUNT_MASK, cpu->icount_budget);
> 
> Can you describe this change a bit please?

Replacing a magic number with its proper symbol.


r~

[Prev in Thread]

Current Thread

[Next in Thread]

[PATCH v2 19/21] accel/tcg: remove CF_NOCACHE and special cases, (continued)
- [PATCH v2 19/21] accel/tcg: remove CF_NOCACHE and special cases, Alex Bennée, 2021/02/10
- [PATCH v2 17/21] accel/tcg: cache single instruction TB on pending replay exception, Alex Bennée, 2021/02/10
  - Re: [PATCH v2 17/21] accel/tcg: cache single instruction TB on pending replay exception, Richard Henderson, 2021/02/11
    - Re: [PATCH v2 17/21] accel/tcg: cache single instruction TB on pending replay exception, Alex Bennée, 2021/02/11
- [PATCH v2 15/21] tests/acceptance: add a new set of tests to exercise plugins, Alex Bennée, 2021/02/10
  - Re: [PATCH v2 15/21] tests/acceptance: add a new set of tests to exercise plugins, Philippe Mathieu-Daudé, 2021/02/11
    - Re: [PATCH v2 15/21] tests/acceptance: add a new set of tests to exercise plugins, Wainer dos Santos Moschetta, 2021/02/11
  - Re: [PATCH v2 15/21] tests/acceptance: add a new set of tests to exercise plugins, Wainer dos Santos Moschetta, 2021/02/11
- [PATCH v2 16/21] accel/tcg: actually cache our partial icount TB, Alex Bennée, 2021/02/10
  - Re: [PATCH v2 16/21] accel/tcg: actually cache our partial icount TB, Philippe Mathieu-Daudé, 2021/02/11
    - Re: [PATCH v2 16/21] accel/tcg: actually cache our partial icount TB, Richard Henderson <=
    - Re: [PATCH v2 16/21] accel/tcg: actually cache our partial icount TB, Philippe Mathieu-Daudé, 2021/02/12
    - Re: [PATCH v2 16/21] accel/tcg: actually cache our partial icount TB, Alex Bennée, 2021/02/12
  - Re: [PATCH v2 16/21] accel/tcg: actually cache our partial icount TB, Richard Henderson, 2021/02/11
- [PATCH v2 14/21] tests/plugin: expand insn test to detect duplicate instructions, Alex Bennée, 2021/02/10
- [PATCH v2 20/21] accel/tcg: allow plugin instrumentation to be disable via cflags, Alex Bennée, 2021/02/10
  - Re: [PATCH v2 20/21] accel/tcg: allow plugin instrumentation to be disable via cflags, Aaron Lindsay, 2021/02/11
    - Re: [PATCH v2 20/21] accel/tcg: allow plugin instrumentation to be disable via cflags, Alex Bennée, 2021/02/12
    - Re: [PATCH v2 20/21] accel/tcg: allow plugin instrumentation to be disable via cflags, Aaron Lindsay, 2021/02/12
    - Re: [PATCH v2 20/21] accel/tcg: allow plugin instrumentation to be disable via cflags, Alex Bennée, 2021/02/12
    - Re: [PATCH v2 20/21] accel/tcg: allow plugin instrumentation to be disable via cflags, Alex Bennée, 2021/02/12

Prev by Date: Re: runaway avocado
Next by Date: Re: [PATCH v2 16/21] accel/tcg: actually cache our partial icount TB
Previous by thread: Re: [PATCH v2 16/21] accel/tcg: actually cache our partial icount TB
Next by thread: Re: [PATCH v2 16/21] accel/tcg: actually cache our partial icount TB
Index(es):
- Date
- Thread