Kernel/x86 - Distinguished tracing of unaligned memcpys from aligned

[tpg/acess2.git] / Kernel / arch / x86 / proc.c

diff --git a/Kernel/arch/x86/proc.c b/Kernel/arch/x86/proc.c
index c0b748f..14eaf3a 100644 (file)
--- a/Kernel/arch/x86/proc.c
+++ b/Kernel/arch/x86/proc.c
@@ -54,10 +54,10 @@ extern int  giNextTID;
 extern tThread gThreadZero;
 extern void    Isr8(void);     // Double Fault
 extern void    Proc_ReturnToUser(tVAddr Handler, Uint Argument, tVAddr KernelStack);
-extern void    scheduler_return;       // Return address in SchedulerBase
-extern void    IRQCommon;      // Common IRQ handler code
-extern void    IRQCommon_handled;      // IRQCommon call return location
-extern void    GetEIP_Sched_ret;       // GetEIP call return location
+extern char    scheduler_return[];     // Return address in SchedulerBase
+extern char    IRQCommon[];    // Common IRQ handler code
+extern char    IRQCommon_handled[];    // IRQCommon call return location
+extern char    GetEIP_Sched_ret[];     // GetEIP call return location
 extern void    Threads_AddToDelete(tThread *Thread);
 extern void    SwitchTasks(Uint NewSP, Uint *OldSP, Uint NewIP, Uint *OldIO, Uint CR3);
 
@@ -71,11 +71,11 @@ void        Proc_IdleThread(void *Ptr);
 //void Proc_Start(void);
 //tThread      *Proc_GetCurThread(void);
 void   Proc_ChangeStack(void);
- int   Proc_NewKThread(void (*Fcn)(void*), void *Data);
+// int Proc_NewKThread(void (*Fcn)(void*), void *Data);
 // int Proc_Clone(Uint *Err, Uint Flags);
 Uint   Proc_MakeUserStack(void);
 //void Proc_StartUser(Uint Entrypoint, Uint *Bases, int ArgC, char **ArgV, char **EnvP, int DataSize);
-void   Proc_StartProcess(Uint16 SS, Uint Stack, Uint Flags, Uint16 CS, Uint IP);
+void   Proc_StartProcess(Uint16 SS, Uint Stack, Uint Flags, Uint16 CS, Uint IP) NORETURN;
  int   Proc_Demote(Uint *Err, int Dest, tRegs *Regs);
 //void Proc_CallFaultHandler(tThread *Thread);
 //void Proc_DumpThreadCPUState(tThread *Thread);
@@ -557,7 +557,7 @@ int Proc_NewKThread(void (*Fcn)(void*), void *Data)
        tThread *newThread, *cur;
        
        cur = Proc_GetCurThread();
-       newThread = Threads_CloneTCB(NULL, 0);
+       newThread = Threads_CloneTCB(0);
        if(!newThread)  return -1;
        
        // Set CR3
@@ -604,7 +604,7 @@ int Proc_Clone(Uint Flags)
        }
        
        // New thread
-       newThread = Threads_CloneTCB(NULL, Flags);
+       newThread = Threads_CloneTCB(Flags);
        if(!newThread)  return -1;
 
        newThread->KernelStack = cur->KernelStack;
@@ -695,52 +695,37 @@ Uint Proc_MakeUserStack(void)
        return base + USER_STACK_SZ;
 }
 
-/**
- * \fn void Proc_StartUser(Uint Entrypoint, Uint *Bases, int ArgC, char **ArgV, char **EnvP, int DataSize)
- * \brief Starts a user task
- */
-void Proc_StartUser(Uint Entrypoint, Uint *Bases, int ArgC, char **ArgV, char **EnvP, int DataSize)
+void Proc_StartUser(Uint Entrypoint, Uint Base, int ArgC, char **ArgV, int DataSize)
 {
-       Uint    *stack = (void*)Proc_MakeUserStack();
+       Uint    *stack;
         int    i;
-       Uint    delta;
+       char    **envp = NULL;
        Uint16  ss, cs;
        
-       //Log("stack = %p", stack);
-       
-       // Copy Arguments
+       // Copy data to the user stack and free original buffer
+       stack = (void*)Proc_MakeUserStack();
        stack -= DataSize/sizeof(*stack);
        memcpy( stack, ArgV, DataSize );
+       free(ArgV);
        
-       //Log("stack = %p", stack);
-       
+       // Adjust Arguments and environment
        if( DataSize )
        {
-               // Adjust Arguments and environment
-               delta = (Uint)stack - (Uint)ArgV;
+               Uint delta = (Uint)stack - (Uint)ArgV;
                ArgV = (char**)stack;
-               for( i = 0; ArgV[i]; i++ )
-                       ArgV[i] += delta;
-               i ++;
-               
-               // Do we care about EnvP?
-               if( EnvP ) {
-                       EnvP = &ArgV[i];
-                       for( i = 0; EnvP[i]; i++ )
-                               EnvP[i] += delta;
-               }
+               for( i = 0; ArgV[i]; i++ )      ArgV[i] += delta;
+               envp = &ArgV[i+1];
+               for( i = 0; envp[i]; i++ )      envp[i] += delta;
        }
        
        // User Mode Segments
        ss = 0x23;      cs = 0x1B;
        
        // Arguments
-       *--stack = (Uint)EnvP;
+       *--stack = (Uint)envp;
        *--stack = (Uint)ArgV;
        *--stack = (Uint)ArgC;
-       while(*Bases)
-               *--stack = *Bases++;
-       *--stack = 0;   // Return Address
+       *--stack = Base;
        
        Proc_StartProcess(ss, (Uint)stack, 0x202, cs, Entrypoint);
 }