Markus Armbruster wrote:
> Juan Quintela writes:
...
>> As qmp command is asynchronous, you can think that -d is *always* on in
>> QMP O:-)
>
> Yes. The existence of "detach" in QMP is owed to limitations of early
> QMP infrastructure. It's flagged as
Juan Quintela writes:
> Markus Armbruster wrote:
>> Juan Quintela writes:
>>
>>> Eric Blake wrote:
>
>
Or is the proposal that we are also going to simplify the QMP 'migrate'
command to get rid of crufty
* Juan Quintela (quint...@redhat.com) wrote:
> "Dr. David Alan Gilbert" wrote:
>
> >> Forget -b/-i.
> >>
> >> migration_set_parameter compression_threads 8
> >>
> >> migrate
> >>
> >> We don't use compression_threads at all
> >>
> >> migrate_set_capability compress
> >>
"Dr. David Alan Gilbert" wrote:
>> Forget -b/-i.
>>
>> migration_set_parameter compression_threads 8
>>
>> migrate
>>
>> We don't use compression_threads at all
>>
>> migrate_set_capability compress
>>
>> migrate
>>
>> Now, we use compression threads.
>>
>> So,
* Juan Quintela (quint...@redhat.com) wrote:
> "Dr. David Alan Gilbert" wrote:
> > * Juan Quintela (quint...@redhat.com) wrote:
> >> "Dr. David Alan Gilbert" wrote:
> >> > * Juan Quintela (quint...@redhat.com) wrote:
> >> >> Markus Armbruster
"Dr. David Alan Gilbert" wrote:
> * Juan Quintela (quint...@redhat.com) wrote:
>> "Dr. David Alan Gilbert" wrote:
>> > * Juan Quintela (quint...@redhat.com) wrote:
>> >> Markus Armbruster wrote:
>> >> > Juan Quintela
* Juan Quintela (quint...@redhat.com) wrote:
> "Dr. David Alan Gilbert" wrote:
> > * Juan Quintela (quint...@redhat.com) wrote:
> >> Markus Armbruster wrote:
> >> > Juan Quintela writes:
> >> >
> >> >> Eric Blake
"Dr. David Alan Gilbert" wrote:
> * Juan Quintela (quint...@redhat.com) wrote:
>> Markus Armbruster wrote:
>> > Juan Quintela writes:
>> >
>> >> Eric Blake wrote:
>>
>>
>> >>> Or is the proposal that we are also
* Juan Quintela (quint...@redhat.com) wrote:
> Markus Armbruster wrote:
> > Juan Quintela writes:
> >
> >> Eric Blake wrote:
>
>
> >>> Or is the proposal that we are also going to simplify the QMP 'migrate'
> >>> command to get rid of
Markus Armbruster wrote:
> Juan Quintela writes:
>
>> Eric Blake wrote:
>>> Or is the proposal that we are also going to simplify the QMP 'migrate'
>>> command to get rid of crufty parameters?
>>
>> I didn't read it that way, but I
Juan Quintela writes:
> Eric Blake wrote:
>> On 05/12/2017 05:55 AM, Juan Quintela wrote:
> @@ -1239,6 +1240,7 @@ void qmp_migrate(const char *uri, bool has_blk,
> bool blk,
> }
>
> if (has_inc && inc) {
> +
Eric Blake wrote:
> [adding Markus]
>
> On 05/15/2017 04:48 AM, Juan Quintela wrote:
>> Eric Blake wrote:
>>> On 05/12/2017 05:55 AM, Juan Quintela wrote:
>> @@ -1239,6 +1240,7 @@ void qmp_migrate(const char *uri, bool has_blk,
>> bool blk,
>>
[adding Markus]
On 05/15/2017 04:48 AM, Juan Quintela wrote:
> Eric Blake wrote:
>> On 05/12/2017 05:55 AM, Juan Quintela wrote:
> @@ -1239,6 +1240,7 @@ void qmp_migrate(const char *uri, bool has_blk,
> bool blk,
> }
>
> if (has_inc && inc) {
* Juan Quintela (quint...@redhat.com) wrote:
> Eric Blake wrote:
> > On 05/12/2017 05:55 AM, Juan Quintela wrote:
> @@ -1239,6 +1240,7 @@ void qmp_migrate(const char *uri, bool has_blk,
> bool blk,
> }
>
> if (has_inc && inc) {
> +
On Fri, May 12, 2017 at 12:55:22PM +0200, Juan Quintela wrote:
> Peter Xu wrote:
> > On Thu, May 11, 2017 at 06:32:27PM +0200, Juan Quintela wrote:
>
> >> @@ -1214,9 +1218,6 @@ void qmp_migrate(const char *uri, bool has_blk, bool
> >> blk,
> >> MigrationParams params;
>
Eric Blake wrote:
> On 05/12/2017 05:55 AM, Juan Quintela wrote:
@@ -1239,6 +1240,7 @@ void qmp_migrate(const char *uri, bool has_blk, bool
blk,
}
if (has_inc && inc) {
+migrate_set_block_enabled(s, true);
On 05/12/2017 05:55 AM, Juan Quintela wrote:
>>> @@ -1239,6 +1240,7 @@ void qmp_migrate(const char *uri, bool has_blk, bool
>>> blk,
>>> }
>>>
>>> if (has_inc && inc) {
>>> +migrate_set_block_enabled(s, true);
>>> migrate_set_block_shared(s, true);
>>
>> [2]
>>
>>
Peter Xu wrote:
> On Thu, May 11, 2017 at 06:32:27PM +0200, Juan Quintela wrote:
>> @@ -1214,9 +1218,6 @@ void qmp_migrate(const char *uri, bool has_blk, bool
>> blk,
>> MigrationParams params;
>> const char *p;
>>
>> -params.blk = has_blk && blk;
>> -
On Thu, May 11, 2017 at 06:32:27PM +0200, Juan Quintela wrote:
> We have change in the previous patch to use migration capabilities for
> it. Notice that we continue using the old command line flags from
> migrate command from the time being. Remove the set_params method as
> now it is empty.
>
On 2017/5/12 0:33, Juan Quintela wrote:
Hailiang Zhang wrote:
Hi,
Hmm you don't seem to have replaced this with anything.
I think that's a behavioural change; the trick COLO did (I'm not sure if this
is still the way it works) is that they initiate the first
Hailiang Zhang wrote:
> Hi,
>
>>> Hmm you don't seem to have replaced this with anything.
>>> I think that's a behavioural change; the trick COLO did (I'm not sure if
>>> this
>>> is still the way it works) is that they initiate the first migration
>>> with block
We have change in the previous patch to use migration capabilities for
it. Notice that we continue using the old command line flags from
migrate command from the time being. Remove the set_params method as
now it is empty.
Signed-off-by: Juan Quintela
---
Hi,
On 2017/5/4 16:51, Juan Quintela wrote:
"Dr. David Alan Gilbert" wrote:
* Juan Quintela (quint...@redhat.com) wrote:
We have change in the previous patch to use migration capabilities for
it. Notice that we continue using the old command line flags from
migrate
"Dr. David Alan Gilbert" wrote:
> * Juan Quintela (quint...@redhat.com) wrote:
>> We have change in the previous patch to use migration capabilities for
>> it. Notice that we continue using the old command line flags from
>> migrate command from the time being. Remove the
On 04/25/2017 05:30 AM, Juan Quintela wrote:
> We have change in the previous patch to use migration capabilities for
> it. Notice that we continue using the old command line flags from
> migrate command from the time being. Remove the set_params method as
> now it is empty.
>
> Signed-off-by:
* Juan Quintela (quint...@redhat.com) wrote:
> We have change in the previous patch to use migration capabilities for
> it. Notice that we continue using the old command line flags from
> migrate command from the time being. Remove the set_params method as
> now it is empty.
>
> Signed-off-by:
We have change in the previous patch to use migration capabilities for
it. Notice that we continue using the old command line flags from
migrate command from the time being. Remove the set_params method as
now it is empty.
Signed-off-by: Juan Quintela
---
27 matches
Mail list logo