migration: simplify state assignmente
Once there, make sure that if we already know that there is one error, just call migration_fd_cleanup() with the ERROR state. Signed-off-by: Juan Quintela <quintela@redhat.com> Reviewed-by: Anthony Liguori <aliguori@us.ibm.com>
This commit is contained in:
		
							parent
							
								
									3a230256e8
								
							
						
					
					
						commit
						c0b2ba46c5
					
				
							
								
								
									
										11
									
								
								migration.c
								
								
								
								
							
							
						
						
									
										11
									
								
								migration.c
								
								
								
								
							| 
						 | 
				
			
			@ -371,7 +371,6 @@ void migrate_fd_put_ready(void *opaque)
 | 
			
		|||
 | 
			
		||||
    DPRINTF("iterate\n");
 | 
			
		||||
    if (qemu_savevm_state_iterate(s->mon, s->file) == 1) {
 | 
			
		||||
        int state;
 | 
			
		||||
        int old_vm_running = runstate_is_running();
 | 
			
		||||
 | 
			
		||||
        DPRINTF("done iterating\n");
 | 
			
		||||
| 
						 | 
				
			
			@ -381,20 +380,18 @@ void migrate_fd_put_ready(void *opaque)
 | 
			
		|||
            if (old_vm_running) {
 | 
			
		||||
                vm_start();
 | 
			
		||||
            }
 | 
			
		||||
            state = MIG_STATE_ERROR;
 | 
			
		||||
        } else {
 | 
			
		||||
            state = MIG_STATE_COMPLETED;
 | 
			
		||||
            s->state = MIG_STATE_ERROR;
 | 
			
		||||
        }
 | 
			
		||||
        if (migrate_fd_cleanup(s) < 0) {
 | 
			
		||||
            if (old_vm_running) {
 | 
			
		||||
                vm_start();
 | 
			
		||||
            }
 | 
			
		||||
            state = MIG_STATE_ERROR;
 | 
			
		||||
            s->state = MIG_STATE_ERROR;
 | 
			
		||||
        }
 | 
			
		||||
        if (state == MIG_STATE_COMPLETED) {
 | 
			
		||||
        if (s->state == MIG_STATE_ACTIVE) {
 | 
			
		||||
            s->state = MIG_STATE_COMPLETED;
 | 
			
		||||
            runstate_set(RUN_STATE_POSTMIGRATE);
 | 
			
		||||
        }
 | 
			
		||||
        s->state = state;
 | 
			
		||||
        notifier_list_notify(&migration_state_notifiers, NULL);
 | 
			
		||||
    }
 | 
			
		||||
}
 | 
			
		||||
| 
						 | 
				
			
			
 | 
			
		|||
		Loading…
	
		Reference in New Issue