Re: [Qemu-devel] [PATCH v5] hmp: add info iothreads command

2015-07-30 Thread Ting Wang
On 2015-7-31 0:05, Luiz Capitulino wrote: > On Fri, 26 Jun 2015 16:07:13 +0800 > Ting Wang wrote: > >> Make "info iothreads" available on the HMP monitor. >> >> For example, the results are as follows when executing qemu >> command with "-objec

Re: [Qemu-devel] [PATCH] blockjob: add block_job_release function

2015-07-05 Thread Ting Wang
Ping? Regards, Ting On 2015-6-26 17:37, Ting Wang wrote: > There is job resource leak in function mirror_start_job, > although bdrv_create_dirty_bitmap is unlikely failed. > Add block_job_release for each release when needed. > > Signed-off-by: Ting Wang > --- > block/mi

Re: [Qemu-devel] [PATCH v5] hmp: add info iothreads command

2015-07-05 Thread Ting Wang
On 2015-7-3 20:43, Markus Armbruster wrote: > Copying the HMP maintainer Luiz. You can feed your patch to > scripts/get_maintainer to find people to cc yourself. OK, thanks. Ting > > Ting Wang writes: > >> Make "info iothreads" available on the HMP monitor. >

[Qemu-devel] [PATCH] blockjob: add block_job_release function

2015-06-26 Thread Ting Wang
There is job resource leak in function mirror_start_job, although bdrv_create_dirty_bitmap is unlikely failed. Add block_job_release for each release when needed. Signed-off-by: Ting Wang --- block/mirror.c | 2 ++ blockjob.c | 20 include/block

[Qemu-devel] [PATCH v5] hmp: add info iothreads command

2015-06-26 Thread Ting Wang
Make "info iothreads" available on the HMP monitor. For example, the results are as follows when executing qemu command with "-object iothread,id=iothread-1 -object iothread,id=iothread-2". (qemu) info iothreads iothread-1: thread_id=123 iothread-2: thread_id=456 Sign

Re: [Qemu-devel] [PATCH v4] hmp: add info iothreads command

2015-06-25 Thread Ting Wang
On 2015-6-23 19:57, Markus Armbruster wrote: > Ting Wang writes: > >> Hi Luiz and Markus, >> >> Would you like to pick up this patch, which has >> been reviewed by Stefan and Fam? > > Looks like this fell through the cracks back in March. You should've

Re: [Qemu-devel] [PATCH v4] hmp: add info iothreads command

2015-06-22 Thread Ting Wang
Hi Luiz and Markus, Would you like to pick up this patch, which has been reviewed by Stefan and Fam? Thanks. Ting On 2015-3-13 16:58, Ting Wang wrote: > Make "info iothreads" available on the HMP monitor. > > The results are as follows: > id1: thread_id=thread_id1 >

[Qemu-devel] [PATCH] nvme: Fix memleak in nvme_dma_read_prp

