From mboxrd@z Thu Jan 1 00:00:00 1970 From: Shriram Rajagopalan Subject: Re: [PATCH 09 of 30] tools: blktap2: copy xenstore/hashtable.h into blktap2 Date: Thu, 24 Mar 2011 10:35:28 -0700 Message-ID: References: <30a1f7d0aa1097fd2b3f.1300718672@localhost.localdomain> <19847.37607.622295.871747@mariner.uk.xensource.com> <1300962566.11560.11.camel@cthulhu.hellion.org.uk> Mime-Version: 1.0 Content-Type: multipart/mixed; boundary="===============0990140258==" Return-path: In-Reply-To: <1300962566.11560.11.camel@cthulhu.hellion.org.uk> List-Unsubscribe: , List-Post: List-Help: List-Subscribe: , Sender: xen-devel-bounces@lists.xensource.com Errors-To: xen-devel-bounces@lists.xensource.com To: Ian Campbell Cc: "xen-devel@lists.xensource.com" , Ian Jackson List-Id: xen-devel@lists.xenproject.org --===============0990140258== Content-Type: multipart/alternative; boundary=0016e6470e649ef1ca049f3de9ad --0016e6470e649ef1ca049f3de9ad Content-Type: text/plain; charset=ISO-8859-1 On Thu, Mar 24, 2011 at 3:29 AM, Ian Campbell wrote: > On Mon, 2011-03-21 at 18:03 +0000, Ian Jackson wrote: > > Ian Campbell writes ("[Xen-devel] [PATCH 09 of 30] tools: blktap2: copy > xenstore/hashtable.h into blktap2"): > > > tools: blktap2: copy xenstore/hashtable.h into blktap2 > > > > > > hashtable.c has already been copied, forked and modified, there > > > doesn't seem much point in avoiding the same for the header until > > > someone feels motivated to properly refactor. > > > > Before we make this any worse, perhaps we should at least leave a > > comment in every copy of hashtable.c referring to every other clone ? > > Sure. I found another one in vtpm_manager while doing so as well :-( > > 8<----------------------- > > # HG changeset patch > # User Ian Campbell > # Date 1300962476 0 > # Node ID 708fdb28b0530e331a6df59cd42b1c3acfc85f8b > # Parent 63f4adae7d7b70d323181ff31fc19f5432d283ff > tools: blktap2: copy xenstore/hashtable.h into blktap2 > > hashtable.c has already been copied, forked and modified, there > doesn't seem much point in avoiding the same for the header until > someone feels motivated to properly refactor. > > Add comments to the various duplicated files cross-referencing each > other for future reference and as a barrier to forking again... > > Signed-off-by: Ian Campbell > > diff -r 63f4adae7d7b -r 708fdb28b053 tools/blktap2/drivers/Makefile > --- a/tools/blktap2/drivers/Makefile Wed Mar 23 17:04:20 2011 +0000 > +++ b/tools/blktap2/drivers/Makefile Thu Mar 24 10:27:56 2011 +0000 > @@ -29,8 +29,6 @@ REMUS-OBJS += hashtable.o > REMUS-OBJS += hashtable.o > REMUS-OBJS += hashtable_itr.o > REMUS-OBJS += hashtable_utility.o > - > -$(REMUS-OBJS): CFLAGS += -I$(XEN_XENSTORE) > > LIBAIO_DIR = $(XEN_ROOT)/tools/libaio/src > MEMSHR_DIR = $(XEN_ROOT)/tools/memshr > diff -r 63f4adae7d7b -r 708fdb28b053 tools/blktap2/drivers/hashtable.c > --- a/tools/blktap2/drivers/hashtable.c Wed Mar 23 17:04:20 2011 +0000 > +++ b/tools/blktap2/drivers/hashtable.c Thu Mar 24 10:27:56 2011 +0000 > @@ -1,4 +1,10 @@ > /* Copyright (C) 2004 Christopher Clark > */ > + > +/* > + * There are duplicates of this code in: > + * - tools/xenstore/hashtable.c > + * - tools/vtpm_manager/util/hashtable.c > + */ > > #include "hashtable.h" > #include "hashtable_private.h" > diff -r 63f4adae7d7b -r 708fdb28b053 tools/blktap2/drivers/hashtable.h > --- /dev/null Thu Jan 01 00:00:00 1970 +0000 > +++ b/tools/blktap2/drivers/hashtable.h Thu Mar 24 10:27:56 2011 +0000 > @@ -0,0 +1,205 @@ > +/* Copyright (C) 2002 Christopher Clark > */ > + > +/* > + * There are duplicates of this code in: > + * - tools/xenstore/hashtable.h > + * - tools/vtpm_manager/util/hashtable.h > + */ > + > +#ifndef __HASHTABLE_CWC22_H__ > +#define __HASHTABLE_CWC22_H__ > + > +struct hashtable; > + > +/* Example of use: > + * > + * struct hashtable *h; > + * struct some_key *k; > + * struct some_value *v; > + * > + * static unsigned int hash_from_key_fn( void *k ); > + * static int keys_equal_fn ( void *key1, void *key2 > ); > + * > + * h = create_hashtable(16, hash_from_key_fn, keys_equal_fn); > + * k = (struct some_key *) malloc(sizeof(struct some_key)); > + * v = (struct some_value *) malloc(sizeof(struct some_value)); > + * > + * (initialise k and v to suitable values) > + * > + * if (! hashtable_insert(h,k,v) ) > + * { exit(-1); } > + * > + * if (NULL == (found = hashtable_search(h,k) )) > + * { printf("not found!"); } > + * > + * if (NULL == (found = hashtable_remove(h,k) )) > + * { printf("Not found\n"); } > + * > + */ > + > +/* Macros may be used to define type-safe(r) hashtable access functions, > with > + * methods specialized to take known key and value types as parameters. > + * > + * Example: > + * > + * Insert this at the start of your file: > + * > + * DEFINE_HASHTABLE_INSERT(insert_some, struct some_key, struct > some_value); > + * DEFINE_HASHTABLE_SEARCH(search_some, struct some_key, struct > some_value); > + * DEFINE_HASHTABLE_REMOVE(remove_some, struct some_key, struct > some_value); > + * > + * This defines the functions 'insert_some', 'search_some' and > 'remove_some'. > + * These operate just like hashtable_insert etc., with the same > parameters, > + * but their function signatures have 'struct some_key *' rather than > + * 'void *', and hence can generate compile time errors if your program is > + * supplying incorrect data as a key (and similarly for value). > + * > + * Note that the hash and key equality functions passed to > create_hashtable > + * still take 'void *' parameters instead of 'some key *'. This shouldn't > be > + * a difficult issue as they're only defined and passed once, and the > other > + * functions will ensure that only valid keys are supplied to them. > + * > + * The cost for this checking is increased code size and runtime overhead > + * - if performance is important, it may be worth switching back to the > + * unsafe methods once your program has been debugged with the safe > methods. > + * This just requires switching to some simple alternative defines - eg: > + * #define insert_some hashtable_insert > + * > + */ > + > > +/***************************************************************************** > + * create_hashtable > + > + * @name create_hashtable > + * @param minsize minimum initial size of hashtable > + * @param hashfunction function for hashing keys > + * @param key_eq_fn function for determining key equality > + * @return newly created hashtable or NULL on failure > + */ > + > +struct hashtable * > +create_hashtable(unsigned int minsize, > + unsigned int (*hashfunction) (void*), > + int (*key_eq_fn) (void*,void*)); > + > > +/***************************************************************************** > + * hashtable_insert > + > + * @name hashtable_insert > + * @param h the hashtable to insert into > + * @param k the key - hashtable claims ownership and will free on > removal > + * @param v the value - does not claim ownership > + * @return non-zero for successful insertion > + * > + * This function will cause the table to expand if the insertion would > take > + * the ratio of entries to table size over the maximum load factor. > + * > + * This function does not check for repeated insertions with a duplicate > key. > + * The value returned when using a duplicate key is undefined -- when > + * the hashtable changes size, the order of retrieval of duplicate key > + * entries is reversed. > + * If in doubt, remove before insert. > + */ > + > +int > +hashtable_insert(struct hashtable *h, void *k, void *v); > + > +#define DEFINE_HASHTABLE_INSERT(fnname, keytype, valuetype) \ > +int fnname (struct hashtable *h, keytype *k, valuetype *v) \ > +{ \ > + return hashtable_insert(h,k,v); \ > +} > + > > +/***************************************************************************** > + * hashtable_search > + > + * @name hashtable_search > + * @param h the hashtable to search > + * @param k the key to search for - does not claim ownership > + * @return the value associated with the key, or NULL if none found > + */ > + > +void * > +hashtable_search(struct hashtable *h, void *k); > + > +#define DEFINE_HASHTABLE_SEARCH(fnname, keytype, valuetype) \ > +valuetype * fnname (struct hashtable *h, keytype *k) \ > +{ \ > + return (valuetype *) (hashtable_search(h,k)); \ > +} > + > > +/***************************************************************************** > + * hashtable_remove > + > + * @name hashtable_remove > + * @param h the hashtable to remove the item from > + * @param k the key to search for - does not claim ownership > + * @return the value associated with the key, or NULL if none found > + */ > + > +void * /* returns value */ > +hashtable_remove(struct hashtable *h, void *k); > + > +#define DEFINE_HASHTABLE_REMOVE(fnname, keytype, valuetype) \ > +valuetype * fnname (struct hashtable *h, keytype *k) \ > +{ \ > + return (valuetype *) (hashtable_remove(h,k)); \ > +} > + > + > > +/***************************************************************************** > + * hashtable_count > + > + * @name hashtable_count > + * @param h the hashtable > + * @return the number of items stored in the hashtable > + */ > +unsigned int > +hashtable_count(struct hashtable *h); > + > + > > +/***************************************************************************** > + * hashtable_destroy > + > + * @name hashtable_destroy > + * @param h the hashtable > + * @param free_values whether to call 'free' on the remaining > values > + */ > + > +void > +hashtable_destroy(struct hashtable *h, int free_values); > + > +#endif /* __HASHTABLE_CWC22_H__ */ > + > +/* > + * Copyright (c) 2002, Christopher Clark > + * All rights reserved. > + * > + * Redistribution and use in source and binary forms, with or without > + * modification, are permitted provided that the following conditions > + * are met: > + * > + * * Redistributions of source code must retain the above copyright > + * notice, this list of conditions and the following disclaimer. > + * > + * * Redistributions in binary form must reproduce the above copyright > + * notice, this list of conditions and the following disclaimer in the > + * documentation and/or other materials provided with the distribution. > + * > + * * Neither the name of the original author; nor the names of any > contributors > + * may be used to endorse or promote products derived from this software > + * without specific prior written permission. > + * > + * > + * THIS SOFTWARE IS PROVIDED BY THE COPYRIGHT HOLDERS AND CONTRIBUTORS > + * "AS IS" AND ANY EXPRESS OR IMPLIED WARRANTIES, INCLUDING, BUT NOT > + * LIMITED TO, THE IMPLIED WARRANTIES OF MERCHANTABILITY AND FITNESS FOR > + * A PARTICULAR PURPOSE ARE DISCLAIMED. IN NO EVENT SHALL THE COPYRIGHT > OWNER > + * OR CONTRIBUTORS BE LIABLE FOR ANY DIRECT, INDIRECT, INCIDENTAL, > SPECIAL, > + * EXEMPLARY, OR CONSEQUENTIAL DAMAGES (INCLUDING, BUT NOT LIMITED TO, > + * PROCUREMENT OF SUBSTITUTE GOODS OR SERVICES; LOSS OF USE, DATA, OR > + * PROFITS; OR BUSINESS INTERRUPTION) HOWEVER CAUSED AND ON ANY THEORY OF > + * LIABILITY, WHETHER IN CONTRACT, STRICT LIABILITY, OR TORT (INCLUDING > + * NEGLIGENCE OR OTHERWISE) ARISING IN ANY WAY OUT OF THE USE OF THIS > + * SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE. > +*/ > diff -r 63f4adae7d7b -r 708fdb28b053 tools/blktap2/drivers/hashtable_itr.c > --- a/tools/blktap2/drivers/hashtable_itr.c Wed Mar 23 17:04:20 2011 > +0000 > +++ b/tools/blktap2/drivers/hashtable_itr.c Thu Mar 24 10:27:56 2011 > +0000 > @@ -1,4 +1,9 @@ > /* Copyright (C) 2002, 2004 Christopher Clark < > firstname.lastname@cl.cam.ac.uk> */ > + > +/* > + * There are duplicates of this code in: > + * - tools/vtpm_manager/util/hashtable_itr.c > + */ > > #include "hashtable.h" > #include "hashtable_private.h" > diff -r 63f4adae7d7b -r 708fdb28b053 tools/blktap2/drivers/hashtable_itr.h > --- a/tools/blktap2/drivers/hashtable_itr.h Wed Mar 23 17:04:20 2011 > +0000 > +++ b/tools/blktap2/drivers/hashtable_itr.h Thu Mar 24 10:27:56 2011 > +0000 > @@ -1,4 +1,9 @@ > /* Copyright (C) 2002, 2004 Christopher Clark < > firstname.lastname@cl.cam.ac.uk> */ > + > +/* > + * There are duplicates of this code in: > + * - tools/vtpm_manager/util/hashtable_itr.h > + */ > > #ifndef __HASHTABLE_ITR_CWC22__ > #define __HASHTABLE_ITR_CWC22__ > diff -r 63f4adae7d7b -r 708fdb28b053 > tools/blktap2/drivers/hashtable_private.h > --- /dev/null Thu Jan 01 00:00:00 1970 +0000 > +++ b/tools/blktap2/drivers/hashtable_private.h Thu Mar 24 10:27:56 2011 > +0000 > @@ -0,0 +1,91 @@ > +/* Copyright (C) 2002, 2004 Christopher Clark < > firstname.lastname@cl.cam.ac.uk> */ > + > +/* > + * There are duplicates of this code in: > + * - tools/xenstore/hashtable_private.h > + * - tools/vtpm_manager/util/hashtable_private.h > + */ > + > +#ifndef __HASHTABLE_PRIVATE_CWC22_H__ > +#define __HASHTABLE_PRIVATE_CWC22_H__ > + > +#include "hashtable.h" > + > > +/*****************************************************************************/ > +struct entry > +{ > + void *k, *v; > + unsigned int h; > + struct entry *next; > +}; > + > +struct hashtable { > + unsigned int tablelength; > + struct entry **table; > + unsigned int entrycount; > + unsigned int loadlimit; > + unsigned int primeindex; > + unsigned int (*hashfn) (void *k); > + int (*eqfn) (void *k1, void *k2); > +}; > + > > +/*****************************************************************************/ > +unsigned int > +hash(struct hashtable *h, void *k); > + > > +/*****************************************************************************/ > +/* indexFor */ > +static inline unsigned int > +indexFor(unsigned int tablelength, unsigned int hashvalue) { > + return (hashvalue % tablelength); > +}; > + > +/* Only works if tablelength == 2^N */ > +/*static inline unsigned int > +indexFor(unsigned int tablelength, unsigned int hashvalue) > +{ > + return (hashvalue & (tablelength - 1u)); > +} > +*/ > + > > +/*****************************************************************************/ > +#define freekey(X) free(X) > +/*define freekey(X) ; */ > + > + > > +/*****************************************************************************/ > + > +#endif /* __HASHTABLE_PRIVATE_CWC22_H__*/ > + > +/* > + * Copyright (c) 2002, Christopher Clark > + * All rights reserved. > + * > + * Redistribution and use in source and binary forms, with or without > + * modification, are permitted provided that the following conditions > + * are met: > + * > + * * Redistributions of source code must retain the above copyright > + * notice, this list of conditions and the following disclaimer. > + * > + * * Redistributions in binary form must reproduce the above copyright > + * notice, this list of conditions and the following disclaimer in the > + * documentation and/or other materials provided with the distribution. > + * > + * * Neither the name of the original author; nor the names of any > contributors > + * may be used to endorse or promote products derived from this software > + * without specific prior written permission. > + * > + * > + * THIS SOFTWARE IS PROVIDED BY THE COPYRIGHT HOLDERS AND CONTRIBUTORS > + * "AS IS" AND ANY EXPRESS OR IMPLIED WARRANTIES, INCLUDING, BUT NOT > + * LIMITED TO, THE IMPLIED WARRANTIES OF MERCHANTABILITY AND FITNESS FOR > + * A PARTICULAR PURPOSE ARE DISCLAIMED. IN NO EVENT SHALL THE COPYRIGHT > OWNER > + * OR CONTRIBUTORS BE LIABLE FOR ANY DIRECT, INDIRECT, INCIDENTAL, > SPECIAL, > + * EXEMPLARY, OR CONSEQUENTIAL DAMAGES (INCLUDING, BUT NOT LIMITED TO, > + * PROCUREMENT OF SUBSTITUTE GOODS OR SERVICES; LOSS OF USE, DATA, OR > + * PROFITS; OR BUSINESS INTERRUPTION) HOWEVER CAUSED AND ON ANY THEORY OF > + * LIABILITY, WHETHER IN CONTRACT, STRICT LIABILITY, OR TORT (INCLUDING > + * NEGLIGENCE OR OTHERWISE) ARISING IN ANY WAY OUT OF THE USE OF THIS > + * SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE. > +*/ > diff -r 63f4adae7d7b -r 708fdb28b053 tools/vtpm_manager/util/hashtable.c > --- a/tools/vtpm_manager/util/hashtable.c Wed Mar 23 17:04:20 2011 > +0000 > +++ b/tools/vtpm_manager/util/hashtable.c Thu Mar 24 10:27:56 2011 > +0000 > @@ -31,6 +31,12 @@ > * NEGLIGENCE OR OTHERWISE) ARISING IN ANY WAY OUT OF THE USE OF THIS > * SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE. > */ > + > +/* > + * There are duplicates of this code in: > + * - tools/xenstore/hashtable.c > + * - tools/blktap2/drivers/hashtable.c > + */ > > #include "hashtable.h" > #include "hashtable_private.h" > diff -r 63f4adae7d7b -r 708fdb28b053 tools/vtpm_manager/util/hashtable.h > --- a/tools/vtpm_manager/util/hashtable.h Wed Mar 23 17:04:20 2011 > +0000 > +++ b/tools/vtpm_manager/util/hashtable.h Thu Mar 24 10:27:56 2011 > +0000 > @@ -32,6 +32,11 @@ > * SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE. > */ > > +/* > + * There are duplicates of this code in: > + * - tools/xenstore/hashtable.h > + * - tools/blktap2/drivers/hashtable.h > + */ > > #ifndef __HASHTABLE_CWC22_H__ > #define __HASHTABLE_CWC22_H__ > diff -r 63f4adae7d7b -r 708fdb28b053 > tools/vtpm_manager/util/hashtable_itr.c > --- a/tools/vtpm_manager/util/hashtable_itr.c Wed Mar 23 17:04:20 2011 > +0000 > +++ b/tools/vtpm_manager/util/hashtable_itr.c Thu Mar 24 10:27:56 2011 > +0000 > @@ -31,6 +31,11 @@ > * NEGLIGENCE OR OTHERWISE) ARISING IN ANY WAY OUT OF THE USE OF THIS > * SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE. > */ > + > +/* > + * There are duplicates of this code in: > + * - tools/blktap2/drivers/hashtable_itr.c > + */ > > #include "hashtable.h" > #include "hashtable_private.h" > diff -r 63f4adae7d7b -r 708fdb28b053 > tools/vtpm_manager/util/hashtable_itr.h > --- a/tools/vtpm_manager/util/hashtable_itr.h Wed Mar 23 17:04:20 2011 > +0000 > +++ b/tools/vtpm_manager/util/hashtable_itr.h Thu Mar 24 10:27:56 2011 > +0000 > @@ -31,6 +31,11 @@ > * NEGLIGENCE OR OTHERWISE) ARISING IN ANY WAY OUT OF THE USE OF THIS > * SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE. > */ > + > +/* > + * There are duplicates of this code in: > + * - tools/blktap2/drivers/hashtable_itr.h > + */ > > > #ifndef __HASHTABLE_ITR_CWC22__ > diff -r 63f4adae7d7b -r 708fdb28b053 > tools/vtpm_manager/util/hashtable_private.h > --- a/tools/vtpm_manager/util/hashtable_private.h Wed Mar 23 17:04:20 > 2011 +0000 > +++ b/tools/vtpm_manager/util/hashtable_private.h Thu Mar 24 10:27:56 > 2011 +0000 > @@ -31,6 +31,12 @@ > * NEGLIGENCE OR OTHERWISE) ARISING IN ANY WAY OUT OF THE USE OF THIS > * SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE. > */ > + > +/* > + * There are duplicates of this code in: > + * - tools/xenstore/hashtable_private.h > + * - tools/blktap2/drivers/hashtable_private.h > + */ > > #ifndef __HASHTABLE_PRIVATE_CWC22_H__ > #define __HASHTABLE_PRIVATE_CWC22_H__ > diff -r 63f4adae7d7b -r 708fdb28b053 tools/xenstore/hashtable.c > --- a/tools/xenstore/hashtable.c Wed Mar 23 17:04:20 2011 +0000 > +++ b/tools/xenstore/hashtable.c Thu Mar 24 10:27:56 2011 +0000 > @@ -1,4 +1,10 @@ > /* Copyright (C) 2004 Christopher Clark > */ > + > +/* > + * There are duplicates of this code in: > + * - tools/blktap2/drivers/hashtable.c > + * - tools/vtpm_manager/util/hashtable.c > + */ > > #include "hashtable.h" > #include "hashtable_private.h" > diff -r 63f4adae7d7b -r 708fdb28b053 tools/xenstore/hashtable.h > --- a/tools/xenstore/hashtable.h Wed Mar 23 17:04:20 2011 +0000 > +++ b/tools/xenstore/hashtable.h Thu Mar 24 10:27:56 2011 +0000 > @@ -1,4 +1,10 @@ > /* Copyright (C) 2002 Christopher Clark > */ > + > +/* > + * There are duplicates of this code in: > + * - tools/blktap2/drivers/hashtable.h > + * - tools/vtpm_manager/util/hashtable.h > + */ > > #ifndef __HASHTABLE_CWC22_H__ > #define __HASHTABLE_CWC22_H__ > diff -r 63f4adae7d7b -r 708fdb28b053 tools/xenstore/hashtable_private.h > --- a/tools/xenstore/hashtable_private.h Wed Mar 23 17:04:20 2011 > +0000 > +++ b/tools/xenstore/hashtable_private.h Thu Mar 24 10:27:56 2011 > +0000 > @@ -1,4 +1,10 @@ > /* Copyright (C) 2002, 2004 Christopher Clark < > firstname.lastname@cl.cam.ac.uk> */ > + > +/* > + * There are duplicates of this code in: > + * - tools/blktap2/drivers/hashtable_private.h > + * - tools/vtpm_manager/util/hashtable_private.h > + */ > > #ifndef __HASHTABLE_PRIVATE_CWC22_H__ > #define __HASHTABLE_PRIVATE_CWC22_H__ > > > > _______________________________________________ > Xen-devel mailing list > Xen-devel@lists.xensource.com > http://lists.xensource.com/xen-devel > I am not sure if my ack is needed (as this touches hashtable* in blktap2/drivers/) :P. If it is needed, Acked-by: Shriram Rajagopalan thanks shriram --0016e6470e649ef1ca049f3de9ad Content-Type: text/html; charset=ISO-8859-1 Content-Transfer-Encoding: quoted-printable
On Thu, Mar 24, 2011 at 3:29 AM, Ian Campbell <Ian.Campbell= @citrix.com> wrote:
On Mon, 2011-03-21 at 18:03 +0000, Ian Jackson wrote:
> Ian Campbell writes ("[Xen-devel] [PATCH 09 of 30] tools: blktap2= : copy xenstore/hashtable.h into blktap2"):
> > tools: blktap2: copy xenstore/hashtable.h into blktap2
> >
> > hashtable.c has already been copied, forked and modified, there > > doesn't seem much point in avoiding the same for the header u= ntil
> > someone feels motivated to properly refactor.
>
> Before we make this any worse, perhaps we should at least leave a
> comment in every copy of hashtable.c referring to every other clone ?<= br>
Sure. I found another one in vtpm_manager while doing so as well :-(<= br>
8<-----------------------

# HG changeset patch
# User Ian Campbell <ian.camp= bell@citrix.com>
# Date 1300962476 0
# Node ID 708fdb28b0530e331a6df59cd42b1c3acfc85f8b
# Parent =A063f4adae7d7b70d323181ff31fc19f5432d283ff
tools: blktap2: copy xenstore/hashtable.h into blktap2
hashtable.c has already been copied, forked and modified, there
doesn't seem much point in avoiding the same for the header until
someone feels motivated to properly refactor.

Add comments to the various duplicated files cross-referencing each other for future reference and as a barrier to forking again...

Signed-off-by: Ian Campbell <= ian.campbell@citrix.com>

diff -r 63f4adae7d7b -r 708fdb28b053 tools/blktap2/drivers/Makefile --- a/tools/blktap2/drivers/Makefile =A0 =A0Wed Mar 23 17:04:20 2011 +0000<= br> +++ b/tools/blktap2/drivers/Makefile =A0 =A0Thu Mar 24 10:27:56 2011 +0000<= br>
@@ -29,8 +29,6 @@ REMUS-OBJS =A0+=3D hashtable.o
=A0REMUS-OBJS =A0+=3D hashtable.o
=A0REMUS-OBJS =A0+=3D hashtable_itr.o
=A0REMUS-OBJS =A0+=3D hashtable_utility.o
-
-$(REMUS-OBJS): CFLAGS +=3D -I$(XEN_XENSTORE)

=A0LIBAIO_DIR =3D $(XEN_ROOT)/tools/libaio/src
=A0MEMSHR_DIR =3D $(XEN_ROOT)/tools/memshr
diff -r 63f4adae7d7b -r 708fdb28b053 tools/blktap2/drivers/hashtable.= c
--- a/tools/blktap2/drivers/hashtable.c Wed Mar 23 17:04:20 2011 +0000
+++ b/tools/blktap2/drivers/hashtable.c Thu Mar 24 10:27:56 2011 +0000
@@ -1,4 +1,10 @@
=A0/* Copyright (C) 2004 Christopher Clark <firstname.lastname@cl.cam.ac.uk> */
+
+/*
+ * There are duplicates of this code in:
+ * =A0- tools/xenstore/hashtable.c
+ * =A0- tools/vtpm_manager/util/hashtable.c
+ */

=A0#include "hashtable.h"
=A0#include "hashtable_private.h"
diff -r 63f4adae7d7b -r 708fdb28b053 tools/blktap2/drivers/hashtable.h
--- /dev/null =A0 Thu Jan 01 00:00:00 1970 +0000
+++ b/tools/blktap2/drivers/hashtable.h Thu Mar 24 10:27:56 2011 +000= 0
@@ -0,0 +1,205 @@
+/* Copyright (C) 2002 Christopher Clark <firstname.lastname@cl.cam.ac.uk&= gt; */
+
+/*
+ * There are duplicates of this code in:
+ * =A0- tools/xenstore/hashtable.h
+ * =A0- tools/vtpm_manager/util/hashtable.h
+ */
+
+#ifndef __HASHTABLE_CWC22_H__
+#define __HASHTABLE_CWC22_H__
+
+struct hashtable;
+
+/* Example of use:
+ *
+ * =A0 =A0 =A0struct hashtable =A0*h;
+ * =A0 =A0 =A0struct some_key =A0 *k;
+ * =A0 =A0 =A0struct some_value *v;
+ *
+ * =A0 =A0 =A0static unsigned int =A0 =A0 =A0 =A0 hash_from_key_fn( void *= k );
+ * =A0 =A0 =A0static int =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0keys_equal_fn = ( void *key1, void *key2 );
+ *
+ * =A0 =A0 =A0h =3D create_hashtable(16, hash_from_key_fn, keys_equal_fn);=
+ * =A0 =A0 =A0k =3D (struct some_key *) =A0 =A0 malloc(sizeof(struct some_= key));
+ * =A0 =A0 =A0v =3D (struct some_value *) =A0 malloc(sizeof(struct some_va= lue));
+ *
+ * =A0 =A0 =A0(initialise k and v to suitable values)
+ *
+ * =A0 =A0 =A0if (! hashtable_insert(h,k,v) )
+ * =A0 =A0 =A0{ =A0 =A0 exit(-1); =A0 =A0 =A0 =A0 =A0 =A0 =A0 }
+ *
+ * =A0 =A0 =A0if (NULL =3D=3D (found =3D hashtable_search(h,k) ))
+ * =A0 =A0 =A0{ =A0 =A0printf("not found!"); =A0 =A0 =A0 =A0 =A0= =A0 =A0 =A0 =A0}
+ *
+ * =A0 =A0 =A0if (NULL =3D=3D (found =3D hashtable_remove(h,k) ))
+ * =A0 =A0 =A0{ =A0 =A0printf("Not found\n"); =A0 =A0 =A0 =A0 = =A0 =A0 =A0 =A0 }
+ *
+ */
+
+/* Macros may be used to define type-safe(r) hashtable access functions, w= ith
+ * methods specialized to take known key and value types as parameters. + *
+ * Example:
+ *
+ * Insert this at the start of your file:
+ *
+ * DEFINE_HASHTABLE_INSERT(insert_some, struct some_key, struct some_value= );
+ * DEFINE_HASHTABLE_SEARCH(search_some, struct some_key, struct some_value= );
+ * DEFINE_HASHTABLE_REMOVE(remove_some, struct some_key, struct some_value= );
+ *
+ * This defines the functions 'insert_some', 'search_some'= and 'remove_some'.
+ * These operate just like hashtable_insert etc., with the same parameters= ,
+ * but their function signatures have 'struct some_key *' rather t= han
+ * 'void *', and hence can generate compile time errors if your pr= ogram is
+ * supplying incorrect data as a key (and similarly for value).
+ *
+ * Note that the hash and key equality functions passed to create_hashtabl= e
+ * still take 'void *' parameters instead of 'some key *'.= This shouldn't be
+ * a difficult issue as they're only defined and passed once, and the = other
+ * functions will ensure that only valid keys are supplied to them.
+ *
+ * The cost for this checking is increased code size and runtime overhead<= br> + * - if performance is important, it may be worth switching back to the + * unsafe methods once your program has been debugged with the safe method= s.
+ * This just requires switching to some simple alternative defines - eg: + * #define insert_some hashtable_insert
+ *
+ */
+
+/*************************************************************************= ****
+ * create_hashtable
+
+ * @name =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0create_hashtable
+ * @param =A0 minsize =A0 =A0 =A0 =A0 minimum initial size of hashtable + * @param =A0 hashfunction =A0 =A0function for hashing keys
+ * @param =A0 key_eq_fn =A0 =A0 =A0 function for determining key equality<= br> + * @return =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0newly created hashtable or N= ULL on failure
+ */
+
+struct hashtable *
+create_hashtable(unsigned int minsize,
+ =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 unsigned int (*hashfunction) (void*),
+ =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 int (*key_eq_fn) (void*,void*));
+
+/*************************************************************************= ****
+ * hashtable_insert
+
+ * @name =A0 =A0 =A0 =A0hashtable_insert
+ * @param =A0 h =A0 the hashtable to insert into
+ * @param =A0 k =A0 the key - hashtable claims ownership and will free on = removal
+ * @param =A0 v =A0 the value - does not claim ownership
+ * @return =A0 =A0 =A0non-zero for successful insertion
+ *
+ * This function will cause the table to expand if the insertion would tak= e
+ * the ratio of entries to table size over the maximum load factor.
+ *
+ * This function does not check for repeated insertions with a duplicate k= ey.
+ * The value returned when using a duplicate key is undefined -- when
+ * the hashtable changes size, the order of retrieval of duplicate key
+ * entries is reversed.
+ * If in doubt, remove before insert.
+ */
+
+int
+hashtable_insert(struct hashtable *h, void *k, void *v);
+
+#define DEFINE_HASHTABLE_INSERT(fnname, keytype, valuetype) \
+int fnname (struct hashtable *h, keytype *k, valuetype *v) \
+{ \
+ =A0 =A0return hashtable_insert(h,k,v); \
+}
+
+/*************************************************************************= ****
+ * hashtable_search
+
+ * @name =A0 =A0 =A0 =A0hashtable_search
+ * @param =A0 h =A0 the hashtable to search
+ * @param =A0 k =A0 the key to search for =A0- does not claim ownership + * @return =A0 =A0 =A0the value associated with the key, or NULL if none f= ound
+ */
+
+void *
+hashtable_search(struct hashtable *h, void *k);
+
+#define DEFINE_HASHTABLE_SEARCH(fnname, keytype, valuetype) \
+valuetype * fnname (struct hashtable *h, keytype *k) \
+{ \
+ =A0 =A0return (valuetype *) (hashtable_search(h,k)); \
+}
+
+/*************************************************************************= ****
+ * hashtable_remove
+
+ * @name =A0 =A0 =A0 =A0hashtable_remove
+ * @param =A0 h =A0 the hashtable to remove the item from
+ * @param =A0 k =A0 the key to search for =A0- does not claim ownership + * @return =A0 =A0 =A0the value associated with the key, or NULL if none f= ound
+ */
+
+void * /* returns value */
+hashtable_remove(struct hashtable *h, void *k);
+
+#define DEFINE_HASHTABLE_REMOVE(fnname, keytype, valuetype) \
+valuetype * fnname (struct hashtable *h, keytype *k) \
+{ \
+ =A0 =A0return (valuetype *) (hashtable_remove(h,k)); \
+}
+
+
+/*************************************************************************= ****
+ * hashtable_count
+
+ * @name =A0 =A0 =A0 =A0hashtable_count
+ * @param =A0 h =A0 the hashtable
+ * @return =A0 =A0 =A0the number of items stored in the hashtable
+ */
+unsigned int
+hashtable_count(struct hashtable *h);
+
+
+/*************************************************************************= ****
+ * hashtable_destroy
+
+ * @name =A0 =A0 =A0 =A0hashtable_destroy
+ * @param =A0 h =A0 the hashtable
+ * @param =A0 =A0 =A0 free_values =A0 =A0 whether to call 'free' o= n the remaining values
+ */
+
+void
+hashtable_destroy(struct hashtable *h, int free_values);
+
+#endif /* __HASHTABLE_CWC22_H__ */
+
+/*
+ * Copyright (c) 2002, Christopher Clark
+ * All rights reserved.
+ *
+ * Redistribution and use in source and binary forms, with or without
+ * modification, are permitted provided that the following conditions
+ * are met:
+ *
+ * * Redistributions of source code must retain the above copyright
+ * notice, this list of conditions and the following disclaimer.
+ *
+ * * Redistributions in binary form must reproduce the above copyright
+ * notice, this list of conditions and the following disclaimer in the
+ * documentation and/or other materials provided with the distribution. + *
+ * * Neither the name of the original author; nor the names of any contrib= utors
+ * may be used to endorse or promote products derived from this software + * without specific prior written permission.
+ *
+ *
+ * THIS SOFTWARE IS PROVIDED BY THE COPYRIGHT HOLDERS AND CONTRIBUTORS
+ * "AS IS" AND ANY EXPRESS OR IMPLIED WARRANTIES, INCLUDING, BUT= NOT
+ * LIMITED TO, THE IMPLIED WARRANTIES OF MERCHANTABILITY AND FITNESS FOR + * A PARTICULAR PURPOSE ARE DISCLAIMED. =A0IN NO EVENT SHALL THE COPYRIGHT= OWNER
+ * OR CONTRIBUTORS BE LIABLE FOR ANY DIRECT, INDIRECT, INCIDENTAL, SPECIAL= ,
+ * EXEMPLARY, OR CONSEQUENTIAL DAMAGES (INCLUDING, BUT NOT LIMITED TO,
+ * PROCUREMENT OF SUBSTITUTE GOODS OR SERVICES; LOSS OF USE, DATA, OR
+ * PROFITS; OR BUSINESS INTERRUPTION) HOWEVER CAUSED AND ON ANY THEORY OF<= br> + * LIABILITY, WHETHER IN CONTRACT, STRICT LIABILITY, OR TORT (INCLUDING + * NEGLIGENCE OR OTHERWISE) ARISING IN ANY WAY OUT OF THE USE OF THIS
+ * SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE.
+*/
diff -r 63f4adae7d7b -r 708fdb28b053 tools/blktap2/drivers/hash= table_itr.c
--- a/tools/blktap2/drivers/hashtable_itr.c =A0 =A0 Wed Mar 23 17:04:20 201= 1 +0000
+++ b/tools/blktap2/drivers/hashtable_itr.c =A0 =A0 Thu Mar 24 10:27:56 201= 1 +0000
@@ -1,4 +1,9 @@
=A0/* Copyright (C) 2002, 2004 Christopher Clark =A0<<= a href=3D"mailto:firstname.lastname@cl.cam.ac.uk">firstname.lastname@cl.cam= .ac.uk> */
+
+/*
+ * There are duplicates of this code in:
+ * =A0- tools/vtpm_manager/util/hashtable_itr.c
+ */

=A0#include "hashtable.h"
=A0#include "hashtable_private.h"
diff -r 63f4adae7d7b -r 708fdb28b053 tools/blktap2/drivers/hashtable_itr.h<= br> --- a/tools/blktap2/drivers/hashtable_itr.h =A0 =A0 Wed Mar 23 17:04:20 201= 1 +0000
+++ b/tools/blktap2/drivers/hashtable_itr.h =A0 =A0 Thu Mar 24 10:27:56 201= 1 +0000
@@ -1,4 +1,9 @@
=A0/* Copyright (C) 2002, 2004 Christopher Clark <firstname.lastname@cl.cam.ac= .uk> */
+
+/*
+ * There are duplicates of this code in:
+ * =A0- tools/vtpm_manager/util/hashtable_itr.h
+ */

=A0#ifndef __HASHTABLE_ITR_CWC22__
=A0#define __HASHTABLE_ITR_CWC22__
diff -r 63f4adae7d7b -r 708fdb28b053 tools/blktap2/drivers/hashtable_privat= e.h
--- /dev/null =A0 Thu Jan 01 00:00:00 1970 +0000
+++ b/tools/blktap2/drivers/hashtable_private.h Thu Mar 24 10:27:56 2= 011 +0000
@@ -0,0 +1,91 @@
+/* Copyright (C) 2002, 2004 Christopher Clark <firstname.lastname@cl.cam.ac.u= k> */
+
+/*
+ * There are duplicates of this code in:
+ * =A0- tools/xenstore/hashtable_private.h
+ * =A0- tools/vtpm_manager/util/hashtable_private.h
+ */
+
+#ifndef __HASHTABLE_PRIVATE_CWC22_H__
+#define __HASHTABLE_PRIVATE_CWC22_H__
+
+#include "hashtable.h"
+
+/*************************************************************************= ****/
+struct entry
+{
+ =A0 =A0void *k, *v;
+ =A0 =A0unsigned int h;
+ =A0 =A0struct entry *next;
+};
+
+struct hashtable {
+ =A0 =A0unsigned int tablelength;
+ =A0 =A0struct entry **table;
+ =A0 =A0unsigned int entrycount;
+ =A0 =A0unsigned int loadlimit;
+ =A0 =A0unsigned int primeindex;
+ =A0 =A0unsigned int (*hashfn) (void *k);
+ =A0 =A0int (*eqfn) (void *k1, void *k2);
+};
+
+/*************************************************************************= ****/
+unsigned int
+hash(struct hashtable *h, void *k);
+
+/*************************************************************************= ****/
+/* indexFor */
+static inline unsigned int
+indexFor(unsigned int tablelength, unsigned int hashvalue) {
+ =A0 =A0return (hashvalue % tablelength);
+};
+
+/* Only works if tablelength =3D=3D 2^N */
+/*static inline unsigned int
+indexFor(unsigned int tablelength, unsigned int hashvalue)
+{
+ =A0 =A0return (hashvalue & (tablelength - 1u));
+}
+*/
+
+/*************************************************************************= ****/
+#define freekey(X) free(X)
+/*define freekey(X) ; */
+
+
+/*************************************************************************= ****/
+
+#endif /* __HASHTABLE_PRIVATE_CWC22_H__*/
+
+/*
+ * Copyright (c) 2002, Christopher Clark
+ * All rights reserved.
+ *
+ * Redistribution and use in source and binary forms, with or without
+ * modification, are permitted provided that the following conditions
+ * are met:
+ *
+ * * Redistributions of source code must retain the above copyright
+ * notice, this list of conditions and the following disclaimer.
+ *
+ * * Redistributions in binary form must reproduce the above copyright
+ * notice, this list of conditions and the following disclaimer in the
+ * documentation and/or other materials provided with the distribution. + *
+ * * Neither the name of the original author; nor the names of any contrib= utors
+ * may be used to endorse or promote products derived from this software + * without specific prior written permission.
+ *
+ *
+ * THIS SOFTWARE IS PROVIDED BY THE COPYRIGHT HOLDERS AND CONTRIBUTORS
+ * "AS IS" AND ANY EXPRESS OR IMPLIED WARRANTIES, INCLUDING, BUT= NOT
+ * LIMITED TO, THE IMPLIED WARRANTIES OF MERCHANTABILITY AND FITNESS FOR + * A PARTICULAR PURPOSE ARE DISCLAIMED. =A0IN NO EVENT SHALL THE COPYRIGHT= OWNER
+ * OR CONTRIBUTORS BE LIABLE FOR ANY DIRECT, INDIRECT, INCIDENTAL, SPECIAL= ,
+ * EXEMPLARY, OR CONSEQUENTIAL DAMAGES (INCLUDING, BUT NOT LIMITED TO,
+ * PROCUREMENT OF SUBSTITUTE GOODS OR SERVICES; LOSS OF USE, DATA, OR
+ * PROFITS; OR BUSINESS INTERRUPTION) HOWEVER CAUSED AND ON ANY THEORY OF<= br> + * LIABILITY, WHETHER IN CONTRACT, STRICT LIABILITY, OR TORT (INCLUDING + * NEGLIGENCE OR OTHERWISE) ARISING IN ANY WAY OUT OF THE USE OF THIS
+ * SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE.
+*/
diff -r 63f4adae7d7b -r 708fdb28b053 tools/vtpm_manager/util/ha= shtable.c
--- a/tools/vtpm_manager/util/hashtable.c =A0 =A0 =A0 Wed Mar 23 17:04:20 2= 011 +0000
+++ b/tools/vtpm_manager/util/hashtable.c =A0 =A0 =A0 Thu Mar 24 10:27:56 2= 011 +0000
@@ -31,6 +31,12 @@
=A0* NEGLIGENCE OR OTHERWISE) ARISING IN ANY WAY OUT OF = THE USE OF THIS
=A0* SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE.
=A0*/
+
+/*
+ * There are duplicates of this code in:
+ * =A0- tools/xenstore/hashtable.c
+ * =A0- tools/blktap2/drivers/hashtable.c
+ */

