提交 04110306 编写于 作者: M Matthias Bolte

remote generator, daemon: Handle more functions

This covers functions that return single wrapped objects and don't
involve complexer code in the body.
上级 0c3cea0b
此差异已折叠。
此差异已折叠。
...@@ -370,7 +370,7 @@ static int remoteDispatchDomainIsUpdated( ...@@ -370,7 +370,7 @@ static int remoteDispatchDomainIsUpdated(
remote_error *rerr, remote_error *rerr,
remote_domain_is_updated_args *args, remote_domain_is_updated_args *args,
remote_domain_is_updated_ret *ret); remote_domain_is_updated_ret *ret);
static int remoteDispatchDomainLookupById( static int remoteDispatchDomainLookupByID(
struct qemud_server *server, struct qemud_server *server,
struct qemud_client *client, struct qemud_client *client,
virConnectPtr conn, virConnectPtr conn,
...@@ -834,7 +834,7 @@ static int remoteDispatchInterfaceIsActive( ...@@ -834,7 +834,7 @@ static int remoteDispatchInterfaceIsActive(
remote_error *rerr, remote_error *rerr,
remote_interface_is_active_args *args, remote_interface_is_active_args *args,
remote_interface_is_active_ret *ret); remote_interface_is_active_ret *ret);
static int remoteDispatchInterfaceLookupByMacString( static int remoteDispatchInterfaceLookupByMACString(
struct qemud_server *server, struct qemud_server *server,
struct qemud_client *client, struct qemud_client *client,
virConnectPtr conn, virConnectPtr conn,
......
...@@ -112,8 +112,8 @@ ...@@ -112,8 +112,8 @@
.args_filter = (xdrproc_t) xdr_remote_list_defined_domains_args, .args_filter = (xdrproc_t) xdr_remote_list_defined_domains_args,
.ret_filter = (xdrproc_t) xdr_remote_list_defined_domains_ret, .ret_filter = (xdrproc_t) xdr_remote_list_defined_domains_ret,
}, },
{ /* DomainLookupById => 22 */ { /* DomainLookupByID => 22 */
.fn = (dispatch_fn) remoteDispatchDomainLookupById, .fn = (dispatch_fn) remoteDispatchDomainLookupByID,
.args_filter = (xdrproc_t) xdr_remote_domain_lookup_by_id_args, .args_filter = (xdrproc_t) xdr_remote_domain_lookup_by_id_args,
.ret_filter = (xdrproc_t) xdr_remote_domain_lookup_by_id_ret, .ret_filter = (xdrproc_t) xdr_remote_domain_lookup_by_id_ret,
}, },
...@@ -647,8 +647,8 @@ ...@@ -647,8 +647,8 @@
.args_filter = (xdrproc_t) xdr_remote_interface_lookup_by_name_args, .args_filter = (xdrproc_t) xdr_remote_interface_lookup_by_name_args,
.ret_filter = (xdrproc_t) xdr_remote_interface_lookup_by_name_ret, .ret_filter = (xdrproc_t) xdr_remote_interface_lookup_by_name_ret,
}, },
{ /* InterfaceLookupByMacString => 129 */ { /* InterfaceLookupByMACString => 129 */
.fn = (dispatch_fn) remoteDispatchInterfaceLookupByMacString, .fn = (dispatch_fn) remoteDispatchInterfaceLookupByMACString,
.args_filter = (xdrproc_t) xdr_remote_interface_lookup_by_mac_string_args, .args_filter = (xdrproc_t) xdr_remote_interface_lookup_by_mac_string_args,
.ret_filter = (xdrproc_t) xdr_remote_interface_lookup_by_mac_string_ret, .ret_filter = (xdrproc_t) xdr_remote_interface_lookup_by_mac_string_ret,
}, },
......
...@@ -33,7 +33,8 @@ sub name_to_ProcName { ...@@ -33,7 +33,8 @@ sub name_to_ProcName {
my @elems = split /_/, $name; my @elems = split /_/, $name;
@elems = map ucfirst, @elems; @elems = map ucfirst, @elems;
@elems = map { $_ =~ s/Nwfilter/NWFilter/; $_ =~ s/Xml/XML/; @elems = map { $_ =~ s/Nwfilter/NWFilter/; $_ =~ s/Xml/XML/;
$_ =~ s/Uri/URI/; $_ =~ s/Uuid/UUID/; $_ } @elems; $_ =~ s/Uri/URI/; $_ =~ s/Uuid/UUID/; $_ =~ s/Id/ID/;
$_ =~ s/Mac/MAC/; $_ } @elems;
join "", @elems join "", @elems
} }
...@@ -262,8 +263,6 @@ elsif ($opt_b) { ...@@ -262,8 +263,6 @@ elsif ($opt_b) {
"DomainBlockPeek", "DomainBlockPeek",
"DomainBlockStats", "DomainBlockStats",
"DomainCreateWithFlags", "DomainCreateWithFlags",
"DomainCreateXML",
"DomainDefineXML",
"DomainEventsDeregister", "DomainEventsDeregister",
"DomainEventsRegister", "DomainEventsRegister",
"DomainGetBlkioParameters", "DomainGetBlkioParameters",
...@@ -276,25 +275,14 @@ elsif ($opt_b) { ...@@ -276,25 +275,14 @@ elsif ($opt_b) {
"DomainGetSecurityLabel", "DomainGetSecurityLabel",
"DomainGetVcpus", "DomainGetVcpus",
"DomainInterfaceStats", "DomainInterfaceStats",
"DomainLookupById",
"DomainLookupByName",
"DomainLookupByUUID",
"DomainMemoryPeek", "DomainMemoryPeek",
"DomainMemoryStats", "DomainMemoryStats",
"DomainMigrateFinish",
"DomainMigrateFinish2",
"DomainMigratePrepare", "DomainMigratePrepare",
"DomainMigratePrepare2", "DomainMigratePrepare2",
"DomainSnapshotCreateXML",
"DomainSnapshotCurrent",
"DomainSnapshotListNames", "DomainSnapshotListNames",
"DomainSnapshotLookupByName",
"FindStoragePoolSources", "FindStoragePoolSources",
"GetMaxVcpus", "GetMaxVcpus",
"GetType", "GetType",
"InterfaceLookupByMacString",
"InterfaceLookupByName",
"InterfaceDefineXML",
"ListDefinedDomains", "ListDefinedDomains",
"ListDefinedInterfaces", "ListDefinedInterfaces",
"ListDefinedNetworks", "ListDefinedNetworks",
...@@ -305,40 +293,17 @@ elsif ($opt_b) { ...@@ -305,40 +293,17 @@ elsif ($opt_b) {
"ListNWFilters", "ListNWFilters",
"ListSecrets", "ListSecrets",
"ListStoragePools", "ListStoragePools",
"NetworkLookupByName",
"NetworkLookupByUUID",
"NodeDeviceCreateXML",
"NetworkCreateXML",
"NodeDeviceGetParent", "NodeDeviceGetParent",
"NodeDeviceListCaps", "NodeDeviceListCaps",
"NodeDeviceLookupByName",
"NodeGetCellsFreeMemory", "NodeGetCellsFreeMemory",
"NodeGetFreeMemory",
"NodeGetInfo", "NodeGetInfo",
"NodeGetSecurityModel", "NodeGetSecurityModel",
"NetworkDefineXML",
"NodeListDevices", "NodeListDevices",
"NodeNumOfDevices", "NodeNumOfDevices",
"NWFilterLookupByName",
"NWFilterLookupByUUID",
"SecretDefineXML",
"SecretGetValue", "SecretGetValue",
"NWFilterDefineXML",
"SecretLookupByUsage",
"SecretLookupByUUID",
"StoragePoolCreateXML",
"StoragePoolDefineXML",
"StoragePoolGetInfo", "StoragePoolGetInfo",
"StoragePoolListVolumes", "StoragePoolListVolumes",
"StoragePoolLookupByName", "StorageVolGetInfo");
"StoragePoolLookupByUUID",
"StoragePoolLookupByVolume",
"StorageVolCreateXML",
"StorageVolCreateXMLFrom",
"StorageVolGetInfo",
"StorageVolLookupByKey",
"StorageVolLookupByName",
"StorageVolLookupByPath");
} elsif ($structprefix eq "qemu") { } elsif ($structprefix eq "qemu") {
@ungeneratable = ("MonitorCommand"); @ungeneratable = ("MonitorCommand");
} }
...@@ -350,7 +315,6 @@ elsif ($opt_b) { ...@@ -350,7 +315,6 @@ elsif ($opt_b) {
# skip things which are REMOTE_MESSAGE # skip things which are REMOTE_MESSAGE
next if $calls{$_}->{msg}; next if $calls{$_}->{msg};
# FIXME: skip functions with explicit return value for now
if (exists($ug{$calls{$_}->{ProcName}})) { if (exists($ug{$calls{$_}->{ProcName}})) {
print "\n/* ${structprefix}Dispatch$calls{$_}->{ProcName} has " . print "\n/* ${structprefix}Dispatch$calls{$_}->{ProcName} has " .
"to be implemented manually */\n"; "to be implemented manually */\n";
...@@ -391,7 +355,9 @@ elsif ($opt_b) { ...@@ -391,7 +355,9 @@ elsif ($opt_b) {
if ($calls{$_}->{args} ne "void") { if ($calls{$_}->{args} ne "void") {
# node device is special, as it's identified by name # node device is special, as it's identified by name
if ($calls{$_}->{args} =~ m/^remote_node_device/) { if ($calls{$_}->{args} =~ m/^remote_node_device_/ and
!($calls{$_}->{args} =~ m/^remote_node_device_lookup_by_name_/) and
!($calls{$_}->{args} =~ m/^remote_node_device_create_xml_/)) {
$has_node_device = 1; $has_node_device = 1;
push(@vars_list, "virNodeDevicePtr dev = NULL"); push(@vars_list, "virNodeDevicePtr dev = NULL");
push(@getters_list, push(@getters_list,
...@@ -407,69 +373,69 @@ elsif ($opt_b) { ...@@ -407,69 +373,69 @@ elsif ($opt_b) {
if ($args_member =~ m/^remote_nonnull_string name;/ and $has_node_device) { if ($args_member =~ m/^remote_nonnull_string name;/ and $has_node_device) {
# ignore the name arg for node devices # ignore the name arg for node devices
next next
} elsif ($args_member =~ m/^remote_nonnull_domain /) { } elsif ($args_member =~ m/^remote_nonnull_domain (\S+);/) {
push(@vars_list, "virDomainPtr dom = NULL"); push(@vars_list, "virDomainPtr $1 = NULL");
push(@getters_list, push(@getters_list,
" if (!(dom = get_nonnull_domain(conn, args->dom)))\n" . " if (!($1 = get_nonnull_domain(conn, args->$1)))\n" .
" goto cleanup;\n"); " goto cleanup;\n");
push(@args_list, "dom"); push(@args_list, "$1");
push(@free_list, push(@free_list,
" if (dom)\n" . " if ($1)\n" .
" virDomainFree(dom);"); " virDomainFree($1);");
} elsif ($args_member =~ m/^remote_nonnull_network /) { } elsif ($args_member =~ m/^remote_nonnull_network (\S+);/) {
push(@vars_list, "virNetworkPtr net = NULL"); push(@vars_list, "virNetworkPtr $1 = NULL");
push(@getters_list, push(@getters_list,
" if (!(net = get_nonnull_network(conn, args->net)))\n" . " if (!($1 = get_nonnull_network(conn, args->$1)))\n" .
" goto cleanup;\n"); " goto cleanup;\n");
push(@args_list, "net"); push(@args_list, "$1");
push(@free_list, push(@free_list,
" if (net)\n" . " if ($1)\n" .
" virNetworkFree(net);"); " virNetworkFree($1);");
} elsif ($args_member =~ m/^remote_nonnull_storage_pool /) { } elsif ($args_member =~ m/^remote_nonnull_storage_pool (\S+);/) {
push(@vars_list, "virStoragePoolPtr pool = NULL"); push(@vars_list, "virStoragePoolPtr $1 = NULL");
push(@getters_list, push(@getters_list,
" if (!(pool = get_nonnull_storage_pool(conn, args->pool)))\n" . " if (!($1 = get_nonnull_storage_pool(conn, args->$1)))\n" .
" goto cleanup;\n"); " goto cleanup;\n");
push(@args_list, "pool"); push(@args_list, "$1");
push(@free_list, push(@free_list,
" if (pool)\n" . " if ($1)\n" .
" virStoragePoolFree(pool);"); " virStoragePoolFree($1);");
} elsif ($args_member =~ m/^remote_nonnull_storage_vol /) { } elsif ($args_member =~ m/^remote_nonnull_storage_vol (\S+);/) {
push(@vars_list, "virStorageVolPtr vol = NULL"); push(@vars_list, "virStorageVolPtr $1 = NULL");
push(@getters_list, push(@getters_list,
" if (!(vol = get_nonnull_storage_vol(conn, args->vol)))\n" . " if (!($1 = get_nonnull_storage_vol(conn, args->$1)))\n" .
" goto cleanup;\n"); " goto cleanup;\n");
push(@args_list, "vol"); push(@args_list, "$1");
push(@free_list, push(@free_list,
" if (vol)\n" . " if ($1)\n" .
" virStorageVolFree(vol);"); " virStorageVolFree($1);");
} elsif ($args_member =~ m/^remote_nonnull_interface /) { } elsif ($args_member =~ m/^remote_nonnull_interface (\S+);/) {
push(@vars_list, "virInterfacePtr iface = NULL"); push(@vars_list, "virInterfacePtr $1 = NULL");
push(@getters_list, push(@getters_list,
" if (!(iface = get_nonnull_interface(conn, args->iface)))\n" . " if (!($1 = get_nonnull_interface(conn, args->$1)))\n" .
" goto cleanup;\n"); " goto cleanup;\n");
push(@args_list, "iface"); push(@args_list, "$1");
push(@free_list, push(@free_list,
" if (iface)\n" . " if ($1)\n" .
" virInterfaceFree(iface);"); " virInterfaceFree($1);");
} elsif ($args_member =~ m/^remote_nonnull_secret /) { } elsif ($args_member =~ m/^remote_nonnull_secret (\S+);/) {
push(@vars_list, "virSecretPtr secret = NULL"); push(@vars_list, "virSecretPtr $1 = NULL");
push(@getters_list, push(@getters_list,
" if (!(secret = get_nonnull_secret(conn, args->secret)))\n" . " if (!($1 = get_nonnull_secret(conn, args->$1)))\n" .
" goto cleanup;\n"); " goto cleanup;\n");
push(@args_list, "secret"); push(@args_list, "$1");
push(@free_list, push(@free_list,
" if (secret)\n" . " if ($1)\n" .
" virSecretFree(secret);"); " virSecretFree($1);");
} elsif ($args_member =~ m/^remote_nonnull_nwfilter /) { } elsif ($args_member =~ m/^remote_nonnull_nwfilter (\S+);/) {
push(@vars_list, "virNWFilterPtr nwfilter = NULL"); push(@vars_list, "virNWFilterPtr $1 = NULL");
push(@getters_list, push(@getters_list,
" if (!(nwfilter = get_nonnull_nwfilter(conn, args->nwfilter)))\n" . " if (!($1 = get_nonnull_nwfilter(conn, args->$1)))\n" .
" goto cleanup;\n"); " goto cleanup;\n");
push(@args_list, "nwfilter"); push(@args_list, "$1");
push(@free_list, push(@free_list,
" if (nwfilter)\n" . " if ($1)\n" .
" virNWFilterFree(nwfilter);"); " virNWFilterFree($1);");
} elsif ($args_member =~ m/^remote_nonnull_domain_snapshot /) { } elsif ($args_member =~ m/^remote_nonnull_domain_snapshot /) {
push(@vars_list, "virDomainPtr dom = NULL"); push(@vars_list, "virDomainPtr dom = NULL");
push(@vars_list, "virDomainSnapshotPtr snapshot = NULL"); push(@vars_list, "virDomainSnapshotPtr snapshot = NULL");
...@@ -497,12 +463,16 @@ elsif ($opt_b) { ...@@ -497,12 +463,16 @@ elsif ($opt_b) {
} }
push(@args_list, "args->$1.$1_len"); push(@args_list, "args->$1.$1_len");
} elsif ($args_member =~ m/.* (\S+);/) { } elsif ($args_member =~ m/(\S+) (\S+);/) {
if (! @args_list) { if (! @args_list) {
push(@args_list, "conn"); push(@args_list, "conn");
} }
push(@args_list, "args->$1"); if ($1 eq "remote_uuid") {
push(@args_list, "(unsigned char *) args->$2");
} else {
push(@args_list, "args->$2");
}
} }
} }
} }
...@@ -522,6 +492,87 @@ elsif ($opt_b) { ...@@ -522,6 +492,87 @@ elsif ($opt_b) {
$single_ret_var = $1; $single_ret_var = $1;
$single_ret_by_ref = 0; $single_ret_by_ref = 0;
$single_ret_check = " == NULL"; $single_ret_check = " == NULL";
} elsif ($ret_member =~ m/remote_nonnull_domain (\S+);/) {
push(@vars_list, "virDomainPtr $1 = NULL");
push(@ret_list, "make_nonnull_domain(&ret->$1, $1);");
push(@free_list,
" if ($1)\n" .
" virDomainFree($1);");
$single_ret_var = $1;
$single_ret_by_ref = 0;
$single_ret_check = " == NULL";
} elsif ($ret_member =~ m/remote_nonnull_network (\S+);/) {
push(@vars_list, "virNetworkPtr $1 = NULL");
push(@ret_list, "make_nonnull_network(&ret->$1, $1);");
push(@free_list,
" if ($1)\n" .
" virNetworkFree($1);");
$single_ret_var = $1;
$single_ret_by_ref = 0;
$single_ret_check = " == NULL";
} elsif ($ret_member =~ m/remote_nonnull_storage_pool (\S+);/) {
push(@vars_list, "virStoragePoolPtr $1 = NULL");
push(@ret_list, "make_nonnull_storage_pool(&ret->$1, $1);");
push(@free_list,
" if ($1)\n" .
" virStoragePoolFree($1);");
$single_ret_var = $1;
$single_ret_by_ref = 0;
$single_ret_check = " == NULL";
} elsif ($ret_member =~ m/remote_nonnull_storage_vol (\S+);/) {
push(@vars_list, "virStorageVolPtr $1 = NULL");
push(@ret_list, "make_nonnull_storage_vol(&ret->$1, $1);");
push(@free_list,
" if ($1)\n" .
" virStorageVolFree($1);");
$single_ret_var = $1;
$single_ret_by_ref = 0;
$single_ret_check = " == NULL";
} elsif ($ret_member =~ m/remote_nonnull_interface (\S+);/) {
push(@vars_list, "virInterfacePtr $1 = NULL");
push(@ret_list, "make_nonnull_interface(&ret->$1, $1);");
push(@free_list,
" if ($1)\n" .
" virInterfaceFree($1);");
$single_ret_var = $1;
$single_ret_by_ref = 0;
$single_ret_check = " == NULL";
} elsif ($ret_member =~ m/remote_nonnull_node_device (\S+);/) {
push(@vars_list, "virNodeDevicePtr $1 = NULL");
push(@ret_list, "make_nonnull_node_device(&ret->$1, $1);");
push(@free_list,
" if ($1)\n" .
" virNodeDeviceFree($1);");
$single_ret_var = $1;
$single_ret_by_ref = 0;
$single_ret_check = " == NULL";
} elsif ($ret_member =~ m/remote_nonnull_secret (\S+);/) {
push(@vars_list, "virSecretPtr $1 = NULL");
push(@ret_list, "make_nonnull_secret(&ret->$1, $1);");
push(@free_list,
" if ($1)\n" .
" virSecretFree($1);");
$single_ret_var = $1;
$single_ret_by_ref = 0;
$single_ret_check = " == NULL";
} elsif ($ret_member =~ m/remote_nonnull_nwfilter (\S+);/) {
push(@vars_list, "virNWFilterPtr $1 = NULL");
push(@ret_list, "make_nonnull_nwfilter(&ret->$1, $1);");
push(@free_list,
" if ($1)\n" .
" virNWFilterFree($1);");
$single_ret_var = $1;
$single_ret_by_ref = 0;
$single_ret_check = " == NULL";
} elsif ($ret_member =~ m/remote_nonnull_domain_snapshot (\S+);/) {
push(@vars_list, "virDomainSnapshotPtr $1 = NULL");
push(@ret_list, "make_nonnull_domain_snapshot(&ret->$1, $1);");
push(@free_list,
" if ($1)\n" .
" virDomainSnapshotFree($1);");
$single_ret_var = $1;
$single_ret_by_ref = 0;
$single_ret_check = " == NULL";
} elsif ($ret_member =~ m/int (\S+);/) { } elsif ($ret_member =~ m/int (\S+);/) {
push(@vars_list, "int $1"); push(@vars_list, "int $1");
push(@ret_list, "ret->$1 = $1;"); push(@ret_list, "ret->$1 = $1;");
...@@ -540,7 +591,8 @@ elsif ($opt_b) { ...@@ -540,7 +591,8 @@ elsif ($opt_b) {
push(@ret_list, "ret->$1 = $1;"); push(@ret_list, "ret->$1 = $1;");
$single_ret_var = $1; $single_ret_var = $1;
if ($calls{$_}->{ProcName} eq "DomainGetMaxMemory") { if ($calls{$_}->{ProcName} eq "DomainGetMaxMemory" or
$calls{$_}->{ProcName} eq "NodeGetFreeMemory") {
$single_ret_by_ref = 0; $single_ret_by_ref = 0;
$single_ret_check = " == 0"; $single_ret_check = " == 0";
} else { } else {
...@@ -579,8 +631,11 @@ elsif ($opt_b) { ...@@ -579,8 +631,11 @@ elsif ($opt_b) {
if (! @args_list) { if (! @args_list) {
push(@args_list, "conn"); push(@args_list, "conn");
if ($calls{$_}->{ProcName} ne "NodeGetFreeMemory") {
$prefix = "Connect" $prefix = "Connect"
} }
}
if ($calls{$_}->{ProcName} eq "GetSysinfo" or if ($calls{$_}->{ProcName} eq "GetSysinfo" or
$calls{$_}->{ProcName} eq "DomainXMLFromNative" or $calls{$_}->{ProcName} eq "DomainXMLFromNative" or
......
Markdown is supported
0% .
You are about to add 0 people to the discussion. Proceed with caution.
先完成此消息的编辑!
想要评论请 注册