[v3] Mini-OS: hide mini-os internal symbols

[PATCH v3 03/32] Mini-OS: add EXPORT_SYMBOL() instances to kernel.c

Posted by Juergen Gross 2 years, 2 months ago

Add the needed instances of EXPORT_SYMBOL() to kernel.c.

Signed-off-by: Juergen Gross <jgross@suse.com>
---
V3:
- new patch
---
 kernel.c | 5 +++++
 1 file changed, 5 insertions(+)

diff --git a/kernel.c b/kernel.c
index 1f97d8dd..0b444940 100644
--- a/kernel.c
+++ b/kernel.c
@@ -50,6 +50,9 @@
 #include <xen/features.h>
 #include <xen/version.h>
 
+void _start(void);
+EXPORT_SYMBOL(_start);
+
 uint8_t xen_features[XENFEAT_NR_SUBMAPS * 32];
 char cmdline[MAX_CMDLINE_SIZE];
 
@@ -181,6 +184,7 @@ void stop_kernel(void)
     /* Reset arch details */
     arch_fini();
 }
+EXPORT_SYMBOL(stop_kernel);
 
 /*
  * do_exit: This is called whenever an IRET fails in entry.S.
@@ -199,3 +203,4 @@ void do_exit(void)
         HYPERVISOR_sched_op(SCHEDOP_shutdown, &sched_shutdown);
     }
 }
+EXPORT_SYMBOL(do_exit);
-- 
2.35.3

Re: [PATCH v3 03/32] Mini-OS: add EXPORT_SYMBOL() instances to kernel.c

Posted by Samuel Thibault 2 years, 2 months ago

Juergen Gross, le lun. 27 nov. 2023 11:24:54 +0100, a ecrit:
> Add the needed instances of EXPORT_SYMBOL() to kernel.c.
> 
> Signed-off-by: Juergen Gross <jgross@suse.com>

Reviewed-by: Samuel Thibault <samuel.thibault@ens-lyon.org>

> ---
> V3:
> - new patch
> ---
>  kernel.c | 5 +++++
>  1 file changed, 5 insertions(+)
> 
> diff --git a/kernel.c b/kernel.c
> index 1f97d8dd..0b444940 100644
> --- a/kernel.c
> +++ b/kernel.c
> @@ -50,6 +50,9 @@
>  #include <xen/features.h>
>  #include <xen/version.h>
>  
> +void _start(void);
> +EXPORT_SYMBOL(_start);
> +
>  uint8_t xen_features[XENFEAT_NR_SUBMAPS * 32];
>  char cmdline[MAX_CMDLINE_SIZE];
>  
> @@ -181,6 +184,7 @@ void stop_kernel(void)
>      /* Reset arch details */
>      arch_fini();
>  }
> +EXPORT_SYMBOL(stop_kernel);
>  
>  /*
>   * do_exit: This is called whenever an IRET fails in entry.S.
> @@ -199,3 +203,4 @@ void do_exit(void)
>          HYPERVISOR_sched_op(SCHEDOP_shutdown, &sched_shutdown);
>      }
>  }
> +EXPORT_SYMBOL(do_exit);
> -- 
> 2.35.3
> 

-- 
Samuel
---
Pour une évaluation indépendante, transparente et rigoureuse !
Je soutiens la Commission d'Évaluation de l'Inria.