RE: [PATCH v4 3/6] net/colo-compare.c: Fix deadlock in compare_chr_send

2020-05-11 Thread Zhang, Chen
> -Original Message- > From: Lukas Straub > Sent: Friday, May 8, 2020 3:56 PM > To: Zhang, Chen > Cc: qemu-devel ; Li Zhijian > ; Jason Wang ; Marc- > André Lureau ; Paolo Bonzini > > Subject: Re: [PATCH v4 3/6] net/colo-compare.c: Fix deadlock in > compa

Re: [PATCH v4 3/6] net/colo-compare.c: Fix deadlock in compare_chr_send

2020-05-08 Thread Lukas Straub
; Paolo Bonzini > > > > Subject: Re: [PATCH v4 3/6] net/colo-compare.c: Fix deadlock in > > compare_chr_send > > > > On Fri, 8 May 2020 02:19:00 + > > "Zhang, Chen" wrote: > > > > > No need to init the notify_sendco each time, bec

RE: [PATCH v4 3/6] net/colo-compare.c: Fix deadlock in compare_chr_send

2020-05-08 Thread Zhang, Chen
> -Original Message- > From: Lukas Straub > Sent: Friday, May 8, 2020 2:08 PM > To: Zhang, Chen > Cc: qemu-devel ; Li Zhijian > ; Jason Wang ; Marc- > André Lureau ; Paolo Bonzini > > Subject: Re: [PATCH v4 3/6] net/colo-compare.c: Fix deadlock in > c

Re: [PATCH v4 3/6] net/colo-compare.c: Fix deadlock in compare_chr_send

2020-05-08 Thread Lukas Straub
On Fri, 8 May 2020 02:19:00 + "Zhang, Chen" wrote: > > > No need to init the notify_sendco each time, because the notify dev just > > an optional parameter. > > > You can use the if (s->notify_dev) here. Just Xen use the chr_notify_dev. > > > > > > > Ok, I will change that and the

RE: [PATCH v4 3/6] net/colo-compare.c: Fix deadlock in compare_chr_send

2020-05-07 Thread Zhang, Chen
> -Original Message- > From: Lukas Straub > Sent: Thursday, May 7, 2020 11:51 PM > To: Zhang, Chen > Cc: qemu-devel ; Li Zhijian > ; Jason Wang ; Marc- > André Lureau ; Paolo Bonzini > > Subject: Re: [PATCH v4 3/6] net/colo-compare.c: Fix deadlock in > c

Re: [PATCH v4 3/6] net/colo-compare.c: Fix deadlock in compare_chr_send

2020-05-07 Thread Lukas Straub
eau ; Paolo Bonzini > > > > Subject: [PATCH v4 3/6] net/colo-compare.c: Fix deadlock in > > compare_chr_send > > > > The chr_out chardev is connected to a filter-redirector running in the > > main loop. qemu_chr_fe_write_all might block here in compare_chr_sen

RE: [PATCH v4 3/6] net/colo-compare.c: Fix deadlock in compare_chr_send

2020-05-07 Thread Zhang, Chen
> -Original Message- > From: Lukas Straub > Sent: Monday, May 4, 2020 6:28 PM > To: qemu-devel > Cc: Zhang, Chen ; Li Zhijian > ; Jason Wang ; Marc- > André Lureau ; Paolo Bonzini > > Subject: [PATCH v4 3/6] net/colo-compare.c: Fix deadlock in > com

[PATCH v4 3/6] net/colo-compare.c: Fix deadlock in compare_chr_send

2020-05-04 Thread Lukas Straub
The chr_out chardev is connected to a filter-redirector running in the main loop. qemu_chr_fe_write_all might block here in compare_chr_send if the (socket-)buffer is full. If another filter-redirector in the main loop want's to send data to chr_pri_in it might also block if the buffer is full.