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] [PATCH] [TESTSUITE] virtual TPM test

To: Ewan Mellor <ewan@xxxxxxxxxxxxx>
Subject: Re: [Xen-devel] [PATCH] [TESTSUITE] virtual TPM test
From: Stefan Berger <stefanb@xxxxxxxxxx>
Date: Wed, 22 Feb 2006 12:05:46 -0500
Cc: xen-devel@xxxxxxxxxxxxxxxxxxx
Delivery-date: Wed, 22 Feb 2006 17:07:13 +0000
Envelope-to: www-data@xxxxxxxxxxxxxxxxxxx
In-reply-to: <20060222154033.GL28961@xxxxxxxxxxxxxxxxxxxxxx>
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>
Sender: xen-devel-bounces@xxxxxxxxxxxxxxxxxxx

xen-devel-bounces@xxxxxxxxxxxxxxxxxxx wrote on 02/22/2006 10:40:33 AM:

> On Tue, Feb 21, 2006 at 11:00:41AM -0500, Stefan Berger wrote:
>


Ewan, if I cannot source the vtpm-hotplug-common.sh from the vtpm-delete anymore, I don't have access to the locking functions
(claim_lock, release_lock) in vtpm-hotplug-common.sh which are needed by my script. May I separte those out to locking.sh so they  can be 'sourced' by other scritps as well?

   Stefan


> > Index: xen/xen-unstable.hg/tools/examples/vtpm-delete
> > ===================================================================
> > --- /dev/null
> > +++ xen/xen-unstable.hg/tools/examples/vtpm-delete
> > @@ -0,0 +1,18 @@
> > +#!/bin/sh
> > +
> > +# This scripts must be called the following way:
> > +# vtpm-delete <domain name>
> > +
> > +if [ $1 != "remove" ]; then
> > +   exec sh -c "bash $0 remove $*"
> > +fi
> > +
> > +
> > +XENBUS_PATH=" "
> > +
> > +dir=$(dirname "$0")
> > +. "$dir/vtpm-common.sh"
> > +
> > +shift
> > +
> > +vtpm_delete_instance $1
>
> You are setting XENBUS_PATH just to hack your way into the hotplug scripts,
> and then relying on the fact that vtpm_delete_instance doesn't actually use
> that value.  This is not a reasonable thing to be do.  If there's a need for a
> tpm-manipulating library, separate from the hotplug infrastructure, then you
> should split vtpm-common.sh into vtpm-common.sh and vtpm-hotplug-common.sh (or
> similar).  Note that vif-common.sh includes xen-hotplug-common.sh and
> xen-network-common.sh -- these are split apart for exactly that reason
> (xen-network-common.sh being used for the network-xyz scripts as well as for
> the vif-xyz hotplugging scripts).
>
> Ewan.
>
> _______________________________________________
> Xen-devel mailing list
> Xen-devel@xxxxxxxxxxxxxxxxxxx
> http://lists.xensource.com/xen-devel
_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxxxxxxxx
http://lists.xensource.com/xen-devel