xen-devel.lists.xenproject.org archive mirror
 help / color / mirror / Atom feed
From: Vincent Hanquez <vincent.hanquez@eu.citrix.com>
To: Ian Campbell <Ian.Campbell@eu.citrix.com>
Cc: "xen-devel@lists.xensource.com" <xen-devel@lists.xensource.com>
Subject: Re: [PATCH] add a way to disable xen's udev script.
Date: Mon, 13 Jun 2011 09:58:16 +0100	[thread overview]
Message-ID: <4DF5D128.4030208@eu.citrix.com> (raw)
In-Reply-To: <1307689982.12738.34.camel@dagon.hellion.org.uk>

On 06/10/2011 08:13 AM, Ian Campbell wrote:
> No it does not go straight to any (completely undefined) step 2 and it
> in no way allows toolstacks to dynamically work in parallel -- it allows
> your current pet project to unilaterally disable functionality in other
> existing toolstacks because that is convenient to you, without any
> consideration for the bigger picture or actual _real_ interop with other
> toolstacks or usecases.

dynamic as in = can be installed on disk in parallel.

There's no way to make two managed toolstacks run in parallel, so i 
wasn't using dynamic in this sense. the only sense i'm talking about is 
that you could have start the init script for XCP or Xend (or xl), and 
it would have clear/set the dummy file to enable/disable the udev script 
depending on whether you want the udev script running or not which is a 
toolstack properties.

So let's be clear, instead of having a way to interop which isn't great 
(not "actual real interop" reusing your word), we have no interop 
whatsoever.

> Furthermore I object to your characterisation of some toolstacks as
> "compat" and the one you are currently interested in as "new"/"modern".
> xl uses these udev scripts and is in no way a "compat" toolstack. You
> cannot simply deny the existence of other toolstacks, ignore their
> requirements and justify breaking their functionality by branding them
> legacy.

I simply characterised the toolstacks using a the new/modern udev 
mechanism i'm talking about as new/modern. compared to the toolstack 
that use the old mechanism as compat, since this was an attempt to keep 
both mechanisms in. this is *not* about the toolstack itself nor about 
branding them legacy.

> So what is your proposal for removing the need for udev (i.e. what is
> the so-called "step 3")? Unless you have specific concrete suggestions
> then we cannot build consensus around any move away from the udev
> scripts and all this talk is just hot air.
>
> And, to be frank, unless you have an plan to back it up with patches
> sooner rather than later[...]

This feature has been discussed with toolstack people, patch has been 
send to maintainers already and it's about to be committed in one case 
(on two) provided it pass my tests i've left running over the weekend.

> You aren't obliged to do any more work than you want. However it does
> not follow that we must take your original patch.

Of course. I never implied as such. Now, Let's give this thread and the 
patch a rest.

-- 
Vincent

  reply	other threads:[~2011-06-13  8:58 UTC|newest]

Thread overview: 14+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2011-06-08 16:55 [PATCH] add a way to disable xen's udev script Vincent Hanquez
2011-06-08 17:42 ` Ian Campbell
2011-06-08 19:23   ` Vincent Hanquez
2011-06-08 19:49     ` Ian Campbell
2011-06-08 20:33       ` Vincent Hanquez
2011-06-09  7:42         ` Ian Campbell
2011-06-09  9:01           ` Vincent Hanquez
2011-06-09  9:23             ` Ian Campbell
2011-06-09 10:07               ` Vincent Hanquez
2011-06-10  7:13                 ` Ian Campbell
2011-06-13  8:58                   ` Vincent Hanquez [this message]
2011-06-14 13:36                     ` Ian Campbell
2011-06-17  7:45                       ` Vincent Hanquez
2011-06-17 17:23                       ` Ian Jackson

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=4DF5D128.4030208@eu.citrix.com \
    --to=vincent.hanquez@eu.citrix.com \
    --cc=Ian.Campbell@eu.citrix.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 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).