Re: [Qemu-devel] [RFC 0/3] Move notdirty handling to cputlb

[David Hildenbrand]

On 18.09.19 07:26, Richard Henderson wrote:
> RFC because this doesn't work, and I don't quite understand why.
> The only failing test is {i386,x86_64} pxe-test -- the other
> migration tests that use notdirty all pass.
> 
> Note that if you try to reproduce this on x86, you'll likely
> have to --disable-kvm, as otherwise the pxe-test will skip tcg.
> 
> Anyone who knows how this works willing to have a look?
> 
> 
> r~
> 
> 
> Richard Henderson (3):
>   exec: Adjust notdirty tracing
>   cputlb: Move NOTDIRTY handling from I/O path to TLB path
>   cputlb: Remove ATOMIC_MMU_DECLS
> 
>  accel/tcg/atomic_template.h    | 12 -----
>  include/exec/cpu-common.h      |  1 -
>  include/exec/memory-internal.h | 53 +++----------------
>  accel/tcg/cputlb.c             | 65 +++++++++++++----------
>  accel/tcg/user-exec.c          |  1 -
>  exec.c                         | 97 +++++++---------------------------
>  memory.c                       | 20 -------
>  trace-events                   |  4 +-
>  8 files changed, 63 insertions(+), 190 deletions(-)
> 

Cool, this might speed up some accesses - and will definetly cleanup
quite some code.

Can you CC me on further iterations, so I can test on s390x? Thanks!

-- 

Thanks,

David / dhildenb