On Fri, 2020-06-26 at 18:49 -0300, Daniel Henrique Barboza wrote:
Commit 076591009ad1 ("conf: fix zPCI address auto-generation on s390") is doing a check for virZPCIDeviceAddressIsIncomplete() prior to checking if the device has a ZPCI address at all. This results in errors like these when starting Libvirt:
error : virDomainDeviceInfoFormat:7527 : internal error: Missing uid or fid attribute of zPCI address
Fix it by moving virZPCIDeviceAddressIsIncomplete() after the check done by virZPCIDeviceAddressIsPresent().
Fixes: 076591009ad11ec108521b52a4945d0f895fa160 CC: Bjoern Walk <bwalk@linux.ibm.com> CC: Boris Fiuczynski <fiuczy@linux.ibm.com> CC: Shalini Chellathurai Saroja <shalini@linux.ibm.com> CC: Andrea Bolognani <abologna@redhat.com> Signed-off-by: Daniel Henrique Barboza <danielhb413@gmail.com> --- src/conf/domain_conf.c | 8 ++++---- 1 file changed, 4 insertions(+), 4 deletions(-)
Good catch! Reviewed-by: Andrea Bolognani <abologna@redhat.com> and pushed with an ever so slightly tweaked commit message. In the future, please don't include CC tags in your commits: removing them results in extra work when picking up a patch, and it's also generally not considered very polite to CC individual developers. Everyone is subscribed to the list anyway :) -- Andrea Bolognani / Red Hat / Virtualization