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

[Kevin Wolf <kevin@xxxxxxxxxx>]

On Sun, Dec 30, 2012 at 08:54:03PM +0100, MNemo wrote:
> 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?

Jap, würde ich bevorzugen. Sich darauf verlassen, was die aufgerufene
Funktion genau mit den Parametern macht, ist jedenfalls gefährlich.

Kevin