Hi Kevin, On Mon, Jul 11, 2011 at 12:31 PM, Kevin Wolf <kw...@redhat.com> wrote: > I would love to try out your code occasionally myself, but so far I have > been to lazy to build a guest kernel only to be able to test it. Having > to deal with the huge kernel git tree just for a small program doesn't > really make it more fun either... Anyway, what I'm trying to say is that > everything in my mails is from a purely theoretical POV. I have only > looked at the code, but never really tried it.
Most distro kernels boot just fine, AFAIK. If you have a kernel tree laying around, you can use git remote add kvm-tool git://github.com/penberg/linux-kvm.git git remote update kvm-tool to fetch the sources. > As Ingo already said, the cache mode is probably the major difference. > From what I can see in your code, cache=writeback would be the > equivalent for what tools/kvm is doing, however cache=none (i.e. > O_DIRECT) is what people usually do with qemu. Yup, I posted 'cache=writeback' results too which are much closer to tools/kvm numbers. > And then there seems to be another big difference. I hope I'm not > missing anything, but you seem to be completely lacking refcount > handling for qcow2. This is okay for read-only image, but with write > access to the image, you're corrupting the images if you don't update > the refcounts. Have you checked qcow2 images with qemu-img check after > tools/kvm having written to it? > > Maintaining the right order between L2 writes and refcount block writes > is another source of flushes in qemu, which of course makes a difference > for performance. Yes, you're absolutely correct. We don't support copy-on-write images and I didn't realize until yesterday evening that we don't even check the 'copied' bit to make sure writes are safe. However, for these particular numbers, it doesn't matter that much because it's all append-only and thus shouldn't trigger any of the copy-on-write paths. Pekka -- To unsubscribe from this list: send the line "unsubscribe kvm" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html