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

[Xen-devel] Re: [PATCH][VNC Auth] Fix qemu-dm becomes defunct when reboo

To: "Daniel P. Berrange" <berrange@xxxxxxxxxx>, Keir Fraser <Keir.Fraser@xxxxxxxxxxxx>
Subject: [Xen-devel] Re: [PATCH][VNC Auth] Fix qemu-dm becomes defunct when rebooting hvm domain
From: Masami Watanabe <masami.watanabe@xxxxxxxxxxxxxx>
Date: Mon, 30 Oct 2006 21:29:27 +0900
Cc: Ian Pratt <m+Ian.Pratt@xxxxxxxxxxxx>, Anthony Liguori <aliguori@xxxxxxxxxx>, xen-devel@xxxxxxxxxxxxxxxxxxx, masami.watanabe@xxxxxxxxxxxxxx, "Zheng, Jeff" <jeff.zheng@xxxxxxxxx>
Delivery-date: Mon, 30 Oct 2006 04:30:45 -0800
Envelope-to: www-data@xxxxxxxxxxxxxxxxxx
In-reply-to: <20061027163942.GM32556@xxxxxxxxxx>
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/cgi-bin/mailman/listinfo/xen-devel>, <mailto:xen-devel-request@lists.xensource.com?subject=subscribe>
List-unsubscribe: <http://lists.xensource.com/cgi-bin/mailman/listinfo/xen-devel>, <mailto:xen-devel-request@lists.xensource.com?subject=unsubscribe>
References: <JO200610260707323.541015@xxxxxxxxxxxxxx> <C167EF89.3855%Keir.Fraser@xxxxxxxxxxxx> <20061027163942.GM32556@xxxxxxxxxx>
Sender: xen-devel-bounces@xxxxxxxxxxxxxxxxxxx
Hi all,

I am sorry for the delayed response.

On Fri, 27 Oct 2006 17:39:42 +0100, Daniel P. Berrange wrote:
> Not sure why qemu would do defunct though - if the vncpasswd is not set
> in xenstore, it ought to have just fallen back to no-password auth.

Yes. Thanks Dan.


On Fri, Oct 27, 2006 at 05:16:57PM +0100, Keir Fraser wrote:
> That would mean that you¹d fall back to the default VNC password though,
> right? Why would it cause qemu to fail to restart and become a defunct
> process?

Thanks Keir, for your correction.
Please refer to the following for accurate processing.
- When qemu-dm start, qemu-dm is sure to need vncpasswd in xenstore.
      main()@vl.c -> xenstore_read_vncpasswd()@xenstore.c
  Null string vncpasswd is allowed. but non  vncpasswd is not allowed.
- If config.remove() in image.py, vncpasswd is not preserved in xend.
  When rebooting guest domain, xend cannot pass qemu-dm vncpasswd.
- When vncpasswd cannot be read from xenstore, qemu-dm is exit(1).
  As a result, qemu-dm becomes defunct process.

Best regards,
Masami



On Fri, 27 Oct 2006 17:39:42 +0100, Daniel P. Berrange wrote:
> On Fri, Oct 27, 2006 at 05:16:57PM +0100, Keir Fraser wrote:
> > On 25/10/06 11:07 pm, "Masami Watanabe" <masami.watanabe@xxxxxxxxxxxxxx>
> > wrote:
> >
> > > Sorry, I should not have removed infomation on xend.
> > >
> > >   image.py, line 399, in configVNC
> > >     config.remove(['vncpasswd', vncpasswd_vmconfig])
> > >
> > > When vncpasswd is removed, qemu-dm cannot be started in reboot of hvm 
> > > domain.
> > > Because necessary information disappears.
> > > It cannot be helped though information in config list remains in xenstore.
> > >
> > > This patch corrects above.
> >
> > That would mean that you¹d fall back to the default VNC password though,
> > right? Why would it cause qemu to fail to restart and become a defunct
> > process?
>
> No, it looks correct to me - we were scrubbing the vncpassword from the
> guest config after using it, so that it didn't appear in the SXPR data
> with doing 'xm list --long' etc. But turns out we do need the password
> later on after all, so we can't scrub it completely. Perhaps we just need
> to scrub it out on the fly when talking to XM & other apps instead - i've
> already been working on a patch to scrub it out when loggin the guest
> data to /var/log/xen/xend.log.
>
> This 'config.remove' thing also impacted my equivalent vncpassword patches
> for the paravirt framebuffer, because it meant when you do a save/restore
> or a migrate, and started the VNC daemon it no longer found a password.
>
> Not sure why qemu would do defunct though - if the vncpasswd is not set
> in xenstore, it ought to have just fallen back to no-password auth.
>
> Dan.
> --
> |=- Red Hat, Engineering, Emerging Technologies, Boston.  +1 978 392 2496 -=|
> |=-           Perl modules: http://search.cpan.org/~danberr/              -=|
> |=-               Projects: http://freshmeat.net/~danielpb/               -=|
> |=-  GnuPG: 7D3B9505   F3C9 553F A1DA 4AC2 5648 23C1 B3DF F742 7D3B 9505  -=|


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