On Tue, Jul 22, 2025 at 13:50:19 -0400, Aaron M. Brown wrote:
One definition per line.
Okay I will remember that going forward!
This is suspicious because this function until now just returned the next eligible port, but now it's also reserving it which doesn't seem to fit well here.
Can you please explain this?
Yes, I agree with you Peter, it is a bit unfitting, we can't process a port assigned to port 0 in virDomainVirtioSerialAddrReserve as it fails the virDomainVirtioSerialAddrIsComplete check, because that check makes sure that `info->addr.vioserial.port != 0` so such a port would show as imcomplete. And if we skip over reserving this port, that means the bitmap would show a clear bit at the first position for every subsequent device on the controller