[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-arm] [PATCH 09/27] Make flatview_access_valid() take a MemTxAt
From: |
Alex Bennée |
Subject: |
Re: [Qemu-arm] [PATCH 09/27] Make flatview_access_valid() take a MemTxAttrs argument |
Date: |
Tue, 22 May 2018 11:58:33 +0100 |
User-agent: |
mu4e 1.1.0; emacs 26.1 |
Peter Maydell <address@hidden> writes:
> As part of plumbing MemTxAttrs down to the IOMMU translate method,
> add MemTxAttrs as an argument to flatview_access_valid().
> Its callers now all have an attrs value to hand, so we can
> correct our earlier temporary use of MEMTXATTRS_UNSPECIFIED.
>
> Signed-off-by: Peter Maydell <address@hidden>
Reviewed-by: Alex Bennée <address@hidden>
> ---
> exec.c | 11 +++++------
> 1 file changed, 5 insertions(+), 6 deletions(-)
>
> diff --git a/exec.c b/exec.c
> index b58eb0fedd..9229fb4058 100644
> --- a/exec.c
> +++ b/exec.c
> @@ -2697,7 +2697,7 @@ static MemTxResult flatview_read(FlatView *fv, hwaddr
> addr,
> static MemTxResult flatview_write(FlatView *fv, hwaddr addr, MemTxAttrs
> attrs,
> const uint8_t *buf, int len);
> static bool flatview_access_valid(FlatView *fv, hwaddr addr, int len,
> - bool is_write);
> + bool is_write, MemTxAttrs attrs);
>
> static MemTxResult subpage_read(void *opaque, hwaddr addr, uint64_t *data,
> unsigned len, MemTxAttrs attrs)
> @@ -2773,7 +2773,7 @@ static bool subpage_accepts(void *opaque, hwaddr addr,
> #endif
>
> return flatview_access_valid(subpage->fv, addr + subpage->base,
> - len, is_write);
> + len, is_write, attrs);
> }
>
> static const MemoryRegionOps subpage_ops = {
> @@ -3461,7 +3461,7 @@ static void cpu_notify_map_clients(void)
> }
>
> static bool flatview_access_valid(FlatView *fv, hwaddr addr, int len,
> - bool is_write)
> + bool is_write, MemTxAttrs attrs)
> {
> MemoryRegion *mr;
> hwaddr l, xlat;
> @@ -3472,8 +3472,7 @@ static bool flatview_access_valid(FlatView *fv, hwaddr
> addr, int len,
> if (!memory_access_is_direct(mr, is_write)) {
> l = memory_access_size(mr, l, addr);
> /* When our callers all have attrs we'll pass them through here
> */
> - if (!memory_region_access_valid(mr, xlat, l, is_write,
> - MEMTXATTRS_UNSPECIFIED)) {
> + if (!memory_region_access_valid(mr, xlat, l, is_write, attrs)) {
> return false;
> }
> }
> @@ -3493,7 +3492,7 @@ bool address_space_access_valid(AddressSpace *as,
> hwaddr addr,
>
> rcu_read_lock();
> fv = address_space_to_flatview(as);
> - result = flatview_access_valid(fv, addr, len, is_write);
> + result = flatview_access_valid(fv, addr, len, is_write, attrs);
> rcu_read_unlock();
> return result;
> }
--
Alex Bennée
- Re: [Qemu-arm] [Qemu-devel] [PATCH 17/27] exec.c: Handle IOMMUs in address_space_translate_for_iotlb(), (continued)
[Qemu-arm] [PATCH 04/27] Make address_space_map() take a MemTxAttrs argument, Peter Maydell, 2018/05/21
[Qemu-arm] [PATCH 24/27] hw/misc/iotkit-secctl.c: Implement SECMPCINTSTATUS, Peter Maydell, 2018/05/21
[Qemu-arm] [PATCH 25/27] hw/arm/iotkit: Instantiate MPC, Peter Maydell, 2018/05/21
[Qemu-arm] [PATCH 09/27] Make flatview_access_valid() take a MemTxAttrs argument, Peter Maydell, 2018/05/21
[Qemu-arm] [PATCH 06/27] Make flatview_extend_translation() take a MemTxAttrs argument, Peter Maydell, 2018/05/21
[Qemu-arm] [PATCH 12/27] Make flatview_do_translate() take a MemTxAttrs argument, Peter Maydell, 2018/05/21
[Qemu-arm] [PATCH 10/27] Make flatview_translate() take a MemTxAttrs argument, Peter Maydell, 2018/05/21