[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH v4 4/6] vhost: add regions to temporary list
From: |
Dr. David Alan Gilbert |
Subject: |
Re: [Qemu-devel] [PATCH v4 4/6] vhost: add regions to temporary list |
Date: |
Thu, 14 Dec 2017 15:53:15 +0000 |
User-agent: |
Mutt/1.9.1 (2017-09-22) |
* Paolo Bonzini (address@hidden) wrote:
> On 13/12/2017 19:08, Dr. David Alan Gilbert (git) wrote:
> > + if (dev->tmp_mem->nregions) {
> > + /* Since we already have at least one region, lets see if
> > + * this extends it; since we're scanning in order, we only
> > + * have to look at the last one, and the FlatView that calls
> > + * us shouldn't have overlaps.
> > + */
> > + struct vhost_memory_region *prev_vmr = dev->tmp_mem->regions +
> > + (dev->tmp_mem->nregions -
> > 1);
> > + uint64_t prev_gpa_start = prev_vmr->guest_phys_addr;
> > + uint64_t prev_gpa_end = range_get_last(prev_gpa_start,
> > + prev_vmr->memory_size);
> > + uint64_t prev_host_start = prev_vmr->userspace_addr;
> > + uint64_t prev_host_end = range_get_last(prev_host_start,
> > + prev_vmr->memory_size);
> > +
> > + if (prev_gpa_end + 1 == mrs_gpa &&
> > + prev_host_end + 1 == mrs_host &&
> > + (!dev->vhost_ops->vhost_backend_can_merge ||
> > + dev->vhost_ops->vhost_backend_can_merge(dev,
> > + mrs_host, mrs_size,
> > + prev_host_start, prev_vmr->memory_size))) {
> > + /* The two regions abut */
> > + need_add = false;
> > + mrs_size = mrs_size + prev_vmr->memory_size;
> > + prev_vmr->memory_size = mrs_size;
> > + trace_vhost_region_add_tmp_abut(section->mr->name, mrs_size);
> > + }
> > + }
>
> Interesting, in which cases does this actually trigger?
:vhost_section vga-lowmem:0
:vhost_section /objects/mem:1
:vhost_region_add_tmp /objects/mem: 0x0+0xa0000 @ 0x7ff1f5a08000
:vhost_section vga.vram:0
:vhost_section vga-lowmem:0
:vhost_section /objects/mem:1
:vhost_region_add_tmp /objects/mem: 0xc0000+0xa000 @ 0x7ff1f5ac8000
:vhost_section /objects/mem:1
:vhost_region_add_tmp /objects/mem: 0xca000+0x3000 @ 0x7ff1f5ad2000
>vhost_region_add_tmp_abut /objects/mem: 0xd000
:vhost_section /objects/mem:1
:vhost_region_add_tmp /objects/mem: 0xcd000+0x1f000 @ 0x7ff1f5ad5000
>vhost_region_add_tmp_abut /objects/mem: 0x2c000
:vhost_section /objects/mem:1
:vhost_region_add_tmp /objects/mem: 0xec000+0x4000 @ 0x7ff1f5af4000
>vhost_region_add_tmp_abut /objects/mem: 0x30000
:vhost_section /objects/mem:1
:vhost_region_add_tmp /objects/mem: 0xf0000+0x10000 @ 0x7ff1f5af8000
>vhost_region_add_tmp_abut /objects/mem: 0x40000
:vhost_section /objects/mem:1
:vhost_region_add_tmp /objects/mem: 0x100000+0x3ff00000 @ 0x7ff1f5b08000
>vhost_region_add_tmp_abut /objects/mem: 0x3ff40000
:vhost_section vga.vram:0
:vhost_section vga ioports remapped:0
:vhost_section bochs dispi interface:0
:vhost_section qemu extended regs:0
:vhost_section msix-table:0
:vhost_section msix-pba:0
:vhost_section kvm-ioapic:0
:vhost_section hpet:0
:vhost_section kvm-apic-msi:0
:vhost_section pc.bios:0
:vhost_commit Started: 1 Changed: 0
So it's not unusual for us to piece them back together into one
chunk.
Dave
> Paolo
--
Dr. David Alan Gilbert / address@hidden / Manchester, UK
- Re: [Qemu-devel] [PATCH v4 1/6] vhost: Move log_dirty check, (continued)
[Qemu-devel] [PATCH v4 2/6] vhost: Simplify ring verification checks, Dr. David Alan Gilbert (git), 2017/12/13
[Qemu-devel] [PATCH v4 3/6] vhost: Add temporary memory structure, Dr. David Alan Gilbert (git), 2017/12/13
[Qemu-devel] [PATCH v4 4/6] vhost: add regions to temporary list, Dr. David Alan Gilbert (git), 2017/12/13
- Re: [Qemu-devel] [PATCH v4 4/6] vhost: add regions to temporary list, Paolo Bonzini, 2017/12/13
- Re: [Qemu-devel] [PATCH v4 4/6] vhost: add regions to temporary list,
Dr. David Alan Gilbert <=
- Re: [Qemu-devel] [PATCH v4 4/6] vhost: add regions to temporary list, Igor Mammedov, 2017/12/14
- Re: [Qemu-devel] [PATCH v4 4/6] vhost: add regions to temporary list, Dr. David Alan Gilbert, 2017/12/15
- Re: [Qemu-devel] [PATCH v4 4/6] vhost: add regions to temporary list, Paolo Bonzini, 2017/12/15
- Re: [Qemu-devel] [PATCH v4 4/6] vhost: add regions to temporary list, Dr. David Alan Gilbert, 2017/12/15
[Qemu-devel] [PATCH v4 5/6] vhost: compare and flip in new memory region list, Dr. David Alan Gilbert (git), 2017/12/13
[Qemu-devel] [PATCH v4 6/6] vhost: Clean out old vhost_set_memory and friends, Dr. David Alan Gilbert (git), 2017/12/13