[libvirt] [PATCH] conf: Add serial target type to ABI stability check

Martin Kletzander mkletzan at redhat.com
Tue Oct 27 09:45:12 UTC 2015


On Wed, Oct 21, 2015 at 03:14:03PM +0800, Luyao Huang wrote:
>https://bugzilla.redhat.com/show_bug.cgi?id=1273686
>
>There is no ABI check for serial target type attribute, just
>add it.
>
>Signed-off-by: Luyao Huang <lhuang at redhat.com>
>---
> src/conf/domain_conf.c | 8 ++++++++
> 1 file changed, 8 insertions(+)
>

ACK && Pushed

>diff --git a/src/conf/domain_conf.c b/src/conf/domain_conf.c
>index 02cc8ad..a31bc05 100644
>--- a/src/conf/domain_conf.c
>+++ b/src/conf/domain_conf.c
>@@ -17225,6 +17225,14 @@ static bool
> virDomainSerialDefCheckABIStability(virDomainChrDefPtr src,
>                                     virDomainChrDefPtr dst)
> {
>+    if (src->targetType != dst->targetType) {
>+        virReportError(VIR_ERR_CONFIG_UNSUPPORTED,
>+                       _("Target serial type %s does not match source %s"),
>+                       virDomainChrSerialTargetTypeToString(dst->targetType),
>+                       virDomainChrSerialTargetTypeToString(src->targetType));
>+        return false;
>+    }
>+
>     if (src->target.port != dst->target.port) {
>         virReportError(VIR_ERR_CONFIG_UNSUPPORTED,
>                        _("Target serial port %d does not match source %d"),
>--
>1.8.3.1
>
>--
>libvir-list mailing list
>libvir-list at redhat.com
>https://www.redhat.com/mailman/listinfo/libvir-list
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 819 bytes
Desc: not available
URL: <http://listman.redhat.com/archives/libvir-list/attachments/20151027/4cb1fb76/attachment-0001.sig>


More information about the libvir-list mailing list