Re: [PATCH 0/3] Refactor AIO to allow multiple AIO implementations

2008-10-03 Thread Ryan Harper
* john cooper <[EMAIL PROTECTED]> [2008-10-02 18:51]: > Ryan Harper wrote: > > >Results: > > These results are with the patch series applied to KVM (plus a small KVM > > only > > change -- KVM patches forthcoming). > > Is the above "KVM only change" a kernel-side kvm patch? > If so could you pro

Re: [PATCH 0/3] Refactor AIO to allow multiple AIO implementations

2008-10-02 Thread john cooper
Ryan Harper wrote: Results: These results are with the patch series applied to KVM (plus a small KVM only change -- KVM patches forthcoming). Is the above "KVM only change" a kernel-side kvm patch? If so could you provide a pointer? Didn't see anything related AFAICT in kvm.git since then.

Re: [Qemu-devel] Re: [PATCH 0/3] Refactor AIO to allow multiple AIO implementations

2008-09-23 Thread Anthony Liguori
Ryan Harper wrote: * Anthony Liguori <[EMAIL PROTECTED]> [2008-09-22 22:44]: Can you run the same performance tests with the following patches (using sync=on instead of cache=off)? You'll need my aio_init fix too. I suspect this will give equally good performance to your patch set. That'

Re: [Qemu-devel] Re: [PATCH 0/3] Refactor AIO to allow multiple AIO implementations

2008-09-23 Thread Anthony Liguori
Ryan Harper wrote: * Anthony Liguori <[EMAIL PROTECTED]> [2008-09-22 22:44]: Can you run the same performance tests with the following patches (using sync=on instead of cache=off)? You'll need my aio_init fix too. I suspect this will give equally good performance to your patch set. That'

Re: [Qemu-devel] Re: [PATCH 0/3] Refactor AIO to allow multiple AIO implementations

2008-09-23 Thread Ryan Harper
* Anthony Liguori <[EMAIL PROTECTED]> [2008-09-22 22:44]: > Can you run the same performance tests with the following patches (using > sync=on instead of cache=off)? > > You'll need my aio_init fix too. I suspect this will give equally good > performance to your patch set. That's not saying yo

Re: [Qemu-devel] [PATCH 0/3] Refactor AIO to allow multiple AIO implementations

2008-09-23 Thread Jamie Lokier
Ryan Harper wrote: > Addtional work has been done on top of QEMU for KVM and virtio-blk > devices. While virtio-blk is not yet upstream in QEMU, the AIO > changes here provide a tremendous performance improvement (from 7.6% > of native, to 100% of randwrite, and 3.9% of native, to 101.4% of > nati

Re: [PATCH 0/3] Refactor AIO to allow multiple AIO implementations

2008-09-22 Thread Anthony Liguori
Can you run the same performance tests with the following patches (using sync=on instead of cache=off)? You'll need my aio_init fix too. I suspect this will give equally good performance to your patch set. That's not saying your patch set isn't useful, but I would like to get performance to

[PATCH 0/3] Refactor AIO to allow multiple AIO implementations

2008-09-22 Thread Ryan Harper
The patchset adds additional AIO driver abstraction to the block raw driver to support multiple aio implementations for each device. The first patch pulls the posix aio implementation out of the block-raw device using a generic call to the newly created AIO Driver structure. The posix aio impleme