dataplane: bail out on unsupported transport
If the virtio transport does not support notifiers (like s390-virtio), we can't use dataplane. Bail out early and let the user know what is wrong. Signed-off-by: Cornelia Huck <cornelia.huck@de.ibm.com> Reviewed-by: Stefan Hajnoczi <stefanha@redhat.com> Signed-off-by: Stefan Hajnoczi <stefanha@redhat.com>
This commit is contained in:
		
				
					committed by
					
						
						Stefan Hajnoczi
					
				
			
			
				
	
			
			
			
						parent
						
							dc80ca6cd6
						
					
				
				
					commit
					a9968c77d5
				
			@@ -127,6 +127,8 @@ void virtio_blk_data_plane_create(VirtIODevice *vdev, VirtIOBlkConf *blk,
 | 
				
			|||||||
    VirtIOBlockDataPlane *s;
 | 
					    VirtIOBlockDataPlane *s;
 | 
				
			||||||
    VirtIOBlock *vblk = VIRTIO_BLK(vdev);
 | 
					    VirtIOBlock *vblk = VIRTIO_BLK(vdev);
 | 
				
			||||||
    Error *local_err = NULL;
 | 
					    Error *local_err = NULL;
 | 
				
			||||||
 | 
					    BusState *qbus = BUS(qdev_get_parent_bus(DEVICE(vdev)));
 | 
				
			||||||
 | 
					    VirtioBusClass *k = VIRTIO_BUS_GET_CLASS(qbus);
 | 
				
			||||||
 | 
					
 | 
				
			||||||
    *dataplane = NULL;
 | 
					    *dataplane = NULL;
 | 
				
			||||||
 | 
					
 | 
				
			||||||
@@ -134,6 +136,14 @@ void virtio_blk_data_plane_create(VirtIODevice *vdev, VirtIOBlkConf *blk,
 | 
				
			|||||||
        return;
 | 
					        return;
 | 
				
			||||||
    }
 | 
					    }
 | 
				
			||||||
 | 
					
 | 
				
			||||||
 | 
					    /* Don't try if transport does not support notifiers. */
 | 
				
			||||||
 | 
					    if (!k->set_guest_notifiers || !k->set_host_notifier) {
 | 
				
			||||||
 | 
					        error_setg(errp,
 | 
				
			||||||
 | 
					                   "device is incompatible with x-data-plane "
 | 
				
			||||||
 | 
					                   "(transport does not support notifiers)");
 | 
				
			||||||
 | 
					        return;
 | 
				
			||||||
 | 
					    }
 | 
				
			||||||
 | 
					
 | 
				
			||||||
    /* If dataplane is (re-)enabled while the guest is running there could be
 | 
					    /* If dataplane is (re-)enabled while the guest is running there could be
 | 
				
			||||||
     * block jobs that can conflict.
 | 
					     * block jobs that can conflict.
 | 
				
			||||||
     */
 | 
					     */
 | 
				
			||||||
 
 | 
				
			|||||||
		Reference in New Issue
	
	Block a user