From patchwork Tue Aug 22 07:45:02 2023 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: =?utf-8?b?SsO8cmdlbiBHcm/Dnw==?= X-Patchwork-Id: 13360305 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org Received: from lists.xenproject.org (lists.xenproject.org [192.237.175.120]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.lore.kernel.org (Postfix) with ESMTPS id 64928EE4993 for ; Tue, 22 Aug 2023 07:45:17 +0000 (UTC) Received: from list by lists.xenproject.org with outflank-mailman.588183.919667 (Exim 4.92) (envelope-from ) id 1qYM4Z-0006KI-Ty; Tue, 22 Aug 2023 07:45:07 +0000 X-Outflank-Mailman: Message body and most headers restored to incoming version Received: by outflank-mailman (output) from mailman id 588183.919667; Tue, 22 Aug 2023 07:45:07 +0000 Received: from localhost ([127.0.0.1] helo=lists.xenproject.org) by lists.xenproject.org with esmtp (Exim 4.92) (envelope-from ) id 1qYM4Z-0006KB-Qu; Tue, 22 Aug 2023 07:45:07 +0000 Received: by outflank-mailman (input) for mailman id 588183; Tue, 22 Aug 2023 07:45:07 +0000 Received: from se1-gles-sth1-in.inumbo.com ([159.253.27.254] helo=se1-gles-sth1.inumbo.com) by lists.xenproject.org with esmtp (Exim 4.92) (envelope-from ) id 1qYM4Z-0006Jy-0G for xen-devel@lists.xenproject.org; Tue, 22 Aug 2023 07:45:07 +0000 Received: from smtp-out2.suse.de (smtp-out2.suse.de [195.135.220.29]) by se1-gles-sth1.inumbo.com (Halon) with ESMTPS id cf0cb860-40bf-11ee-8782-cb3800f73035; Tue, 22 Aug 2023 09:45:05 +0200 (CEST) Received: from imap2.suse-dmz.suse.de (imap2.suse-dmz.suse.de [192.168.254.74]) (using TLSv1.3 with cipher TLS_AES_256_GCM_SHA384 (256/256 bits) key-exchange X25519 server-signature ECDSA (P-521) server-digest SHA512) (No client certificate requested) by smtp-out2.suse.de (Postfix) with ESMTPS id 846441F381; Tue, 22 Aug 2023 07:45:04 +0000 (UTC) Received: from imap2.suse-dmz.suse.de (imap2.suse-dmz.suse.de [192.168.254.74]) (using TLSv1.3 with cipher TLS_AES_256_GCM_SHA384 (256/256 bits) key-exchange X25519 server-signature ECDSA (P-521) server-digest SHA512) (No client certificate requested) by imap2.suse-dmz.suse.de (Postfix) with ESMTPS id 3135513919; Tue, 22 Aug 2023 07:45:04 +0000 (UTC) Received: from dovecot-director2.suse.de ([192.168.254.65]) by imap2.suse-dmz.suse.de with ESMTPSA id +CKnCoBn5GQ3WgAAMHmgww (envelope-from ); Tue, 22 Aug 2023 07:45:04 +0000 X-BeenThere: xen-devel@lists.xenproject.org List-Id: Xen developer discussion List-Unsubscribe: , List-Post: List-Help: List-Subscribe: , Errors-To: xen-devel-bounces@lists.xenproject.org Precedence: list Sender: "Xen-devel" X-Inumbo-ID: cf0cb860-40bf-11ee-8782-cb3800f73035 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=suse.com; s=susede1; t=1692690304; h=from:from:reply-to:date:date:message-id:message-id:to:to:cc:cc: mime-version:mime-version: content-transfer-encoding:content-transfer-encoding; bh=O9RtAv9jURfsxw/veCsBpJ0O2mx8GT/ncDaTN+R0Xzc=; b=G9XjytzerdEdZgkgqRJWZDdWBcIiNMj9nE9E++snrqsRP+feZspupuh20JEoYV+5lkitBm twF/wGu+9pWDHLftHv59qn/GxZqYjDwt6sZsVlHxw/V/DAQiMo3alzJAl/BeUiTmmFVhlf Bbvq/sZdR11h4K+q7COqyI+skc081vI= From: Juergen Gross To: xen-devel@lists.xenproject.org Cc: Juergen Gross , Andrew Cooper , George Dunlap , Jan Beulich , Julien Grall , Stefano Stabellini , Wei Liu , Anthony PERARD , Samuel Thibault , Julien Grall Subject: [PATCH v7] tools/xenstore: move xenstored sources into dedicated directory Date: Tue, 22 Aug 2023 09:45:02 +0200 Message-Id: <20230822074502.5543-1-jgross@suse.com> X-Mailer: git-send-email 2.35.3 MIME-Version: 1.0 In tools/xenstore there are living xenstored and xenstore clients. They are no longer sharing anything apart from the "xenstore" in their names. Move the xenstored sources into a new directory tools/xenstored while dropping the "xenstored_" prefix from their names. This will make it clearer that xenstore clients and xenstored are independent from each other. In order to avoid two very similar named directories below tools, rename tools/xenstore to tools/xs-clients. Drop the make targets [un]install-clients as those are not used in the Xen tree. Signed-off-by: Juergen Gross Reviewed-by: Julien Grall Acked-by: Anthony PERARD --- After the large overhaul of xenstored I think such a reorg would make sense to go into the same Xen version. Delaying it until the next version would make potential backports for 4.18 harder. V4: - new patch V5: - rename xenstore directory to xs-clients (Julien Grall) V6: - use local .gitignore files (Andrew Cooper) V7: - prepend local .gitignore entries with "/" (Anthony PERARD) - drop [un]install-clients make targets (Anthony PERARD) - init xenstored TARGETS make variable with ":=" (Anthony PERARD) --- .gitignore | 11 ----- MAINTAINERS | 3 +- stubdom/Makefile | 4 +- tools/Makefile | 3 +- tools/xenstored/.gitignore | 1 + tools/xenstored/Makefile | 48 +++++++++++++++++++ tools/{xenstore => xenstored}/Makefile.common | 13 +++-- tools/{xenstore => xenstored}/README | 0 .../control.c} | 8 ++-- .../control.h} | 0 .../xenstored_core.c => xenstored/core.c} | 14 +++--- .../xenstored_core.h => xenstored/core.h} | 0 .../xenstored_domain.c => xenstored/domain.c} | 10 ++-- .../xenstored_domain.h => xenstored/domain.h} | 0 tools/{xenstore => xenstored}/hashtable.c | 0 tools/{xenstore => xenstored}/hashtable.h | 0 tools/{xenstore => xenstored}/list.h | 0 .../xenstored_lu.c => xenstored/lu.c} | 8 ++-- .../xenstored_lu.h => xenstored/lu.h} | 0 .../lu_daemon.c} | 4 +- .../lu_minios.c} | 2 +- .../xenstored_minios.c => xenstored/minios.c} | 2 +- .../xenstored_osdep.h => xenstored/osdep.h} | 0 .../xenstored_posix.c => xenstored/posix.c} | 4 +- tools/{xenstore => xenstored}/talloc.c | 0 tools/{xenstore => xenstored}/talloc.h | 0 .../{xenstore => xenstored}/talloc_guide.txt | 0 .../transaction.c} | 6 +-- .../transaction.h} | 2 +- tools/{xenstore => xenstored}/utils.c | 0 tools/{xenstore => xenstored}/utils.h | 0 .../xenstored_watch.c => xenstored/watch.c} | 6 +-- .../xenstored_watch.h => xenstored/watch.h} | 2 +- .../include => xenstored}/xenstore_state.h | 0 tools/xs-clients/.gitignore | 10 ++++ tools/{xenstore => xs-clients}/Makefile | 42 ++-------------- .../xenstore_client.c | 0 .../xenstore_control.c | 0 38 files changed, 110 insertions(+), 93 deletions(-) create mode 100644 tools/xenstored/.gitignore create mode 100644 tools/xenstored/Makefile rename tools/{xenstore => xenstored}/Makefile.common (50%) rename tools/{xenstore => xenstored}/README (100%) rename tools/{xenstore/xenstored_control.c => xenstored/control.c} (98%) rename tools/{xenstore/xenstored_control.h => xenstored/control.h} (100%) rename tools/{xenstore/xenstored_core.c => xenstored/core.c} (99%) rename tools/{xenstore/xenstored_core.h => xenstored/core.h} (100%) rename tools/{xenstore/xenstored_domain.c => xenstored/domain.c} (99%) rename tools/{xenstore/xenstored_domain.h => xenstored/domain.h} (100%) rename tools/{xenstore => xenstored}/hashtable.c (100%) rename tools/{xenstore => xenstored}/hashtable.h (100%) rename tools/{xenstore => xenstored}/list.h (100%) rename tools/{xenstore/xenstored_lu.c => xenstored/lu.c} (98%) rename tools/{xenstore/xenstored_lu.h => xenstored/lu.h} (100%) rename tools/{xenstore/xenstored_lu_daemon.c => xenstored/lu_daemon.c} (97%) rename tools/{xenstore/xenstored_lu_minios.c => xenstored/lu_minios.c} (98%) rename tools/{xenstore/xenstored_minios.c => xenstored/minios.c} (97%) rename tools/{xenstore/xenstored_osdep.h => xenstored/osdep.h} (100%) rename tools/{xenstore/xenstored_posix.c => xenstored/posix.c} (98%) rename tools/{xenstore => xenstored}/talloc.c (100%) rename tools/{xenstore => xenstored}/talloc.h (100%) rename tools/{xenstore => xenstored}/talloc_guide.txt (100%) rename tools/{xenstore/xenstored_transaction.c => xenstored/transaction.c} (99%) rename tools/{xenstore/xenstored_transaction.h => xenstored/transaction.h} (98%) rename tools/{xenstore => xenstored}/utils.c (100%) rename tools/{xenstore => xenstored}/utils.h (100%) rename tools/{xenstore/xenstored_watch.c => xenstored/watch.c} (98%) rename tools/{xenstore/xenstored_watch.h => xenstored/watch.h} (98%) rename tools/{xenstore/include => xenstored}/xenstore_state.h (100%) create mode 100644 tools/xs-clients/.gitignore rename tools/{xenstore => xs-clients}/Makefile (62%) rename tools/{xenstore => xs-clients}/xenstore_client.c (100%) rename tools/{xenstore => xs-clients}/xenstore_control.c (100%) diff --git a/.gitignore b/.gitignore index c1b73b0968..50273adb8d 100644 --- a/.gitignore +++ b/.gitignore @@ -237,17 +237,6 @@ tools/xenmon/xentrace_setmask tools/xenmon/xenbaked tools/xenpaging/xenpaging tools/xenpmd/xenpmd -tools/xenstore/xenstore -tools/xenstore/xenstore-chmod -tools/xenstore/xenstore-control -tools/xenstore/xenstore-exists -tools/xenstore/xenstore-list -tools/xenstore/xenstore-ls -tools/xenstore/xenstore-read -tools/xenstore/xenstore-rm -tools/xenstore/xenstore-watch -tools/xenstore/xenstore-write -tools/xenstore/xenstored tools/xentop/xentop tools/xentrace/xentrace_setsize tools/xentrace/tbctl diff --git a/MAINTAINERS b/MAINTAINERS index a0805d35cd..bf71ac144c 100644 --- a/MAINTAINERS +++ b/MAINTAINERS @@ -669,7 +669,8 @@ F: tools/include/xenstore.h F: tools/include/xenstore_lib.h F: tools/include/xen-tools/xenstore-common.h F: tools/libs/store/ -F: tools/xenstore/ +F: tools/xenstored/ +F: tools/xs-clients/ XENTRACE M: George Dunlap diff --git a/stubdom/Makefile b/stubdom/Makefile index d5fb354e7e..52bf7f0e21 100644 --- a/stubdom/Makefile +++ b/stubdom/Makefile @@ -371,10 +371,10 @@ endef $(foreach lib,$(STUB_LIBS),$(eval $(call BUILD_lib,$(lib)))) -xenstore/stamp: $(XEN_ROOT)/tools/xenstore/Makefile.common +xenstore/stamp: $(XEN_ROOT)/tools/xenstored/Makefile.common $(do_links) -xenstorepvh/stamp: $(XEN_ROOT)/tools/xenstore/Makefile.common +xenstorepvh/stamp: $(XEN_ROOT)/tools/xenstored/Makefile.common $(do_links) LINK_DIRS := xenstore xenstorepvh $(foreach dir,$(STUB_LIBS),libs-$(XEN_TARGET_ARCH)/$(dir)) diff --git a/tools/Makefile b/tools/Makefile index 311a9098d7..3a510663a0 100644 --- a/tools/Makefile +++ b/tools/Makefile @@ -8,7 +8,8 @@ SUBDIRS-y := SUBDIRS-y += libs SUBDIRS-y += flask SUBDIRS-y += fuzz -SUBDIRS-y += xenstore +SUBDIRS-y += xs-clients +SUBDIRS-$(XENSTORE_XENSTORED) += xenstored SUBDIRS-y += misc SUBDIRS-y += examples SUBDIRS-y += hotplug diff --git a/tools/xenstored/.gitignore b/tools/xenstored/.gitignore new file mode 100644 index 0000000000..cec482f21f --- /dev/null +++ b/tools/xenstored/.gitignore @@ -0,0 +1 @@ +/xenstored diff --git a/tools/xenstored/Makefile b/tools/xenstored/Makefile new file mode 100644 index 0000000000..e0897ed1ba --- /dev/null +++ b/tools/xenstored/Makefile @@ -0,0 +1,48 @@ +XEN_ROOT=$(CURDIR)/../.. +include $(XEN_ROOT)/tools/Rules.mk + +include Makefile.common + +xenstored: LDLIBS += $(LDLIBS_libxenevtchn) +xenstored: LDLIBS += $(LDLIBS_libxengnttab) +xenstored: LDLIBS += $(LDLIBS_libxenctrl) +xenstored: LDLIBS += -lrt +xenstored: LDLIBS += $(SOCKET_LIBS) + +ifeq ($(CONFIG_SYSTEMD),y) +$(XENSTORED_OBJS-y): CFLAGS += $(SYSTEMD_CFLAGS) +xenstored: LDLIBS += $(SYSTEMD_LIBS) +endif + +TARGETS := xenstored + +.PHONY: all +all: $(TARGETS) + +xenstored: $(XENSTORED_OBJS-y) + $(CC) $(LDFLAGS) $^ $(LDLIBS) -o $@ $(APPEND_LDFLAGS) + +.PHONY: clean +clean:: + $(RM) $(TARGETS) $(DEPS_RM) + +.PHONY: distclean +distclean: clean + +.PHONY: TAGS +TAGS: + etags `find . -name '*.[ch]'` + +.PHONY: install +install: all + $(INSTALL_DIR) $(DESTDIR)$(sbindir) + $(INSTALL_PROG) xenstored $(DESTDIR)$(sbindir) + +.PHONY: uninstall +uninstall: + rm -f $(DESTDIR)$(sbindir)/xenstored + if [ -d $(DESTDIR)$(includedir)/xenstore-compat ]; then \ + rmdir --ignore-fail-on-non-empty $(DESTDIR)$(includedir)/xenstore-compat; \ + fi + +-include $(DEPS_INCLUDE) diff --git a/tools/xenstore/Makefile.common b/tools/xenstored/Makefile.common similarity index 50% rename from tools/xenstore/Makefile.common rename to tools/xenstored/Makefile.common index 41973a8a5e..189ab81b8d 100644 --- a/tools/xenstore/Makefile.common +++ b/tools/xenstored/Makefile.common @@ -1,17 +1,16 @@ # Makefile shared with stubdom -XENSTORED_OBJS-y := xenstored_core.o xenstored_watch.o xenstored_domain.o -XENSTORED_OBJS-y += xenstored_transaction.o xenstored_control.o xenstored_lu.o +XENSTORED_OBJS-y := core.o watch.o domain.o +XENSTORED_OBJS-y += transaction.o control.o lu.o XENSTORED_OBJS-y += talloc.o utils.o hashtable.o -XENSTORED_OBJS-$(CONFIG_Linux) += xenstored_posix.o xenstored_lu_daemon.o -XENSTORED_OBJS-$(CONFIG_NetBSD) += xenstored_posix.o xenstored_lu_daemon.o -XENSTORED_OBJS-$(CONFIG_FreeBSD) += xenstored_posix.o xenstored_lu_daemon.o -XENSTORED_OBJS-$(CONFIG_MiniOS) += xenstored_minios.o xenstored_lu_minios.o +XENSTORED_OBJS-$(CONFIG_Linux) += posix.o lu_daemon.o +XENSTORED_OBJS-$(CONFIG_NetBSD) += posix.o lu_daemon.o +XENSTORED_OBJS-$(CONFIG_FreeBSD) += posix.o lu_daemon.o +XENSTORED_OBJS-$(CONFIG_MiniOS) += minios.o lu_minios.o # Include configure output (config.h) CFLAGS += -include $(XEN_ROOT)/tools/config.h -CFLAGS += -I./include CFLAGS += $(CFLAGS_libxenevtchn) CFLAGS += $(CFLAGS_libxenctrl) CFLAGS += $(CFLAGS_libxenguest) diff --git a/tools/xenstore/README b/tools/xenstored/README similarity index 100% rename from tools/xenstore/README rename to tools/xenstored/README diff --git a/tools/xenstore/xenstored_control.c b/tools/xenstored/control.c similarity index 98% rename from tools/xenstore/xenstored_control.c rename to tools/xenstored/control.c index 3bdf2edc34..b2f64d674f 100644 --- a/tools/xenstore/xenstored_control.c +++ b/tools/xenstored/control.c @@ -27,10 +27,10 @@ #include "utils.h" #include "talloc.h" -#include "xenstored_core.h" -#include "xenstored_control.h" -#include "xenstored_domain.h" -#include "xenstored_lu.h" +#include "core.h" +#include "control.h" +#include "domain.h" +#include "lu.h" struct cmd_s { char *cmd; diff --git a/tools/xenstore/xenstored_control.h b/tools/xenstored/control.h similarity index 100% rename from tools/xenstore/xenstored_control.h rename to tools/xenstored/control.h diff --git a/tools/xenstore/xenstored_core.c b/tools/xenstored/core.c similarity index 99% rename from tools/xenstore/xenstored_core.c rename to tools/xenstored/core.c index 7de4df2f28..092de76a2e 100644 --- a/tools/xenstore/xenstored_core.c +++ b/tools/xenstored/core.c @@ -47,12 +47,12 @@ #include "utils.h" #include "list.h" #include "talloc.h" -#include "xenstored_core.h" -#include "xenstored_watch.h" -#include "xenstored_transaction.h" -#include "xenstored_domain.h" -#include "xenstored_control.h" -#include "xenstored_lu.h" +#include "core.h" +#include "watch.h" +#include "transaction.h" +#include "domain.h" +#include "control.h" +#include "lu.h" #ifndef NO_SOCKETS #if defined(HAVE_SYSTEMD) @@ -64,7 +64,7 @@ #include #endif -extern xenevtchn_handle *xce_handle; /* in xenstored_domain.c */ +extern xenevtchn_handle *xce_handle; /* in domain.c */ static int xce_pollfd_idx = -1; static struct pollfd *fds; static unsigned int current_array_size; diff --git a/tools/xenstore/xenstored_core.h b/tools/xenstored/core.h similarity index 100% rename from tools/xenstore/xenstored_core.h rename to tools/xenstored/core.h diff --git a/tools/xenstore/xenstored_domain.c b/tools/xenstored/domain.c similarity index 99% rename from tools/xenstore/xenstored_domain.c rename to tools/xenstored/domain.c index 1bf138c8b1..a6cd199fdc 100644 --- a/tools/xenstore/xenstored_domain.c +++ b/tools/xenstored/domain.c @@ -27,11 +27,11 @@ #include "utils.h" #include "talloc.h" -#include "xenstored_core.h" -#include "xenstored_domain.h" -#include "xenstored_transaction.h" -#include "xenstored_watch.h" -#include "xenstored_control.h" +#include "core.h" +#include "domain.h" +#include "transaction.h" +#include "watch.h" +#include "control.h" #include #include diff --git a/tools/xenstore/xenstored_domain.h b/tools/xenstored/domain.h similarity index 100% rename from tools/xenstore/xenstored_domain.h rename to tools/xenstored/domain.h diff --git a/tools/xenstore/hashtable.c b/tools/xenstored/hashtable.c similarity index 100% rename from tools/xenstore/hashtable.c rename to tools/xenstored/hashtable.c diff --git a/tools/xenstore/hashtable.h b/tools/xenstored/hashtable.h similarity index 100% rename from tools/xenstore/hashtable.h rename to tools/xenstored/hashtable.h diff --git a/tools/xenstore/list.h b/tools/xenstored/list.h similarity index 100% rename from tools/xenstore/list.h rename to tools/xenstored/list.h diff --git a/tools/xenstore/xenstored_lu.c b/tools/xenstored/lu.c similarity index 98% rename from tools/xenstore/xenstored_lu.c rename to tools/xenstored/lu.c index f7f76acbf9..2f41d10c95 100644 --- a/tools/xenstore/xenstored_lu.c +++ b/tools/xenstored/lu.c @@ -13,10 +13,10 @@ #include #include "talloc.h" -#include "xenstored_core.h" -#include "xenstored_domain.h" -#include "xenstored_lu.h" -#include "xenstored_watch.h" +#include "core.h" +#include "domain.h" +#include "lu.h" +#include "watch.h" #ifndef NO_LIVE_UPDATE struct live_update *lu_status; diff --git a/tools/xenstore/xenstored_lu.h b/tools/xenstored/lu.h similarity index 100% rename from tools/xenstore/xenstored_lu.h rename to tools/xenstored/lu.h diff --git a/tools/xenstore/xenstored_lu_daemon.c b/tools/xenstored/lu_daemon.c similarity index 97% rename from tools/xenstore/xenstored_lu_daemon.c rename to tools/xenstored/lu_daemon.c index 8c7522b0e1..71bcabadd3 100644 --- a/tools/xenstore/xenstored_lu_daemon.c +++ b/tools/xenstored/lu_daemon.c @@ -13,8 +13,8 @@ #include #include "talloc.h" -#include "xenstored_core.h" -#include "xenstored_lu.h" +#include "core.h" +#include "lu.h" #ifndef NO_LIVE_UPDATE void lu_get_dump_state(struct lu_dump_state *state) diff --git a/tools/xenstore/xenstored_lu_minios.c b/tools/xenstored/lu_minios.c similarity index 98% rename from tools/xenstore/xenstored_lu_minios.c rename to tools/xenstored/lu_minios.c index ae0483575e..ede8b4dd47 100644 --- a/tools/xenstore/xenstored_lu_minios.c +++ b/tools/xenstored/lu_minios.c @@ -14,7 +14,7 @@ #include #include "talloc.h" -#include "xenstored_lu.h" +#include "lu.h" /* Mini-OS only knows about MAP_ANON. */ #ifndef MAP_ANONYMOUS diff --git a/tools/xenstore/xenstored_minios.c b/tools/xenstored/minios.c similarity index 97% rename from tools/xenstore/xenstored_minios.c rename to tools/xenstored/minios.c index aa384e50c8..b5c3a205e6 100644 --- a/tools/xenstore/xenstored_minios.c +++ b/tools/xenstored/minios.c @@ -17,7 +17,7 @@ */ #include #include -#include "xenstored_core.h" +#include "core.h" #include void write_pidfile(const char *pidfile) diff --git a/tools/xenstore/xenstored_osdep.h b/tools/xenstored/osdep.h similarity index 100% rename from tools/xenstore/xenstored_osdep.h rename to tools/xenstored/osdep.h diff --git a/tools/xenstore/xenstored_posix.c b/tools/xenstored/posix.c similarity index 98% rename from tools/xenstore/xenstored_posix.c rename to tools/xenstored/posix.c index b20504d1b6..6ac45fdb45 100644 --- a/tools/xenstore/xenstored_posix.c +++ b/tools/xenstored/posix.c @@ -24,8 +24,8 @@ #include #include "utils.h" -#include "xenstored_core.h" -#include "xenstored_osdep.h" +#include "core.h" +#include "osdep.h" void write_pidfile(const char *pidfile) { diff --git a/tools/xenstore/talloc.c b/tools/xenstored/talloc.c similarity index 100% rename from tools/xenstore/talloc.c rename to tools/xenstored/talloc.c diff --git a/tools/xenstore/talloc.h b/tools/xenstored/talloc.h similarity index 100% rename from tools/xenstore/talloc.h rename to tools/xenstored/talloc.h diff --git a/tools/xenstore/talloc_guide.txt b/tools/xenstored/talloc_guide.txt similarity index 100% rename from tools/xenstore/talloc_guide.txt rename to tools/xenstored/talloc_guide.txt diff --git a/tools/xenstore/xenstored_transaction.c b/tools/xenstored/transaction.c similarity index 99% rename from tools/xenstore/xenstored_transaction.c rename to tools/xenstored/transaction.c index 1f892b002d..167cd597fd 100644 --- a/tools/xenstore/xenstored_transaction.c +++ b/tools/xenstored/transaction.c @@ -30,9 +30,9 @@ #include #include "talloc.h" #include "list.h" -#include "xenstored_transaction.h" -#include "xenstored_watch.h" -#include "xenstored_domain.h" +#include "transaction.h" +#include "watch.h" +#include "domain.h" #include "xenstore_lib.h" #include "utils.h" diff --git a/tools/xenstore/xenstored_transaction.h b/tools/xenstored/transaction.h similarity index 98% rename from tools/xenstore/xenstored_transaction.h rename to tools/xenstored/transaction.h index b196b1ab07..90435b4fc9 100644 --- a/tools/xenstore/xenstored_transaction.h +++ b/tools/xenstored/transaction.h @@ -17,7 +17,7 @@ */ #ifndef _XENSTORED_TRANSACTION_H #define _XENSTORED_TRANSACTION_H -#include "xenstored_core.h" +#include "core.h" enum node_access_type { NODE_ACCESS_READ, diff --git a/tools/xenstore/utils.c b/tools/xenstored/utils.c similarity index 100% rename from tools/xenstore/utils.c rename to tools/xenstored/utils.c diff --git a/tools/xenstore/utils.h b/tools/xenstored/utils.h similarity index 100% rename from tools/xenstore/utils.h rename to tools/xenstored/utils.h diff --git a/tools/xenstore/xenstored_watch.c b/tools/xenstored/watch.c similarity index 98% rename from tools/xenstore/xenstored_watch.c rename to tools/xenstored/watch.c index 7d4d097cf9..b66a9f1a39 100644 --- a/tools/xenstore/xenstored_watch.c +++ b/tools/xenstored/watch.c @@ -25,11 +25,11 @@ #include #include "talloc.h" #include "list.h" -#include "xenstored_watch.h" +#include "watch.h" #include "xenstore_lib.h" #include "utils.h" -#include "xenstored_domain.h" -#include "xenstored_transaction.h" +#include "domain.h" +#include "transaction.h" struct watch { diff --git a/tools/xenstore/xenstored_watch.h b/tools/xenstored/watch.h similarity index 98% rename from tools/xenstore/xenstored_watch.h rename to tools/xenstored/watch.h index ea247997ad..d9ac6a334a 100644 --- a/tools/xenstore/xenstored_watch.h +++ b/tools/xenstored/watch.h @@ -19,7 +19,7 @@ #ifndef _XENSTORED_WATCH_H #define _XENSTORED_WATCH_H -#include "xenstored_core.h" +#include "core.h" int do_watch(const void *ctx, struct connection *conn, struct buffered_data *in); diff --git a/tools/xenstore/include/xenstore_state.h b/tools/xenstored/xenstore_state.h similarity index 100% rename from tools/xenstore/include/xenstore_state.h rename to tools/xenstored/xenstore_state.h diff --git a/tools/xs-clients/.gitignore b/tools/xs-clients/.gitignore new file mode 100644 index 0000000000..6a0a7941d5 --- /dev/null +++ b/tools/xs-clients/.gitignore @@ -0,0 +1,10 @@ +/xenstore +/xenstore-chmod +/xenstore-control +/xenstore-exists +/xenstore-list +/xenstore-ls +/xenstore-read +/xenstore-rm +/xenstore-watch +/xenstore-write diff --git a/tools/xenstore/Makefile b/tools/xs-clients/Makefile similarity index 62% rename from tools/xenstore/Makefile rename to tools/xs-clients/Makefile index dc39b6cb31..4e78d365e0 100644 --- a/tools/xenstore/Makefile +++ b/tools/xs-clients/Makefile @@ -1,18 +1,11 @@ XEN_ROOT=$(CURDIR)/../.. include $(XEN_ROOT)/tools/Rules.mk -include Makefile.common - -xenstored: LDLIBS += $(LDLIBS_libxenevtchn) -xenstored: LDLIBS += $(LDLIBS_libxengnttab) -xenstored: LDLIBS += $(LDLIBS_libxenctrl) -xenstored: LDLIBS += -lrt -xenstored: LDLIBS += $(SOCKET_LIBS) - -ifeq ($(CONFIG_SYSTEMD),y) -$(XENSTORED_OBJS-y): CFLAGS += $(SYSTEMD_CFLAGS) -xenstored: LDLIBS += $(SYSTEMD_LIBS) -endif +CFLAGS += -include $(XEN_ROOT)/tools/config.h +CFLAGS += $(CFLAGS_libxenctrl) +CFLAGS += $(CFLAGS_libxenguest) +CFLAGS += $(CFLAGS_libxentoolcore) +CFLAGS += $(CFLAGS_libxenstore) xenstore: LDLIBS += $(LDLIBS_libxenstore) xenstore: LDLIBS += $(LDLIBS_libxentoolcore) @@ -28,9 +21,6 @@ CLIENTS := xenstore-exists xenstore-list xenstore-read xenstore-rm xenstore-chmo CLIENTS += xenstore-write xenstore-ls xenstore-watch TARGETS := xenstore $(CLIENTS) xenstore-control -ifeq ($(XENSTORE_XENSTORED),y) -TARGETS += xenstored -endif .PHONY: all all: $(TARGETS) @@ -38,9 +28,6 @@ all: $(TARGETS) .PHONY: clients clients: xenstore $(CLIENTS) xenstore-control -xenstored: $(XENSTORED_OBJS-y) - $(CC) $(LDFLAGS) $^ $(LDLIBS) -o $@ $(APPEND_LDFLAGS) - $(CLIENTS): xenstore ln -f xenstore $@ @@ -64,10 +51,6 @@ TAGS: .PHONY: install install: all $(INSTALL_DIR) $(DESTDIR)$(bindir) -ifeq ($(XENSTORE_XENSTORED),y) - $(INSTALL_DIR) $(DESTDIR)$(sbindir) - $(INSTALL_PROG) xenstored $(DESTDIR)$(sbindir) -endif $(INSTALL_PROG) xenstore-control $(DESTDIR)$(bindir) $(INSTALL_PROG) xenstore $(DESTDIR)$(bindir) set -e ; for c in $(CLIENTS) ; do \ @@ -79,23 +62,8 @@ uninstall: rm -f $(addprefix $(DESTDIR)$(bindir)/, $(CLIENTS)) rm -f $(DESTDIR)$(bindir)/xenstore rm -f $(DESTDIR)$(bindir)/xenstore-control -ifeq ($(XENSTORE_XENSTORED),y) - rm -f $(DESTDIR)$(sbindir)/xenstored -endif if [ -d $(DESTDIR)$(includedir)/xenstore-compat ]; then \ rmdir --ignore-fail-on-non-empty $(DESTDIR)$(includedir)/xenstore-compat; \ fi -.PHONY: clients-uninstall -clients-uninstall: - rm -f $(addprefix $(DESTDIR)$(bindir)/, $(CLIENTS)) - -.PHONY: clients-install -clients-install: clients - $(INSTALL_DIR) $(DESTDIR)$(bindir) - $(INSTALL_PROG) xenstore $(DESTDIR)$(bindir) - set -e ; for c in $(CLIENTS) ; do \ - ln -f $(DESTDIR)$(bindir)/xenstore $(DESTDIR)$(bindir)/$${c} ; \ - done - -include $(DEPS_INCLUDE) diff --git a/tools/xenstore/xenstore_client.c b/tools/xs-clients/xenstore_client.c similarity index 100% rename from tools/xenstore/xenstore_client.c rename to tools/xs-clients/xenstore_client.c diff --git a/tools/xenstore/xenstore_control.c b/tools/xs-clients/xenstore_control.c similarity index 100% rename from tools/xenstore/xenstore_control.c rename to tools/xs-clients/xenstore_control.c