Re: [Qemu-devel] [PULL 07/12] migration: Start of multiple fd work

2017-02-14 Thread Paolo Bonzini
On 14/02/2017 14:52, Juan Quintela wrote: > will get some general documentation in /doc/. > > Basically what we had on the only stream was: > > > {[page header][page]}+ > > > And we moved to: > > {[page header]+[where to receive]}: on the principal stream > > [page]+: on the rest of the mu

Re: [Qemu-devel] [PULL 07/12] migration: Start of multiple fd work

2017-02-14 Thread Juan Quintela
Paolo Bonzini wrote: > On 14/02/2017 14:12, Juan Quintela wrote: >>> On 13/02/2017 18:19, Juan Quintela wrote: +qemu_sem_init(&p->init, 0); p->quit = false; +p->c = socket_send_channel_create(); +if (!p->c) { +error_report("Erro

Re: [Qemu-devel] [PULL 07/12] migration: Start of multiple fd work

2017-02-14 Thread Paolo Bonzini
On 14/02/2017 14:12, Juan Quintela wrote: >> On 13/02/2017 18:19, Juan Quintela wrote: >>> +qemu_sem_init(&p->init, 0); >>> p->quit = false; >>> +p->c = socket_send_channel_create(); >>> +if (!p->c) { >>> +error_report("Error creating a send channel");

Re: [Qemu-devel] [PULL 07/12] migration: Start of multiple fd work

2017-02-14 Thread Juan Quintela
Paolo Bonzini wrote: > On 13/02/2017 18:19, Juan Quintela wrote: >> +qemu_sem_init(&p->init, 0); >> p->quit = false; >> +p->c = socket_send_channel_create(); >> +if (!p->c) { >> +error_report("Error creating a send channel"); >> +exit(0); >>

Re: [Qemu-devel] [PULL 07/12] migration: Start of multiple fd work

2017-02-14 Thread Paolo Bonzini
On 13/02/2017 18:19, Juan Quintela wrote: > +qemu_sem_init(&p->init, 0); > p->quit = false; > +p->c = socket_send_channel_create(); > +if (!p->c) { > +error_report("Error creating a send channel"); > +exit(0); > +} > snprin

Re: [Qemu-devel] [PULL 07/12] migration: Start of multiple fd work

2017-02-14 Thread Daniel P. Berrange
On Mon, Feb 13, 2017 at 06:19:43PM +0100, Juan Quintela wrote: > We create new channels for each new thread created. We only send through > them a character to be sure that we are creating the channels in the > right order. > > Signed-off-by: Juan Quintela > --- > include/migration/migration.h |