=A0#include "hashtable.h"
=A0#include "hashtable_private.h"
diff -r 63f4adae7d7b -r 708fdb28b053 tools/vtpm_manager/util/hashtable.h --- a/tools/vtpm_manager/util/hashtable.h =A0 =A0 =A0 Wed Mar 23 17:04:20 2= 011 +0000
+++ b/tools/vtpm_manager/util/hashtable.h =A0 =A0 =A0 Thu Mar 24 10:27:56 2= 011 +0000
@@ -32,6 +32,11 @@
=A0* SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUC= H DAMAGE.
=A0*/

+/*
+ * There are duplicates of this code in:
+ * =A0- tools/xenstore/hashtable.h
+ * =A0- tools/blktap2/drivers/hashtable.h
+ */

=A0#ifndef __HASHTABLE_CWC22_H__
=A0#define __HASHTABLE_CWC22_H__
diff -r 63f4adae7d7b -r 708fdb28b053 tools/vtpm_manager/util/hashtabl= e_itr.c
--- a/tools/vtpm_manager/util/hashtable_itr.c =A0 Wed Mar 23 17:04:20 2011 = +0000
+++ b/tools/vtpm_manager/util/hashtable_itr.c =A0 Thu Mar 24 10:27:56 2011 = +0000
@@ -31,6 +31,11 @@
=A0* NEGLIGENCE OR OTHERWISE) ARISING IN ANY WAY OUT OF = THE USE OF THIS
=A0* SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE.
=A0*/
+
+/*
+ * There are duplicates of this code in:
+ * =A0- tools/blktap2/drivers/hashtable_itr.c
+ */

