Re: [Xen-devel] [PATCH] [TESTSUITE] virtual TPM test

[Stefan Berger <stefanb@xxxxxxxxxx>]

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