2015-06-18 Thread Ting Wang
From: Lu Lina Signed-off-by: Lu Lina --- hw/block/nvme.c | 1 + 1 file changed, 1 insertion(+) diff --git a/hw/block/nvme.c b/hw/block/nvme.c index 4b6d5e6..c6a6a0e 100644 --- a/hw/block/nvme.c +++ b/hw/block/nvme.c @@ -154,6 +154,7 @@ static uint16_t nvme_dma_read_prp(NvmeCtrl *n, uint8_t *p

[Qemu-devel] [PATCH] virtio-scsi-dataplane: fix memory leak for VirtIOSCSIVring

2015-03-26 Thread Ting Wang
VirtIOSCSIVring which allocated in virtio_scsi_vring_init should be free when dataplane has been stopped or failed to start. Signed-off-by: Ting Wang --- hw/scsi/virtio-scsi-dataplane.c | 6 ++ 1 file changed, 6 insertions(+) diff --git a/hw/scsi/virtio-scsi-dataplane.c b/hw/scsi/virtio

[Qemu-devel] [PATCH v4] hmp: add info iothreads command

2015-03-13 Thread Ting Wang
Make "info iothreads" available on the HMP monitor. The results are as follows: id1: thread_id=thread_id1 id2: thread_id=thread_id2 Signed-off-by: Ting Wang --- v4: use the PRId64 format specifier macro for the int64_t thread_id v3: fix comment and the trailing whitespace v2: add bra

Re: [Qemu-devel] [PATCH v2] hmp: add info iothreads command

2015-03-12 Thread Ting Wang
On 2015-3-12 22:03, Stefan Hajnoczi wrote: > On Thu, Mar 12, 2015 at 04:35:24PM +0800, Ting Wang wrote: >> +void hmp_info_iothreads(Monitor *mon, const QDict *qdict) >> +{ >> +IOThreadInfoList *head = NULL, *elem = NULL; >> + >> +head = qmp_query_i

[Qemu-devel] [PATCH v3] hmp: add info iothreads command

2015-03-12 Thread Ting Wang
Make "info iothreads" available on the HMP monitor. The results are as follows: id1: thread_id=thread_id1 id2: thread_id=thread_id2 Signed-off-by: Ting Wang --- v3: fix comment and the trailing whitespace v2: add braces for if --- hmp-commands.hx | 2 ++ hmp.c

Re: [Qemu-devel] [PATCH v2] hmp: add info iothreads command

2015-03-12 Thread Ting Wang
On 2015-3-12 17:05, Fam Zheng wrote: > On Thu, 03/12 16:35, Ting Wang wrote: >> Make "info iothreads" available on the HMP monitor. >> >> The results are as follows: >> id1: thread_id1 >> id2: thread_id2 > > Does this match the actual output form

Re: [Qemu-devel] [PATCH v2] hmp: add info iothreads command

2015-03-12 Thread Ting Wang
On 2015-3-12 17:05, Fam Zheng wrote: > On Thu, 03/12 16:35, Ting Wang wrote: >> Make "info iothreads" available on the HMP monitor. >> >> The results are as follows: >> id1: thread_id1 >> id2: thread_id2 > > Does this match the actual output form

[Qemu-devel] [PATCH v2] hmp: add info iothreads command

2015-03-12 Thread Ting Wang
Make "info iothreads" available on the HMP monitor. The results are as follows: id1: thread_id1 id2: thread_id2 Signed-off-by: Ting Wang --- v2: add braces for if --- hmp-commands.hx | 2 ++ hmp.c | 22 ++ hmp.h | 1 + monitor.c | 7

[Qemu-devel] [PATCH] hmp: add info iothreads command

2015-03-12 Thread Ting Wang
Make "info iothreads" available on the HMP monitor. The results are as follows: id1: thread_id1 id2: thread_id2 Signed-off-by: Ting Wang --- hmp-commands.hx | 2 ++ hmp.c | 21 + hmp.h | 1 + monitor.c | 7 +++ 4 files changed, 31

[Qemu-devel] [PATCH] qdev: The sequence of devices added to bus

2015-01-18 Thread Ting Wang
lun 0 is always named as sda, device with lun 1 is named as sdc, and device with lun 2 is named as sdb. So the order is different from the lun number. Signed-off-by: Ting Wang --- hw/core/qdev.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/hw/core/qdev.c b/hw/core/qdev.c

[Qemu-devel] [PATCH] NetKVM: fix for indirectc mode when vring is full

2014-12-09 Thread Ting Wang
In function vring_add_indirect, there is no limiti about free entry in vring. If vring is full, vq->num_free will be less than zero, and the address of vq->vring.desc becomes illegal. Signed-off-by: Ting Wang --- NetKVM/NDIS5/VirtIO/VirtIORing.c | 2 +- 1 file changed, 1 insertion

[Qemu-devel] [PATCH] virtio-scsi: sense in virtio_scsi_command_complete

2014-10-27 Thread Ting Wang
If req->resp.cmd.status is not GOOD, the address of sense for qemu_iovec_from_buf should be modified from &req->resp to sense. Signed-off-by: Ting Wang --- hw/scsi/virtio-scsi.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/hw/scsi/virtio-scsi.c b/hw/scsi/vir