[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

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




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

 


Rackspace

Lists.xenproject.org is hosted with RackSpace, monitoring our
servers 24x7x365 and backed by RackSpace's Fanatical Support®.