On 1/13/22 15:43, Michal Prívozník wrote:
On 1/13/22 08:33, Divya Garg wrote:
> Issue
>
> Divya Garg (2):
> Add the port allocation logic for isa-serial devices.
> qemu: add index for isa-serial device using target.port
> 61 files changed, 156 insertions(+), 90 deletions(-)
>
Hey, couple of points. Usually, when sending new version we do so in a
new thread. And we don't CC random people. Us, developers, are
subscribed to the list.
Anyway, I'm raising only very basic nitpicks with your patches. I'll
squash in the fixes I'm suggesting before pushing. Speaking of which, we
are currently in freeze, preparing for tomorrow's release. so I'll push
these tomorrow.
Reviewed-by: Michal Privoznik <mprivozn(a)redhat.com>
This is now pushed. Congratulations on your first libvirt contribution!
Michal