hmp: Add nbd_server_remove to mirror QMP command
Since everything else about the nbd-server-* QMP commands is accessible from HMP, we might as well make removing an export available as well. For now, I went with a bool flag rather than a mode string for choosing between safe (default) and hard modes. Signed-off-by: Eric Blake <eblake@redhat.com> Message-Id: <20180125144557.25502-1-eblake@redhat.com> Reviewed-by: Dr. David Alan Gilbert <dgilbert@redhat.com> Signed-off-by: Eric Blake <eblake@redhat.com>
This commit is contained in:
		
							parent
							
								
									2dadedce2b
								
							
						
					
					
						commit
						08fb10a729
					
				| 
						 | 
					@ -1565,6 +1565,23 @@ Export a block device through QEMU's NBD server, which must be started
 | 
				
			||||||
beforehand with @command{nbd_server_start}.  The @option{-w} option makes the
 | 
					beforehand with @command{nbd_server_start}.  The @option{-w} option makes the
 | 
				
			||||||
exported device writable too.  The export name is controlled by @var{name},
 | 
					exported device writable too.  The export name is controlled by @var{name},
 | 
				
			||||||
defaulting to @var{device}.
 | 
					defaulting to @var{device}.
 | 
				
			||||||
 | 
					ETEXI
 | 
				
			||||||
 | 
					
 | 
				
			||||||
 | 
					    {
 | 
				
			||||||
 | 
					        .name       = "nbd_server_remove",
 | 
				
			||||||
 | 
					        .args_type  = "force:-f,name:s",
 | 
				
			||||||
 | 
					        .params     = "nbd_server_remove [-f] name",
 | 
				
			||||||
 | 
					        .help       = "remove an export previously exposed via NBD",
 | 
				
			||||||
 | 
					        .cmd        = hmp_nbd_server_remove,
 | 
				
			||||||
 | 
					    },
 | 
				
			||||||
 | 
					STEXI
 | 
				
			||||||
 | 
					@item nbd_server_remove [-f] @var{name}
 | 
				
			||||||
 | 
					@findex nbd_server_remove
 | 
				
			||||||
 | 
					Stop exporting a block device through QEMU's NBD server, which was
 | 
				
			||||||
 | 
					previously started with @command{nbd_server_add}.  The @option{-f}
 | 
				
			||||||
 | 
					option forces the server to drop the export immediately even if
 | 
				
			||||||
 | 
					clients are connected; otherwise the command fails unless there are no
 | 
				
			||||||
 | 
					clients.
 | 
				
			||||||
ETEXI
 | 
					ETEXI
 | 
				
			||||||
 | 
					
 | 
				
			||||||
    {
 | 
					    {
 | 
				
			||||||
| 
						 | 
					
 | 
				
			||||||
							
								
								
									
										14
									
								
								hmp.c
								
								
								
								
							
							
						
						
									
										14
									
								
								hmp.c
								
								
								
								
							| 
						 | 
					@ -2226,10 +2226,18 @@ void hmp_nbd_server_add(Monitor *mon, const QDict *qdict)
 | 
				
			||||||
    Error *local_err = NULL;
 | 
					    Error *local_err = NULL;
 | 
				
			||||||
 | 
					
 | 
				
			||||||
    qmp_nbd_server_add(device, !!name, name, true, writable, &local_err);
 | 
					    qmp_nbd_server_add(device, !!name, name, true, writable, &local_err);
 | 
				
			||||||
 | 
					    hmp_handle_error(mon, &local_err);
 | 
				
			||||||
 | 
					}
 | 
				
			||||||
 | 
					
 | 
				
			||||||
    if (local_err != NULL) {
 | 
					void hmp_nbd_server_remove(Monitor *mon, const QDict *qdict)
 | 
				
			||||||
        hmp_handle_error(mon, &local_err);
 | 
					{
 | 
				
			||||||
    }
 | 
					    const char *name = qdict_get_str(qdict, "name");
 | 
				
			||||||
 | 
					    bool force = qdict_get_try_bool(qdict, "force", false);
 | 
				
			||||||
 | 
					    Error *err = NULL;
 | 
				
			||||||
 | 
					
 | 
				
			||||||
 | 
					    /* Rely on NBD_SERVER_REMOVE_MODE_SAFE being the default */
 | 
				
			||||||
 | 
					    qmp_nbd_server_remove(name, force, NBD_SERVER_REMOVE_MODE_HARD, &err);
 | 
				
			||||||
 | 
					    hmp_handle_error(mon, &err);
 | 
				
			||||||
}
 | 
					}
 | 
				
			||||||
 | 
					
 | 
				
			||||||
void hmp_nbd_server_stop(Monitor *mon, const QDict *qdict)
 | 
					void hmp_nbd_server_stop(Monitor *mon, const QDict *qdict)
 | 
				
			||||||
| 
						 | 
					
 | 
				
			||||||
							
								
								
									
										1
									
								
								hmp.h
								
								
								
								
							
							
						
						
									
										1
									
								
								hmp.h
								
								
								
								
							| 
						 | 
					@ -101,6 +101,7 @@ void hmp_sendkey(Monitor *mon, const QDict *qdict);
 | 
				
			||||||
void hmp_screendump(Monitor *mon, const QDict *qdict);
 | 
					void hmp_screendump(Monitor *mon, const QDict *qdict);
 | 
				
			||||||
void hmp_nbd_server_start(Monitor *mon, const QDict *qdict);
 | 
					void hmp_nbd_server_start(Monitor *mon, const QDict *qdict);
 | 
				
			||||||
void hmp_nbd_server_add(Monitor *mon, const QDict *qdict);
 | 
					void hmp_nbd_server_add(Monitor *mon, const QDict *qdict);
 | 
				
			||||||
 | 
					void hmp_nbd_server_remove(Monitor *mon, const QDict *qdict);
 | 
				
			||||||
void hmp_nbd_server_stop(Monitor *mon, const QDict *qdict);
 | 
					void hmp_nbd_server_stop(Monitor *mon, const QDict *qdict);
 | 
				
			||||||
void hmp_chardev_add(Monitor *mon, const QDict *qdict);
 | 
					void hmp_chardev_add(Monitor *mon, const QDict *qdict);
 | 
				
			||||||
void hmp_chardev_change(Monitor *mon, const QDict *qdict);
 | 
					void hmp_chardev_change(Monitor *mon, const QDict *qdict);
 | 
				
			||||||
| 
						 | 
					
 | 
				
			||||||
		Loading…
	
		Reference in New Issue