On 03/28/2018 11:18 AM, Daniel P. Berrangé wrote:
> Signed-off-by: Daniel P. Berrangé <[email protected]>
> ---
> src/remote/remote_daemon.h | 1 +
> src/remote/remote_daemon_dispatch.c | 19 +++++++++++--------
> src/rpc/gendispatch.pl | 6 ++++++
> 3 files changed, 18 insertions(+), 8 deletions(-)
>
[...]
> diff --git a/src/rpc/gendispatch.pl b/src/rpc/gendispatch.pl
> index 5bab13bb7b..0e7567fbde 100755
> --- a/src/rpc/gendispatch.pl
> +++ b/src/rpc/gendispatch.pl
> @@ -131,6 +131,9 @@ sub get_conn_arg {
> if ($type =~ /remote_nonnull_network/) {
> return "priv->networkConn";
> }
> + if ($type =~ /remote_nonnull_node_device/) {
> + return "priv->nodedevConn";
> + }
> }
>
> # This is for the few virConnect APIs that
> @@ -142,6 +145,9 @@ sub get_conn_arg {
> if ($proc =~ /Connect.*Network/) {
> return "priv->networkConn";
> }
> + if ($proc =~ /Node.*Device/) {
Originally I thought this should be NodeDevice; however, I see you're
picking up the NumOfDevices and ListDevices by this syntax.
Hopefully some day there isn't a *NodeGet*Device* entry for those Node,
but not NodeDevice API's... That'd be a bugger to find.
Reviewed-by: John Ferlan <[email protected]>
John
> + return "priv->nodedevConn";
> + }
>
> return "priv->conn";
> }
>
--
libvir-list mailing list
[email protected]
https://www.redhat.com/mailman/listinfo/libvir-list
