[PATCH v1 0/7] various tcg and linux-user updates

Alex Bennée posted 7 patches 4 years ago
Test docker-mingw@fedora passed
Test checkpatch passed
Test asan passed
Test docker-quick@centos7 passed
Test FreeBSD passed
Failed in applying to current master (apply log)
include/disas/disas.h     |   2 +-
include/exec/cpu-all.h    |  25 +-
include/exec/log.h        |   4 +-
linux-user/qemu.h         |  31 ++-
target/alpha/cpu-param.h  |  15 +-
accel/tcg/translate-all.c |  55 ++++-
bsd-user/main.c           |   4 +-
disas.c                   |  35 ++-
linux-user/elfload.c      | 503 +++++++++++++++++++-------------------
linux-user/flatload.c     |   6 +
linux-user/main.c         |  27 +-
tcg/tcg.c                 |   5 +-
accel/tcg/trace-events    |   8 +-
13 files changed, 379 insertions(+), 341 deletions(-)
[PATCH v1 0/7] various tcg and linux-user updates
Posted by Alex Bennée 4 years ago
Hi,

Cleaning up my queues into more focused trees these are all tweaks to
TCG related stuff. The guest_base changes where posted before but
where a little radical for 5.0 but I think are worth getting in early
as it enables the sanitizer builds for a range of linux-user targets
we couldn't run before. Finally there is a little tweak made to the
out_asm handling which makes it a bit easier to see which guest
instructions are being emulated by which host code.

The following need review:

 - translate-all: include guest address in out_asm output
 - disas: add optional note support to cap_disas
 - disas: include an optional note for the start of disassembly
 - accel/tcg: don't disable exec_tb trace events
 - linux-user: completely re-write init_guest_space

Alex Bennée (5):
  linux-user: completely re-write init_guest_space
  accel/tcg: don't disable exec_tb trace events
  disas: include an optional note for the start of disassembly
  disas: add optional note support to cap_disas
  translate-all: include guest address in out_asm output

Richard Henderson (2):
  exec/cpu-all: Use bool for have_guest_base
  accel/tcg: Relax va restrictions on 64-bit guests

 include/disas/disas.h     |   2 +-
 include/exec/cpu-all.h    |  25 +-
 include/exec/log.h        |   4 +-
 linux-user/qemu.h         |  31 ++-
 target/alpha/cpu-param.h  |  15 +-
 accel/tcg/translate-all.c |  55 ++++-
 bsd-user/main.c           |   4 +-
 disas.c                   |  35 ++-
 linux-user/elfload.c      | 503 +++++++++++++++++++-------------------
 linux-user/flatload.c     |   6 +
 linux-user/main.c         |  27 +-
 tcg/tcg.c                 |   5 +-
 accel/tcg/trace-events    |   8 +-
 13 files changed, 379 insertions(+), 341 deletions(-)

-- 
2.20.1


Re: [PATCH v1 0/7] various tcg and linux-user updates
Posted by Alex Bennée 3 years, 11 months ago
Alex Bennée <alex.bennee@linaro.org> writes:

> Hi,
>
> Cleaning up my queues into more focused trees these are all tweaks to
> TCG related stuff. The guest_base changes where posted before but
> where a little radical for 5.0 but I think are worth getting in early
> as it enables the sanitizer builds for a range of linux-user targets
> we couldn't run before. Finally there is a little tweak made to the
> out_asm handling which makes it a bit easier to see which guest
> instructions are being emulated by which host code.
>
> The following need review:
>
>  - translate-all: include guest address in out_asm output
>  - disas: add optional note support to cap_disas
>  - disas: include an optional note for the start of disassembly
>  - accel/tcg: don't disable exec_tb trace events
>  - linux-user: completely re-write init_guest_space

Gentle ping,

I would especially like some feed-back on the guest base updates from
the linux-user maintainers so we can get the sanitizers more widely
used.

If your happy for me to include them in my next PR I'll just take some
Acked-by's ;-)

-- 
Alex Bennée

Re: [PATCH v1 0/7] various tcg and linux-user updates
Posted by Laurent Vivier 3 years, 11 months ago
Le 11/05/2020 à 13:12, Alex Bennée a écrit :
> 
> Alex Bennée <alex.bennee@linaro.org> writes:
> 
>> Hi,
>>
>> Cleaning up my queues into more focused trees these are all tweaks to
>> TCG related stuff. The guest_base changes where posted before but
>> where a little radical for 5.0 but I think are worth getting in early
>> as it enables the sanitizer builds for a range of linux-user targets
>> we couldn't run before. Finally there is a little tweak made to the
>> out_asm handling which makes it a bit easier to see which guest
>> instructions are being emulated by which host code.
>>
>> The following need review:
>>
>>  - translate-all: include guest address in out_asm output
>>  - disas: add optional note support to cap_disas
>>  - disas: include an optional note for the start of disassembly
>>  - accel/tcg: don't disable exec_tb trace events
>>  - linux-user: completely re-write init_guest_space
> 
> Gentle ping,
> 
> I would especially like some feed-back on the guest base updates from
> the linux-user maintainers so we can get the sanitizers more widely
> used.
> 
> If your happy for me to include them in my next PR I'll just take some
> Acked-by's ;-)
> 
I don't have enough time to review the changes, but if you are confident
with your patch you can add:

Acked-by: Laurent Vivier <laurent@vivier.eu>

Thanks,
Laurent