The original extra= was not removed, so there were two extra= in the resulting config file.
It wasn't a problem for xl because the second extra= took precedence. However libvirt tests would only pick up the first extra= -- they worked by chance. Fix this issue by merging the original. Signed-off-by: Wei Liu <wei.l...@citrix.com> --- ts-debian-fixup | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/ts-debian-fixup b/ts-debian-fixup index f29971d..288766d 100755 --- a/ts-debian-fixup +++ b/ts-debian-fixup @@ -175,7 +175,7 @@ sub otherfixupcfg () { $extra .= " iommu=soft"; } - $cfg .= "\nextra='$extra'\n"; + $cfg =~ s/^extra\s*=\s*['"](.*)['"]/extra = '$1 $extra'/mg; }; sub writecfg () { -- 2.11.0 _______________________________________________ Xen-devel mailing list Xen-devel@lists.xen.org https://lists.xen.org/xen-devel