WARNING - OLD ARCHIVES

This is an archived copy of the Xen.org mailing list, which we have preserved to ensure that existing links to archives are not broken. The live archive, which contains the latest emails, can be found at http://lists.xen.org/
   
 
 
Xen 
 
Home Products Support Community News
 
   
 

xen-devel

Re: [Xen-devel] Re: [Qemu-devel] [PATCH 12/13] set vnc password from xen

To: Anthony Liguori <anthony@xxxxxxxxxxxxx>
Subject: Re: [Xen-devel] Re: [Qemu-devel] [PATCH 12/13] set vnc password from xenstore.
From: "Daniel P. Berrange" <berrange@xxxxxxxxxx>
Date: Thu, 21 Aug 2008 21:26:40 +0100
Cc: xen-devel@xxxxxxxxxxxxxxxxxxx, qemu-devel@xxxxxxxxxx, Gerd Hoffmann <kraxel@xxxxxxxxxx>
Delivery-date: Thu, 21 Aug 2008 13:27:28 -0700
Envelope-to: www-data@xxxxxxxxxxxxxxxxxxx
In-reply-to: <48ADCE91.2070602@xxxxxxxxxxxxx>
List-help: <mailto:xen-devel-request@lists.xensource.com?subject=help>
List-id: Xen developer discussion <xen-devel.lists.xensource.com>
List-post: <mailto:xen-devel@lists.xensource.com>
List-subscribe: <http://lists.xensource.com/mailman/listinfo/xen-devel>, <mailto:xen-devel-request@lists.xensource.com?subject=subscribe>
List-unsubscribe: <http://lists.xensource.com/mailman/listinfo/xen-devel>, <mailto:xen-devel-request@lists.xensource.com?subject=unsubscribe>
References: <1219336054-15919-1-git-send-email-kraxel@xxxxxxxxxx> <1219336054-15919-13-git-send-email-kraxel@xxxxxxxxxx> <48ADCCA2.8050201@xxxxxxxxxxxxx> <20080821201955.GG1531@xxxxxxxxxx> <48ADCE91.2070602@xxxxxxxxxxxxx>
Reply-to: "Daniel P. Berrange" <berrange@xxxxxxxxxx>
Sender: xen-devel-bounces@xxxxxxxxxxxxxxxxxxx
User-agent: Mutt/1.4.1i
On Thu, Aug 21, 2008 at 03:22:41PM -0500, Anthony Liguori wrote:
> Daniel P. Berrange wrote:
> >On Thu, Aug 21, 2008 at 03:14:26PM -0500, Anthony Liguori wrote:
> >  
> >>I don't really like this approach.  I don't want to have a totally 
> >>separate management interface just for Xend to use (via xenstore).
> >>
> >>Can't Xend use the monitor like every other management tool?
> >>    
> >
> >If the monitor really isn't sufficient, then I'd suggest allowing an
> >open file descriptor to be specified on the -vnc arg, eg
> >
> >  -vnc localhost:3,passwdfd=6
> >  
> 
> In general, I strongly dislike passing file descriptors like this.
> 
> >And then let QEMU read the password from this file descriptor. I'd rather
> >XenD just used the monitor though - it'll ultimately need to use it anyway
> >to perform disk/network hotplug amongst other things.
> >  
> 
> If there are reasons that Xend doesn't want to interact with the monitor 
> (for instance, if you want to support monitor redirection), we can 
> certainly discuss how to enhance the monitor (like having multiple 
> monitor instances).  Ultimately, we need to have everyone use the same 
> interface to interact with QEMU.

Based on my understanding of XenD code & requirements I think it is mostly
just a historical accident that it doesn't use the monitor - quite simply
when XenD first picked up QEMU there wasn't much of use the monitor could
provide. There were no disk/network/vnc commands for it to use, so it
did the easiest thing and read the data out of xenstore, since it was
there already.

Daniel
-- 
|: Red Hat, Engineering, London   -o-   http://people.redhat.com/berrange/ :|
|: http://libvirt.org  -o-  http://virt-manager.org  -o-  http://ovirt.org :|
|: http://autobuild.org       -o-         http://search.cpan.org/~danberr/ :|
|: GnuPG: 7D3B9505  -o-  F3C9 553F A1DA 4AC2 5648 23C1 B3DF F742 7D3B 9505 :|

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

<Prev in Thread] Current Thread [Next in Thread>