On 11/17/24 15:44, shenjiatong wrote: > Virtio-serial-pci device is hot pluggable, losen the restriction and > allow user to hot plug it. > --- > src/qemu/qemu_hotplug.c | 3 ++- > 1 file changed, 2 insertions(+), 1 deletion(-) > > diff --git a/src/qemu/qemu_hotplug.c b/src/qemu/qemu_hotplug.c > index bddd553c88..55512476e4 100644 > --- a/src/qemu/qemu_hotplug.c > +++ b/src/qemu/qemu_hotplug.c > @@ -837,7 +837,8 @@ qemuDomainAttachControllerDevice(virDomainObj *vm, > { .controller = controller } }; > bool releaseaddr = false; > > - if (controller->type != VIR_DOMAIN_CONTROLLER_TYPE_SCSI) { > + if (controller->type != VIR_DOMAIN_CONTROLLER_TYPE_SCSI && \ > + controller->type != VIR_DOMAIN_CONTROLLER_TYPE_VIRTIO_SERIAL) { > virReportError(VIR_ERR_OPERATION_UNSUPPORTED, > _("'%1$s' controller cannot be hot plugged."), > virDomainControllerTypeToString(controller->type)); Patch looks good, but we require Signed-off-by line in commit messages [1]. Can you reply to this e-mail with your SoB line? I'll append it to the commit message. 1: https://libvirt.org/hacking.html#developer-certificate-of-origin Michal