On 4/24/26 7:36 AM, Bruno Martins via Devel wrote:
Applied this in my dom0 (QubesOS) and tested. It's working for me.
You had flag some potential improvements for more verbosity in the run-time non-debug error handling. Let me know if you still would like those.
If you mean checking the return from g_path_get_dirname(), I realized after sending my message that we don't check the return from that anywhere else in the code, and that it can't fail - it will always return something non-null (if nothing else it will return "."), so the proper change there was just to remove the check. And in the callers of the new function, as long as virPCIDeviceIsOnRootBus() is logging an error in all cases when it returns failure, we don't need to (and shouldn't) log anything in the caller (since it's already been logged). Is there something else I've missed?
-- Best