include/xen-x86: Remove os.h on x86

Remove os.h from the project as it was unused.
mm_pv.h included os.h but did not use anything from
it. The rest of the changes have been made in sources
which need the extern definition for the shared info
page.

Signed-off-by: Radu Nicolau <radunicolau102@gmail.com>
Reviewed-by: Costin Lupu <costin.lupu@cs.pub.ro>

diff --git a/plat/xen/gnttab.c b/plat/xen/gnttab.c
index ddfff992f130921c7c9db6093cd03953185240a1..edc3341e1376fab07252947a86ea72f8cd3f63f3 100644 (file)
--- a/plat/xen/gnttab.c
+++ b/plat/xen/gnttab.c
@@ -27,6 +27,7 @@
 #include <common/gnttab.h>
 #include <xen-x86/mm.h>
 
+#include <xen-x86/hypercall.h>
 
 /* NR_GRANT_FRAMES must be less than or equal to that configured in Xen */
 #define NR_GRANT_FRAMES         4

diff --git a/plat/xen/include/xen-x86/mm_pv.h b/plat/xen/include/xen-x86/mm_pv.h
index 21f91695d94cb3ea21abd3d5e953c5f53915ad4f..f78e62f29461e925b4655730dfc7199e2750ee83 100644 (file)
--- a/plat/xen/include/xen-x86/mm_pv.h
+++ b/plat/xen/include/xen-x86/mm_pv.h
@@ -24,7 +24,6 @@
 #ifndef _MM_PV_H
 #define _MM_PV_H
 
-#include <xen-x86/os.h>
 #include <xen-x86/setup.h>
 
 #ifdef __x86_64__

diff --git a/plat/xen/include/xen-x86/os.h b/plat/xen/include/xen-x86/os.h
deleted file mode 100644 (file)
index 308d91a..0000000
--- a/plat/xen/include/xen-x86/os.h
+++ /dev/null
@@ -1,90 +0,0 @@
-/* SPDX-License-Identifier: BSD-2-Clause */
-/*
- * Redistribution and use in source and binary forms, with or without
- * modification, are permitted provided that the following conditions
- * are met:
- * 1. Redistributions of source code must retain the above copyright
- *    notice, this list of conditions and the following disclaimer.
- * 2. 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.
- *
- * THIS SOFTWARE IS PROVIDED BY AUTHOR 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 AUTHOR 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.
- */
-/* Taken from Mini-OS */
-/******************************************************************************
- * os.h
- *
- * random collection of macros and definition
- */
-
-#ifndef _OS_H_
-#define _OS_H_
-
-#include <xen-x86/smp.h>
-
-#include <stdint.h>
-#include <stddef.h>
-#include <limits.h>
-#include <uk/arch/types.h>
-#include <uk/arch/limits.h>
-#include <uk/arch/time.h>
-#include <xen/xen.h>
-
-#ifndef __ASSEMBLY__
-
-typedef __s64 quad_t;
-typedef __u64 u_quad_t;
-
-typedef __sptr intptr_t;
-typedef __uptr uintptr_t;
-typedef __sptr ptrdiff_t;
-
-typedef unsigned char u_char;
-typedef unsigned int u_int;
-typedef unsigned long u_long;
-
-#include <xen-x86/hypercall.h>
-
-#include <xen/event_channel.h>
-#include <xen/xsm/flask_op.h>
-#endif
-
-#include <x86/cpu_defs.h>
-
-
-#define LOCK_PREFIX ""
-#define ADDR (*(volatile long *)addr)
-
-/* Everything below this point is not included by assembler (.S) files. */
-#ifndef __ASSEMBLY__
-
-extern shared_info_t *HYPERVISOR_shared_info;
-
-#include <xen-x86/irq.h>
-
-
-/*
- * Make sure gcc doesn't try to be clever and move things around
- * on us. We need to use _exactly_ the address the user gave us,
- * not some alias that contains the same information.
- */
-typedef struct {
-       volatile int counter;
-} atomic_t;
-
-
-void block_domain(__snsec until);
-
-#endif /* not assembly */
-#endif /* _OS_H_ */

diff --git a/plat/xen/include/xen-x86/setup.h b/plat/xen/include/xen-x86/setup.h
index 6eaa4f16aad1578c7e5d4b9ede3ee6b2ab140fc3..7a57190143f00b4006d867ec653fe0a3551e3608 100644 (file)
--- a/plat/xen/include/xen-x86/setup.h
+++ b/plat/xen/include/xen-x86/setup.h
@@ -46,7 +46,6 @@ extern unsigned int _libxenplat_mrd_num;
 extern struct ukplat_memregion_desc _libxenplat_mrd[];
 extern char _libxenplat_bootstack[];
 
-/* Note: HYPERVISOR_shared_info is defined in os.h */
 extern start_info_t *HYPERVISOR_start_info;
 
 #endif /* __SETUP_H__ */

diff --git a/plat/xen/lcpu.c b/plat/xen/lcpu.c
index f0650da252b6f8ee1a25589836b21a603d445056..d7158dfa91c299f7c212bb02110843c9a2563c1d 100644 (file)
--- a/plat/xen/lcpu.c
+++ b/plat/xen/lcpu.c
@@ -34,7 +34,6 @@
 
 #include <stdint.h>
 #if defined(__X86_32__) || defined(__x86_64__)
-#include <xen-x86/os.h>
 #include <xen-x86/irq.h>
 #elif (defined __ARM_32__) || (defined __ARM_64__)
 #include <xen-arm/os.h>

diff --git a/plat/xen/x86/mm.c b/plat/xen/x86/mm.c
index b7b080efa606e7248ccedc6de1720bdcfd009edf..b89384f6858ad27e0682877483df2fe2e62733e2 100644 (file)
--- a/plat/xen/x86/mm.c
+++ b/plat/xen/x86/mm.c
@@ -39,6 +39,7 @@
 #include <errno.h>
 #include <uk/alloc.h>
 #include <uk/plat/config.h>
+#include <common/hypervisor.h>
 #include <xen-x86/mm.h>
 #include <xen/memory.h>
 #include <uk/print.h>

diff --git a/plat/xen/xenbus/xs_comms.c b/plat/xen/xenbus/xs_comms.c
index 274e44b5be7292a5e36bacd8643d4712a0ebf4f9..0589d48b768db011e0da3e61819f3506d7a7dec1 100644 (file)
--- a/plat/xen/xenbus/xs_comms.c
+++ b/plat/xen/xenbus/xs_comms.c
@@ -46,6 +46,7 @@
 #include <uk/wait.h>
 #include <uk/arch/spinlock.h>
 #include <common/events.h>
+#include <common/hypervisor.h>
 #include <xen-x86/mm.h>
 #include <xen-x86/setup.h>
 #include <xenbus/client.h>