[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

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



On 30.12.2012 20:31, Kevin Wolf wrote:
> 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.
Ja. Das ist richtig. Es ist vielleicht nicht die sauberste Lösung, aber
die Parameter werden nur vom Bootstrap gebraucht.

Meinst du die sollte man auch auf Null setzen?

> 
> void init(int multiboot_magic, struct multiboot_info *boot_info, bool bsp)
> 
> 
>> +
>> +    .extern init
>> +    call init
>>  
>>      // Geschieht nie
>>      hlt
> 
> Kevin
> _______________________________________________
> tyndur-devel mailing list
> tyndur-devel@xxxxxxxxxx
> http://list.tyndur.org/mailman/listinfo/tyndur-devel
>