From: Ian Campbell <ian.campbell@citrix.com>
To: "Roger Pau Monné" <roger.pau@citrix.com>,
"Wei Liu" <wei.liu2@citrix.com>
Cc: George Dunlap <george.dunlap@eu.citrix.com>,
Konrad Rzeszutek Wilk <konrad@darnok.org>,
xen-devel@lists.xen.org, Ian Jackson <ian.jackson@citrix.com>
Subject: Re: [PATCH 2/6] libxl: Remove linux udev rules
Date: Thu, 23 Jul 2015 13:30:15 +0100 [thread overview]
Message-ID: <1437654615.19412.101.camel@citrix.com> (raw)
In-Reply-To: <55B0D61A.8000603@citrix.com>
On Thu, 2015-07-23 at 13:55 +0200, Roger Pau Monné wrote:
> El 14/07/15 a les 18.48, Ian Campbell ha escrit:
> > On Tue, 2015-07-14 at 17:40 +0100, Wei Liu wrote:
> > > On Tue, Jul 14, 2015 at 05:35:23PM +0100, George Dunlap wrote:
> > > > On 07/14/2015 05:21 PM, Ian Campbell wrote:
> > > > > On Tue, 2015-07-14 at 12:13 -0400, Konrad Rzeszutek Wilk
> > > > > wrote:
> > > > > > On Tue, Jul 07, 2015 at 12:39:52PM +0100, Wei Liu wrote:
> > > > > > > On Mon, Jul 06, 2015 at 11:51:39AM +0100, George Dunlap
> > > > > > > wrote:
> > > > > > > > They are no longer needed, having been replaced by a
> > > > > > > > daemon for
> > > > > > > > driverdomains which will run scripts as necessary.
> > > > > >
> > > > > > This introduces an regression. The 'daemon for
> > > > > > driverdomains'
> > > > > > is xenbackendd - which only gets built for NetBSD.
> > > > >
> > > > > It's not (any more), it's "xl devd", which should be built
> > > > > everywhere.
> > > >
> > > > Aha! I knew I'd seen it somewhere.
> > > >
> > > > In that case, do we actually still need xenbackendd? Or does
> > > > xenbackendd do something slightly different?
> > > >
> > >
> > > That's NetBSD-only thing as far as I remember. Not sure if what
> > > it does
> > > is NetBSD specific.
> >
> > My understanding was it had been subsumed by "xl devd", but I don't
> > know
> > why it hasn't been removed. Probably best to wait until Roger is
> > back...
>
> Some time ago I proposed to the NetBSD folks to remove xenbackendd,
> because I think it's useless now, but the reply was to leave it as
> is:
>
> http://mail-index.netbsd.org/port-xen/2015/01/29/msg008483.html
It's redundant with "xl devd", I don't see any reason for use to
continue carrying it in xen.git. If xl devd has some short coming
relative to xenbackendd then we should address that. But otherwise
if folks want to keep xenbackend despite it having been replaced then
they can carry the patch IMHO.
Ian.
_______________________________________________
Xen-devel mailing list
Xen-devel@lists.xen.org
http://lists.xen.org/xen-devel
next prev parent reply other threads:[~2015-07-23 12:30 UTC|newest]
Thread overview: 45+ messages / expand[flat|nested] mbox.gz Atom feed top
2015-07-06 10:51 [PATCH 0/6] Use system blktap George Dunlap
2015-07-06 10:51 ` [PATCH 1/6] libxl: Make local_initiate_attach more rational George Dunlap
2015-07-06 10:51 ` [PATCH 2/6] libxl: Remove linux udev rules George Dunlap
2015-07-07 11:39 ` Wei Liu
2015-07-14 16:13 ` Konrad Rzeszutek Wilk
2015-07-14 16:21 ` Ian Campbell
2015-07-14 16:35 ` George Dunlap
2015-07-14 16:40 ` Wei Liu
2015-07-14 16:48 ` Ian Campbell
2015-07-23 11:55 ` Roger Pau Monné
2015-07-23 12:30 ` Ian Campbell [this message]
2015-07-15 11:07 ` [PATCH for-4.6] tools/hotplug: Add an initscript to start "xl devd" in a driver domain Ian Campbell
2015-07-15 13:26 ` Wei Liu
2015-07-15 13:40 ` Ian Campbell
2015-07-15 15:25 ` George Dunlap
2015-07-15 15:32 ` Ian Jackson
2015-07-15 15:35 ` George Dunlap
2015-07-15 15:37 ` Ian Campbell
2015-07-16 16:58 ` [PATCH for-4.6 v2] " Ian Campbell
2015-07-16 17:09 ` Ian Jackson
2015-07-16 17:48 ` Wei Liu
2015-07-17 9:05 ` Wei Liu
2015-07-17 11:36 ` Ian Campbell
2015-07-20 14:16 ` Roger Pau Monné
2015-07-20 14:28 ` Ian Campbell
2015-07-06 10:51 ` [PATCH 3/6] tools: Add a block-tap script for setting up tapdisks via tap-ctl George Dunlap
2015-07-07 12:03 ` Wei Liu
2015-07-07 12:35 ` Wei Liu
2015-07-06 10:51 ` [PATCH 4/6] libxl: Use the block-tap script for LIBXL_DISK_BACKEND_TAP George Dunlap
2015-07-06 11:01 ` Andrew Cooper
2015-07-06 12:39 ` George Dunlap
2015-07-07 12:29 ` Wei Liu
2015-07-07 13:41 ` George Dunlap
2015-07-07 14:20 ` Ian Campbell
2015-07-07 14:27 ` George Dunlap
2015-07-06 10:51 ` [PATCH 5/6] tools: Remove in-tree blktap2 George Dunlap
2015-07-07 11:55 ` Wei Liu
2015-07-06 10:51 ` [PATCH 6/6] libxl: Add more logging to hotplug script path George Dunlap
2015-07-07 11:55 ` Wei Liu
2015-07-07 14:21 ` [PATCH 0/6] Use system blktap Ian Campbell
2015-07-07 14:24 ` George Dunlap
2015-07-07 14:52 ` Ian Campbell
2015-07-07 14:59 ` George Dunlap
2015-07-07 15:04 ` Ian Campbell
2015-07-07 15:20 ` 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=1437654615.19412.101.camel@citrix.com \
--to=ian.campbell@citrix.com \
--cc=george.dunlap@eu.citrix.com \
--cc=ian.jackson@citrix.com \
--cc=konrad@darnok.org \
--cc=roger.pau@citrix.com \
--cc=wei.liu2@citrix.com \
--cc=xen-devel@lists.xen.org \
/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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).