Re: [tyndur-devel] [PATCH 1/4] kernel2: vm86 ist i386-spezifisch

[Kevin Wolf <kevin@xxxxxxxxxx>]

On Sun, Dec 30, 2012 at 08:01:31PM +0100, Andreas Freimuth wrote:
>  * syscalls/vm86.c: nach arch/i386 verschoben
>  * tasks.h: #ifdef für vm86-Flag in pm_thread_t
> 
> Signed-off-by: Andreas Freimuth <m.nemo@xxxxxxx>
> ---
>  src/kernel2/include/tasks.h                     | 2 ++
>  src/kernel2/src/{ => arch/i386}/syscalls/vm86.c | 0
>  2 files changed, 2 insertions(+)
>  rename src/kernel2/src/{ => arch/i386}/syscalls/vm86.c (100%)
> 
> diff --git a/src/kernel2/include/tasks.h b/src/kernel2/include/tasks.h
> index 1b719d7..1347fd5 100644
> --- a/src/kernel2/include/tasks.h
> +++ b/src/kernel2/include/tasks.h
> @@ -138,8 +138,10 @@ typedef struct {
>      /// Wird gesperrt, wenn aenderungen am Thread vorgenommen werden
>      lock_t lock;
>  
> +#if CONFIG_ARCH == ARCH_I386
>      /// Gesetzt, wenn der Thread ein VM86-Thread mit allem Drum und Dran ist
>      bool vm86;
> +#endif
>  } pm_thread_t;
>  
>  typedef void (*pm_process_destroy_handler)(pm_process_t* process, void* prv);
> diff --git a/src/kernel2/src/syscalls/vm86.c b/src/kernel2/src/arch/i386/syscalls/vm86.c
> similarity index 100%
> rename from src/kernel2/src/syscalls/vm86.c
> rename to src/kernel2/src/arch/i386/syscalls/vm86.c

Ich gebe zu, dass mein Ansatz von wegen "wenn's noch bootet, ist es
okay" ein bisschen zu optimistisch war, weil es ja immer noch nicht
baut. Aber bisher ging es erst beim Linken schief, wäre schön, wenn wir
den Zustand nicht verschlechtern. Konkret:

thread.c: In function ‘pm_thread_create’:
thread.c:71:11: error: ‘pm_thread_t’ has no member named ‘vm86’

Ich schätze, du hast noch ein paar #ifdefs vergessen, damit es auch nach
diesem Patch wieder kompiliert.

Kevin