All of lore.kernel.org
 help / color / mirror / Atom feed
From: Christian Limpach <christian.limpach@gmail.com>
To: Anthony Liguori <aliguori@us.ibm.com>
Cc: xen-devel@lists.xensource.com, Robert Read <robert@xensource.com>
Subject: Re: [Xen-changelog] New console transport and update xenconsoled.
Date: Wed, 31 Aug 2005 09:52:47 +0100	[thread overview]
Message-ID: <3d8eece205083101527735dc23@mail.gmail.com> (raw)
In-Reply-To: <4314BECA.5070104@us.ibm.com>

On 8/30/05, Anthony Liguori <aliguori@us.ibm.com> wrote:
> >>considerably more robust.  The virq's aren't always delivered on domain
> >>destruction (they are only delivered if a domain crashes or is shutdown).
> I see from your latest checkins that you found out what I was talking about.

Yes, I hope that all cases are now covered.

> Although, I'm a bit confused at the semantics now.  Previously, a
> DOM_EXC VIRQ was delivered on shutdown or crash.  In both circumstances,
> there's always going to be a destroy that occurs after it (when Xend
> decides to clean up).
> 
> Now it appears that a VIRQ is delivered on shutdown, crash, and then
> again for destroy?  This means that for some domains it's delivered
> twice (when shutdown or crashed) and some domains it's delivered only
> once (if they are manually xm destroyed)?

Yes, I guess you can actually get upto 3 VIRQs now when a domain dies:
- shutdown/crash
- destroy
- final cleanup

This is ok though, anybody who listens for the VIRQ needs to handle
spurious VIRQs anyway since multiple domains ending at the same time
can raise an arbitrary number of VIRQs.  The VIRQ only tells you that
something has changed.

> I think I understand what you guys were trying to do, in consoled today
> domains are reaped only when 1) they are dead (or dying) and 2) their
> output buffers are empty.  This is to ensure that if you're connected to
> a console and the domain crashes, you still see all the console output.
> 
> For the domain to completely reap (in the new console code), you need to
> unmap the shared frame.  I think the right way to do this is when the
> domain destroy is detected (in the polling loop), to unmap the shared
> frame but not actually reap until the above conditions are satisified.

Yeah, at the moment we're still missing the code which lets the output
buffer get drained after we detect that a domain has ended.

    christian

  reply	other threads:[~2005-08-31  8:52 UTC|newest]

Thread overview: 5+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
     [not found] <E1EA8qJ-0002Zl-23@xenbits.xensource.com>
2005-08-30 17:01 ` [Xen-changelog] New console transport and update xenconsoled Anthony Liguori
2005-08-30 17:41   ` Christian Limpach
2005-08-30 20:17     ` Anthony Liguori
2005-08-31  8:52       ` Christian Limpach [this message]
2005-08-30 19:20   ` Nivedita Singhvi

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=3d8eece205083101527735dc23@mail.gmail.com \
    --to=christian.limpach@gmail.com \
    --cc=Christian.Limpach@cl.cam.ac.uk \
    --cc=aliguori@us.ibm.com \
    --cc=robert@xensource.com \
    --cc=xen-devel@lists.xensource.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.