All of lore.kernel.org
 help / color / mirror / Atom feed
From: Anthony Liguori <aliguori@us.ibm.com>
To: Anthony Liguori <aliguori@us.ibm.com>
Cc: Rusty Russell <rusty@rustcorp.com.au>,
	xen-devel <xen-devel@lists.xensource.com>,
	Christian Limpach <Christian.Limpach@cl.cam.ac.uk>
Subject: Re: [PATCH 0/2] Change xs_read_watch interfaces to support variable arguments
Date: Mon, 03 Oct 2005 23:48:17 -0500	[thread overview]
Message-ID: <43420991.3070905@us.ibm.com> (raw)
In-Reply-To: <4342087F.5010505@us.ibm.com>

I should mention that I ran the xenstore testsuite (which rocks btw) and 
tested the block devices and network devices.  I haven't tested blktap 
and vtpm but the changes are identical (and I don't have any test cases 
for those).

Regards,

Anthony Liguori

Anthony Liguori wrote:

> The following set of changes modify the read_watch interface in 
> userspace and in the kernel to support variable arguments.  This is in 
> preparation for adding domid to @{introduce,release}Domain watches.  
> This will allow backends to automatically reap on domain destruction 
> which should eliminate a large number of the zombie problems we're 
> having on domain destruction since Xend no longer plays a role in 
> freeing that memory.
>
> The interface for read_watch is now consistent between userspace and 
> kernel space.  It's modelled after xs_read_directory.  The returned 
> array is indexable via XS_WATCH_* macros.
>
> While slightly awkward, IMHO it seems like the cleanest and most 
> flexible option of the proposed interfaces.
>
> Signed-off-by: Anthony Liguori <aliguori@us.ibm.com>
>
> Regards,
>
> Anthony Liguori
>
> _______________________________________________
> Xen-devel mailing list
> Xen-devel@lists.xensource.com
> http://lists.xensource.com/xen-devel
>

  reply	other threads:[~2005-10-04  4:48 UTC|newest]

Thread overview: 5+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2005-10-04  4:43 [PATCH 0/2] Change xs_read_watch interfaces to support variable arguments Anthony Liguori
2005-10-04  4:48 ` Anthony Liguori [this message]
2005-10-04  6:55 ` Ewan Mellor
2005-10-04 14:34   ` Anthony Liguori
2005-10-04 14:51     ` Ewan Mellor

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=43420991.3070905@us.ibm.com \
    --to=aliguori@us.ibm.com \
    --cc=Christian.Limpach@cl.cam.ac.uk \
    --cc=rusty@rustcorp.com.au \
    --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.