[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [virtio-dev] Re: [PATCH v2 repost 4/7] virtio-balloon:
From: |
Li, Liang Z |
Subject: |
Re: [Qemu-devel] [virtio-dev] Re: [PATCH v2 repost 4/7] virtio-balloon: speed up inflate/deflate process |
Date: |
Thu, 28 Jul 2016 06:36:18 +0000 |
> > > This ends up doing a 1MB kmalloc() right? That seems a _bit_ big.
> > > How big was the pfn buffer before?
> >
> > Yes, it is if the max pfn is more than 32GB.
> > The size of the pfn buffer use before is 256*4 = 1024 Bytes, it's too
> > small, and it's the main reason for bad performance.
> > Use the max 1MB kmalloc is a balance between performance and
> > flexibility, a large page bitmap covers the range of all the memory is
> > no good for a system with huge amount of memory. If the bitmap is too
> > small, it means we have to traverse a long list for many times, and it's bad
> for performance.
> >
> > Thanks!
> > Liang
>
> There are all your implementation decisions though.
>
> If guest memory is so fragmented that you only have order 0 4k pages, then
> allocating a huge 1M contigious chunk is very problematic in and of itself.
>
The memory is allocated in the probe stage. This will not happen if the driver
is
loaded when booting the guest.
> Most people rarely migrate and do not care how fast that happens.
> Wasting a large chunk of memory (and it's zeroed for no good reason, so you
> actually request host memory for it) for everyone to speed it up when it
> does happen is not really an option.
>
If people don't plan to do inflating/deflating, they should not enable the
virtio-balloon
at the beginning, once they decide to use it, the driver should provide better
performance
as much as possible.
1MB is a very small portion for a VM with more than 32GB memory and it's the
*worst case*,
for VM with less than 32GB memory, the amount of RAM depends on VM's memory size
and will be less than 1MB.
If 1MB is too big, how about 512K, or 256K? 32K seems too small.
Liang
> --
> MST
>
> ---------------------------------------------------------------------
> To unsubscribe, e-mail: address@hidden
> For additional commands, e-mail: address@hidden
- [Qemu-devel] [PATCH v2 repost 2/7] virtio-balloon: define new feature bit and page bitmap head, (continued)
- [Qemu-devel] [PATCH v2 repost 2/7] virtio-balloon: define new feature bit and page bitmap head, Liang Li, 2016/07/26
- [Qemu-devel] [PATCH v2 repost 1/7] virtio-balloon: rework deflate to add page to a list, Liang Li, 2016/07/26
- [Qemu-devel] [PATCH v2 repost 4/7] virtio-balloon: speed up inflate/deflate process, Liang Li, 2016/07/26
- Re: [Qemu-devel] [PATCH v2 repost 4/7] virtio-balloon: speed up inflate/deflate process, Dave Hansen, 2016/07/27
- Re: [Qemu-devel] [PATCH v2 repost 4/7] virtio-balloon: speed up inflate/deflate process, Michael S. Tsirkin, 2016/07/27
- Re: [Qemu-devel] [PATCH v2 repost 4/7] virtio-balloon: speed up inflate/deflate process, Li, Liang Z, 2016/07/27
- Re: [Qemu-devel] [PATCH v2 repost 4/7] virtio-balloon: speed up inflate/deflate process, Michael S. Tsirkin, 2016/07/28
- Re: [Qemu-devel] [virtio-dev] Re: [PATCH v2 repost 4/7] virtio-balloon: speed up inflate/deflate process, Li, Liang Z, 2016/07/28
- Re: [Qemu-devel] [PATCH v2 repost 4/7] virtio-balloon: speed up inflate/deflate process, Li, Liang Z, 2016/07/27
- Re: [Qemu-devel] [PATCH v2 repost 4/7] virtio-balloon: speed up inflate/deflate process, Michael S. Tsirkin, 2016/07/27
- Re: [Qemu-devel] [virtio-dev] Re: [PATCH v2 repost 4/7] virtio-balloon: speed up inflate/deflate process,
Li, Liang Z <=
- Re: [Qemu-devel] [virtio-dev] Re: [PATCH v2 repost 4/7] virtio-balloon: speed up inflate/deflate process, Michael S. Tsirkin, 2016/07/28
- Re: [Qemu-devel] [virtio-dev] Re: [PATCH v2 repost 4/7] virtio-balloon: speed up inflate/deflate process, Li, Liang Z, 2016/07/28
- Re: [Qemu-devel] [virtio-dev] Re: [PATCH v2 repost 4/7] virtio-balloon: speed up inflate/deflate process, Dave Hansen, 2016/07/29
Re: [Qemu-devel] [PATCH v2 repost 4/7] virtio-balloon: speed up inflate/deflate process, Michael S. Tsirkin, 2016/07/27
Re: [Qemu-devel] [PATCH v2 repost 4/7] virtio-balloon: speed up inflate/deflate process, Michael S. Tsirkin, 2016/07/27
[Qemu-devel] [PATCH v2 repost 5/7] virtio-balloon: define feature bit and head for misc virt queue, Liang Li, 2016/07/26