Drop ABI versioning for asm_switch_context(), as it could be easily done by renaming...
authorbernie <bernie@38d2e660-2303-0410-9eaa-f027e97ec537>
Sun, 10 Aug 2008 09:26:29 +0000 (09:26 +0000)
committerbernie <bernie@38d2e660-2303-0410-9eaa-f027e97ec537>
Sun, 10 Aug 2008 09:26:29 +0000 (09:26 +0000)
git-svn-id: https://src.develer.com/svnoss/bertos/trunk@1590 38d2e660-2303-0410-9eaa-f027e97ec537

bertos/cpu/arm/hw/switch_arm.S
bertos/cpu/avr/hw/switch_avr.S
bertos/cpu/dsp56k/hw/switch_dsp56k.c
bertos/emul/switch_i386.S
bertos/emul/switch_x86_64.s
bertos/kern/proc.c

index 531b18a54f6d9b1a229e9436309a5d460054b4ee..bde843cda734cd14ed7066fcb69dfa096dc2c4b2 100644 (file)
@@ -26,7 +26,7 @@
  * invalidate any other reasons why the executable file might be covered by
  * the GNU General Public License.
  *
- * Copyright 2007 Develer S.r.l. (http://www.develer.com/)
+ * Copyright 2007, 2008 Develer S.r.l. (http://www.develer.com/)
  *
  * -->
  *
  * \author Francesco Sacchi <batt@develer.com>
  */
 
-
-/*
- * NOTE: At each change of this function affecting proc.c
- * (i.e. arguments, data stored in the stack) bump up version
- * number in asm_switch_version().
- */
-
-/* void asm_switch_context(void **new_sp, void **save_sp) */
+/* void asm_switch_context(void **new_sp [r0], void **save_sp [r1]) */
 .globl asm_switch_context
 asm_switch_context:
        mrs     r2, cpsr               /* Save status. */
@@ -58,10 +51,3 @@ asm_switch_context:
        msr     cpsr, r2               /* restore flags reg. */
 
        mov     pc, lr
-
-
-/* int asm_switch_version(void) */
-.globl asm_switch_version
-asm_switch_version:
-       mov     r0, #1
-       mov     pc, lr
index b34e3f5470f99b1d60d6669c6a83e3ae9f14672a..50280292ad7a2e90f73aabc4b8f557a380071bc2 100644 (file)
@@ -1,28 +1,46 @@
 /*!
  * \file
  * <!--
- * Copyright 2004 Develer S.r.l. (http://www.develer.com/)
- * Copyright 1999,2000,2001 Bernie Innocenti <bernie@codewiz.org>
- * This file is part of DevLib - See devlib/README for information.
- * -->
+ * This file is part of BeRTOS.
  *
- * \brief AVR context switch
+ * Bertos is free software; you can redistribute it and/or modify
+ * it under the terms of the GNU General Public License as published by
+ * the Free Software Foundation; either version 2 of the License, or
+ * (at your option) any later version.
  *
- * \version $Id$
+ * This program is distributed in the hope that it will be useful,
+ * but WITHOUT ANY WARRANTY; without even the implied warranty of
+ * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
+ * GNU General Public License for more details.
+ *
+ * You should have received a copy of the GNU General Public License
+ * along with this program; if not, write to the Free Software
+ * Foundation, Inc., 51 Franklin St, Fifth Floor, Boston, MA  02110-1301  USA
+ *
+ * As a special exception, you may use this file as part of a free software
+ * library without restriction.  Specifically, if other files instantiate
+ * templates or use macros or inline functions from this file, or you compile
+ * this file and link it with other files to produce an executable, this
+ * file does not by itself cause the resulting executable to be covered by
+ * the GNU General Public License.  This exception does not however
+ * invalidate any other reasons why the executable file might be covered by
+ * the GNU General Public License.
  *
+ * Copyright 2004, 2008 Develer S.r.l. (http://www.develer.com/)
+ * Copyright 1999, 2000, 2001 Bernie Innocenti <bernie@codewiz.org>
+ * -->
+ *
+ * \version $Id$
  * \author Bernie Innocenti <bernie@codewiz.org>
  * \author Stefano Fedrigo <aleph@develer.com>
+ *
+ * \brief AVR context switch
+ *
  */
 
 #include <avr/io.h>
 
-/*
- * NOTE: At each change of this function affecting proc.c
- * (i.e. arguments, data stored in the stack) bump up version
- * number in asm_switch_version().
- */
-
-/* void asm_switch_context(void **new_sp, void **save_sp) */
+/* void asm_switch_context(void **new_sp [r24:r25], void **save_sp [r22:r23]) */
 .globl asm_switch_context
 asm_switch_context:
 
@@ -66,8 +84,6 @@ asm_switch_context:
 ;      push    r30     ;caller-save
 ;      push    r31     ;caller-save
 
-;      First parameter (new_sp) is in r24:r25, second (save_sp) in r22:r23
-
        in      r18,SPL-__SFR_OFFSET    ; r18:r19 = SP
        in      r19,SPH-__SFR_OFFSET
        movw    r26,r22                 ; X = save_sp
@@ -124,12 +140,3 @@ asm_switch_context:
        out     SREG-__SFR_OFFSET,r0
 
        ret
-
-
-/* int asm_switch_version(void) */
-.globl asm_switch_version
-asm_switch_version:
-       ldi r24,lo8(1)
-       ldi r25,hi8(1)
-       ret
-
index 41a12056679482af0324e3dbc5fe0fa63361020f..116483e24581cb73ea03ea1cf9e4b9b9c24c1539 100644 (file)
  * invalidate any other reasons why the executable file might be covered by
  * the GNU General Public License.
  *
- * Copyright 2004 Develer S.r.l. (http://www.develer.com/)
- *
+ * Copyright 2004, 2008 Develer S.r.l. (http://www.develer.com/)
  * -->
  *
  * \version $Id$
- *
  * \author Giovanni Bajo <rasky@develer.com>
  *
  * \brief DSP5680x task switching support
  */
 
-/*#*
- *#* $Log$
- *#* Revision 1.7  2006/07/19 12:56:27  bernie
- *#* Convert to new Doxygen style.
- *#*
- *#* Revision 1.6  2006/02/24 01:17:05  bernie
- *#* Update for new emulator.
- *#*
- *#* Revision 1.5  2005/11/04 16:20:02  bernie
- *#* Fix reference to README.devlib in header.
- *#*
- *#* Revision 1.4  2004/08/25 14:12:09  rasky
- *#* Aggiornato il comment block dei log RCS
- *#*
- *#* Revision 1.3  2004/07/30 14:24:16  rasky
- *#* Task switching con salvataggio perfetto stato di interrupt (SR)
- *#* Kernel monitor per dump informazioni su stack dei processi
- *#*
- *#* Revision 1.2  2004/06/03 11:27:09  bernie
- *#* Add dual-license information.
- *#*
- *#* Revision 1.1  2004/05/23 17:27:00  bernie
- *#* Import kern/ subdirectory.
- *#*
- *#*/
-
-void asm_switch_context(void ** new_sp/*R2*/, void ** save_sp/*R3*/);
+void asm_switch_context(void ** new_sp /* R2 */, void ** save_sp /* R3 */);
 asm void asm_switch_context(void ** new_sp, void ** save_sp)
 {
        lea   (SP)+
@@ -127,9 +99,3 @@ asm void asm_switch_context(void ** new_sp, void ** save_sp)
        ; it, so that interrupt status is preserved across the tasks.
        rti
 }
-
-int asm_switch_version(void);
-int asm_switch_version(void)
-{
-       return 1;
-}
index 0b2f0937e07c2fed3251a32fc4d5c723aa7a249e..5032b31c2556d5fcb9bb60b0a7af8ad2d6a2ec59 100644 (file)
@@ -26,9 +26,8 @@
  * invalidate any other reasons why the executable file might be covered by
  * the GNU General Public License.
  *
- * Copyright 2004 Develer S.r.l. (http://www.develer.com/)
+ * Copyright 2004, 2008 Develer S.r.l. (http://www.develer.com/)
  * Copyright 1999, 2000, 2001 Bernie Innocenti <bernie@codewiz.org>
- * This file is part of DevLib - See devlib/README for information.
  * -->
  *
  * \version $Id$
  * \brief i386 context switch
  */
 
-
-
 #ifdef __APPLE__
-    //This workaround is necessary to compile under OS X assembler.
+    // This workaround is necessary to compile under OS X assembler.
     #define SWITCH_CONTEXT _asm_switch_context
-    #define SWITCH_VERSION _asm_switch_version
 #else
     #define SWITCH_CONTEXT asm_switch_context
-    #define SWITCH_VERSION asm_switch_version
 #endif
 
-/* void asm_switch_context(void ** new_sp, void ** save_sp) */
+/* void asm_switch_context(void ** new_sp [sp+4], void ** save_sp [sp+8]) */
 .globl SWITCH_CONTEXT
 SWITCH_CONTEXT:
        pushl   %eax
@@ -70,10 +65,3 @@ SWITCH_CONTEXT:
        popl    %ebx
        popl    %eax
        ret
-
-/* int asm_switch_version(void) */
-.globl SWITCH_VERSION
-SWITCH_VERSION:
-       mov     $1,%eax
-       ret
-
index 61859834cb95f50dfa41aa219fa6c2c5b584544f..1632c816d4b01512c7fb554043aed9d7b8a259aa 100644 (file)
@@ -26,9 +26,8 @@
  * invalidate any other reasons why the executable file might be covered by
  * the GNU General Public License.
  *
- * Copyright 2004 Develer S.r.l. (http://www.develer.com/)
+ * Copyright 2004, 2008 Develer S.r.l. (http://www.develer.com/)
  * Copyright 1999, 2000, 2001 Bernie Innocenti <bernie@codewiz.org>
- * This file is part of DevLib - See devlib/README for information.
  * -->
  *
  * \version $Id$
  * \brief i386 context switch
  */
 
-/* I know it's ugly... */
-/* .intel_syntax */
-
-/* void AsmSwitchContext(void **new_sp, void **save_sp) */
-/*                       %rdi           %rsi            */
+/* void asm_switch_context(void **new_sp [%rdi], void **save_sp [%rsi]) */
 .globl asm_switch_context
 asm_switch_context:
        pushq   %rax
@@ -61,10 +56,3 @@ asm_switch_context:
        popq    %rbx
        popq    %rax
        ret
-
-/* int asm_switch_version(void) */
-.globl asm_switch_version
-asm_switch_version:
-       mov     $1,%rax
-       ret
-
index 32e3afe8a2f50b317af65291fa9f0d86fcc3f7b2..37b36e07d664c9509f9d920c51357b6b5c027f75 100644 (file)
@@ -63,7 +63,6 @@
  * \note This function *MUST* preserve also the status of the interrupts.
  */
 EXTERN_C void asm_switch_context(cpustack_t **new_sp, cpustack_t **save_sp);
-EXTERN_C int asm_switch_version(void);
 
 /*
  * The scheduer tracks ready and waiting processes
@@ -129,8 +128,6 @@ void proc_init(void)
        proc_init_struct(&MainProcess);
        CurrentProcess = &MainProcess;
 
-       /* Make sure the assembly routine is up-to-date with us */
-       ASSERT(asm_switch_version() == 1);
        MOD_INIT(proc);
 }