<div dir="ltr"><div>Hello Peter,<br><br></div><div>Thanks for the reply. I have made changes according to your suggestion and re-submitted the patch as [patch v2] on the mailing list. <br>Link:- <a href="https://www.redhat.com/archives/libvir-list/2016-February/msg01071.html">https://www.redhat.com/archives/libvir-list/2016-February/msg01071.html</a><br><br></div><div>Warm Regards,<br></div><div>Nitesh Konkar. <br></div></div><div class="gmail_extra"><br><div class="gmail_quote">On Fri, Feb 19, 2016 at 8:57 PM, Peter Krempa <span dir="ltr"><<a href="mailto:pkrempa@redhat.com" target="_blank">pkrempa@redhat.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><span class="">On Fri, Feb 19, 2016 at 07:24:17 -0500, Nitesh Konkar wrote:<br>
><br>
> This patch gives an error when migration is attempted with both<br>
> --live and --offline options.<br>
><br>
> Signed-off-by: Nitesh Konkar <<a href="mailto:nitkon12@linux.vnet.ibm.com">nitkon12@linux.vnet.ibm.com</a>><br>
> ---<br>
>  tools/virsh-domain.c | 6 ++++++<br>
>  1 file changed, 6 insertions(+)<br>
><br>
> diff --git a/tools/virsh-domain.c b/tools/virsh-domain.c<br>
> index 62acecb..527db38 100644<br>
> --- a/tools/virsh-domain.c<br>
> +++ b/tools/virsh-domain.c<br>
> @@ -9795,6 +9795,12 @@ doMigrate(void *opaque)<br>
>      } else {<br>
>          /* For traditional live migration, connect to the destination host directly. */<br>
>          virDomainPtr ddom = NULL;<br>
> +<br>
> +        if ((flags & VIR_MIGRATE_OFFLINE) && (flags & VIR_MIGRATE_LIVE)){<br>
> +            vshError(ctl, "%s", _("migrate: Using both --live and --offline option together is not permitted."));<br>
> +            goto out;<br>
> +        }<br>
<br>
<br>
</span>This should be validated in the callers using one of the<br>
VSH_EXCLUSIVE_OPTIONS_* macros.<br>
<span class="HOEnZb"><font color="#888888"><br>
Peter<br>
</font></span></blockquote></div><br></div>