Re: [tyndur-devel] [PATCH 2/4] smp: init ist das neue smp_entry

[Kevin Wolf <kevin@xxxxxxxxxx>]

On Sun, Dec 30, 2012 at 08:01:32PM +0100, Andreas Freimuth wrote:
>  ! amd64/smp/trampoline.S smp_entry wurde scheinbar durch init ersetzt
> 
> Signed-off-by: Andreas Freimuth <m.nemo@xxxxxxx>
> ---
>  src/kernel2/src/arch/amd64/smp/trampoline.S | 9 ++++++---
>  1 file changed, 6 insertions(+), 3 deletions(-)
> 
> diff --git a/src/kernel2/src/arch/amd64/smp/trampoline.S b/src/kernel2/src/arch/amd64/smp/trampoline.S
> index 13b27c3..0edb775 100644
> --- a/src/kernel2/src/arch/amd64/smp/trampoline.S
> +++ b/src/kernel2/src/arch/amd64/smp/trampoline.S
> @@ -105,9 +105,12 @@ smp_entry_longmode:
>      
>      // Den Lock wieder aufheben
>      movl $0, smp_entry_lock
> -    
> -    .extern smp_entry
> -    call smp_entry
> +
> +    // Das ist nicht der Bootstrap Processor bzw. Core
> +    xorl %edx, %edx

Fehlen da nicht noch ein paar Parameter? Wenn ich es richtig sehe,
steht in %rdi und %rsi einfach irgendwas.

void init(int multiboot_magic, struct multiboot_info *boot_info, bool bsp)


> +
> +    .extern init
> +    call init
>  
>      // Geschieht nie
>      hlt

Kevin