=A0#include "hashtable.h"
=A0#include "hashtable_private.h"
diff -r 63f4adae7d7b -r 708fdb28b053 tools/vtpm_manager/util/hashtable_itr.= h
--- a/tools/vtpm_manager/util/hashtable_itr.h =A0 Wed Mar 23 17:04:20 2011 = +0000
+++ b/tools/vtpm_manager/util/hashtable_itr.h =A0 Thu Mar 24 10:27:56 2011 = +0000
@@ -31,6 +31,11 @@
=A0* NEGLIGENCE OR OTHERWISE) ARISING IN ANY WAY OUT OF = THE USE OF THIS
=A0* SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE.
=A0*/
+
+/*
+ * There are duplicates of this code in:
+ * =A0- tools/blktap2/drivers/hashtable_itr.h
+ */


=A0#ifndef __HASHTABLE_ITR_CWC22__
diff -r 63f4adae7d7b -r 708fdb28b053 tools/vtpm_manager/util/hashtable_priv= ate.h
--- a/tools/vtpm_manager/util/hashtable_private.h =A0 =A0 =A0 Wed Mar 23 17= :04:20 2011 +0000
+++ b/tools/vtpm_manager/util/hashtable_private.h =A0 =A0 =A0 Thu Mar 24 10= :27:56 2011 +0000
@@ -31,6 +31,12 @@
=A0* NEGLIGENCE OR OTHERWISE) ARISING IN ANY WAY OUT OF = THE USE OF THIS
=A0* SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE.
=A0*/
+
+/*
+ * There are duplicates of this code in:
+ * =A0- tools/xenstore/hashtable_private.h
+ * =A0- tools/blktap2/drivers/hashtable_private.h
+ */

