[libvirt] [PATCH 02/20] remote: Don't hard code the feature VIR_DRV_FEATURE_REMOTE_CLOSE_CALLBACK as available

John Ferlan jferlan at redhat.com
Wed Mar 14 21:30:40 UTC 2018



On 03/08/2018 07:20 AM, Marc Hartmayer wrote:
> Don't assume that the feature VIR_DRV_FEATURE_REMOTE_CLOSE_CALLBACK is
> available for every driver used for the connection.
> 
> Signed-off-by: Marc Hartmayer <mhartmay at linux.vnet.ibm.com>
> Reviewed-by: Bjoern Walk <bwalk at linux.vnet.ibm.com>
> Reviewed-by: Boris Fiuczynski <fiuczy at linux.vnet.ibm.com>
> ---
>  src/remote/remote_daemon_dispatch.c | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 

Something that is not clear about this one - since this was added for
'vz' driver by commit id 'f484310a', then shouldn't
vzConnectSupportsFeature be updated to indicate support?

If I'm right and you add the feature to the vz routine along with a
reference to the commit id that forgot to in your commit message, then

Reviewed-by: John Ferlan <jferlan at redhat.com>

If I'm wrong - then help me understand!

John

> diff --git a/src/remote/remote_daemon_dispatch.c b/src/remote/remote_daemon_dispatch.c
> index 82f6400ca49d..bf6c00348a5e 100644
> --- a/src/remote/remote_daemon_dispatch.c
> +++ b/src/remote/remote_daemon_dispatch.c
> @@ -4667,7 +4667,6 @@ static int remoteDispatchConnectSupportsFeature(virNetServerPtr server ATTRIBUTE
>      switch ((virDrvFeature) args->feature) {
>      case VIR_DRV_FEATURE_FD_PASSING:
>      case VIR_DRV_FEATURE_REMOTE_EVENT_CALLBACK:
> -    case VIR_DRV_FEATURE_REMOTE_CLOSE_CALLBACK:
>          supported = 1;
>          break;
>      case VIR_DRV_FEATURE_MIGRATION_V1:
> @@ -4681,6 +4680,7 @@ static int remoteDispatchConnectSupportsFeature(virNetServerPtr server ATTRIBUTE
>      case VIR_DRV_FEATURE_XML_MIGRATABLE:
>      case VIR_DRV_FEATURE_MIGRATION_OFFLINE:
>      case VIR_DRV_FEATURE_MIGRATION_PARAMS:
> +    case VIR_DRV_FEATURE_REMOTE_CLOSE_CALLBACK:
>      default:
>          if ((supported = virConnectSupportsFeature(priv->conn, args->feature)) < 0)
>              goto cleanup;
> 




More information about the libvir-list mailing list