* [PATCH RESEND 0/5] Add vhost-blk support
@ 2012-07-13 8:55 Asias He
0 siblings, 0 replies; 20+ messages in thread
From: Asias He @ 2012-07-13 8:55 UTC (permalink / raw)
To: linux-kernel
Cc: linux-aio, kvm, Michael S. Tsirkin, virtualization,
James Bottomley, Jeff Moyer, Benjamin LaHaise, Alexander Viro,
linux-fsdevel
Hi folks,
[I am resending to fix the broken thread in the previous one.]
This patchset adds vhost-blk support. vhost-blk is a in kernel virito-blk
device accelerator. Compared to userspace virtio-blk implementation, vhost-blk
gives about 5% to 15% performance improvement.
Asias He (5):
aio: Export symbols and struct kiocb_batch for in kernel aio usage
eventfd: Export symbol eventfd_file_create()
vhost: Make vhost a separate module
vhost-net: Use VHOST_NET_FEATURES for vhost-net
vhost-blk: Add vhost-blk support
drivers/vhost/Kconfig | 20 +-
drivers/vhost/Makefile | 6 +-
drivers/vhost/blk.c | 600 ++++++++++++++++++++++++++++++++++++++++++++++++
drivers/vhost/net.c | 4 +-
drivers/vhost/test.c | 4 +-
drivers/vhost/vhost.c | 48 ++++
drivers/vhost/vhost.h | 18 +-
fs/aio.c | 37 ++-
fs/eventfd.c | 1 +
include/linux/aio.h | 21 ++
include/linux/vhost.h | 3 +
11 files changed, 731 insertions(+), 31 deletions(-)
create mode 100644 drivers/vhost/blk.c
--
1.7.10.4
^ permalink raw reply [flat|nested] 20+ messages in thread
* Re: [PATCH RESEND 0/5] Add vhost-blk support
2012-07-13 8:55 ` Asias He
@ 2012-07-20 19:30 ` Michael S. Tsirkin
-1 siblings, 0 replies; 20+ messages in thread
From: Michael S. Tsirkin @ 2012-07-20 19:30 UTC (permalink / raw)
To: Asias He
Cc: linux-kernel, Alexander Viro, Benjamin LaHaise, James Bottomley,
Jeff Moyer, kvm, linux-aio, linux-fsdevel, virtualization,
gregkh
On Fri, Jul 13, 2012 at 04:55:06PM +0800, Asias He wrote:
>
> Hi folks,
>
> [I am resending to fix the broken thread in the previous one.]
>
> This patchset adds vhost-blk support. vhost-blk is a in kernel virito-blk
> device accelerator. Compared to userspace virtio-blk implementation, vhost-blk
> gives about 5% to 15% performance improvement.
>
> Asias He (5):
> aio: Export symbols and struct kiocb_batch for in kernel aio usage
> eventfd: Export symbol eventfd_file_create()
> vhost: Make vhost a separate module
> vhost-net: Use VHOST_NET_FEATURES for vhost-net
> vhost-blk: Add vhost-blk support
OK so given the state it's in, and assuming you think it is helpful
to let it mature in tree and not out of tree, I think it's
reasonable to try to do it like tcm_vhost is going to do it:
- send me changes to vhost core ASAP (and keep it minimal, e.g.
use your own header file to export to userspace)
- for other stuff - put in drivers/staging, and ask Greg to merge
--
MST
^ permalink raw reply [flat|nested] 20+ messages in thread
* Re: [PATCH RESEND 0/5] Add vhost-blk support
@ 2012-07-20 19:30 ` Michael S. Tsirkin
0 siblings, 0 replies; 20+ messages in thread
From: Michael S. Tsirkin @ 2012-07-20 19:30 UTC (permalink / raw)
To: Asias He
Cc: linux-kernel, Alexander Viro, Benjamin LaHaise, James Bottomley,
Jeff Moyer, kvm, linux-aio, linux-fsdevel, virtualization,
gregkh
On Fri, Jul 13, 2012 at 04:55:06PM +0800, Asias He wrote:
>
> Hi folks,
>
> [I am resending to fix the broken thread in the previous one.]
>
> This patchset adds vhost-blk support. vhost-blk is a in kernel virito-blk
> device accelerator. Compared to userspace virtio-blk implementation, vhost-blk
> gives about 5% to 15% performance improvement.
>
> Asias He (5):
> aio: Export symbols and struct kiocb_batch for in kernel aio usage
> eventfd: Export symbol eventfd_file_create()
> vhost: Make vhost a separate module
> vhost-net: Use VHOST_NET_FEATURES for vhost-net
> vhost-blk: Add vhost-blk support
OK so given the state it's in, and assuming you think it is helpful
to let it mature in tree and not out of tree, I think it's
reasonable to try to do it like tcm_vhost is going to do it:
- send me changes to vhost core ASAP (and keep it minimal, e.g.
use your own header file to export to userspace)
- for other stuff - put in drivers/staging, and ask Greg to merge
--
MST
--
To unsubscribe, send a message with 'unsubscribe linux-aio' in
the body to majordomo@kvack.org. For more info on Linux AIO,
see: http://www.kvack.org/aio/
Don't email: <a href=mailto:"aart@kvack.org">aart@kvack.org</a>
^ permalink raw reply [flat|nested] 20+ messages in thread
* Re: [PATCH RESEND 0/5] Add vhost-blk support
2012-07-13 8:55 ` Asias He
` (5 preceding siblings ...)
(?)
@ 2012-07-20 19:30 ` Michael S. Tsirkin
-1 siblings, 0 replies; 20+ messages in thread
From: Michael S. Tsirkin @ 2012-07-20 19:30 UTC (permalink / raw)
To: Asias He
Cc: linux-aio, kvm, gregkh, linux-kernel, virtualization,
James Bottomley, Jeff Moyer, Benjamin LaHaise, Alexander Viro,
linux-fsdevel
On Fri, Jul 13, 2012 at 04:55:06PM +0800, Asias He wrote:
>
> Hi folks,
>
> [I am resending to fix the broken thread in the previous one.]
>
> This patchset adds vhost-blk support. vhost-blk is a in kernel virito-blk
> device accelerator. Compared to userspace virtio-blk implementation, vhost-blk
> gives about 5% to 15% performance improvement.
>
> Asias He (5):
> aio: Export symbols and struct kiocb_batch for in kernel aio usage
> eventfd: Export symbol eventfd_file_create()
> vhost: Make vhost a separate module
> vhost-net: Use VHOST_NET_FEATURES for vhost-net
> vhost-blk: Add vhost-blk support
OK so given the state it's in, and assuming you think it is helpful
to let it mature in tree and not out of tree, I think it's
reasonable to try to do it like tcm_vhost is going to do it:
- send me changes to vhost core ASAP (and keep it minimal, e.g.
use your own header file to export to userspace)
- for other stuff - put in drivers/staging, and ask Greg to merge
--
MST
^ permalink raw reply [flat|nested] 20+ messages in thread
* Re: [PATCH RESEND 0/5] Add vhost-blk support
2012-07-17 15:09 ` Michael S. Tsirkin
` (2 preceding siblings ...)
(?)
@ 2012-07-18 11:42 ` Stefan Hajnoczi
-1 siblings, 0 replies; 20+ messages in thread
From: Stefan Hajnoczi @ 2012-07-18 11:42 UTC (permalink / raw)
To: Michael S. Tsirkin
Cc: Asias He, Jens Axboe, linux-aio, target-devel, linux-scsi, kvm,
linux-kernel, lf-virt, James Bottomley, Anthony Liguori,
Jeff Moyer, Benjamin LaHaise, Alexander Viro, linux-fsdevel,
Paolo Bonzini, Zhi Yong Wu, Christoph Hellwig, Stefan Hajnoczi
On Tue, Jul 17, 2012 at 4:09 PM, Michael S. Tsirkin <mst@redhat.com> wrote:
> On Fri, Jul 13, 2012 at 04:55:06PM +0800, Asias He wrote:
>>
>> Hi folks,
>>
>> [I am resending to fix the broken thread in the previous one.]
>>
>> This patchset adds vhost-blk support. vhost-blk is a in kernel virito-blk
>> device accelerator. Compared to userspace virtio-blk implementation, vhost-blk
>> gives about 5% to 15% performance improvement.
>
> Same thing as tcm_host comment:
>
> It seems not 100% clear whether this driver will have major
> userspace using it. And if not, it would be very hard to support a
> driver when recent userspace does not use it in the end.
>
> I think a good idea for 3.6 would be to make it depend on
> CONFIG_STAGING. Then we don't commit to an ABI. For this, you can add
> a separate Kconfig and source it from drivers/staging/Kconfig. Maybe it
> needs to be in a separate directory drivers/vhost/staging/Kconfig.
>
> I Cc'd the list of tcm_host in the hope that you can cooperate on this.
>
Adding it to staging allows more people to try it out, so that's a
good thing. If I get a moment to play with it I'll let you know the
results.
Stefan
^ permalink raw reply [flat|nested] 20+ messages in thread
* Re: [PATCH RESEND 0/5] Add vhost-blk support
2012-07-17 15:09 ` Michael S. Tsirkin
` (3 preceding siblings ...)
(?)
@ 2012-07-18 11:42 ` Stefan Hajnoczi
-1 siblings, 0 replies; 20+ messages in thread
From: Stefan Hajnoczi @ 2012-07-18 11:42 UTC (permalink / raw)
To: Michael S. Tsirkin
Cc: Jens Axboe, linux-aio, linux-scsi, kvm, linux-kernel, lf-virt,
James Bottomley, Anthony Liguori, Jeff Moyer, target-devel,
Alexander Viro, Paolo Bonzini, Benjamin LaHaise, linux-fsdevel,
Zhi Yong Wu, Christoph Hellwig, Stefan Hajnoczi
On Tue, Jul 17, 2012 at 4:09 PM, Michael S. Tsirkin <mst@redhat.com> wrote:
> On Fri, Jul 13, 2012 at 04:55:06PM +0800, Asias He wrote:
>>
>> Hi folks,
>>
>> [I am resending to fix the broken thread in the previous one.]
>>
>> This patchset adds vhost-blk support. vhost-blk is a in kernel virito-blk
>> device accelerator. Compared to userspace virtio-blk implementation, vhost-blk
>> gives about 5% to 15% performance improvement.
>
> Same thing as tcm_host comment:
>
> It seems not 100% clear whether this driver will have major
> userspace using it. And if not, it would be very hard to support a
> driver when recent userspace does not use it in the end.
>
> I think a good idea for 3.6 would be to make it depend on
> CONFIG_STAGING. Then we don't commit to an ABI. For this, you can add
> a separate Kconfig and source it from drivers/staging/Kconfig. Maybe it
> needs to be in a separate directory drivers/vhost/staging/Kconfig.
>
> I Cc'd the list of tcm_host in the hope that you can cooperate on this.
>
Adding it to staging allows more people to try it out, so that's a
good thing. If I get a moment to play with it I'll let you know the
results.
Stefan
^ permalink raw reply [flat|nested] 20+ messages in thread
* Re: [PATCH RESEND 0/5] Add vhost-blk support
2012-07-17 15:09 ` Michael S. Tsirkin
@ 2012-07-18 2:09 ` Asias He
-1 siblings, 0 replies; 20+ messages in thread
From: Asias He @ 2012-07-18 2:09 UTC (permalink / raw)
To: Michael S. Tsirkin
Cc: linux-kernel, Alexander Viro, Benjamin LaHaise, James Bottomley,
Jeff Moyer, kvm, linux-aio, linux-fsdevel, target-devel,
linux-scsi, lf-virt, Stefan Hajnoczi, Zhi Yong Wu,
Anthony Liguori, Paolo Bonzini, Christoph Hellwig, Jens Axboe,
Hannes Reinecke
On 07/17/2012 11:09 PM, Michael S. Tsirkin wrote:
> On Fri, Jul 13, 2012 at 04:55:06PM +0800, Asias He wrote:
>>
>> Hi folks,
>>
>> [I am resending to fix the broken thread in the previous one.]
>>
>> This patchset adds vhost-blk support. vhost-blk is a in kernel virito-blk
>> device accelerator. Compared to userspace virtio-blk implementation, vhost-blk
>> gives about 5% to 15% performance improvement.
>
> Same thing as tcm_host comment:
>
> It seems not 100% clear whether this driver will have major
> userspace using it. And if not, it would be very hard to support a
> driver when recent userspace does not use it in the end.
>
> I think a good idea for 3.6 would be to make it depend on
> CONFIG_STAGING. Then we don't commit to an ABI. For this, you can add
> a separate Kconfig and source it from drivers/staging/Kconfig. Maybe it
> needs to be in a separate directory drivers/vhost/staging/Kconfig.
OK.
> I Cc'd the list of tcm_host in the hope that you can cooperate on this.
Sure.
--
Asias
^ permalink raw reply [flat|nested] 20+ messages in thread
* Re: [PATCH RESEND 0/5] Add vhost-blk support
@ 2012-07-18 2:09 ` Asias He
0 siblings, 0 replies; 20+ messages in thread
From: Asias He @ 2012-07-18 2:09 UTC (permalink / raw)
To: Michael S. Tsirkin
Cc: linux-kernel, Alexander Viro, Benjamin LaHaise, James Bottomley,
Jeff Moyer, kvm, linux-aio, linux-fsdevel, target-devel,
linux-scsi, lf-virt, Stefan Hajnoczi, Zhi Yong Wu,
Anthony Liguori, Paolo Bonzini, Christoph Hellwig, Jens Axboe,
Hannes Reinecke
On 07/17/2012 11:09 PM, Michael S. Tsirkin wrote:
> On Fri, Jul 13, 2012 at 04:55:06PM +0800, Asias He wrote:
>>
>> Hi folks,
>>
>> [I am resending to fix the broken thread in the previous one.]
>>
>> This patchset adds vhost-blk support. vhost-blk is a in kernel virito-blk
>> device accelerator. Compared to userspace virtio-blk implementation, vhost-blk
>> gives about 5% to 15% performance improvement.
>
> Same thing as tcm_host comment:
>
> It seems not 100% clear whether this driver will have major
> userspace using it. And if not, it would be very hard to support a
> driver when recent userspace does not use it in the end.
>
> I think a good idea for 3.6 would be to make it depend on
> CONFIG_STAGING. Then we don't commit to an ABI. For this, you can add
> a separate Kconfig and source it from drivers/staging/Kconfig. Maybe it
> needs to be in a separate directory drivers/vhost/staging/Kconfig.
OK.
> I Cc'd the list of tcm_host in the hope that you can cooperate on this.
Sure.
--
Asias
--
To unsubscribe, send a message with 'unsubscribe linux-aio' in
the body to majordomo@kvack.org. For more info on Linux AIO,
see: http://www.kvack.org/aio/
Don't email: <a href=mailto:"aart@kvack.org">aart@kvack.org</a>
^ permalink raw reply [flat|nested] 20+ messages in thread
* Re: [PATCH RESEND 0/5] Add vhost-blk support
2012-07-17 15:09 ` Michael S. Tsirkin
(?)
@ 2012-07-18 2:09 ` Asias He
-1 siblings, 0 replies; 20+ messages in thread
From: Asias He @ 2012-07-18 2:09 UTC (permalink / raw)
To: Michael S. Tsirkin
Cc: Jens Axboe, linux-aio, target-devel, linux-scsi, kvm,
linux-kernel, lf-virt, James Bottomley, Anthony Liguori,
Jeff Moyer, Benjamin LaHaise, Alexander Viro, linux-fsdevel,
Paolo Bonzini, Zhi Yong Wu, Christoph Hellwig, Stefan Hajnoczi
On 07/17/2012 11:09 PM, Michael S. Tsirkin wrote:
> On Fri, Jul 13, 2012 at 04:55:06PM +0800, Asias He wrote:
>>
>> Hi folks,
>>
>> [I am resending to fix the broken thread in the previous one.]
>>
>> This patchset adds vhost-blk support. vhost-blk is a in kernel virito-blk
>> device accelerator. Compared to userspace virtio-blk implementation, vhost-blk
>> gives about 5% to 15% performance improvement.
>
> Same thing as tcm_host comment:
>
> It seems not 100% clear whether this driver will have major
> userspace using it. And if not, it would be very hard to support a
> driver when recent userspace does not use it in the end.
>
> I think a good idea for 3.6 would be to make it depend on
> CONFIG_STAGING. Then we don't commit to an ABI. For this, you can add
> a separate Kconfig and source it from drivers/staging/Kconfig. Maybe it
> needs to be in a separate directory drivers/vhost/staging/Kconfig.
OK.
> I Cc'd the list of tcm_host in the hope that you can cooperate on this.
Sure.
--
Asias
^ permalink raw reply [flat|nested] 20+ messages in thread
* Re: [PATCH RESEND 0/5] Add vhost-blk support
2012-07-13 8:55 ` Asias He
@ 2012-07-17 15:09 ` Michael S. Tsirkin
-1 siblings, 0 replies; 20+ messages in thread
From: Michael S. Tsirkin @ 2012-07-17 15:09 UTC (permalink / raw)
To: Asias He
Cc: linux-kernel, Alexander Viro, Benjamin LaHaise, James Bottomley,
Jeff Moyer, kvm, linux-aio, linux-fsdevel, target-devel,
linux-scsi, lf-virt, Stefan Hajnoczi, Zhi Yong Wu,
Anthony Liguori, Paolo Bonzini, Christoph Hellwig, Jens Axboe,
Hannes Reinecke
On Fri, Jul 13, 2012 at 04:55:06PM +0800, Asias He wrote:
>
> Hi folks,
>
> [I am resending to fix the broken thread in the previous one.]
>
> This patchset adds vhost-blk support. vhost-blk is a in kernel virito-blk
> device accelerator. Compared to userspace virtio-blk implementation, vhost-blk
> gives about 5% to 15% performance improvement.
Same thing as tcm_host comment:
It seems not 100% clear whether this driver will have major
userspace using it. And if not, it would be very hard to support a
driver when recent userspace does not use it in the end.
I think a good idea for 3.6 would be to make it depend on
CONFIG_STAGING. Then we don't commit to an ABI. For this, you can add
a separate Kconfig and source it from drivers/staging/Kconfig. Maybe it
needs to be in a separate directory drivers/vhost/staging/Kconfig.
I Cc'd the list of tcm_host in the hope that you can cooperate on this.
--
MST
^ permalink raw reply [flat|nested] 20+ messages in thread
* Re: [PATCH RESEND 0/5] Add vhost-blk support
@ 2012-07-17 15:09 ` Michael S. Tsirkin
0 siblings, 0 replies; 20+ messages in thread
From: Michael S. Tsirkin @ 2012-07-17 15:09 UTC (permalink / raw)
To: Asias He
Cc: linux-kernel, Alexander Viro, Benjamin LaHaise, James Bottomley,
Jeff Moyer, kvm, linux-aio, linux-fsdevel, target-devel,
linux-scsi, lf-virt, Stefan Hajnoczi, Zhi Yong Wu,
Anthony Liguori, Paolo Bonzini, Christoph Hellwig, Jens Axboe,
Hannes Reinecke
On Fri, Jul 13, 2012 at 04:55:06PM +0800, Asias He wrote:
>
> Hi folks,
>
> [I am resending to fix the broken thread in the previous one.]
>
> This patchset adds vhost-blk support. vhost-blk is a in kernel virito-blk
> device accelerator. Compared to userspace virtio-blk implementation, vhost-blk
> gives about 5% to 15% performance improvement.
Same thing as tcm_host comment:
It seems not 100% clear whether this driver will have major
userspace using it. And if not, it would be very hard to support a
driver when recent userspace does not use it in the end.
I think a good idea for 3.6 would be to make it depend on
CONFIG_STAGING. Then we don't commit to an ABI. For this, you can add
a separate Kconfig and source it from drivers/staging/Kconfig. Maybe it
needs to be in a separate directory drivers/vhost/staging/Kconfig.
I Cc'd the list of tcm_host in the hope that you can cooperate on this.
--
MST
--
To unsubscribe, send a message with 'unsubscribe linux-aio' in
the body to majordomo@kvack.org. For more info on Linux AIO,
see: http://www.kvack.org/aio/
Don't email: <a href=mailto:"aart@kvack.org">aart@kvack.org</a>
^ permalink raw reply [flat|nested] 20+ messages in thread
* Re: [PATCH RESEND 0/5] Add vhost-blk support
2012-07-13 8:55 ` Asias He
` (3 preceding siblings ...)
(?)
@ 2012-07-17 15:09 ` Michael S. Tsirkin
-1 siblings, 0 replies; 20+ messages in thread
From: Michael S. Tsirkin @ 2012-07-17 15:09 UTC (permalink / raw)
To: Asias He
Cc: Jens Axboe, linux-aio, target-devel, linux-scsi, kvm,
linux-kernel, lf-virt, James Bottomley, Anthony Liguori,
Jeff Moyer, Benjamin LaHaise, Alexander Viro, linux-fsdevel,
Paolo Bonzini, Zhi Yong Wu, Christoph Hellwig, Stefan Hajnoczi
On Fri, Jul 13, 2012 at 04:55:06PM +0800, Asias He wrote:
>
> Hi folks,
>
> [I am resending to fix the broken thread in the previous one.]
>
> This patchset adds vhost-blk support. vhost-blk is a in kernel virito-blk
> device accelerator. Compared to userspace virtio-blk implementation, vhost-blk
> gives about 5% to 15% performance improvement.
Same thing as tcm_host comment:
It seems not 100% clear whether this driver will have major
userspace using it. And if not, it would be very hard to support a
driver when recent userspace does not use it in the end.
I think a good idea for 3.6 would be to make it depend on
CONFIG_STAGING. Then we don't commit to an ABI. For this, you can add
a separate Kconfig and source it from drivers/staging/Kconfig. Maybe it
needs to be in a separate directory drivers/vhost/staging/Kconfig.
I Cc'd the list of tcm_host in the hope that you can cooperate on this.
--
MST
^ permalink raw reply [flat|nested] 20+ messages in thread
* Re: [PATCH RESEND 0/5] Add vhost-blk support
2012-07-14 7:49 ` Christoph Hellwig
@ 2012-07-16 9:05 ` Asias He
-1 siblings, 0 replies; 20+ messages in thread
From: Asias He @ 2012-07-16 9:05 UTC (permalink / raw)
To: Christoph Hellwig
Cc: linux-kernel, Alexander Viro, Benjamin LaHaise, James Bottomley,
Jeff Moyer, kvm, linux-aio, linux-fsdevel, Michael S. Tsirkin,
virtualization, Dave Kleikamp, Zach Brown
Hi Christoph,
On 07/14/2012 03:49 PM, Christoph Hellwig wrote:
> Please send a version that does direct block I/O similar to xen-blkback
> for now.
Seems xen-blkback converts the guest IO request to host bio and submit
them directly. I was wondering whether this has a performance gain
compared to AIO implementation.
> If we get proper in-kernel aio support one day you can add
> back file backend support.
I talked with Dave and Zack on the in-kernel aio patch which James
pointed out:
http://marc.info/?l=linux-fsdevel&m=133312234313122
Dave will post a new version soon. I will wait for it.
--
Asias
^ permalink raw reply [flat|nested] 20+ messages in thread
* Re: [PATCH RESEND 0/5] Add vhost-blk support
@ 2012-07-16 9:05 ` Asias He
0 siblings, 0 replies; 20+ messages in thread
From: Asias He @ 2012-07-16 9:05 UTC (permalink / raw)
To: Christoph Hellwig
Cc: linux-kernel, Alexander Viro, Benjamin LaHaise, James Bottomley,
Jeff Moyer, kvm, linux-aio, linux-fsdevel, Michael S. Tsirkin,
virtualization, Dave Kleikamp, Zach Brown
Hi Christoph,
On 07/14/2012 03:49 PM, Christoph Hellwig wrote:
> Please send a version that does direct block I/O similar to xen-blkback
> for now.
Seems xen-blkback converts the guest IO request to host bio and submit
them directly. I was wondering whether this has a performance gain
compared to AIO implementation.
> If we get proper in-kernel aio support one day you can add
> back file backend support.
I talked with Dave and Zack on the in-kernel aio patch which James
pointed out:
http://marc.info/?l=linux-fsdevel&m=133312234313122
Dave will post a new version soon. I will wait for it.
--
Asias
--
To unsubscribe, send a message with 'unsubscribe linux-aio' in
the body to majordomo@kvack.org. For more info on Linux AIO,
see: http://www.kvack.org/aio/
Don't email: <a href=mailto:"aart@kvack.org">aart@kvack.org</a>
^ permalink raw reply [flat|nested] 20+ messages in thread
* Re: [PATCH RESEND 0/5] Add vhost-blk support
2012-07-14 7:49 ` Christoph Hellwig
(?)
(?)
@ 2012-07-16 9:05 ` Asias He
-1 siblings, 0 replies; 20+ messages in thread
From: Asias He @ 2012-07-16 9:05 UTC (permalink / raw)
To: Christoph Hellwig
Cc: linux-aio, Dave Kleikamp, Zach Brown, kvm, Michael S. Tsirkin,
linux-kernel, virtualization, James Bottomley, Jeff Moyer,
Benjamin LaHaise, Alexander Viro, linux-fsdevel
Hi Christoph,
On 07/14/2012 03:49 PM, Christoph Hellwig wrote:
> Please send a version that does direct block I/O similar to xen-blkback
> for now.
Seems xen-blkback converts the guest IO request to host bio and submit
them directly. I was wondering whether this has a performance gain
compared to AIO implementation.
> If we get proper in-kernel aio support one day you can add
> back file backend support.
I talked with Dave and Zack on the in-kernel aio patch which James
pointed out:
http://marc.info/?l=linux-fsdevel&m=133312234313122
Dave will post a new version soon. I will wait for it.
--
Asias
^ permalink raw reply [flat|nested] 20+ messages in thread
* Re: [PATCH RESEND 0/5] Add vhost-blk support
2012-07-13 8:55 ` Asias He
@ 2012-07-14 7:49 ` Christoph Hellwig
-1 siblings, 0 replies; 20+ messages in thread
From: Christoph Hellwig @ 2012-07-14 7:49 UTC (permalink / raw)
To: Asias He
Cc: linux-kernel, Alexander Viro, Benjamin LaHaise, James Bottomley,
Jeff Moyer, kvm, linux-aio, linux-fsdevel, Michael S. Tsirkin,
virtualization
Please send a version that does direct block I/O similar to xen-blkback
for now. If we get proper in-kernel aio support one day you can add
back file backend support.
^ permalink raw reply [flat|nested] 20+ messages in thread
* Re: [PATCH RESEND 0/5] Add vhost-blk support
@ 2012-07-14 7:49 ` Christoph Hellwig
0 siblings, 0 replies; 20+ messages in thread
From: Christoph Hellwig @ 2012-07-14 7:49 UTC (permalink / raw)
To: Asias He
Cc: linux-kernel, Alexander Viro, Benjamin LaHaise, James Bottomley,
Jeff Moyer, kvm, linux-aio, linux-fsdevel, Michael S. Tsirkin,
virtualization
Please send a version that does direct block I/O similar to xen-blkback
for now. If we get proper in-kernel aio support one day you can add
back file backend support.
--
To unsubscribe, send a message with 'unsubscribe linux-aio' in
the body to majordomo@kvack.org. For more info on Linux AIO,
see: http://www.kvack.org/aio/
Don't email: <a href=mailto:"aart@kvack.org">aart@kvack.org</a>
^ permalink raw reply [flat|nested] 20+ messages in thread
* Re: [PATCH RESEND 0/5] Add vhost-blk support
2012-07-13 8:55 ` Asias He
(?)
(?)
@ 2012-07-14 7:49 ` Christoph Hellwig
-1 siblings, 0 replies; 20+ messages in thread
From: Christoph Hellwig @ 2012-07-14 7:49 UTC (permalink / raw)
To: Asias He
Cc: linux-aio, kvm, Michael S. Tsirkin, linux-kernel, virtualization,
James Bottomley, Jeff Moyer, Benjamin LaHaise, Alexander Viro,
linux-fsdevel
Please send a version that does direct block I/O similar to xen-blkback
for now. If we get proper in-kernel aio support one day you can add
back file backend support.
^ permalink raw reply [flat|nested] 20+ messages in thread
* [PATCH RESEND 0/5] Add vhost-blk support
@ 2012-07-13 8:55 ` Asias He
0 siblings, 0 replies; 20+ messages in thread
From: Asias He @ 2012-07-13 8:55 UTC (permalink / raw)
To: linux-kernel
Cc: Alexander Viro, Benjamin LaHaise, James Bottomley, Jeff Moyer,
kvm, linux-aio, linux-fsdevel, Michael S. Tsirkin,
virtualization
Hi folks,
[I am resending to fix the broken thread in the previous one.]
This patchset adds vhost-blk support. vhost-blk is a in kernel virito-blk
device accelerator. Compared to userspace virtio-blk implementation, vhost-blk
gives about 5% to 15% performance improvement.
Asias He (5):
aio: Export symbols and struct kiocb_batch for in kernel aio usage
eventfd: Export symbol eventfd_file_create()
vhost: Make vhost a separate module
vhost-net: Use VHOST_NET_FEATURES for vhost-net
vhost-blk: Add vhost-blk support
drivers/vhost/Kconfig | 20 +-
drivers/vhost/Makefile | 6 +-
drivers/vhost/blk.c | 600 ++++++++++++++++++++++++++++++++++++++++++++++++
drivers/vhost/net.c | 4 +-
drivers/vhost/test.c | 4 +-
drivers/vhost/vhost.c | 48 ++++
drivers/vhost/vhost.h | 18 +-
fs/aio.c | 37 ++-
fs/eventfd.c | 1 +
include/linux/aio.h | 21 ++
include/linux/vhost.h | 3 +
11 files changed, 731 insertions(+), 31 deletions(-)
create mode 100644 drivers/vhost/blk.c
--
1.7.10.4
^ permalink raw reply [flat|nested] 20+ messages in thread
* [PATCH RESEND 0/5] Add vhost-blk support
@ 2012-07-13 8:55 ` Asias He
0 siblings, 0 replies; 20+ messages in thread
From: Asias He @ 2012-07-13 8:55 UTC (permalink / raw)
To: linux-kernel
Cc: Alexander Viro, Benjamin LaHaise, James Bottomley, Jeff Moyer,
kvm, linux-aio, linux-fsdevel, Michael S. Tsirkin,
virtualization
Hi folks,
[I am resending to fix the broken thread in the previous one.]
This patchset adds vhost-blk support. vhost-blk is a in kernel virito-blk
device accelerator. Compared to userspace virtio-blk implementation, vhost-blk
gives about 5% to 15% performance improvement.
Asias He (5):
aio: Export symbols and struct kiocb_batch for in kernel aio usage
eventfd: Export symbol eventfd_file_create()
vhost: Make vhost a separate module
vhost-net: Use VHOST_NET_FEATURES for vhost-net
vhost-blk: Add vhost-blk support
drivers/vhost/Kconfig | 20 +-
drivers/vhost/Makefile | 6 +-
drivers/vhost/blk.c | 600 ++++++++++++++++++++++++++++++++++++++++++++++++
drivers/vhost/net.c | 4 +-
drivers/vhost/test.c | 4 +-
drivers/vhost/vhost.c | 48 ++++
drivers/vhost/vhost.h | 18 +-
fs/aio.c | 37 ++-
fs/eventfd.c | 1 +
include/linux/aio.h | 21 ++
include/linux/vhost.h | 3 +
11 files changed, 731 insertions(+), 31 deletions(-)
create mode 100644 drivers/vhost/blk.c
--
1.7.10.4
--
To unsubscribe, send a message with 'unsubscribe linux-aio' in
the body to majordomo@kvack.org. For more info on Linux AIO,
see: http://www.kvack.org/aio/
Don't email: <a href=mailto:"aart@kvack.org">aart@kvack.org</a>
^ permalink raw reply [flat|nested] 20+ messages in thread
end of thread, other threads:[~2012-07-20 19:30 UTC | newest]
Thread overview: 20+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2012-07-13 8:55 [PATCH RESEND 0/5] Add vhost-blk support Asias He
2012-07-13 8:55 Asias He
2012-07-13 8:55 ` Asias He
2012-07-14 7:49 ` Christoph Hellwig
2012-07-14 7:49 ` Christoph Hellwig
2012-07-16 9:05 ` Asias He
2012-07-16 9:05 ` Asias He
2012-07-16 9:05 ` Asias He
2012-07-14 7:49 ` Christoph Hellwig
2012-07-17 15:09 ` Michael S. Tsirkin
2012-07-17 15:09 ` Michael S. Tsirkin
2012-07-18 2:09 ` Asias He
2012-07-18 2:09 ` Asias He
2012-07-18 2:09 ` Asias He
2012-07-18 11:42 ` Stefan Hajnoczi
2012-07-18 11:42 ` Stefan Hajnoczi
2012-07-17 15:09 ` Michael S. Tsirkin
2012-07-20 19:30 ` Michael S. Tsirkin
2012-07-20 19:30 ` Michael S. Tsirkin
2012-07-20 19:30 ` Michael S. Tsirkin
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.