[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH 2/4] exec: rename resolve_subpage
From: |
Paolo Bonzini |
Subject: |
Re: [Qemu-devel] [PATCH 2/4] exec: rename resolve_subpage |
Date: |
Mon, 15 May 2017 11:03:49 +0200 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:52.0) Gecko/20100101 Thunderbird/52.1.0 |
On 15/05/2017 10:50, Peter Xu wrote:
> It is not easy for people to know "what this parameter does" before
> knowing "what is subpage". Let's use "is_mmio" to make it easier to
> understand.
>
> Signed-off-by: Peter Xu <address@hidden>
Maybe invert the direction and call it for_iotlb?
Paolo
> ---
> exec.c | 8 ++++----
> 1 file changed, 4 insertions(+), 4 deletions(-)
>
> diff --git a/exec.c b/exec.c
> index 1d76c63..0adae94 100644
> --- a/exec.c
> +++ b/exec.c
> @@ -403,7 +403,7 @@ bool memory_region_is_unassigned(MemoryRegion *mr)
> /* Called from RCU critical section */
> static MemoryRegionSection *address_space_lookup_region(AddressSpaceDispatch
> *d,
> hwaddr addr,
> - bool resolve_subpage)
> + bool is_mmio)
> {
> MemoryRegionSection *section = atomic_read(&d->mru_section);
> subpage_t *subpage;
> @@ -416,7 +416,7 @@ static MemoryRegionSection
> *address_space_lookup_region(AddressSpaceDispatch *d,
> section = phys_page_find(d, addr);
> update = true;
> }
> - if (resolve_subpage && section->mr->subpage) {
> + if (is_mmio && section->mr->subpage) {
> subpage = container_of(section->mr, subpage_t, iomem);
> section = &d->map.sections[subpage->sub_section[SUBPAGE_IDX(addr)]];
> }
> @@ -429,13 +429,13 @@ static MemoryRegionSection
> *address_space_lookup_region(AddressSpaceDispatch *d,
> /* Called from RCU critical section */
> static MemoryRegionSection *
> address_space_translate_internal(AddressSpaceDispatch *d, hwaddr addr,
> hwaddr *xlat,
> - hwaddr *plen, bool resolve_subpage)
> + hwaddr *plen, bool is_mmio)
> {
> MemoryRegionSection *section;
> MemoryRegion *mr;
> Int128 diff;
>
> - section = address_space_lookup_region(d, addr, resolve_subpage);
> + section = address_space_lookup_region(d, addr, is_mmio);
> /* Compute offset within MemoryRegionSection */
> addr -= section->offset_within_address_space;
>
>
- [Qemu-devel] [PATCH 0/4] exec: address space translation cleanups, Peter Xu, 2017/05/15
- [Qemu-devel] [PATCH 1/4] exec: simplify phys_page_find() params, Peter Xu, 2017/05/15
- [Qemu-devel] [PATCH 2/4] exec: rename resolve_subpage, Peter Xu, 2017/05/15
- Re: [Qemu-devel] [PATCH 2/4] exec: rename resolve_subpage,
Paolo Bonzini <=
- [Qemu-devel] [PATCH 3/4] exec: further use is_mmio, Peter Xu, 2017/05/15
- [Qemu-devel] [PATCH 4/4] exec: abstract address_space_do_translate(), Peter Xu, 2017/05/15
- Re: [Qemu-devel] [PATCH 0/4] exec: address space translation cleanups, Maxime Coquelin, 2017/05/16