[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH 1/4] exec: simplify phys_page_find() params
From: |
David Gibson |
Subject: |
Re: [Qemu-devel] [PATCH 1/4] exec: simplify phys_page_find() params |
Date: |
Tue, 16 May 2017 12:53:00 +1000 |
User-agent: |
Mutt/1.8.0 (2017-02-23) |
On Mon, May 15, 2017 at 04:50:57PM +0800, Peter Xu wrote:
> It really only plays with the dispatchers, so the parameter list does
> not need that complexity. This helps for readability at least.
>
> Signed-off-by: Peter Xu <address@hidden>
Reviewed-by: David Gibson <address@hidden>
> ---
> exec.c | 13 ++++++-------
> 1 file changed, 6 insertions(+), 7 deletions(-)
>
> diff --git a/exec.c b/exec.c
> index eac6085..1d76c63 100644
> --- a/exec.c
> +++ b/exec.c
> @@ -371,10 +371,11 @@ static inline bool section_covers_addr(const
> MemoryRegionSection *section,
> int128_getlo(section->size), addr);
> }
>
> -static MemoryRegionSection *phys_page_find(PhysPageEntry lp, hwaddr addr,
> - Node *nodes, MemoryRegionSection
> *sections)
> +static MemoryRegionSection *phys_page_find(AddressSpaceDispatch *d, hwaddr
> addr)
> {
> - PhysPageEntry *p;
> + PhysPageEntry lp = d->phys_map, *p;
> + Node *nodes = d->map.nodes;
> + MemoryRegionSection *sections = d->map.sections;
> hwaddr index = addr >> TARGET_PAGE_BITS;
> int i;
>
> @@ -412,8 +413,7 @@ static MemoryRegionSection
> *address_space_lookup_region(AddressSpaceDispatch *d,
> section_covers_addr(section, addr)) {
> update = false;
> } else {
> - section = phys_page_find(d->phys_map, addr, d->map.nodes,
> - d->map.sections);
> + section = phys_page_find(d, addr);
> update = true;
> }
> if (resolve_subpage && section->mr->subpage) {
> @@ -1246,8 +1246,7 @@ static void register_subpage(AddressSpaceDispatch *d,
> MemoryRegionSection *secti
> subpage_t *subpage;
> hwaddr base = section->offset_within_address_space
> & TARGET_PAGE_MASK;
> - MemoryRegionSection *existing = phys_page_find(d->phys_map, base,
> - d->map.nodes,
> d->map.sections);
> + MemoryRegionSection *existing = phys_page_find(d, base);
> MemoryRegionSection subsection = {
> .offset_within_address_space = base,
> .size = int128_make64(TARGET_PAGE_SIZE),
--
David Gibson | I'll have my music baroque, and my code
david AT gibson.dropbear.id.au | minimalist, thank you. NOT _the_ _other_
| _way_ _around_!
http://www.ozlabs.org/~dgibson
signature.asc
Description: PGP signature
- [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
- Re: [Qemu-devel] [PATCH 1/4] exec: simplify phys_page_find() params,
David Gibson <=
- [Qemu-devel] [PATCH 2/4] exec: rename resolve_subpage, Peter Xu, 2017/05/15
- [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