|
From: | Almudena Garcia |
Subject: | Re: Enable SMP support |
Date: | Thu, 28 Jun 2018 01:12:07 +0200 |
Yes, the type for processor is needed, see the patch I have now commited.
> /* More-specific code must define cpu_number() and CPU_NUMBER. */
> #define CX(addr, reg) addr(,reg,4)
> +#define CPU_NUMBER(reg) /*XXX: we think this is not right, but we wanted to bypass the compiling error. Please review. */
It will have to contain the assembler code to get the CPU number indeed.
Hello,
Almudena Garcia, le mer. 27 juin 2018 22:37:55 +0200, a ecrit:
> ../kern/sched_prim.h:188:1: warning: parameter names (without types) in function declaration
> extern void cause_ast_check(processor); /*new*/
Yes, the type for processor is needed, see the patch I have now commited.
> /* More-specific code must define cpu_number() and CPU_NUMBER. */
> #define CX(addr, reg) addr(,reg,4)
> +#define CPU_NUMBER(reg) /*XXX: we think this is not right, but we wanted to bypass the compiling error. Please review. */
It will have to contain the assembler code to get the CPU number indeed.
> diff -Naur '--exclude=.git' gnumach/kern/sched_prim.h gnumach-dev/kern/sched_prim.h
> --- gnumach/kern/sched_prim.h 2018-06-27 16:52:47.000000000 -0400
> +++ gnumach-dev/kern/sched_prim.h 2018-06-27 15:36:46.000000000 -0400
> +extern void cause_ast_check(processor); /*new*/
> diff -Naur '--exclude=.git' gnumach/kern/thread.h gnumach-dev/kern/thread.h
> --- gnumach/kern/thread.h 2018-06-27 16:52:47.000000000 -0400
> +++ gnumach-dev/kern/thread.h 2018-06-27 15:36:46.000000000 -0400
> +extern void cause_ast_check(processor); /*new*/
There is no need to declare it twice. I have just declared it in a
proper place instead.
Samuel
[Prev in Thread] | Current Thread | [Next in Thread] |