qemu-devel
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [Qemu-devel] [PATCH v2 09/12] sheepdog: move coroutine send/recv fun


From: Paolo Bonzini
Subject: Re: [Qemu-devel] [PATCH v2 09/12] sheepdog: move coroutine send/recv function to generic code
Date: Tue, 13 Sep 2011 17:16:53 +0200
User-agent: Mozilla/5.0 (X11; Linux x86_64; rv:6.0.2) Gecko/20110906 Thunderbird/6.0.2

On 09/13/2011 04:14 PM, Kevin Wolf wrote:
>    block/sheepdog.c |  225 
++++++------------------------------------------------
>    cutils.c         |  103 +++++++++++++++++++++++++
>    qemu-common.h    |    3 +
>    qemu-coroutine.c |   70 +++++++++++++++++
>    qemu-coroutine.h |   26 ++++++

Can we move the code somewhere else? This is not core coroutine
infrastructure. I would suggest qemu_socket.h/qemu-sockets.c.

It's not really socket-specific either (it uses recv/send only because of Windows brokenness---it could use read/write if it wasn't for that). I hoped sooner or later it could become a qemu_co_readv/writev, hence the choice of qemu-coroutine.c.

Paolo

ps: I also hope that the Earth will start spinning slower and will give me 32 hour days, so just tell me if you really want that outside qemu-coroutine.c.



reply via email to

[Prev in Thread] Current Thread [Next in Thread]