=A0#ifndef __HASHTABLE_PRIVATE_CWC22_H__
=A0#define __HASHTABLE_PRIVATE_CWC22_H__
diff -r 63f4adae7d7b -r 708fdb28b053 tools/xenstore/hashtable.c
--- a/tools/xenstore/hashtable.c =A0 =A0 =A0 =A0Wed Mar 23 17:04:20 2011 +0= 000
+++ b/tools/xenstore/hashtable.c =A0 =A0 =A0 =A0Thu Mar 24 10:27:56 2011 +0= 000
@@ -1,4 +1,10 @@
=A0/* Copyright (C) 2004 Christopher Clark <firstname.lastname@cl.cam.ac.uk> */
+
+/*
+ * There are duplicates of this code in:
+ * =A0- tools/blktap2/drivers/hashtable.c
+ * =A0- tools/vtpm_manager/util/hashtable.c
+ */

=A0#include "hashtable.h"
=A0#include "hashtable_private.h"
diff -r 63f4adae7d7b -r 708fdb28b053 tools/xenstore/hashtable.h
--- a/tools/xenstore/hashtable.h =A0 =A0 =A0 =A0Wed Mar 23 17:04:20 2011 +0= 000
+++ b/tools/xenstore/hashtable.h =A0 =A0 =A0 =A0Thu Mar 24 10:27:56 2011 +0= 000
@@ -1,4 +1,10 @@
=A0/* Copyright (C) 2002 Christopher Clark <firstname.lastname@cl.cam.ac.uk> */
+
+/*
+ * There are duplicates of this code in:
+ * =A0- tools/blktap2/drivers/hashtable.h
+ * =A0- tools/vtpm_manager/util/hashtable.h
+ */

