From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([208.118.235.92]:41608) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1UIhzc-00020h-8e for qemu-devel@nongnu.org; Thu, 21 Mar 2013 12:05:31 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1UIhzT-0005wi-Hj for qemu-devel@nongnu.org; Thu, 21 Mar 2013 12:05:28 -0400 Received: from mx1.redhat.com ([209.132.183.28]:20297) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1UIhzT-0005wA-8F for qemu-devel@nongnu.org; Thu, 21 Mar 2013 12:05:19 -0400 From: Orit Wasserman Date: Thu, 21 Mar 2013 18:05:40 +0200 Message-Id: <1363881940-27505-10-git-send-email-owasserm@redhat.com> In-Reply-To: <1363881940-27505-1-git-send-email-owasserm@redhat.com> References: <1363881940-27505-1-git-send-email-owasserm@redhat.com> Subject: [Qemu-devel] [PATCH v3 9/9] coalesce adjacent iovecs List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: qemu-devel@nongnu.org Cc: Orit Wasserman , pbonzini@redhat.com, mst@redhat.com, chegu_vinod@hp.com, quintela@redhat.com This way we send one big buffer instead of many small ones Signed-off-by: Orit Wasserman --- savevm.c | 21 +++++++++++++++++---- 1 file changed, 17 insertions(+), 4 deletions(-) diff --git a/savevm.c b/savevm.c index 50e8fb2..13a533b 100644 --- a/savevm.c +++ b/savevm.c @@ -634,8 +634,14 @@ void qemu_put_buffer_no_copy(QEMUFile *f, const uint8_t *buf, int size) abort(); } - f->iov[f->iovcnt].iov_base = (uint8_t *)buf; - f->iov[f->iovcnt++].iov_len = size; + /* check for adjoint buffer and colace them */ + if (f->iovcnt > 0 && buf == f->iov[f->iovcnt - 1].iov_base + + f->iov[f->iovcnt - 1].iov_len) { + f->iov[f->iovcnt - 1].iov_len += size; + } else { + f->iov[f->iovcnt].iov_base = (uint8_t *)buf; + f->iov[f->iovcnt++].iov_len = size; + } f->is_write = 1; f->bytes_xfer += size; @@ -690,8 +696,15 @@ void qemu_put_byte(QEMUFile *f, int v) f->buf[f->buf_index++] = v; f->is_write = 1; f->bytes_xfer += 1; - f->iov[f->iovcnt].iov_base = f->buf + (f->buf_index - 1); - f->iov[f->iovcnt++].iov_len = 1; + + /* check for adjoint buffer and colace them */ + if (f->iovcnt > 0 && f->buf + (f->buf_index - 1) == + f->iov[f->iovcnt - 1].iov_base + f->iov[f->iovcnt - 1].iov_len) { + f->iov[f->iovcnt - 1].iov_len += 1; + } else { + f->iov[f->iovcnt].iov_base = f->buf + (f->buf_index - 1); + f->iov[f->iovcnt++].iov_len = 1; + } if (f->buf_index >= IO_BUF_SIZE || f->iovcnt >= MAX_IOV_SIZE) { qemu_fflush(f); -- 1.7.11.7