[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [qemu-s390x] [Qemu-devel] [PATCH v1 4/9] tcg: Enforce single page ac
From: |
Richard Henderson |
Subject: |
Re: [qemu-s390x] [Qemu-devel] [PATCH v1 4/9] tcg: Enforce single page access in probe_write() for !CONFIG_USER_ONLY |
Date: |
Fri, 23 Aug 2019 08:28:58 -0700 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:60.0) Gecko/20100101 Thunderbird/60.8.0 |
On 8/23/19 3:07 AM, David Hildenbrand wrote:
> While the CONFIG_USER_ONLY variant can handle multiple pages (no MMU), the
> !CONFIG_USER_ONLY variant can't and won't. We'll want to convert
> probe_write() to return a host address (similar to tlb_vaddr_to_host())
> soon. This only works on page granularity.
>
> Signed-off-by: David Hildenbrand <address@hidden>
> ---
> accel/tcg/cputlb.c | 2 ++
> 1 file changed, 2 insertions(+)
As I just mentioned in the previous, I think the two implementations should
match.
Anyway, the "multiple pages" thing above still means exactly two, since if
there were three pages involved we were only probing two of them, and the third
could still be unmapped.
r~
- [qemu-s390x] [PATCH v1 0/9] tcg: probe_write() refactorings and watchpoints, David Hildenbrand, 2019/08/23
- [qemu-s390x] [PATCH v1 1/9] s390x/tcg: Use guest_addr_valid() instead of h2g_valid() in probe_write_access(), David Hildenbrand, 2019/08/23
- [qemu-s390x] [PATCH v1 2/9] s390x/tcg: Fix length calculation in probe_write_access(), David Hildenbrand, 2019/08/23
- [qemu-s390x] [PATCH v1 3/9] tcg: Factor out CONFIG_USER_ONLY probe_write() from s390x code, David Hildenbrand, 2019/08/23
- [qemu-s390x] [PATCH v1 4/9] tcg: Enforce single page access in probe_write() for !CONFIG_USER_ONLY, David Hildenbrand, 2019/08/23
- Re: [qemu-s390x] [Qemu-devel] [PATCH v1 4/9] tcg: Enforce single page access in probe_write() for !CONFIG_USER_ONLY,
Richard Henderson <=
- [qemu-s390x] [PATCH v1 5/9] mips/tcg: Call probe_write() for CONFIG_USER_ONLY as well, David Hildenbrand, 2019/08/23
- [qemu-s390x] [PATCH v1 6/9] hppa/tcg: Call probe_write() also for CONFIG_USER_ONLY, David Hildenbrand, 2019/08/23
- [qemu-s390x] [PATCH v1 7/9] s390x/tcg: Pass a size to probe_write() in do_csst(), David Hildenbrand, 2019/08/23
- [qemu-s390x] [PATCH v1 8/9] exec.c: Factor out core logic of check_watchpoint(), David Hildenbrand, 2019/08/23