Re: [PATCH v2 00/21] Add qemu RDP server support

So I tried testing it and I found a couple of issues. Those might be on my side, maybe caused by misunderstanding of it all. First of all <graphics type='dbus'/> fails with these patches for me even without

<graphics type='rdp'/>. That might be the source of all other issues, but since I was not sure whether to use p2p='yes' or address='unix:path=...' I tried both. When address is specified qemu tries to connect to it, so that's not the right approach. When I specify p2p='yes' then the VM starts. If I add the RDP graphics I get a "confusing" error message: error: unsupported configuration: qemu-rdp support requires a D-Bus bus graphics device. which should probably say "qemu-rdp support requires a non-p2p dbus graphics device" instead since it is looking for a p2p='no' dbus graphics.

If I delay the checking of qemu-rdp running I find out that it fails right away. It cannot connect to the unix socket, so I tried just running qemu VM with dbus graphics and qemu-rdp manually, but that failed on the missing certificate files. That could be checked before blindly passing it down to the helper, too (even though I was running it manually this time it reminded me of that possibility).

Ideally some handshake could confirm it is actually running. If not a handshake, then maybe waiting whether it listens on where it is supposed to or some other indication of it being started while checking that the PID exists, so that it is not a dumb delay. Even running it manually I cannot verify it works, but that's probably a client-side issue? Server output when trying rdesktop: Waiting for org.qemu... Starting RDP server, args: ServerArgs { bind_address: 0.0.0.0:3389, cert: None, key: None, remotefx: Disable } Cert: "/home/nert/.config/qemu-rdp/server-cert.pem", Key: "/home/nert/.config/qemu-rdp/server-key.pem" 2025-03-11T13:21:00.350891Z ERROR ironrdp_server::server: Connection error error=[no credentials while doing credssp] general error 2025-03-11T13:21:00.352401Z ERROR ironrdp_server::server: Connection error error=accept_begin failed Caused by: [failed to negotiate security protocol] general error sdl-freerdp client output (no error on the server side reported): [14:24:35:046] [449399:0006db7b] [ERROR][com.freerdp.core.transport] - [transport_default_write]: BIO_should_retry returned a system error 32: Broken pipe rdesktop client output: Failed to initialize NLA, do you have correct Kerberos TGT initialized ? Core(error): rcp_recv(), connection closed by peer

=== Anyway, I'm clearly doing something wrong, but there are some issues in the code as well and from the greater picture it should not be this complicated to test it out, should it? I can test out other patches from you if you want, and I am open to hearing tips on what I'm doing wrong.