All of lore.kernel.org
 help / color / mirror / Atom feed
From: Anthony Liguori <aliguori@us.ibm.com>
To: Keir Fraser <Keir.Fraser@cl.cam.ac.uk>
Cc: Dan Smith <danms@us.ibm.com>,
	Xen Developers <xen-devel@lists.xensource.com>
Subject: Re: [PATCH] Add an ioctl interface for simple xenstore access
Date: Sat, 18 Mar 2006 12:02:08 -0600	[thread overview]
Message-ID: <441C4B20.4000202@us.ibm.com> (raw)
In-Reply-To: <01789afa761fc8e2b1f398148de7370c@cl.cam.ac.uk>

Keir Fraser wrote:
>> Hmm, wouldn't a virtual file system be a much better fit for xenstore
>> access? the kernel community considers ioctls Evil with a capital E.
>
> Also, we already support xenstore access via read/write on that device 
> file. And libxenstore knows how to target that read/write interface. 
> Why would we add an ioctl to do the same thing?
Hi Keir,

We had discussed this in a previous thread and I thought you were okay 
with this approach.  The /proc/xen/xenbus interface requires full 
parsing of the XenBus protocol.  This requires libxenstore in domU.  
Today, libxenstore is shipped with the rest of the tools package on most 
distros.  If one wants to write a Xen-aware application for a domU, that 
brings in a pretty large number of unnecessary dependencies.  Also, for 
32 bit management apps in a 64 bit environment, it makes things very 
nasty (should we build a 32 bit and 64 bit version of libxenstore?  Is 
the protocol 32/64 bit safe?).

Ideally, an app could just use a simple interface to /proc/xen/xenbus to 
access XenStore.  That solves all of the above problems elegantly.  An 
ioctl() interface seemed like the most obvious approach that wouldn't 
break existing apps.  Of course, any suggestion for a better interface 
would be appreciated.

The general problem of xenstore access in domU is a big issue for us 
(and I assume it will be for most people building Xen management 
infrastructure).  It would really help to have a good solution for 3.0.2.

Regards,

Anthony Liguori

>  -- Keir
>
>
> _______________________________________________
> Xen-devel mailing list
> Xen-devel@lists.xensource.com
> http://lists.xensource.com/xen-devel

  reply	other threads:[~2006-03-18 18:02 UTC|newest]

Thread overview: 11+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2006-03-17 22:33 [PATCH] Add an ioctl interface for simple xenstore access Dan Smith
2006-03-18  7:14 ` Muli Ben-Yehuda
2006-03-18  9:50   ` Keir Fraser
2006-03-18 18:02     ` Anthony Liguori [this message]
2006-03-18 19:32       ` Keir Fraser
2006-03-20  8:38       ` Gerd Hoffmann
2006-03-20  8:52         ` Ewan Mellor
2006-03-20 10:08           ` Keir Fraser
2006-03-20 10:10           ` Gerd Hoffmann
2006-03-18 17:56   ` Anthony Liguori
2006-03-19  0:10     ` Muli Ben-Yehuda

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=441C4B20.4000202@us.ibm.com \
    --to=aliguori@us.ibm.com \
    --cc=Keir.Fraser@cl.cam.ac.uk \
    --cc=danms@us.ibm.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.