From: Wanlong Gao <gaowanlong@cn.fujitsu.com>
To: Asias He <asias.hejun@gmail.com>
Cc: linux-kernel@vger.kernel.org,
"James E.J. Bottomley" <JBottomley@parallels.com>,
Paolo Bonzini <pbonzini@redhat.com>,
Rusty Russell <rusty@rustcorp.com.au>,
linux-scsi@vger.kernel.org, kvm@vger.kernel.org,
Wanlong Gao <gaowanlong@cn.fujitsu.com>
Subject: Re: [PATCH 0/5] virtio-scsi: create a separate workqueue
Date: Wed, 07 Nov 2012 20:40:22 +0800 [thread overview]
Message-ID: <509A56B6.2010909@cn.fujitsu.com> (raw)
In-Reply-To: <CAFO3S42dT_j9pQSNSUrdmkqu_pgxthx0zBy3qjdiuXrbxf5H+g@mail.gmail.com>
On 11/07/2012 08:23 PM, Asias He wrote:
> On Wed, Nov 7, 2012 at 6:18 PM, Wanlong Gao <gaowanlong@cn.fujitsu.com> wrote:
>> patch 1-3,5 are some cleanups.
>> patch 4: create a separate work queue for virtio-scsi
>> to improve the performance, I tested with tmpfs backed
>> disk, the config file is like below,
>> [global]
>> bsrange=4k-64k
>> ioengine=libaio
>> direct=1
>> iodepth=4
>> loops=10
>> size=64M
>>
>> Before:
>> Disk stats (read/write):
>> sda: ios=6547/5275, merge=37/39, ticks=3144/2645, in_queue=5780, util=81.62%
>> sdb: ios=6542/5277, merge=54/59, ticks=3234/3143, in_queue=6364, util=84.50%
>> sdc: ios=6532/5244, merge=65/62, ticks=3440/3076, in_queue=6505, util=86.83%
>
> The result is about the disk stat. Can you please post the iops and
> the bw instead?
Sure,
before:
write: io=589788KB, bw=122211KB/s, iops=3584 , runt= 4826msec
bw (KB/s) : min=24208, max=116272, per=57.47%, avg=70236.40, stdev=35496.33
WRITE: io=589788KB, aggrb=122210KB/s, minb=122210KB/s, maxb=122210KB/s, mint=4826msec, maxt=4826msec
after:
write: io=589788KB, bw=124244KB/s, iops=3530 , runt= 4747msec
bw (KB/s) : min=63496, max=117943, per=73.39%, avg=91184.20, stdev=20062.42
WRITE: io=589788KB, aggrb=124244KB/s, minb=124244KB/s, maxb=124244KB/s, mint=4747msec, maxt=4747msec
Thanks,
Wanlong Gao
>
>> After:
>> Disk stats (read/write):
>> sda: ios=6349/5318, merge=77/89, ticks=4820/3169, in_queue=7991, util=90.62%
>> sdb: ios=6364/5358, merge=39/53, ticks=3810/2816, in_queue=6615, util=84.63%
>> sdc: ios=6458/5352, merge=74/47, ticks=4425/3048, in_queue=7459, util=88.41%
>>
>>
>> Wanlong Gao (5):
>> virtio-scsi: remove the useless assignment
>> virtio-scsi: remove the needless variable gfp_mask
>> virtio-scsi: use pr_err instead of printk
>> virtio-scsi: create a separate work queue for virtio-scsi
>> virtio-scsi: tidy up the goto label in init()
>>
>>
>>
>> Cc: James E.J. Bottomley <JBottomley@parallels.com>
>> Cc: Paolo Bonzini <pbonzini@redhat.com>
>> Cc: Rusty Russell <rusty@rustcorp.com.au>
>> Cc: linux-scsi@vger.kernel.org
>> Cc: kvm@vger.kernel.org
>> drivers/scsi/virtio_scsi.c | 43 +++++++++++++++++++++++--------------------
>> 1 file changed, 23 insertions(+), 20 deletions(-)
>>
>> --
>> 1.8.0
>>
>> --
>> To unsubscribe from this list: send the line "unsubscribe kvm" in
>> the body of a message to majordomo@vger.kernel.org
>> More majordomo info at http://vger.kernel.org/majordomo-info.html
>
>
>
next prev parent reply other threads:[~2012-11-07 12:40 UTC|newest]
Thread overview: 10+ messages / expand[flat|nested] mbox.gz Atom feed top
2012-11-07 10:18 [PATCH 0/5] virtio-scsi: create a separate workqueue Wanlong Gao
2012-11-07 10:18 ` [PATCH 1/5] virtio-scsi: remove the useless assignment Wanlong Gao
2012-11-07 10:18 ` [PATCH 2/5] virtio-scsi: remove the needless variable gfp_mask Wanlong Gao
2012-11-07 10:18 ` [PATCH 3/5] virtio-scsi: use pr_err instead of printk Wanlong Gao
2012-11-07 10:18 ` [PATCH 4/5] virtio-scsi: create a separate work queue for virtio-scsi Wanlong Gao
2012-11-07 10:18 ` [PATCH 5/5] virtio-scsi: tidy up the goto label in init() Wanlong Gao
2012-11-07 12:23 ` [PATCH 0/5] virtio-scsi: create a separate workqueue Asias He
2012-11-07 12:40 ` Wanlong Gao [this message]
2012-11-07 22:12 ` Paolo Bonzini
2012-11-07 23:38 ` Wanlong Gao
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=509A56B6.2010909@cn.fujitsu.com \
--to=gaowanlong@cn.fujitsu.com \
--cc=JBottomley@parallels.com \
--cc=asias.hejun@gmail.com \
--cc=kvm@vger.kernel.org \
--cc=linux-kernel@vger.kernel.org \
--cc=linux-scsi@vger.kernel.org \
--cc=pbonzini@redhat.com \
--cc=rusty@rustcorp.com.au \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
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.