Re: [Qemu-devel] [PATCH 1/1] qemu_char: socket backend: disconnect on EPIPE

2017-02-02 Thread Daniel P. Berrange
On Thu, Feb 02, 2017 at 02:43:18PM +0300, Denis V. Lunev wrote: > On 02/02/2017 12:47 PM, Daniel P. Berrange wrote: > > On Thu, Feb 02, 2017 at 11:59:22AM +0300, Denis V. Lunev wrote: > >> On 02/01/2017 04:53 PM, Daniel P. Berrange wrote: > >>> On Wed, Feb 01, 2017 at 04:15:28PM +0300, Denis V. Lun

Re: [Qemu-devel] [PATCH 1/1] qemu_char: socket backend: disconnect on EPIPE

2017-02-02 Thread Denis V. Lunev
On 02/02/2017 12:47 PM, Daniel P. Berrange wrote: > On Thu, Feb 02, 2017 at 11:59:22AM +0300, Denis V. Lunev wrote: >> On 02/01/2017 04:53 PM, Daniel P. Berrange wrote: >>> On Wed, Feb 01, 2017 at 04:15:28PM +0300, Denis V. Lunev wrote: From: Anton Nefedov Socket backend read handle

Re: [Qemu-devel] [PATCH 1/1] qemu_char: socket backend: disconnect on EPIPE

2017-02-02 Thread Daniel P. Berrange
On Wed, Feb 01, 2017 at 04:54:28PM -0800, Paolo Bonzini wrote: > > > On 01/02/2017 05:53, Daniel P. Berrange wrote: > >> > >> +if (ret < 0 && errno == EPIPE) { > > IMHO you should just remove "&& errno == EPIPE" and it'd be fine. > > It should be errno != EAGAIN. Oh, yes, thats right.

Re: [Qemu-devel] [PATCH 1/1] qemu_char: socket backend: disconnect on EPIPE

2017-02-02 Thread Daniel P. Berrange
On Thu, Feb 02, 2017 at 11:59:22AM +0300, Denis V. Lunev wrote: > On 02/01/2017 04:53 PM, Daniel P. Berrange wrote: > > On Wed, Feb 01, 2017 at 04:15:28PM +0300, Denis V. Lunev wrote: > >> From: Anton Nefedov > >> > >> Socket backend read handler should normally perform a disconnect, however > >>

Re: [Qemu-devel] [PATCH 1/1] qemu_char: socket backend: disconnect on EPIPE

2017-02-02 Thread Denis V. Lunev
On 02/01/2017 04:53 PM, Daniel P. Berrange wrote: > On Wed, Feb 01, 2017 at 04:15:28PM +0300, Denis V. Lunev wrote: >> From: Anton Nefedov >> >> Socket backend read handler should normally perform a disconnect, however >> the read handler may not get a chance to run if the frontend is not ready >>

Re: [Qemu-devel] [PATCH 1/1] qemu_char: socket backend: disconnect on EPIPE

2017-02-01 Thread Paolo Bonzini
On 01/02/2017 05:53, Daniel P. Berrange wrote: >> >> +if (ret < 0 && errno == EPIPE) { > IMHO you should just remove "&& errno == EPIPE" and it'd be fine. It should be errno != EAGAIN. Paolo >> +if (tcp_chr_read_poll(chr) <= 0) { >> +tcp_chr_disconnect(chr

Re: [Qemu-devel] [PATCH 1/1] qemu_char: socket backend: disconnect on EPIPE

2017-02-01 Thread Daniel P. Berrange
On Wed, Feb 01, 2017 at 04:15:28PM +0300, Denis V. Lunev wrote: > From: Anton Nefedov > > Socket backend read handler should normally perform a disconnect, however > the read handler may not get a chance to run if the frontend is not ready > (qemu_chr_be_can_write == 0). > > This means that in v