All of lore.kernel.org
 help / color / mirror / Atom feed
From: Kip Macy <kip.macy@gmail.com>
To: Keir Fraser <Keir.Fraser@cl.cam.ac.uk>
Cc: xen-devel <xen-devel@lists.xensource.com>
Subject: Re: semantics of GETDOMAININFO
Date: Sun, 8 May 2005 16:17:48 -0700	[thread overview]
Message-ID: <b1fa291705050816172964a665@mail.gmail.com> (raw)
In-Reply-To: <59b0cc0feef1cf941df30e24b38dc143@cl.cam.ac.uk>

That seems reasonable - but I find its lack of uniformity a bit grating. 

Maybe instead of having the current domain be first fit, we could add
a "next_domid" field. That way when "xm list" and the like want to
enumerate, provided they start zero, they can keep querying until
next_domid == -1. I can't think of a case when one wouldn't enumerate
the domains starting at zero. So, at least superficially, it seems
like a reasonable alternative.

I'm willing to accept the possiblity that I'm just nitpicking so I
won't push it.

                                          -Kip


On 5/8/05, Keir Fraser <Keir.Fraser@cl.cam.ac.uk> wrote:
> 
> On 8 May 2005, at 22:39, Kip Macy wrote:
> 
> > One of the things that I've always thought was weird, but didn't pay
> > too close attention to is the fact that GETDOMAININFO will return a
> > valid result even if we give it a domid that is no longer valid.
> > Looking at the code we get back the first valid domain after the domid
> > we pass in.
> >
> > What is the reason for this design choice? When I request the
> > attributes of a process I don't get the attributes for the next pid up
> > with a pid field set to the process id of the process I actually got
> > the attributes for.
> 
> Easy enumeration when we don't know which domain id's are in use. There
> are other ways to allow for enumeration that might be neater.
> 
>   -- Keir
> 
>

      reply	other threads:[~2005-05-08 23:17 UTC|newest]

Thread overview: 3+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2005-05-08 21:39 semantics of GETDOMAININFO Kip Macy
2005-05-08 22:10 ` Keir Fraser
2005-05-08 23:17   ` Kip Macy [this message]

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=b1fa291705050816172964a665@mail.gmail.com \
    --to=kip.macy@gmail.com \
    --cc=Keir.Fraser@cl.cam.ac.uk \
    --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.