All of lore.kernel.org
 help / color / mirror / Atom feed
From: Michael Tokarev <mjt@tls.msk.ru>
To: Anthony Liguori <aliguori@us.ibm.com>
Cc: qemu-devel <qemu-devel@nongnu.org>
Subject: [Qemu-devel] [PULL] iov* function rework (one more time)
Date: Thu, 07 Jun 2012 21:53:04 +0400	[thread overview]
Message-ID: <4FD0EA80.1000302@msgid.tls.msk.ru> (raw)

The following changes since commit 083dbf489d1b0592e910ecfb90b3858c23e49ab7:

  target-microblaze: fix swx build breakage (2012-06-07 10:12:00 +0200)

are available in the git repository at:

  git://git.corpit.ru/qemu.git mjt-iov

for you to fetch changes up to 85c5ece9bc0bd30f0573e43d1ddd756154ac6849:

  rewrite iov_send_recv() and move it to iov.c (2012-06-07 21:22:54 +0400)

----------------------------------------------------------------
Michael Tokarev (11):
      virtio-serial-bus: use correct lengths in control_out() message
      change iov_* function prototypes to be more appropriate
      rewrite iov_* functions
      consolidate qemu_iovec_memset{,_skip}() into single function and use existing iov_memset()
      allow qemu_iovec_from_buffer() to specify offset from which to start copying
      consolidate qemu_iovec_copy() and qemu_iovec_concat() and make them consistent
      change qemu_iovec_to_buf() to match other to,from_buf functions
      rename qemu_sendv to iov_send, change proto and move declarations to iov.h
      export iov_send_recv() and use it in iov_send() and iov_recv()
      cleanup qemu_co_sendv(), qemu_co_recvv() and friends
      rewrite iov_send_recv() and move it to iov.c

This is the same patchset which has been submitted, resubmitted,
rewritten, resubmitted, split, resubmitted, merged, rewritten again,
resubmitted and finally received a promise from Anthony to be looked
at.  Today, months later, I'm still awaiiting for the answer, and,
while I said before that the last resubmission was, well, last,
since I bothered so many people so many times, I decided to break
my promise, rebase against current qemy/master and resend it,
now as a git pull request, to not mailbomb people again.  The rebase
took quite some work again, since the code changed (obviously) since
the patchset was ready before.  During rebase I reviewed all the
commit messages too, and removed all the Cc lines they had.

As far as I remember, all comments about the patchset has been
addressed or answered, the last event was the above mentioned
promise.

I ensured that each commit in the series compiles and works (this
time I didn't perform extensive tests as before, just very basic
tests), so it is bisectable as before.

The description for the patchset is the same as before, can be found
at http://thread.gmane.org/gmane.comp.emulators.qemu/142079 .
New diffstat is below.

The patchest can be browsed online at
http://git.corpit.ru/?p=qemu.git;a=shortlog;h=refs/heads/mjt-iov

Please pull.

Thanks,

/mjt

 Makefile.objs          |    2 +-
 block.c                |   12 ++---
 block/curl.c           |    6 +--
 block/iscsi.c          |    3 +-
 block/nbd.c            |   18 ++++---
 block/qcow.c           |    4 +-
 block/qcow2.c          |   21 ++++----
 block/qed.c            |   10 ++--
 block/rbd.c            |    4 +-
 block/sheepdog.c       |    6 +--
 cutils.c               |  234 ++++++++++++---------------------------------------------------------------------------
 hw/9pfs/virtio-9p.c    |    8 +--
 hw/rtl8139.c           |    2 +-
 hw/usb/core.c          |    6 +--
 hw/virtio-balloon.c    |    4 +-
 hw/virtio-net.c        |    4 +-
 hw/virtio-serial-bus.c |   10 ++--
 iov.c                  |  192 ++++++++++++++++++++++++++++++++++++++++++++++++++++-------------------
 iov.h                  |   77 ++++++++++++++++++++++++++---
 linux-aio.c            |    4 +-
 net.c                  |    2 +-
 posix-aio-compat.c     |    8 ++-
 qemu-common.h          |   56 ++++++++++-----------
 qemu-coroutine-io.c    |   83 ++++++++++---------------------
 tests/Makefile         |    2 +
 tests/test-iov.c       |  260 +++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++
 26 files changed, 620 insertions(+), 418 deletions(-)
 create mode 100644 tests/test-iov.c

             reply	other threads:[~2012-06-07 17:53 UTC|newest]

Thread overview: 8+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2012-06-07 17:53 Michael Tokarev [this message]
2012-06-07 23:20 ` [Qemu-devel] [PULL] iov* function rework (one more time) Paolo Bonzini
2012-06-10 15:50   ` Paolo Bonzini
2012-06-08 10:13 ` Kevin Wolf
2012-06-11 18:31 ` Anthony Liguori
2012-06-11 19:19   ` Michael Tokarev
2012-07-09 17:14     ` Michael Tokarev
2012-07-10 13:37     ` Anthony Liguori

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=4FD0EA80.1000302@msgid.tls.msk.ru \
    --to=mjt@tls.msk.ru \
    --cc=aliguori@us.ibm.com \
    --cc=qemu-devel@nongnu.org \
    /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.