- Devel - libvirt List Archives

[PATCH] remote: Initialize args variable
by Michal Privoznik 26 Jan '23

26 Jan '23

Recently, in v9.0.0-7-gb2034bb04c we've dropped initialization of @args variable. The reasoning was that eventually, all members of the variable will be set. Well, this is not correct. For instance, in remoteConnectGetAllDomainStats() the args.doms.doms_val pointer is set iff @ndoms != 0. However, regardless of that, the pointer is then passed to VIR_FREE(). Worse, the whole args is passed to xdr_remote_connect_get_all_domain_stats_args() which then calls xdr_array, which tests the (uninitialized) pointer against NULL. This effectively reverts b2034bb04c61c75ddbfbed46879d641b6f8ca8dc. Signed-off-by: Michal Privoznik <mprivozn(a)redhat.com> --- src/remote/remote_driver.c | 178 ++++++++++++++++++------------------- 1 file changed, 89 insertions(+), 89 deletions(-) diff --git a/src/remote/remote_driver.c b/src/remote/remote_driver.c index 7f84050f70..9f3a9520e4 100644 --- a/src/remote/remote_driver.c +++ b/src/remote/remote_driver.c @@ -1434,7 +1434,7 @@ remoteNodeGetCPUStats(virConnectPtr conn, unsigned int flags) { int rv = -1; - remote_node_get_cpu_stats_args args; + remote_node_get_cpu_stats_args args = {0}; remote_node_get_cpu_stats_ret ret = {0}; size_t i; struct private_data *priv = conn->privateData; @@ -1496,7 +1496,7 @@ remoteNodeGetMemoryStats(virConnectPtr conn, unsigned int flags) { int rv = -1; - remote_node_get_memory_stats_args args; + remote_node_get_memory_stats_args args = {0}; remote_node_get_memory_stats_ret ret = {0}; size_t i; struct private_data *priv = conn->privateData; @@ -1554,7 +1554,7 @@ remoteNodeGetCellsFreeMemory(virConnectPtr conn, int startCell, int maxCells) { - remote_node_get_cells_free_memory_args args; + remote_node_get_cells_free_memory_args args = {0}; remote_node_get_cells_free_memory_ret ret = {0}; size_t i; struct private_data *priv = conn->privateData; @@ -1588,7 +1588,7 @@ remoteConnectListDomains(virConnectPtr conn, int *ids, int maxids) { int rv = -1; size_t i; - remote_connect_list_domains_args args; + remote_connect_list_domains_args args = {0}; remote_connect_list_domains_ret ret = {0}; struct private_data *priv = conn->privateData; VIR_LOCK_GUARD lock = remoteDriverLock(priv); @@ -1661,7 +1661,7 @@ remoteDomainBlockStatsFlags(virDomainPtr domain, unsigned int flags) { int rv = -1; - remote_domain_block_stats_flags_args args; + remote_domain_block_stats_flags_args args = {0}; remote_domain_block_stats_flags_ret ret = {0}; struct private_data *priv = domain->conn->privateData; VIR_LOCK_GUARD lock = remoteDriverLock(priv); @@ -1718,7 +1718,7 @@ remoteDomainGetMemoryParameters(virDomainPtr domain, unsigned int flags) { int rv = -1; - remote_domain_get_memory_parameters_args args; + remote_domain_get_memory_parameters_args args = {0}; remote_domain_get_memory_parameters_ret ret = {0}; struct private_data *priv = domain->conn->privateData; VIR_LOCK_GUARD lock = remoteDriverLock(priv); @@ -1762,7 +1762,7 @@ remoteDomainGetNumaParameters(virDomainPtr domain, unsigned int flags) { int rv = -1; - remote_domain_get_numa_parameters_args args; + remote_domain_get_numa_parameters_args args = {0}; remote_domain_get_numa_parameters_ret ret = {0}; struct private_data *priv = domain->conn->privateData; VIR_LOCK_GUARD lock = remoteDriverLock(priv); @@ -1807,7 +1807,7 @@ remoteDomainGetLaunchSecurityInfo(virDomainPtr domain, unsigned int flags) { int rv = -1; - remote_domain_get_launch_security_info_args args; + remote_domain_get_launch_security_info_args args = {0}; remote_domain_get_launch_security_info_ret ret = {0}; struct private_data *priv = domain->conn->privateData; VIR_LOCK_GUARD lock = remoteDriverLock(priv); @@ -1842,7 +1842,7 @@ remoteDomainGetPerfEvents(virDomainPtr domain, unsigned int flags) { int rv = -1; - remote_domain_get_perf_events_args args; + remote_domain_get_perf_events_args args = {0}; remote_domain_get_perf_events_ret ret = {0}; struct private_data *priv = domain->conn->privateData; VIR_LOCK_GUARD lock = remoteDriverLock(priv); @@ -1876,7 +1876,7 @@ remoteDomainGetBlkioParameters(virDomainPtr domain, unsigned int flags) { int rv = -1; - remote_domain_get_blkio_parameters_args args; + remote_domain_get_blkio_parameters_args args = {0}; remote_domain_get_blkio_parameters_ret ret = {0}; struct private_data *priv = domain->conn->privateData; VIR_LOCK_GUARD lock = remoteDriverLock(priv); @@ -1923,7 +1923,7 @@ remoteDomainGetVcpuPinInfo(virDomainPtr domain, { int rv = -1; size_t i; - remote_domain_get_vcpu_pin_info_args args; + remote_domain_get_vcpu_pin_info_args args = {0}; remote_domain_get_vcpu_pin_info_ret ret = {0}; struct private_data *priv = domain->conn->privateData; VIR_LOCK_GUARD lock = remoteDriverLock(priv); @@ -1988,7 +1988,7 @@ remoteDomainPinEmulator(virDomainPtr dom, unsigned int flags) { struct private_data *priv = dom->conn->privateData; - remote_domain_pin_emulator_args args; + remote_domain_pin_emulator_args args = {0}; VIR_LOCK_GUARD lock = remoteDriverLock(priv); if (cpumaplen > REMOTE_CPUMAP_MAX) { @@ -2021,7 +2021,7 @@ remoteDomainGetEmulatorPinInfo(virDomainPtr domain, { int rv = -1; size_t i; - remote_domain_get_emulator_pin_info_args args; + remote_domain_get_emulator_pin_info_args args = {0}; remote_domain_get_emulator_pin_info_ret ret = {0}; struct private_data *priv = domain->conn->privateData; VIR_LOCK_GUARD lock = remoteDriverLock(priv); @@ -2074,7 +2074,7 @@ remoteDomainGetVcpus(virDomainPtr domain, { int rv = -1; size_t i; - remote_domain_get_vcpus_args args; + remote_domain_get_vcpus_args args = {0}; remote_domain_get_vcpus_ret ret = {0}; struct private_data *priv = domain->conn->privateData; VIR_LOCK_GUARD lock = remoteDriverLock(priv); @@ -2143,7 +2143,7 @@ remoteDomainGetIOThreadInfo(virDomainPtr dom, int rv = -1; size_t i; struct private_data *priv = dom->conn->privateData; - remote_domain_get_iothread_info_args args; + remote_domain_get_iothread_info_args args = {0}; remote_domain_get_iothread_info_ret ret = {0}; remote_domain_iothread_info *src; VIR_LOCK_GUARD lock = remoteDriverLock(priv); @@ -2202,7 +2202,7 @@ remoteDomainGetIOThreadInfo(virDomainPtr dom, static int remoteDomainGetSecurityLabel(virDomainPtr domain, virSecurityLabelPtr seclabel) { - remote_domain_get_security_label_args args; + remote_domain_get_security_label_args args = {0}; remote_domain_get_security_label_ret ret = {0}; struct private_data *priv = domain->conn->privateData; int rv = -1; @@ -2235,7 +2235,7 @@ remoteDomainGetSecurityLabel(virDomainPtr domain, virSecurityLabelPtr seclabel) static int remoteDomainGetSecurityLabelList(virDomainPtr domain, virSecurityLabelPtr* seclabels) { - remote_domain_get_security_label_list_args args; + remote_domain_get_security_label_list_args args = {0}; remote_domain_get_security_label_list_ret ret = {0}; struct private_data *priv = domain->conn->privateData; size_t i; @@ -2276,7 +2276,7 @@ remoteDomainGetState(virDomainPtr domain, int *reason, unsigned int flags) { - remote_domain_get_state_args args; + remote_domain_get_state_args args = {0}; remote_domain_get_state_ret ret = {0}; struct private_data *priv = domain->conn->privateData; VIR_LOCK_GUARD lock = remoteDriverLock(priv); @@ -2341,7 +2341,7 @@ remoteDomainMigratePrepare(virConnectPtr dconn, unsigned long flags, const char *dname, unsigned long resource) { - remote_domain_migrate_prepare_args args; + remote_domain_migrate_prepare_args args = {0}; remote_domain_migrate_prepare_ret ret = {0}; struct private_data *priv = dconn->privateData; VIR_LOCK_GUARD lock = remoteDriverLock(priv); @@ -2376,7 +2376,7 @@ remoteDomainMigratePrepare2(virConnectPtr dconn, const char *dom_xml) { int rv = -1; - remote_domain_migrate_prepare2_args args; + remote_domain_migrate_prepare2_args args = {0}; remote_domain_migrate_prepare2_ret ret = {0}; struct private_data *priv = dconn->privateData; VIR_LOCK_GUARD lock = remoteDriverLock(priv); @@ -2426,8 +2426,8 @@ remoteDomainMigratePrepare2(virConnectPtr dconn, static int remoteDomainCreate(virDomainPtr domain) { - remote_domain_create_args args; - remote_domain_lookup_by_uuid_args args2; + remote_domain_create_args args = {0}; + remote_domain_lookup_by_uuid_args args2 = {0}; remote_domain_lookup_by_uuid_ret ret2 = {0}; struct private_data *priv = domain->conn->privateData; VIR_LOCK_GUARD lock = remoteDriverLock(priv); @@ -2458,7 +2458,7 @@ remoteDomainCreate(virDomainPtr domain) static char * remoteDomainGetSchedulerType(virDomainPtr domain, int *nparams) { - remote_domain_get_scheduler_type_args args; + remote_domain_get_scheduler_type_args args = {0}; remote_domain_get_scheduler_type_ret ret = {0}; struct private_data *priv = domain->conn->privateData; VIR_LOCK_GUARD lock = remoteDriverLock(priv); @@ -2483,7 +2483,7 @@ remoteDomainMemoryStats(virDomainPtr domain, unsigned int flags) { int rv = -1; - remote_domain_memory_stats_args args; + remote_domain_memory_stats_args args = {0}; remote_domain_memory_stats_ret ret = {0}; struct private_data *priv = domain->conn->privateData; size_t i; @@ -2525,7 +2525,7 @@ remoteDomainBlockPeek(virDomainPtr domain, unsigned int flags) { int rv = -1; - remote_domain_block_peek_args args; + remote_domain_block_peek_args args = {0}; remote_domain_block_peek_ret ret = {0}; struct private_data *priv = domain->conn->privateData; VIR_LOCK_GUARD lock = remoteDriverLock(priv); @@ -2572,7 +2572,7 @@ remoteDomainMemoryPeek(virDomainPtr domain, unsigned int flags) { int rv = -1; - remote_domain_memory_peek_args args; + remote_domain_memory_peek_args args = {0}; remote_domain_memory_peek_ret ret = {0}; struct private_data *priv = domain->conn->privateData; VIR_LOCK_GUARD lock = remoteDriverLock(priv); @@ -2616,7 +2616,7 @@ static int remoteDomainGetBlockJobInfo(virDomainPtr domain, virDomainBlockJobInfoPtr info, unsigned int flags) { - remote_domain_get_block_job_info_args args; + remote_domain_get_block_job_info_args args = {0}; remote_domain_get_block_job_info_ret ret = {0}; struct private_data *priv = domain->conn->privateData; VIR_LOCK_GUARD lock = remoteDriverLock(priv); @@ -2650,7 +2650,7 @@ static int remoteDomainGetBlockIoTune(virDomainPtr domain, unsigned int flags) { int rv = -1; - remote_domain_get_block_io_tune_args args; + remote_domain_get_block_io_tune_args args = {0}; remote_domain_get_block_io_tune_ret ret = {0}; struct private_data *priv = domain->conn->privateData; VIR_LOCK_GUARD lock = remoteDriverLock(priv); @@ -2700,7 +2700,7 @@ static int remoteDomainGetCPUStats(virDomainPtr domain, unsigned int flags) { struct private_data *priv = domain->conn->privateData; - remote_domain_get_cpu_stats_args args; + remote_domain_get_cpu_stats_args args = {0}; remote_domain_get_cpu_stats_ret ret = {0}; int rv = -1; int cpu; @@ -2789,7 +2789,7 @@ remoteConnectNetworkEventRegisterAny(virConnectPtr conn, virFreeCallback freecb) { struct private_data *priv = conn->privateData; - remote_connect_network_event_register_any_args args; + remote_connect_network_event_register_any_args args = {0}; remote_connect_network_event_register_any_ret ret = {0}; int callbackID; int count; @@ -2833,7 +2833,7 @@ remoteConnectNetworkEventDeregisterAny(virConnectPtr conn, int callbackID) { struct private_data *priv = conn->privateData; - remote_connect_network_event_deregister_any_args args; + remote_connect_network_event_deregister_any_args args = {0}; int eventID; int remoteID; int count; @@ -2870,7 +2870,7 @@ remoteConnectStoragePoolEventRegisterAny(virConnectPtr conn, virFreeCallback freecb) { struct private_data *priv = conn->privateData; - remote_connect_storage_pool_event_register_any_args args; + remote_connect_storage_pool_event_register_any_args args = {0}; remote_connect_storage_pool_event_register_any_ret ret = {0}; int callbackID; int count; @@ -2914,7 +2914,7 @@ remoteConnectStoragePoolEventDeregisterAny(virConnectPtr conn, int callbackID) { struct private_data *priv = conn->privateData; - remote_connect_storage_pool_event_deregister_any_args args; + remote_connect_storage_pool_event_deregister_any_args args = {0}; int eventID; int remoteID; int count; @@ -2953,7 +2953,7 @@ remoteConnectNodeDeviceEventRegisterAny(virConnectPtr conn, virFreeCallback freecb) { struct private_data *priv = conn->privateData; - remote_connect_node_device_event_register_any_args args; + remote_connect_node_device_event_register_any_args args = {0}; remote_connect_node_device_event_register_any_ret ret = {0}; int callbackID; int count; @@ -2998,7 +2998,7 @@ remoteConnectNodeDeviceEventDeregisterAny(virConnectPtr conn, int callbackID) { struct private_data *priv = conn->privateData; - remote_connect_node_device_event_deregister_any_args args; + remote_connect_node_device_event_deregister_any_args args = {0}; int eventID; int remoteID; int count; @@ -3037,7 +3037,7 @@ remoteConnectSecretEventRegisterAny(virConnectPtr conn, virFreeCallback freecb) { struct private_data *priv = conn->privateData; - remote_connect_secret_event_register_any_args args; + remote_connect_secret_event_register_any_args args = {0}; remote_connect_secret_event_register_any_ret ret = {0}; int callbackID; int count; @@ -3082,7 +3082,7 @@ remoteConnectSecretEventDeregisterAny(virConnectPtr conn, int callbackID) { struct private_data *priv = conn->privateData; - remote_connect_secret_event_deregister_any_args args; + remote_connect_secret_event_deregister_any_args args = {0}; int eventID; int remoteID; int count; @@ -3204,7 +3204,7 @@ remoteConnectFindStoragePoolSources(virConnectPtr conn, unsigned int flags) { char *rv = NULL; - remote_connect_find_storage_pool_sources_args args; + remote_connect_find_storage_pool_sources_args args = {0}; remote_connect_find_storage_pool_sources_ret ret = {0}; struct private_data *priv = conn->privateData; VIR_LOCK_GUARD lock = remoteDriverLock(priv); @@ -3229,7 +3229,7 @@ remoteConnectFindStoragePoolSources(virConnectPtr conn, static int remoteNodeDeviceDettach(virNodeDevicePtr dev) { - remote_node_device_dettach_args args; + remote_node_device_dettach_args args = {0}; struct private_data *priv = dev->conn->privateData; VIR_LOCK_GUARD lock = remoteDriverLock(priv); @@ -3248,7 +3248,7 @@ remoteNodeDeviceDetachFlags(virNodeDevicePtr dev, const char *driverName, unsigned int flags) { - remote_node_device_detach_flags_args args; + remote_node_device_detach_flags_args args = {0}; struct private_data *priv = dev->conn->privateData; VIR_LOCK_GUARD lock = remoteDriverLock(priv); @@ -3267,7 +3267,7 @@ remoteNodeDeviceDetachFlags(virNodeDevicePtr dev, static int remoteNodeDeviceReAttach(virNodeDevicePtr dev) { - remote_node_device_re_attach_args args; + remote_node_device_re_attach_args args = {0}; struct private_data *priv = dev->conn->privateData; VIR_LOCK_GUARD lock = remoteDriverLock(priv); @@ -3284,7 +3284,7 @@ remoteNodeDeviceReAttach(virNodeDevicePtr dev) static int remoteNodeDeviceReset(virNodeDevicePtr dev) { - remote_node_device_reset_args args; + remote_node_device_reset_args args = {0}; /* This method is unusual in that it uses the HV driver, not the devMon driver * hence its use of privateData, instead of nodeDevicePrivateData */ struct private_data *priv = dev->conn->privateData; @@ -3702,9 +3702,9 @@ remoteAuthSASL(virConnectPtr conn, struct private_data *priv, virConnectAuthPtr auth, const char *wantmech) { remote_auth_sasl_init_ret iret; - remote_auth_sasl_start_args sargs; + remote_auth_sasl_start_args sargs = {0}; remote_auth_sasl_start_ret sret; - remote_auth_sasl_step_args pargs; + remote_auth_sasl_step_args pargs = {0}; remote_auth_sasl_step_ret pret; const char *clientout; char *serverin = NULL; @@ -3976,7 +3976,7 @@ remoteConnectDomainEventRegister(virConnectPtr conn, if (count == 1) { /* Tell the server when we are the first callback registering */ if (priv->serverEventFilter) { - remote_connect_domain_event_callback_register_any_args args; + remote_connect_domain_event_callback_register_any_args args = {0}; remote_connect_domain_event_callback_register_any_ret ret = {0}; args.eventID = VIR_DOMAIN_EVENT_ID_LIFECYCLE; @@ -4012,7 +4012,7 @@ remoteConnectDomainEventDeregister(virConnectPtr conn, virConnectDomainEventCallback callback) { struct private_data *priv = conn->privateData; - remote_connect_domain_event_callback_deregister_any_args args; + remote_connect_domain_event_callback_deregister_any_args args = {0}; int callbackID; int remoteID; int count; @@ -5085,7 +5085,7 @@ static unsigned char * remoteSecretGetValue(virSecretPtr secret, size_t *value_size, unsigned int flags) { - remote_secret_get_value_args args; + remote_secret_get_value_args args = {0}; remote_secret_get_value_ret ret = {0}; struct private_data *priv = secret->conn->privateData; VIR_LOCK_GUARD lock = remoteDriverLock(priv); @@ -5492,7 +5492,7 @@ remoteConnectDomainEventRegisterAny(virConnectPtr conn, * events on the server */ if (count == 1) { if (priv->serverEventFilter) { - remote_connect_domain_event_callback_register_any_args args; + remote_connect_domain_event_callback_register_any_args args = {0}; remote_connect_domain_event_callback_register_any_ret ret = {0}; args.eventID = eventID; @@ -5513,7 +5513,7 @@ remoteConnectDomainEventRegisterAny(virConnectPtr conn, virObjectEventStateSetRemote(conn, priv->eventState, callbackID, ret.callbackID); } else { - remote_connect_domain_event_register_any_args args; + remote_connect_domain_event_register_any_args args = {0}; args.eventID = eventID; @@ -5553,7 +5553,7 @@ remoteConnectDomainEventDeregisterAny(virConnectPtr conn, * events on the server */ if (count == 0) { if (priv->serverEventFilter) { - remote_connect_domain_event_callback_deregister_any_args args; + remote_connect_domain_event_callback_deregister_any_args args = {0}; args.callbackID = remoteID; @@ -5563,7 +5563,7 @@ remoteConnectDomainEventDeregisterAny(virConnectPtr conn, (xdrproc_t) xdr_void, (char *) NULL) == -1) return -1; } else { - remote_connect_domain_event_deregister_any_args args; + remote_connect_domain_event_deregister_any_args args = {0}; args.eventID = eventID; @@ -5668,7 +5668,7 @@ remoteDomainMigrateBegin3(virDomainPtr domain, unsigned long resource) { char *rv = NULL; - remote_domain_migrate_begin3_args args; + remote_domain_migrate_begin3_args args = {0}; remote_domain_migrate_begin3_ret ret = {0}; struct private_data *priv = domain->conn->privateData; VIR_LOCK_GUARD lock = remoteDriverLock(priv); @@ -5719,7 +5719,7 @@ remoteDomainMigratePrepare3(virConnectPtr dconn, const char *dom_xml) { int rv = -1; - remote_domain_migrate_prepare3_args args; + remote_domain_migrate_prepare3_args args = {0}; remote_domain_migrate_prepare3_ret ret = {0}; struct private_data *priv = dconn->privateData; VIR_LOCK_GUARD lock = remoteDriverLock(priv); @@ -5781,7 +5781,7 @@ remoteDomainMigratePrepareTunnel3(virConnectPtr dconn, const char *dom_xml) { struct private_data *priv = dconn->privateData; - remote_domain_migrate_prepare_tunnel3_args args; + remote_domain_migrate_prepare_tunnel3_args args = {0}; remote_domain_migrate_prepare_tunnel3_ret ret = {0}; virNetClientStream *netst; VIR_LOCK_GUARD lock = remoteDriverLock(priv); @@ -5847,7 +5847,7 @@ remoteDomainMigratePerform3(virDomainPtr dom, const char *dname, unsigned long resource) { - remote_domain_migrate_perform3_args args; + remote_domain_migrate_perform3_args args = {0}; remote_domain_migrate_perform3_ret ret = {0}; struct private_data *priv = dom->conn->privateData; VIR_LOCK_GUARD lock = remoteDriverLock(priv); @@ -5898,7 +5898,7 @@ remoteDomainMigrateFinish3(virConnectPtr dconn, unsigned long flags, int cancelled) { - remote_domain_migrate_finish3_args args; + remote_domain_migrate_finish3_args args = {0}; remote_domain_migrate_finish3_ret ret = {0}; struct private_data *priv = dconn->privateData; virDomainPtr rv = NULL; @@ -5947,7 +5947,7 @@ remoteDomainMigrateConfirm3(virDomainPtr domain, unsigned long flags, int cancelled) { - remote_domain_migrate_confirm3_args args; + remote_domain_migrate_confirm3_args args = {0}; struct private_data *priv = domain->conn->privateData; VIR_LOCK_GUARD lock = remoteDriverLock(priv); @@ -5975,7 +5975,7 @@ remoteConnectGetCPUModelNames(virConnectPtr conn, int rv = -1; size_t i; g_auto(GStrv) retmodels = NULL; - remote_connect_get_cpu_model_names_args args; + remote_connect_get_cpu_model_names_args args = {0}; remote_connect_get_cpu_model_names_ret ret = {0}; struct private_data *priv = conn->privateData; VIR_LOCK_GUARD lock = remoteDriverLock(priv); @@ -6024,7 +6024,7 @@ remoteDomainOpenGraphics(virDomainPtr dom, int fd, unsigned int flags) { - remote_domain_open_graphics_args args; + remote_domain_open_graphics_args args = {0}; struct private_data *priv = dom->conn->privateData; int fdin[] = { fd }; size_t fdinlen = G_N_ELEMENTS(fdin); @@ -6052,7 +6052,7 @@ remoteDomainOpenGraphicsFD(virDomainPtr dom, unsigned int flags) { int rv = -1; - remote_domain_open_graphics_fd_args args; + remote_domain_open_graphics_fd_args args = {0}; struct private_data *priv = dom->conn->privateData; int *fdout = NULL; size_t fdoutlen = 0; @@ -6139,7 +6139,7 @@ remoteDomainGetDiskErrors(virDomainPtr dom, { int rv = -1; struct private_data *priv = dom->conn->privateData; - remote_domain_get_disk_errors_args args; + remote_domain_get_disk_errors_args args = {0}; remote_domain_get_disk_errors_ret ret = {0}; VIR_LOCK_GUARD lock = remoteDriverLock(priv); @@ -6244,7 +6244,7 @@ remoteDomainGetInterfaceParameters(virDomainPtr domain, unsigned int flags) { int rv = -1; - remote_domain_get_interface_parameters_args args; + remote_domain_get_interface_parameters_args args = {0}; remote_domain_get_interface_parameters_ret ret = {0}; struct private_data *priv = domain->conn->privateData; VIR_LOCK_GUARD lock = remoteDriverLock(priv); @@ -6291,7 +6291,7 @@ remoteNodeGetMemoryParameters(virConnectPtr conn, unsigned int flags) { int rv = -1; - remote_node_get_memory_parameters_args args; + remote_node_get_memory_parameters_args args = {0}; remote_node_get_memory_parameters_ret ret = {0}; struct private_data *priv = conn->privateData; VIR_LOCK_GUARD lock = remoteDriverLock(priv); @@ -6336,7 +6336,7 @@ remoteNodeGetSEVInfo(virConnectPtr conn, unsigned int flags) { int rv = -1; - remote_node_get_sev_info_args args; + remote_node_get_sev_info_args args = {0}; remote_node_get_sev_info_ret ret = {0}; struct private_data *priv = conn->privateData; VIR_LOCK_GUARD lock = remoteDriverLock(priv); @@ -6370,7 +6370,7 @@ remoteNodeGetCPUMap(virConnectPtr conn, unsigned int flags) { int rv = -1; - remote_node_get_cpu_map_args args; + remote_node_get_cpu_map_args args = {0}; remote_node_get_cpu_map_ret ret = {0}; struct private_data *priv = conn->privateData; VIR_LOCK_GUARD lock = remoteDriverLock(priv); @@ -6439,7 +6439,7 @@ remoteDomainGetJobStats(virDomainPtr domain, unsigned int flags) { int rv = -1; - remote_domain_get_job_stats_args args; + remote_domain_get_job_stats_args args = {0}; remote_domain_get_job_stats_ret ret = {0}; struct private_data *priv = domain->conn->privateData; VIR_LOCK_GUARD lock = remoteDriverLock(priv); @@ -6478,7 +6478,7 @@ remoteDomainMigrateBegin3Params(virDomainPtr domain, unsigned int flags) { char *rv = NULL; - remote_domain_migrate_begin3_params_args args; + remote_domain_migrate_begin3_params_args args = {0}; remote_domain_migrate_begin3_params_ret ret = {0}; struct private_data *priv = domain->conn->privateData; VIR_LOCK_GUARD lock = remoteDriverLock(priv); @@ -6536,7 +6536,7 @@ remoteDomainMigratePrepare3Params(virConnectPtr dconn, unsigned int flags) { int rv = -1; - remote_domain_migrate_prepare3_params_args args; + remote_domain_migrate_prepare3_params_args args = {0}; remote_domain_migrate_prepare3_params_ret ret = {0}; struct private_data *priv = dconn->privateData; VIR_LOCK_GUARD lock = remoteDriverLock(priv); @@ -6607,7 +6607,7 @@ remoteDomainMigratePrepareTunnel3Params(virConnectPtr dconn, { struct private_data *priv = dconn->privateData; int rv = -1; - remote_domain_migrate_prepare_tunnel3_params_args args; + remote_domain_migrate_prepare_tunnel3_params_args args = {0}; remote_domain_migrate_prepare_tunnel3_params_ret ret = {0}; virNetClientStream *netst; VIR_LOCK_GUARD lock = remoteDriverLock(priv); @@ -6684,7 +6684,7 @@ remoteDomainMigratePerform3Params(virDomainPtr dom, unsigned int flags) { int rv = -1; - remote_domain_migrate_perform3_params_args args; + remote_domain_migrate_perform3_params_args args = {0}; remote_domain_migrate_perform3_params_ret ret = {0}; struct private_data *priv = dom->conn->privateData; VIR_LOCK_GUARD lock = remoteDriverLock(priv); @@ -6744,7 +6744,7 @@ remoteDomainMigrateFinish3Params(virConnectPtr dconn, unsigned int flags, int cancelled) { - remote_domain_migrate_finish3_params_args args; + remote_domain_migrate_finish3_params_args args = {0}; remote_domain_migrate_finish3_params_ret ret = {0}; struct private_data *priv = dconn->privateData; virDomainPtr rv = NULL; @@ -6807,7 +6807,7 @@ remoteDomainMigrateConfirm3Params(virDomainPtr domain, int cancelled) { int rv = -1; - remote_domain_migrate_confirm3_params_args args; + remote_domain_migrate_confirm3_params_args args = {0}; struct private_data *priv = domain->conn->privateData; VIR_LOCK_GUARD lock = remoteDriverLock(priv); @@ -6845,7 +6845,7 @@ remoteDomainCreateXMLWithFiles(virConnectPtr conn, const char *xml_desc, { virDomainPtr rv = NULL; struct private_data *priv = conn->privateData; - remote_domain_create_xml_with_files_args args; + remote_domain_create_xml_with_files_args args = {0}; remote_domain_create_xml_with_files_ret ret = {0}; VIR_LOCK_GUARD lock = remoteDriverLock(priv); @@ -6873,7 +6873,7 @@ remoteDomainCreateWithFiles(virDomainPtr dom, unsigned int flags) { struct private_data *priv = dom->conn->privateData; - remote_domain_create_with_files_args args; + remote_domain_create_with_files_args args = {0}; remote_domain_create_with_files_ret ret = {0}; VIR_LOCK_GUARD lock = remoteDriverLock(priv); @@ -6900,7 +6900,7 @@ remoteDomainGetTime(virDomainPtr dom, unsigned int flags) { struct private_data *priv = dom->conn->privateData; - remote_domain_get_time_args args; + remote_domain_get_time_args args = {0}; remote_domain_get_time_ret ret = {0}; VIR_LOCK_GUARD lock = remoteDriverLock(priv); @@ -6930,7 +6930,7 @@ remoteNodeGetFreePages(virConnectPtr conn, unsigned long long *counts, unsigned int flags) { - remote_node_get_free_pages_args args; + remote_node_get_free_pages_args args = {0}; remote_node_get_free_pages_ret ret = {0}; struct private_data *priv = conn->privateData; VIR_LOCK_GUARD lock = remoteDriverLock(priv); @@ -7010,7 +7010,7 @@ remoteNetworkGetDHCPLeases(virNetworkPtr net, int rv = -1; size_t i; struct private_data *priv = net->conn->privateData; - remote_network_get_dhcp_leases_args args; + remote_network_get_dhcp_leases_args args = {0}; remote_network_get_dhcp_leases_ret ret = {0}; virNetworkDHCPLeasePtr *leases_ret = NULL; VIR_LOCK_GUARD lock = remoteDriverLock(priv); @@ -7072,7 +7072,7 @@ remoteConnectGetAllDomainStats(virConnectPtr conn, struct private_data *priv = conn->privateData; int rv = -1; size_t i; - remote_connect_get_all_domain_stats_args args; + remote_connect_get_all_domain_stats_args args = {0}; remote_connect_get_all_domain_stats_ret ret = {0}; virDomainStatsRecordPtr elem = NULL; virDomainStatsRecordPtr *tmpret = NULL; @@ -7150,7 +7150,7 @@ remoteNodeAllocPages(virConnectPtr conn, unsigned int cellCount, unsigned int flags) { - remote_node_alloc_pages_args args; + remote_node_alloc_pages_args args = {0}; remote_node_alloc_pages_ret ret = {0}; struct private_data *priv = conn->privateData; VIR_LOCK_GUARD lock = remoteDriverLock(priv); @@ -7187,7 +7187,7 @@ remoteDomainGetFSInfo(virDomainPtr dom, int rv = -1; size_t i, j, len; struct private_data *priv = dom->conn->privateData; - remote_domain_get_fsinfo_args args; + remote_domain_get_fsinfo_args args = {0}; remote_domain_get_fsinfo_ret ret = {0}; remote_domain_fsinfo *src; VIR_LOCK_GUARD lock = remoteDriverLock(priv); @@ -7261,7 +7261,7 @@ remoteDomainInterfaceAddresses(virDomainPtr dom, size_t i, j; virDomainInterfacePtr *ifaces_ret = NULL; - remote_domain_interface_addresses_args args; + remote_domain_interface_addresses_args args = {0}; remote_domain_interface_addresses_ret ret = {0}; struct private_data *priv = dom->conn->privateData; VIR_LOCK_GUARD lock = remoteDriverLock(priv); @@ -7393,7 +7393,7 @@ remoteDomainRename(virDomainPtr dom, const char *new_name, unsigned int flags) { int rv = -1; struct private_data *priv = dom->conn->privateData; - remote_domain_rename_args args; + remote_domain_rename_args args = {0}; remote_domain_rename_ret ret = {0}; char *tmp = NULL; VIR_LOCK_GUARD lock = remoteDriverLock(priv); @@ -7427,7 +7427,7 @@ remoteStorageVolGetInfoFlags(virStorageVolPtr vol, unsigned int flags) { struct private_data *priv = vol->conn->privateData; - remote_storage_vol_get_info_flags_args args; + remote_storage_vol_get_info_flags_args args = {0}; remote_storage_vol_get_info_flags_ret ret = {0}; VIR_LOCK_GUARD lock = remoteDriverLock(priv); @@ -7457,7 +7457,7 @@ remoteNetworkPortGetParameters(virNetworkPortPtr port, { int rv = -1; struct private_data *priv = port->net->conn->privateData; - remote_network_port_get_parameters_args args; + remote_network_port_get_parameters_args args = {0}; remote_network_port_get_parameters_ret ret = {0}; VIR_LOCK_GUARD lock = remoteDriverLock(priv); @@ -7492,7 +7492,7 @@ remoteDomainGetGuestInfo(virDomainPtr dom, { int rv = -1; struct private_data *priv = dom->conn->privateData; - remote_domain_get_guest_info_args args; + remote_domain_get_guest_info_args args = {0}; remote_domain_get_guest_info_ret ret = {0}; VIR_LOCK_GUARD lock = remoteDriverLock(priv); @@ -7530,7 +7530,7 @@ remoteDomainAuthorizedSSHKeysGet(virDomainPtr domain, int rv = -1; size_t i; struct private_data *priv = domain->conn->privateData; - remote_domain_authorized_ssh_keys_get_args args; + remote_domain_authorized_ssh_keys_get_args args = {0}; remote_domain_authorized_ssh_keys_get_ret ret = {0}; VIR_LOCK_GUARD lock = remoteDriverLock(priv); @@ -7570,7 +7570,7 @@ remoteDomainAuthorizedSSHKeysSet(virDomainPtr domain, unsigned int flags) { struct private_data *priv = domain->conn->privateData; - remote_domain_authorized_ssh_keys_set_args args; + remote_domain_authorized_ssh_keys_set_args args = {0}; VIR_LOCK_GUARD lock = remoteDriverLock(priv); if (nkeys > REMOTE_DOMAIN_AUTHORIZED_SSH_KEYS_MAX) { @@ -7603,7 +7603,7 @@ remoteDomainGetMessages(virDomainPtr domain, int rv = -1; size_t i; struct private_data *priv = domain->conn->privateData; - remote_domain_get_messages_args args; + remote_domain_get_messages_args args = {0}; remote_domain_get_messages_ret ret = {0}; VIR_LOCK_GUARD lock = remoteDriverLock(priv); @@ -7643,7 +7643,7 @@ remoteDomainFDAssociate(virDomainPtr domain, int *fds, unsigned int flags) { - remote_domain_fd_associate_args args; + remote_domain_fd_associate_args args = {0}; struct private_data *priv = domain->conn->privateData; VIR_LOCK_GUARD lock = remoteDriverLock(priv); -- 2.39.1

