From: Zhi Yong Wu <zwu.kernel@gmail.com> To: Marcelo Tosatti <mtosatti@redhat.com> Cc: Zhi Yong Wu <wuzhy@linux.vnet.ibm.com>, qemu-devel@nongnu.org, kvm@vger.kernel.org, stefanha@linux.vnet.ibm.com, aliguori@us.ibm.com, ryanh@us.ibm.com, kwolf@redhat.com, pair@us.ibm.com Subject: Re: [PATCH v8 3/4] block: add block timer and throttling algorithm Date: Mon, 19 Sep 2011 17:55:41 +0800 [thread overview] Message-ID: <CAEH94LhRJ4QTfobxVt0XUp4_Wrp9zRJ+B_bGp_ELcbMgkwpu7w@mail.gmail.com> (raw) In-Reply-To: <20110914105024.GA18910@amt.cnet> On Wed, Sep 14, 2011 at 6:50 PM, Marcelo Tosatti <mtosatti@redhat.com> wrote: > On Tue, Sep 13, 2011 at 11:09:46AM +0800, Zhi Yong Wu wrote: >> On Fri, Sep 9, 2011 at 10:44 PM, Marcelo Tosatti <mtosatti@redhat.com> wrote: >> > On Thu, Sep 08, 2011 at 06:11:07PM +0800, Zhi Yong Wu wrote: >> >> Note: >> >> 1.) When bps/iops limits are specified to a small value such as 511 bytes/s, this VM will hang up. We are considering how to handle this senario. >> > >> > You can increase the length of the slice, if the request is larger than >> > slice_time * bps_limit. >> Yeah, but it is a challenge for how to increase it. Do you have some nice idea? > > If the queue is empty, and the request being processed does not fit the > queue, increase the slice so that the request fits. Sorry for late reply. actually, do you think that this scenario is meaningful for the user? Since we implement this, if the user limits the bps below 512 bytes/second, the VM can also not run every task. Can you let us know why we need to make such effort? > > That is, make BLOCK_IO_SLICE_TIME dynamic and adjust it as described > above (if the bps or io limits change, reset it to the default > BLOCK_IO_SLICE_TIME). > >> >> 2.) When "dd" command is issued in guest, if its option bs is set to a large value such as "bs=1024K", the result speed will slightly bigger than the limits. >> > >> > Why? >> This issue has not existed. I will remove it. >> When drive bps=1000000, i did some testings on guest VM. >> 1.) bs=1024K >> 18+0 records in >> 18+0 records out >> 18874368 bytes (19 MB) copied, 26.6268 s, 709 kB/s >> 2.) bs=2048K >> 18+0 records in >> 18+0 records out >> 37748736 bytes (38 MB) copied, 46.5336 s, 811 kB/s >> >> > >> > There is lots of debugging leftovers in the patch. >> sorry, i forgot to remove them. >> > >> > > > -- Regards, Zhi Yong Wu
WARNING: multiple messages have this Message-ID (diff)
From: Zhi Yong Wu <zwu.kernel@gmail.com> To: Marcelo Tosatti <mtosatti@redhat.com> Cc: kwolf@redhat.com, aliguori@us.ibm.com, stefanha@linux.vnet.ibm.com, kvm@vger.kernel.org, Zhi Yong Wu <wuzhy@linux.vnet.ibm.com>, qemu-devel@nongnu.org, pair@us.ibm.com, ryanh@us.ibm.com Subject: Re: [Qemu-devel] [PATCH v8 3/4] block: add block timer and throttling algorithm Date: Mon, 19 Sep 2011 17:55:41 +0800 [thread overview] Message-ID: <CAEH94LhRJ4QTfobxVt0XUp4_Wrp9zRJ+B_bGp_ELcbMgkwpu7w@mail.gmail.com> (raw) In-Reply-To: <20110914105024.GA18910@amt.cnet> On Wed, Sep 14, 2011 at 6:50 PM, Marcelo Tosatti <mtosatti@redhat.com> wrote: > On Tue, Sep 13, 2011 at 11:09:46AM +0800, Zhi Yong Wu wrote: >> On Fri, Sep 9, 2011 at 10:44 PM, Marcelo Tosatti <mtosatti@redhat.com> wrote: >> > On Thu, Sep 08, 2011 at 06:11:07PM +0800, Zhi Yong Wu wrote: >> >> Note: >> >> 1.) When bps/iops limits are specified to a small value such as 511 bytes/s, this VM will hang up. We are considering how to handle this senario. >> > >> > You can increase the length of the slice, if the request is larger than >> > slice_time * bps_limit. >> Yeah, but it is a challenge for how to increase it. Do you have some nice idea? > > If the queue is empty, and the request being processed does not fit the > queue, increase the slice so that the request fits. Sorry for late reply. actually, do you think that this scenario is meaningful for the user? Since we implement this, if the user limits the bps below 512 bytes/second, the VM can also not run every task. Can you let us know why we need to make such effort? > > That is, make BLOCK_IO_SLICE_TIME dynamic and adjust it as described > above (if the bps or io limits change, reset it to the default > BLOCK_IO_SLICE_TIME). > >> >> 2.) When "dd" command is issued in guest, if its option bs is set to a large value such as "bs=1024K", the result speed will slightly bigger than the limits. >> > >> > Why? >> This issue has not existed. I will remove it. >> When drive bps=1000000, i did some testings on guest VM. >> 1.) bs=1024K >> 18+0 records in >> 18+0 records out >> 18874368 bytes (19 MB) copied, 26.6268 s, 709 kB/s >> 2.) bs=2048K >> 18+0 records in >> 18+0 records out >> 37748736 bytes (38 MB) copied, 46.5336 s, 811 kB/s >> >> > >> > There is lots of debugging leftovers in the patch. >> sorry, i forgot to remove them. >> > >> > > > -- Regards, Zhi Yong Wu
next prev parent reply other threads:[~2011-09-19 9:55 UTC|newest] Thread overview: 68+ messages / expand[flat|nested] mbox.gz Atom feed top 2011-09-08 10:11 [PATCH v8 0/4] The intro of QEMU block I/O throttling Zhi Yong Wu 2011-09-08 10:11 ` [Qemu-devel] " Zhi Yong Wu 2011-09-08 10:11 ` [PATCH v8 1/4] block: add the block queue support Zhi Yong Wu 2011-09-08 10:11 ` [Qemu-devel] " Zhi Yong Wu 2011-09-23 15:32 ` Kevin Wolf 2011-09-23 15:32 ` [Qemu-devel] " Kevin Wolf 2011-09-26 8:01 ` Zhi Yong Wu 2011-09-26 8:01 ` Zhi Yong Wu 2011-10-17 10:17 ` Kevin Wolf 2011-10-17 10:17 ` Kevin Wolf 2011-10-17 10:17 ` Paolo Bonzini 2011-10-18 7:00 ` Zhi Yong Wu 2011-10-18 7:00 ` Zhi Yong Wu 2011-10-18 8:07 ` Zhi Yong Wu 2011-10-18 8:07 ` [Qemu-devel] " Zhi Yong Wu 2011-10-18 8:36 ` Kevin Wolf 2011-10-18 8:36 ` Kevin Wolf 2011-10-18 9:29 ` Zhi Yong Wu 2011-10-18 9:29 ` Zhi Yong Wu 2011-10-18 9:56 ` Kevin Wolf 2011-10-18 9:56 ` Kevin Wolf 2011-10-18 13:29 ` Zhi Yong Wu 2011-10-18 13:29 ` Zhi Yong Wu 2011-09-08 10:11 ` [PATCH v8 2/4] block: add the command line support Zhi Yong Wu 2011-09-08 10:11 ` [Qemu-devel] " Zhi Yong Wu 2011-09-23 15:54 ` Kevin Wolf 2011-09-23 15:54 ` [Qemu-devel] " Kevin Wolf 2011-09-26 6:15 ` Zhi Yong Wu 2011-09-26 6:15 ` [Qemu-devel] " Zhi Yong Wu 2011-10-17 10:19 ` Kevin Wolf 2011-10-17 10:19 ` Kevin Wolf 2011-10-18 8:17 ` Zhi Yong Wu 2011-10-18 8:17 ` [Qemu-devel] " Zhi Yong Wu 2011-09-08 10:11 ` [PATCH v8 3/4] block: add block timer and throttling algorithm Zhi Yong Wu 2011-09-08 10:11 ` [Qemu-devel] " Zhi Yong Wu 2011-09-09 14:44 ` Marcelo Tosatti 2011-09-09 14:44 ` [Qemu-devel] " Marcelo Tosatti 2011-09-13 3:09 ` Zhi Yong Wu 2011-09-13 3:09 ` [Qemu-devel] " Zhi Yong Wu 2011-09-14 10:50 ` Marcelo Tosatti 2011-09-14 10:50 ` [Qemu-devel] " Marcelo Tosatti 2011-09-19 9:55 ` Zhi Yong Wu [this message] 2011-09-19 9:55 ` Zhi Yong Wu 2011-09-20 12:34 ` Marcelo Tosatti 2011-09-20 12:34 ` [Qemu-devel] " Marcelo Tosatti 2011-09-21 3:14 ` Zhi Yong Wu 2011-09-21 3:14 ` [Qemu-devel] " Zhi Yong Wu 2011-09-21 5:54 ` Zhi Yong Wu 2011-09-21 5:54 ` [Qemu-devel] " Zhi Yong Wu 2011-09-21 7:03 ` Zhi Yong Wu 2011-09-21 7:03 ` [Qemu-devel] " Zhi Yong Wu 2011-09-26 8:15 ` Zhi Yong Wu 2011-09-26 8:15 ` [Qemu-devel] " Zhi Yong Wu 2011-09-23 16:19 ` Kevin Wolf 2011-09-23 16:19 ` [Qemu-devel] " Kevin Wolf 2011-09-26 7:24 ` Zhi Yong Wu 2011-09-26 7:24 ` [Qemu-devel] " Zhi Yong Wu 2011-10-17 10:26 ` Kevin Wolf 2011-10-17 10:26 ` Kevin Wolf 2011-10-17 15:54 ` Stefan Hajnoczi 2011-10-17 15:54 ` Stefan Hajnoczi 2011-10-18 8:29 ` Zhi Yong Wu 2011-10-18 8:29 ` Zhi Yong Wu 2011-10-18 8:43 ` Zhi Yong Wu 2011-10-18 8:43 ` Zhi Yong Wu 2011-09-08 10:11 ` [PATCH v8 4/4] qmp/hmp: add block_set_io_throttle Zhi Yong Wu 2011-09-08 10:11 ` [Qemu-devel] " Zhi Yong Wu -- strict thread matches above, loose matches on Subject: below -- 2011-09-07 12:41 [PATCH v8 3/4] block: add block timer and throttling algorithm Zhi Yong Wu
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=CAEH94LhRJ4QTfobxVt0XUp4_Wrp9zRJ+B_bGp_ELcbMgkwpu7w@mail.gmail.com \ --to=zwu.kernel@gmail.com \ --cc=aliguori@us.ibm.com \ --cc=kvm@vger.kernel.org \ --cc=kwolf@redhat.com \ --cc=mtosatti@redhat.com \ --cc=pair@us.ibm.com \ --cc=qemu-devel@nongnu.org \ --cc=ryanh@us.ibm.com \ --cc=stefanha@linux.vnet.ibm.com \ --cc=wuzhy@linux.vnet.ibm.com \ /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: linkBe 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.