=A0#ifndef __HASHTABLE_CWC22_H__
=A0#define __HASHTABLE_CWC22_H__
diff -r 63f4adae7d7b -r 708fdb28b053 tools/xenstore/hashtable_private= .h
--- a/tools/xenstore/hashtable_private.h =A0 =A0 =A0 =A0Wed Mar 23 17:04:20= 2011 +0000
+++ b/tools/xenstore/hashtable_private.h =A0 =A0 =A0 =A0Thu Mar 24 10:27:56= 2011 +0000
@@ -1,4 +1,10 @@
=A0/* Copyright (C) 2002, 2004 Christopher Clark <firstname.lastname@cl.cam.ac= .uk> */
+
+/*
+ * There are duplicates of this code in:
+ * =A0- tools/blktap2/drivers/hashtable_private.h
+ * =A0- tools/vtpm_manager/util/hashtable_private.h
+ */

=A0#ifndef __HASHTABLE_PRIVATE_CWC22_H__
=A0#define __HASHTABLE_PRIVATE_CWC22_H__



___________________________________= ____________
Xen-devel mailing list
Xen-devel@lists.xensource.= com
http://l= ists.xensource.com/xen-devel
I am not sure if my ack is needed (as this t= ouches hashtable* in blktap2/drivers/) :P.
If it is needed,
Acked-by:= Shriram Rajagopalan <rshriram@cs.= ubc.ca>

thanks
shriram
--0016e6470e649ef1ca049f3de9ad-- --===============0990140258== Content-Type: text/plain; charset="us-ascii" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit Content-Disposition: inline _______________________________________________ Xen-devel mailing list Xen-devel@lists.xensource.com http://lists.xensource.com/xen-devel --===============0990140258==--