4 4

[libvirt PATCH] remote: Fix version annotation for remoteDomainFDAssociate
by Jiri Denemark 26 Jan '23

26 Jan '23

The API was added in libvirt 9.0.0. Signed-off-by: Jiri Denemark <jdenemar(a)redhat.com> --- src/remote/remote_driver.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/remote/remote_driver.c b/src/remote/remote_driver.c index 9f3a9520e4..2f421fb5e0 100644 --- a/src/remote/remote_driver.c +++ b/src/remote/remote_driver.c @@ -8101,7 +8101,7 @@ static virHypervisorDriver hypervisor_driver = { .domainGetMessages = remoteDomainGetMessages, /* 7.1.0 */ .domainStartDirtyRateCalc = remoteDomainStartDirtyRateCalc, /* 7.2.0 */ .domainSetLaunchSecurityState = remoteDomainSetLaunchSecurityState, /* 8.0.0 */ - .domainFDAssociate = remoteDomainFDAssociate, /* 8.9.0 */ + .domainFDAssociate = remoteDomainFDAssociate, /* 9.0.0 */ }; static virNetworkDriver network_driver = { -- 2.39.1

2 1

[libvirt PATCH 0/4] remote: Random RPC fixes and cleanups
by Jiri Denemark 26 Jan '23

26 Jan '23

Jiri Denemark (4): remote: Propagate error from virDomainGetSecurityLabelList via RPC remote: Drop useless check in remoteDispatchDomainGetIOThreadInfo remote: Avoid leaking uri_out remote: Drop useless cleanup in remoteDispatchNodeGet{CPU,Memory}Stats src/remote/remote_daemon_dispatch.c | 48 +++++++++-------------------- 1 file changed, 15 insertions(+), 33 deletions(-) -- 2.39.1

3 7

[libvirt PATCH 0/7] Extract the integration job commands to a shell scripts
by Erik Skultety 26 Jan '23

26 Jan '23

Using shell scripts rather than inlining shell commands to YAML feels more natural, more readable, and will keep all different variations of execution consistent. Essentially the only disadvantage is that we won't see each command listed one-by-one in gitlab's log output (unless we set -x that is), but given that shell would complain if something was wrong with the script, it's fairly easy to identify the problem. Here's a successful pipeline after the change: https://gitlab.com/eskultety/libvirt/-/pipelines/753090691 Erik Skultety (7): ci: integration: Extract several hidden job definitions to a script ci: integration: Drop the 'install-deps' hidden job and reference ci: integration-template: Drop Fedora 35 check ci: integration.sh: Replace most 'sudo' usage with running as root ci: integration.sh: Add/Rewrite/Reformat commentaries ci: integration.sh: Replace 'test' with '[' operator ci: integration.sh: Define the SCRATCH_DIR variable for local execution ci/integration-template.yml | 42 ++---------------------------- ci/integration.sh | 51 +++++++++++++++++++++++++++++++++++++ 2 files changed, 53 insertions(+), 40 deletions(-) create mode 100644 ci/integration.sh -- 2.39.0

3 23

[PATCH] qemu_domain: Don't unref NULL hash table in qemuDomainRefreshStatsSchema()
by Michal Privoznik 26 Jan '23

26 Jan '23

The g_hash_table_unref() function does not accept NULL. Passing NULL results in a glib warning being triggered. Check whether the hash table is not NULL and unref it only then. Signed-off-by: Michal Privoznik <mprivozn(a)redhat.com> --- src/qemu/qemu_domain.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/src/qemu/qemu_domain.c b/src/qemu/qemu_domain.c index b6ad118f1f..881c88d54c 100644 --- a/src/qemu/qemu_domain.c +++ b/src/qemu/qemu_domain.c @@ -11923,7 +11923,8 @@ qemuDomainRefreshStatsSchema(virDomainObj *dom) if (!schema) return -1; - g_hash_table_unref(priv->statsSchema); + if (priv->statsSchema) + g_hash_table_unref(priv->statsSchema); priv->statsSchema = schema; return 0; -- 2.39.1

2 1

[PATCH 0/7] qemu: Fix implicit q35 watchdog
by Martin Kletzander 26 Jan '23

26 Jan '23

More info in commit messages (duh). Martin Kletzander (7): Avoid unnecessary nesting Support multiple watchdog devices Add iTCO watchdog support qemu: Enable iTCO watchdog by disabling its noreboot pin strap qemu: Add implicit watchdog for q35 machine types Document change to multiple watchdogs news: Add information about iTCO watchdog changes NEWS.rst | 6 ++ docs/formatdomain.rst | 10 ++- src/conf/domain_conf.c | 83 +++++++++++++------ src/conf/domain_conf.h | 9 +- src/conf/schemas/domaincommon.rng | 5 +- src/libvirt_private.syms | 1 + src/qemu/qemu_alias.c | 26 ++++-- src/qemu/qemu_alias.h | 5 +- src/qemu/qemu_command.c | 34 ++++++-- src/qemu/qemu_domain.c | 44 ++++++++++ src/qemu/qemu_domain_address.c | 9 +- src/qemu/qemu_driver.c | 14 ++-- src/qemu/qemu_hotplug.c | 71 +++++++--------- src/qemu/qemu_process.c | 3 +- src/qemu/qemu_validate.c | 50 +++++++++++ .../qemustatusxml2xmldata/vcpus-multi-in.xml | 1 + tests/qemuxml2argvdata/autoindex.args | 2 + .../boot-floppy-q35.x86_64-latest.args | 2 + tests/qemuxml2argvdata/boot-floppy-q35.xml | 1 + tests/qemuxml2argvdata/cpu-host-model.args | 2 + .../cpu-host-model.x86_64-4.2.0.args | 2 + .../cpu-host-model.x86_64-5.0.0.args | 2 + .../cpu-host-model.x86_64-5.1.0.args | 2 + .../cpu-host-model.x86_64-5.2.0.args | 2 + .../cpu-host-model.x86_64-6.0.0.args | 2 + .../cpu-host-model.x86_64-6.1.0.args | 2 + .../cpu-host-model.x86_64-latest.args | 2 + .../disk-floppy-q35.x86_64-latest.args | 2 + ...are-auto-bios-stateless.x86_64-latest.args | 2 + .../firmware-auto-bios.x86_64-latest.args | 2 + ...-auto-efi-enrolled-keys.x86_64-latest.args | 2 + ...-auto-efi-loader-secure.x86_64-latest.args | 2 + ...to-efi-no-enrolled-keys.x86_64-latest.args | 2 + ...are-auto-efi-no-secboot.x86_64-latest.args | 2 + ...firmware-auto-efi-nvram.x86_64-latest.args | 2 + ...rmware-auto-efi-secboot.x86_64-latest.args | 2 + ...ware-auto-efi-stateless.x86_64-latest.args | 2 + .../firmware-auto-efi.x86_64-latest.args | 2 + .../firmware-manual-efi-acpi-q35.args | 2 + .../firmware-manual-efi-secure.args | 2 + .../firmware-manual-noefi-acpi-q35.args | 2 + .../firmware-manual-noefi-noacpi-q35.args | 2 + ...dev-scsi-vhost-scsi-pcie.x86_64-4.2.0.args | 2 + ...ev-scsi-vhost-scsi-pcie.x86_64-latest.args | 2 + .../hvf-x86_64-q35-headless.args | 2 + .../intel-iommu-aw-bits.x86_64-latest.args | 2 + .../qemuxml2argvdata/intel-iommu-aw-bits.xml | 1 + ...ntel-iommu-caching-mode.x86_64-latest.args | 2 + .../intel-iommu-caching-mode.xml | 1 + ...ntel-iommu-device-iotlb.x86_64-latest.args | 2 + .../intel-iommu-device-iotlb.xml | 1 + .../intel-iommu-eim.x86_64-latest.args | 2 + tests/qemuxml2argvdata/intel-iommu-eim.xml | 1 + .../intel-iommu.x86_64-latest.args | 2 + tests/qemuxml2argvdata/intel-iommu.xml | 1 + ...othreads-ids-pool-sizes.x86_64-latest.args | 2 + .../iothreads-ids-pool-sizes.xml | 1 + .../machine-smm-off.x86_64-latest.args | 2 + .../machine-smm-on.x86_64-latest.args | 2 + tests/qemuxml2argvdata/pcie-expander-bus.args | 2 + .../pcie-root-port-model-generic.args | 2 + .../pcie-root-port-model-ioh3420.args | 2 + ...cie-root-port-nohotplug.x86_64-latest.args | 2 + tests/qemuxml2argvdata/pcie-root-port.args | 2 + tests/qemuxml2argvdata/pcie-root.args | 2 + .../pcie-switch-downstream-port.args | 2 + .../pcie-switch-upstream-port.args | 2 + tests/qemuxml2argvdata/pcihole64-q35.args | 2 + ...q35-default-devices-only.x86_64-4.2.0.args | 2 + ...35-default-devices-only.x86_64-latest.args | 2 + .../q35-multifunction.x86_64-4.2.0.args | 2 + .../q35-multifunction.x86_64-latest.args | 2 + .../q35-pci-force-address.args | 2 + .../q35-pcie-autoadd.x86_64-4.2.0.args | 2 + .../q35-pcie-autoadd.x86_64-latest.args | 2 + .../q35-pcie.x86_64-4.2.0.args | 2 + .../q35-pcie.x86_64-latest.args | 2 + .../q35-pm-disable-fallback.args | 2 + tests/qemuxml2argvdata/q35-pm-disable.args | 2 + tests/qemuxml2argvdata/q35-usb2-multi.args | 2 + tests/qemuxml2argvdata/q35-usb2-reorder.args | 2 + tests/qemuxml2argvdata/q35-usb2.args | 2 + .../q35-virt-manager-basic.x86_64-4.2.0.args | 2 + .../q35-virt-manager-basic.x86_64-latest.args | 2 + tests/qemuxml2argvdata/q35.args | 2 + .../sgx-epc.x86_64-7.0.0.args | 2 + tests/qemuxml2argvdata/sgx-epc.xml | 1 + .../tseg-explicit-size.x86_64-latest.args | 2 + .../usb-controller-default-q35.args | 2 + .../usb-controller-explicit-q35.args | 2 + .../user-aliases2.x86_64-latest.args | 2 + .../vcpu-placement-static.args | 2 + .../vcpu-placement-static.xml | 1 + ...vhost-user-fs-hugepages.x86_64-latest.args | 2 + .../vhost-user-fs-hugepages.xml | 1 + .../virtio-iommu-x86_64.x86_64-latest.args | 2 + ...virtio-non-transitional.x86_64-latest.args | 2 + .../virtio-transitional.x86_64-latest.args | 2 + .../watchdog-q35-multiple.x86_64-latest.args | 40 +++++++++ .../watchdog-q35-multiple.xml | 25 ++++++ ...default-cpu-kvm-q35-4.2.x86_64-latest.args | 2 + ...efault-cpu-tcg-features.x86_64-latest.args | 2 + .../x86_64-default-cpu-tcg-features.xml | 1 + ...default-cpu-tcg-q35-4.2.x86_64-latest.args | 2 + .../x86_64-q35-graphics.x86_64-latest.args | 2 + .../x86_64-q35-headless.x86_64-latest.args | 2 + tests/qemuxml2argvtest.c | 1 + tests/qemuxml2xmloutdata/autoindex.xml | 1 + ...ware-auto-bios-stateless.x86_64-latest.xml | 1 + .../firmware-auto-bios.x86_64-latest.xml | 1 + ...e-auto-efi-enrolled-keys.x86_64-latest.xml | 1 + ...e-auto-efi-loader-secure.x86_64-latest.xml | 1 + ...uto-efi-no-enrolled-keys.x86_64-latest.xml | 1 + ...ware-auto-efi-no-secboot.x86_64-latest.xml | 1 + .../firmware-auto-efi-nvram.x86_64-latest.xml | 1 + ...irmware-auto-efi-secboot.x86_64-latest.xml | 1 + .../firmware-auto-efi.x86_64-latest.xml | 1 + ...dev-scsi-vhost-scsi-pcie.x86_64-latest.xml | 1 + .../hvf-x86_64-q35-headless.xml | 1 + .../machine-smm-off.x86_64-latest.xml | 1 + .../machine-smm-on.x86_64-latest.xml | 1 + .../net-isolated-port.x86_64-latest.xml | 1 + .../qemuxml2xmloutdata/pcie-expander-bus.xml | 1 + .../pcie-root-port-model-generic.xml | 1 + .../pcie-root-port-model-ioh3420.xml | 1 + ...pcie-root-port-nohotplug.x86_64-latest.xml | 1 + tests/qemuxml2xmloutdata/pcie-root-port.xml | 1 + tests/qemuxml2xmloutdata/pcie-root.xml | 1 + .../pcie-switch-downstream-port.xml | 1 + .../pcie-switch-upstream-port.xml | 1 + tests/qemuxml2xmloutdata/pcihole64-q35.xml | 1 + ...q35-default-devices-only.x86_64-latest.xml | 1 + .../q35-multifunction.x86_64-latest.xml | 1 + .../q35-pci-force-address.xml | 1 + .../q35-pcie-autoadd.x86_64-latest.xml | 1 + .../q35-pcie.x86_64-latest.xml | 1 + tests/qemuxml2xmloutdata/q35-usb2-multi.xml | 1 + tests/qemuxml2xmloutdata/q35-usb2-reorder.xml | 1 + tests/qemuxml2xmloutdata/q35-usb2.xml | 1 + .../q35-virt-manager-basic.x86_64-latest.xml | 1 + tests/qemuxml2xmloutdata/q35.xml | 1 + .../tseg-explicit-size.x86_64-latest.xml | 1 + .../usb-controller-default-q35.xml | 1 + .../usb-controller-explicit-q35.xml | 1 + .../virtio-iommu-x86_64.x86_64-latest.xml | 1 + .../virtio-non-transitional.x86_64-latest.xml | 1 + .../virtio-transitional.x86_64-latest.xml | 1 + .../watchdog-q35-multiple.x86_64-latest.xml | 51 ++++++++++++ ...-default-cpu-kvm-q35-4.2.x86_64-latest.xml | 1 + ...-default-cpu-tcg-q35-4.2.x86_64-latest.xml | 1 + .../x86_64-q35-graphics.x86_64-latest.xml | 1 + .../x86_64-q35-headless.x86_64-latest.xml | 1 + tests/qemuxml2xmltest.c | 1 + 153 files changed, 601 insertions(+), 97 deletions(-) create mode 100644 tests/qemuxml2argvdata/watchdog-q35-multiple.x86_64-latest.args create mode 100644 tests/qemuxml2argvdata/watchdog-q35-multiple.xml create mode 100644 tests/qemuxml2xmloutdata/watchdog-q35-multiple.x86_64-latest.xml -- 2.39.1

3 13

[PATCH v2 0/5] Support crypto device
by zhenwei pi 25 Jan '23

25 Jan '23

v1 -> v2: - Rebase code against the latest commit: 46aee2a9255adf842ab44a9292acb46189a726f7 - Merge previous 1/7 and 2/7 into a single patch: conf: introduce crypto device - Add missing qemuxml2xmltest. - Merge previous 3/7 and 4/7 into a single patch: capabilities: introduce crypto - Add missing formatdomaincaps.rst. - Several fixes from Michal Prívozník' crypto_review branch. - Add caps and test for qemu-8.0. Thanks to Michal for lots of suggestions! v1: This series supports crypto device, also add support for QEMU. The basic xml schema looks like: <crypto model='virtio' type='qemu'> <backend model='builtin' queues='1'/> <address type='pci' domain='0x0000' bus='0x00' slot='0x0a' function='0x0'/> </crypto> <crypto model='virtio' type='qemu'> <backend model='lkcf'/> <address type='pci' domain='0x0000' bus='0x00' slot='0x0b' function='0x0'/> </crypto> Each patch has been tested by 'ninja -C build test', and launch a QEMU with crypto devices, it works fine. zhenwei pi (5): conf: introduce crypto device capabilities: introduce crypto qemu: alias: support crypto device qemu: command: support crypto device NEWS: Document 'crypto' device NEWS.rst | 3 + docs/formatdomain.rst | 21 +++ docs/formatdomaincaps.rst | 33 ++++ src/ch/ch_domain.c | 1 + src/conf/domain_capabilities.c | 15 ++ src/conf/domain_capabilities.h | 12 ++ src/conf/domain_conf.c | 158 ++++++++++++++++++ src/conf/domain_conf.h | 39 +++++ src/conf/domain_postparse.c | 1 + src/conf/domain_validate.c | 18 ++ src/conf/schemas/domaincaps.rng | 10 ++ src/conf/schemas/domaincommon.rng | 58 +++++++ src/conf/virconftypes.h | 2 + src/libvirt_private.syms | 1 + src/qemu/qemu_alias.c | 23 +++ src/qemu/qemu_capabilities.c | 30 ++++ src/qemu/qemu_capabilities.h | 6 + src/qemu/qemu_command.c | 109 ++++++++++++ src/qemu/qemu_domain.c | 3 + src/qemu/qemu_domain_address.c | 26 +++ src/qemu/qemu_driver.c | 5 + src/qemu/qemu_hotplug.c | 3 + src/qemu/qemu_validate.c | 22 +++ .../domaincapsdata/qemu_4.2.0-q35.x86_64.xml | 11 ++ .../domaincapsdata/qemu_4.2.0-tcg.x86_64.xml | 11 ++ .../qemu_4.2.0-virt.aarch64.xml | 11 ++ tests/domaincapsdata/qemu_4.2.0.aarch64.xml | 11 ++ tests/domaincapsdata/qemu_4.2.0.ppc64.xml | 11 ++ tests/domaincapsdata/qemu_4.2.0.s390x.xml | 11 ++ tests/domaincapsdata/qemu_4.2.0.x86_64.xml | 11 ++ .../domaincapsdata/qemu_5.0.0-q35.x86_64.xml | 11 ++ .../domaincapsdata/qemu_5.0.0-tcg.x86_64.xml | 11 ++ .../qemu_5.0.0-virt.aarch64.xml | 11 ++ tests/domaincapsdata/qemu_5.0.0.aarch64.xml | 11 ++ tests/domaincapsdata/qemu_5.0.0.ppc64.xml | 11 ++ tests/domaincapsdata/qemu_5.0.0.x86_64.xml | 11 ++ .../domaincapsdata/qemu_5.1.0-q35.x86_64.xml | 11 ++ .../domaincapsdata/qemu_5.1.0-tcg.x86_64.xml | 11 ++ tests/domaincapsdata/qemu_5.1.0.sparc.xml | 9 + tests/domaincapsdata/qemu_5.1.0.x86_64.xml | 11 ++ .../domaincapsdata/qemu_5.2.0-q35.x86_64.xml | 11 ++ .../domaincapsdata/qemu_5.2.0-tcg.x86_64.xml | 11 ++ .../qemu_5.2.0-virt.aarch64.xml | 11 ++ tests/domaincapsdata/qemu_5.2.0.aarch64.xml | 11 ++ tests/domaincapsdata/qemu_5.2.0.ppc64.xml | 11 ++ tests/domaincapsdata/qemu_5.2.0.s390x.xml | 11 ++ tests/domaincapsdata/qemu_5.2.0.x86_64.xml | 11 ++ .../domaincapsdata/qemu_6.0.0-q35.x86_64.xml | 11 ++ .../domaincapsdata/qemu_6.0.0-tcg.x86_64.xml | 11 ++ .../qemu_6.0.0-virt.aarch64.xml | 11 ++ tests/domaincapsdata/qemu_6.0.0.aarch64.xml | 11 ++ tests/domaincapsdata/qemu_6.0.0.s390x.xml | 11 ++ tests/domaincapsdata/qemu_6.0.0.x86_64.xml | 11 ++ .../domaincapsdata/qemu_6.1.0-q35.x86_64.xml | 11 ++ .../domaincapsdata/qemu_6.1.0-tcg.x86_64.xml | 11 ++ tests/domaincapsdata/qemu_6.1.0.x86_64.xml | 11 ++ .../domaincapsdata/qemu_6.2.0-q35.x86_64.xml | 11 ++ .../domaincapsdata/qemu_6.2.0-tcg.x86_64.xml | 11 ++ .../qemu_6.2.0-virt.aarch64.xml | 11 ++ tests/domaincapsdata/qemu_6.2.0.aarch64.xml | 11 ++ tests/domaincapsdata/qemu_6.2.0.ppc64.xml | 11 ++ tests/domaincapsdata/qemu_6.2.0.x86_64.xml | 11 ++ .../domaincapsdata/qemu_7.0.0-q35.x86_64.xml | 11 ++ .../domaincapsdata/qemu_7.0.0-tcg.x86_64.xml | 11 ++ .../qemu_7.0.0-virt.aarch64.xml | 11 ++ tests/domaincapsdata/qemu_7.0.0.aarch64.xml | 11 ++ tests/domaincapsdata/qemu_7.0.0.ppc64.xml | 11 ++ tests/domaincapsdata/qemu_7.0.0.x86_64.xml | 11 ++ .../domaincapsdata/qemu_7.1.0-q35.x86_64.xml | 11 ++ .../domaincapsdata/qemu_7.1.0-tcg.x86_64.xml | 11 ++ tests/domaincapsdata/qemu_7.1.0.ppc64.xml | 11 ++ tests/domaincapsdata/qemu_7.1.0.x86_64.xml | 11 ++ .../domaincapsdata/qemu_7.2.0-q35.x86_64.xml | 12 ++ .../domaincapsdata/qemu_7.2.0-tcg.x86_64.xml | 12 ++ tests/domaincapsdata/qemu_7.2.0.x86_64.xml | 12 ++ .../domaincapsdata/qemu_8.0.0-q35.x86_64.xml | 12 ++ .../domaincapsdata/qemu_8.0.0-tcg.x86_64.xml | 12 ++ tests/domaincapsdata/qemu_8.0.0.x86_64.xml | 12 ++ .../caps_4.2.0.aarch64.xml | 2 + .../qemucapabilitiesdata/caps_4.2.0.ppc64.xml | 2 + .../qemucapabilitiesdata/caps_4.2.0.s390x.xml | 2 + .../caps_4.2.0.x86_64.xml | 2 + .../caps_5.0.0.aarch64.xml | 2 + .../qemucapabilitiesdata/caps_5.0.0.ppc64.xml | 2 + .../caps_5.0.0.riscv64.xml | 2 + .../caps_5.0.0.x86_64.xml | 2 + .../qemucapabilitiesdata/caps_5.1.0.sparc.xml | 1 + .../caps_5.1.0.x86_64.xml | 2 + .../caps_5.2.0.aarch64.xml | 2 + .../qemucapabilitiesdata/caps_5.2.0.ppc64.xml | 2 + .../caps_5.2.0.riscv64.xml | 2 + .../qemucapabilitiesdata/caps_5.2.0.s390x.xml | 2 + .../caps_5.2.0.x86_64.xml | 2 + .../caps_6.0.0.aarch64.xml | 2 + .../qemucapabilitiesdata/caps_6.0.0.s390x.xml | 2 + .../caps_6.0.0.x86_64.xml | 2 + .../caps_6.1.0.x86_64.xml | 2 + .../caps_6.2.0.aarch64.xml | 2 + .../qemucapabilitiesdata/caps_6.2.0.ppc64.xml | 2 + .../caps_6.2.0.x86_64.xml | 2 + .../caps_7.0.0.aarch64.xml | 2 + .../qemucapabilitiesdata/caps_7.0.0.ppc64.xml | 2 + .../caps_7.0.0.x86_64.xml | 2 + .../qemucapabilitiesdata/caps_7.1.0.ppc64.xml | 2 + .../caps_7.1.0.x86_64.xml | 2 + .../caps_7.2.0.x86_64.xml | 3 + .../caps_8.0.0.x86_64.xml | 3 + tests/qemuxml2argvdata/crypto-builtin.xml | 51 ++++++ .../crypto-builtin.x86_64-latest.xml | 1 + tests/qemuxml2xmltest.c | 2 + 110 files changed, 1321 insertions(+) create mode 100644 tests/qemuxml2argvdata/crypto-builtin.xml create mode 120000 tests/qemuxml2xmloutdata/crypto-builtin.x86_64-latest.xml -- 2.34.1

2 15

[libvirt PATCH] rpc: don't try to spawn non-existant daemon
by Daniel P. Berrangé 25 Jan '23

25 Jan '23

If libvirt is built in client only mode, the libvirtd/virtqemud/etc daemons won't exist. If the client is told to connect to a local hypervisor, it'll see the socket doesn't exist, try to spawn the daemon and then re-try connecting to the socket for a few seconds. Ultimately this will fail because the daemon doesn't exist and the user gets an error message error: Failed to connect socket to '/run/user/1000/libvirt/virtqemud-sock': No such file or directory technically this is accurate, but it doesn't help identify the root cause. With this change it will now report error: binary 'virtqemud' does not exist in $PATH: No such file or directory and will skip all the socket connect retries Signed-off-by: Daniel P. Berrangé <berrange(a)redhat.com> --- src/rpc/virnetsocket.c | 16 +++++++++++++--- 1 file changed, 13 insertions(+), 3 deletions(-) diff --git a/src/rpc/virnetsocket.c b/src/rpc/virnetsocket.c index 8280bda007..bb2e0c5d3d 100644 --- a/src/rpc/virnetsocket.c +++ b/src/rpc/virnetsocket.c @@ -123,9 +123,19 @@ VIR_ONCE_GLOBAL_INIT(virNetSocket); #ifndef WIN32 static int virNetSocketForkDaemon(const char *binary) { - g_autoptr(virCommand) cmd = virCommandNewArgList(binary, - "--timeout=120", - NULL); + g_autofree char *binarypath = virFindFileInPath(binary); + g_autoptr(virCommand) cmd = NULL; + + if (!binarypath) { + virReportSystemError(ENOENT, + _("binary '%s' does not exist in $PATH"), + binary); + return -1; + } + + cmd = virCommandNewArgList(binarypath, + "--timeout=120", + NULL); virCommandAddEnvPassCommon(cmd); virCommandAddEnvPass(cmd, "XDG_CACHE_HOME"); -- 2.39.0

2 4

[PATCH 0/3] update RISC-V QEMU caps for QEMU 8.0.0
by Daniel Henrique Barboza 24 Jan '23

24 Jan '23

Hi, This series updates RISC-V QEMU caps that weren't being updated since QEMU 5.0.0. To do that I had to add a RISC-V stub driver in src/cpu to avoid test failures after the caps update. This is a very bare-bone driver that is basically a no-op. I also took the opportunity to enable the RISC-V 'virt' machine type in domaincapstest.c to increase the test coverage for the architecture. Daniel Henrique Barboza (3): src/cpu: add a basic RiscV64 cpu driver tests: update RISC-V QEMU caps for QEMU 8.0.0 tests: add QEMU RISC-V "virt" machine in domaincapstest src/cpu/cpu.c | 2 + src/cpu/cpu_riscv64.c | 59 + src/cpu/cpu_riscv64.h | 25 + src/cpu/meson.build | 1 + .../qemu_5.0.0-tcg-virt.riscv64.xml | 142 + .../qemu_5.0.0-virt.riscv64.xml | 145 + .../qemu_5.2.0-tcg-virt.riscv64.xml | 142 + .../qemu_5.2.0-virt.riscv64.xml | 145 + .../qemu_8.0.0-tcg-virt.riscv64.xml | 148 + .../qemu_8.0.0-virt.riscv64.xml | 151 + tests/domaincapstest.c | 14 +- .../caps_8.0.0.riscv64.replies | 28554 ++++++++++++++++ .../caps_8.0.0.riscv64.xml | 157 + ...ult-video-type-riscv64.riscv64-latest.args | 11 +- .../riscv64-virt-graphics.riscv64-latest.args | 45 +- .../riscv64-virt-headless.riscv64-latest.args | 37 +- tests/testutilshostcpus.h | 10 + 17 files changed, 29741 insertions(+), 47 deletions(-) create mode 100644 src/cpu/cpu_riscv64.c create mode 100644 src/cpu/cpu_riscv64.h create mode 100644 tests/domaincapsdata/qemu_5.0.0-tcg-virt.riscv64.xml create mode 100644 tests/domaincapsdata/qemu_5.0.0-virt.riscv64.xml create mode 100644 tests/domaincapsdata/qemu_5.2.0-tcg-virt.riscv64.xml create mode 100644 tests/domaincapsdata/qemu_5.2.0-virt.riscv64.xml create mode 100644 tests/domaincapsdata/qemu_8.0.0-tcg-virt.riscv64.xml create mode 100644 tests/domaincapsdata/qemu_8.0.0-virt.riscv64.xml create mode 100644 tests/qemucapabilitiesdata/caps_8.0.0.riscv64.replies create mode 100644 tests/qemucapabilitiesdata/caps_8.0.0.riscv64.xml -- 2.39.0

2 6

[PATCH] rpc: Fix error message in virNetServerSetClientLimits
by Martin Kletzander 24 Jan '23

24 Jan '23

Commit f007940cb25a tried to change the error message so that it is unified later in 35afa1d2d6c1, but various rewrites missed this particular error message which does not make sense. Fix it so that it is the same as the other two messages checking the same thing in this file. Resolves: https://bugzilla.redhat.com/show_bug.cgi?id=2033879 Signed-off-by: Martin Kletzander <mkletzan(a)redhat.com> --- src/rpc/virnetserver.c | 5 ++--- 1 file changed, 2 insertions(+), 3 deletions(-) diff --git a/src/rpc/virnetserver.c b/src/rpc/virnetserver.c index bf0fda04ee89..e97dfe81369f 100644 --- a/src/rpc/virnetserver.c +++ b/src/rpc/virnetserver.c @@ -1127,9 +1127,8 @@ virNetServerSetClientLimits(virNetServer *srv, if (max < max_unauth) { virReportError(VIR_ERR_INVALID_ARG, "%s", - _("The overall maximum number of clients waiting " - "for authentication must not be less than the overall " - "maximum number of clients")); + _("The overall maximum number of clients must not be less " + "than the number of clients waiting for authentication")); return -1; } -- 2.39.1

2 1