Re: [Qemu-devel] [PATCH v2 3/5] migration: remove unnecessary variables len in QIOChannelRDMA

2018-04-27 Thread Daniel P . Berrangé
On Wed, Apr 25, 2018 at 10:35:32PM +0800, Lidong Chen wrote: > Because qio_channel_rdma_writev and qio_channel_rdma_readv maybe invoked > by different threads concurrently, this patch removes unnecessary variables > len in QIOChannelRDMA and use local variable instead. > > Signed-off-by: Lidong Ch

Re: [Qemu-devel] [PATCH v2 3/5] migration: remove unnecessary variables len in QIOChannelRDMA

2018-04-27 Thread Daniel P . Berrangé
On Thu, Apr 26, 2018 at 05:40:11PM +0100, Dr. David Alan Gilbert wrote: > * Lidong Chen (jemmy858...@gmail.com) wrote: > > Because qio_channel_rdma_writev and qio_channel_rdma_readv maybe invoked > > by different threads concurrently, this patch removes unnecessary variables > > len in QIOChannelRD

Re: [Qemu-devel] [PATCH v2 3/5] migration: remove unnecessary variables len in QIOChannelRDMA

2018-04-26 Thread 858585 jemmy
On Fri, Apr 27, 2018 at 12:40 AM, Dr. David Alan Gilbert wrote: > * Lidong Chen (jemmy858...@gmail.com) wrote: >> Because qio_channel_rdma_writev and qio_channel_rdma_readv maybe invoked >> by different threads concurrently, this patch removes unnecessary variables >> len in QIOChannelRDMA and use

Re: [Qemu-devel] [PATCH v2 3/5] migration: remove unnecessary variables len in QIOChannelRDMA

2018-04-26 Thread Dr. David Alan Gilbert
* Lidong Chen (jemmy858...@gmail.com) wrote: > Because qio_channel_rdma_writev and qio_channel_rdma_readv maybe invoked > by different threads concurrently, this patch removes unnecessary variables > len in QIOChannelRDMA and use local variable instead. > > Signed-off-by: Lidong Chen I'm OK with

[Qemu-devel] [PATCH v2 3/5] migration: remove unnecessary variables len in QIOChannelRDMA

2018-04-25 Thread Lidong Chen
Because qio_channel_rdma_writev and qio_channel_rdma_readv maybe invoked by different threads concurrently, this patch removes unnecessary variables len in QIOChannelRDMA and use local variable instead. Signed-off-by: Lidong Chen --- migration/rdma.c | 15 +++ 1 file changed, 7 inser