All of lore.kernel.org
 help / color / mirror / Atom feed
From: Wei Liu <wei.liu2@citrix.com>
To: Zhigang Wang <zhigang.x.wang@oracle.com>
Cc: xen-devel <xen-devel@lists.xenproject.org>,
	Wei Liu <wei.liu2@citrix.com>
Subject: Re: xl list -l doesn't work for incoming domain
Date: Tue, 11 Nov 2014 15:20:11 +0000	[thread overview]
Message-ID: <20141111152011.GA21312@zion.uk.xensource.com> (raw)
In-Reply-To: <5462201C.302@oracle.com>

On Tue, Nov 11, 2014 at 09:41:32AM -0500, Zhigang Wang wrote:
> On 11/11/2014 06:01 AM, Wei Liu wrote:
> > On Mon, Nov 10, 2014 at 12:54:18PM -0500, Zhigang Wang wrote:
> > [...]
> >> Could you please explain what does "no configuration" means?
> >>
> >> Do you mean no info for the domain at all? If this is the case, it seems not consistent with xl list without -l.
> >>
> > 
> > That means no configuration at all. I think a skeleton can be provided
> > at best (in xl level) to be consistent with "xl list", which should
> > include domid and domain name etc. Since nothing else exists in
> > xenstore yet, there's no point poking there.
> 
> This approach should works for me.
> 
> > [...]
> >> Currently we want our APIs to get domain info by invoking xl list -l, but we can change them to get necessary info from other places.
> >>
> > 
> > Hmm... I don't think poking around in different places is a good idea.
> > This is prone to breakage in the future.
> 
> I agree.
>  
> > Since xenstore is not filled in when your tool looks at it, what makes
> > it different to wait a bit until migration finishes?
> 
> The logic is: when migration started, high level management console will check
> destination side to make sure the VM is running there 
> (currently call xl list -l <domain>).
> 
> If I can get enough runtime info (even some are missing), I think it should be OK.
> 

What runtime info do you need?

As I understand it's something in the long output -- otherwise you would
have used the short output already if you only need to check the
existence and / or state of a domain.

> > And, from your earlier reply, you're implying Xend fires
> > @introduceDomain at the same point as xl, but your tool can work with
> > it?
> 
> For xm/xend, VM xenstore entries already populated before @introduceDomain.
> "xm list -l" will show the right information.
> 
> Anyone knows what prevents us from populating VM xenstore entries during migration in libxl?
> 

FWIW in libxl @introduceDomain is fired after domain is built, before
adding devices. If you're after device information, it's a bit late.

Xend might have done it in different order. I don't know.

Whether we can change libxl to do so, I'm not sure. But it's definitely
not 4.5 material.

Wei.

> Thanks,
> 
> Zhigang

  reply	other threads:[~2014-11-11 15:23 UTC|newest]

Thread overview: 25+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2014-11-06 19:14 xl list -l doesn't work for incoming domain Zhigang Wang
2014-11-07 10:47 ` Wei Liu
2014-11-07 16:34   ` Zhigang Wang
2014-11-10 12:35     ` Wei Liu
2014-11-10 12:38       ` Ian Campbell
2014-11-10 13:54         ` Wei Liu
2014-11-10 14:05           ` Ian Campbell
2014-11-10 14:16             ` Wei Liu
2014-11-10 15:01       ` Zhigang Wang
2014-11-10 15:25         ` Wei Liu
2014-11-10 17:08           ` Zhigang Wang
2014-11-10 17:24             ` Wei Liu
2014-11-10 17:54               ` Zhigang Wang
2014-11-11 11:01                 ` Wei Liu
2014-11-11 14:41                   ` Zhigang Wang
2014-11-11 15:20                     ` Wei Liu [this message]
2014-11-11 16:42                       ` Zhigang Wang
2014-11-12 11:31                         ` Wei Liu
2014-11-12 14:36                           ` Zhigang Wang
2014-11-12 14:40                             ` Ian Campbell
2014-11-12 14:48                               ` Zhigang Wang
2014-11-12 14:52                                 ` Ian Campbell
2014-11-12 15:04                                   ` Zhigang Wang
2014-11-12 15:40                                     ` Wei Liu
2014-11-13 11:44                                     ` Ian Campbell

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20141111152011.GA21312@zion.uk.xensource.com \
    --to=wei.liu2@citrix.com \
    --cc=xen-devel@lists.xenproject.org \
    --cc=zhigang.x.wang@oracle.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.