[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-block] [PATCH v3] virtio-blk: trivial code optimization
From: |
Paolo Bonzini |
Subject: |
Re: [Qemu-block] [PATCH v3] virtio-blk: trivial code optimization |
Date: |
Mon, 9 Nov 2015 14:28:59 +0100 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:38.0) Gecko/20100101 Thunderbird/38.3.0 |
On 09/11/2015 10:03, address@hidden wrote:
> From: Gonglei <address@hidden>
>
> 1. avoid possible superflous checking
> 2. make code more robustness
>
> Signed-off-by: Gonglei <address@hidden>
> Reviewed-by: Fam Zheng <address@hidden>
> ---
> v3: change the third condition too [Paolo]
> add Fam's R-by
> ---
> hw/block/virtio-blk.c | 27 +++++++++------------------
> 1 file changed, 9 insertions(+), 18 deletions(-)
>
> diff --git a/hw/block/virtio-blk.c b/hw/block/virtio-blk.c
> index 093e475..9124358 100644
> --- a/hw/block/virtio-blk.c
> +++ b/hw/block/virtio-blk.c
> @@ -404,24 +404,15 @@ void virtio_blk_submit_multireq(BlockBackend *blk,
> MultiReqBuffer *mrb)
> for (i = 0; i < mrb->num_reqs; i++) {
> VirtIOBlockReq *req = mrb->reqs[i];
> if (num_reqs > 0) {
> - bool merge = true;
> -
> - /* merge would exceed maximum number of IOVs */
> - if (niov + req->qiov.niov > IOV_MAX) {
> - merge = false;
> - }
> -
> - /* merge would exceed maximum transfer length of backend device
> */
> - if (req->qiov.size / BDRV_SECTOR_SIZE + nb_sectors >
> max_xfer_len) {
> - merge = false;
> - }
> -
> - /* requests are not sequential */
> - if (sector_num + nb_sectors != req->sector_num) {
> - merge = false;
> - }
> -
> - if (!merge) {
> + /*
> + * NOTE: We cannot merge the requests in below situations:
> + * 1. requests are not sequential
> + * 2. merge would exceed maximum number of IOVs
> + * 3. merge would exceed maximum transfer length of backend
> device
> + */
> + if (sector_num + nb_sectors != req->sector_num ||
> + niov > IOV_MAX - req->qiov.niov ||
> + nb_sectors > max_xfer_len - req->qiov.size /
> BDRV_SECTOR_SIZE) {
> submit_requests(blk, mrb, start, num_reqs, niov);
> num_reqs = 0;
> }
>
Reviewed-by: Paolo Bonzini <address@hidden>