SMP: Clean CPU0 D-Cache before starting new CPU; Invalidate D-Cache when new CPU started.

arch/arm/src/armv7-a/arm_addrenv.c

@@ -141,10 +141,6 @@
 #  error CONFIG_ARCH_HEAP_VBASE not aligned to section boundary
 #endif
 
-/****************************************************************************
- * Private Data
- ****************************************************************************/
-
 /****************************************************************************
  * Private Functions
  ****************************************************************************/

arch/arm/src/armv7-a/arm_addrenv_kstack.c

@@ -121,18 +121,6 @@
 
 #if defined(CONFIG_ARCH_ADDRENV) && defined(CONFIG_ARCH_KERNEL_STACK)
 
-/****************************************************************************
- * Pre-processor Definitions
- ****************************************************************************/
-
-/****************************************************************************
- * Private Data
- ****************************************************************************/
-
-/****************************************************************************
- * Private Functions
- ****************************************************************************/
-
 /****************************************************************************
  * Public Functions
  ****************************************************************************/

arch/arm/src/armv7-a/arm_addrenv_shm.c

@@ -55,18 +55,6 @@
 
 #if defined(CONFIG_BUILD_KERNEL) && defined(CONFIG_MM_SHM)
 
-/****************************************************************************
- * Pre-processor Definitions
- ****************************************************************************/
-
-/****************************************************************************
- * Private Data
- ****************************************************************************/
-
-/****************************************************************************
- * Private Functions
- ****************************************************************************/
-
 /****************************************************************************
  * Public Functions
  ****************************************************************************/

arch/arm/src/armv7-a/arm_addrenv_ustack.c

@@ -116,14 +116,6 @@
 #  error CONFIG_ARCH_STACK_VBASE not aligned to section boundary
 #endif
 
-/****************************************************************************
- * Private Data
- ****************************************************************************/
-
-/****************************************************************************
- * Private Functions
- ****************************************************************************/
-
 /****************************************************************************
  * Public Functions
  ****************************************************************************/

arch/arm/src/armv7-a/arm_addrenv_utils.c

@@ -53,18 +53,6 @@
 
 #ifdef CONFIG_ARCH_ADDRENV
 
-/****************************************************************************
- * Pre-processor Definitions
- ****************************************************************************/
-
-/****************************************************************************
- * Private Data
- ****************************************************************************/
-
-/****************************************************************************
- * Private Functions
- ****************************************************************************/
-
 /****************************************************************************
  * Public Functions
  ****************************************************************************/

arch/arm/src/armv7-a/arm_allocpage.c

@@ -54,10 +54,6 @@
 #include "pg_macros.h"
 #include "up_internal.h"
 
-/****************************************************************************
- * Pre-processor Definitions
- ****************************************************************************/
-
 /****************************************************************************
  * Private Types
  ****************************************************************************/
@@ -108,10 +104,6 @@ static L1ndx_t g_ptemap[CONFIG_PAGING_NPPAGED];
 
 static bool g_pgwrap;
 
-/****************************************************************************
- * Private Functions
- ****************************************************************************/
-
 /****************************************************************************
  * Public Functions
  ****************************************************************************/

arch/arm/src/armv7-a/arm_checkmapping.c

@@ -51,18 +51,6 @@
 
 #ifdef CONFIG_PAGING
 
-/****************************************************************************
- * Pre-processor Definitions
- ****************************************************************************/
-
-/****************************************************************************
- * Private Data
- ****************************************************************************/
-
-/****************************************************************************
- * Private Functions
- ****************************************************************************/
-
 /****************************************************************************
  * Public Functions
  ****************************************************************************/

arch/arm/src/armv7-a/arm_coherent_dcache.c

@@ -46,18 +46,6 @@
 
 #include <nuttx/arch.h>
 
-/****************************************************************************
- * Pre-processor Definitions
- ****************************************************************************/
-
-/****************************************************************************
- * Private Data
- ****************************************************************************/
-
-/****************************************************************************
- * Private Functions
- ****************************************************************************/
-
 /****************************************************************************
  * Public Functions
  ****************************************************************************/

arch/arm/src/armv7-a/arm_copyfullstate.c

@@ -44,18 +44,6 @@
 
 #include "up_internal.h"
 
-/****************************************************************************
- * Pre-processor Definitions
- ****************************************************************************/
-
-/****************************************************************************
- * Private Data
- ****************************************************************************/
-
-/****************************************************************************
- * Private Functions
- ****************************************************************************/
-
 /****************************************************************************
  * Public Functions
  ****************************************************************************/

arch/arm/src/armv7-a/arm_cpustart.c

@@ -46,6 +46,7 @@
 
 #include "up_internal.h"
 #include "gic.h"
+#include "cp15_cacheops.h"
 #include "sched/sched.h"
 
 #ifdef CONFIG_SMP
@@ -74,6 +75,10 @@ int arm_start_handler(int irq, FAR void *context)
 {
   FAR struct tcb_s *tcb = this_task();
 
+  /* Invalidate CPUn L1 so that is will be reloaded from coherent L2. */
+
+  cp15_invalidate_dcache_all();
+
   /* Reset scheduler parameters */
 
   sched_resume_scheduler(tcb);
@@ -118,6 +123,10 @@ int up_cpu_start(int cpu)
 {
   DEBUGASSERT(cpu >= 0 && cpu < CONFIG_SMP_NCPUS && cpu != this_cpu());
 
+  /* Make the content of CPU0 L1 cache has been written to coherent L2 */
+
+  cp15_clean_dcache(CONFIG_RAM_START, CONFIG_RAM_END - 1);
+
   /* Execute SGI1 */
 
   return arm_cpu_sgi(GIC_IRQ_SGI1, (1 << cpu));

arch/arm/src/armv7-a/arm_doirq.c

@@ -52,22 +52,6 @@
 
 #include "group/group.h"
 
-/****************************************************************************
- * Pre-processor Definitions
- ****************************************************************************/
-
-/****************************************************************************
- * Public Data
- ****************************************************************************/
-
-/****************************************************************************
- * Private Data
- ****************************************************************************/
-
-/****************************************************************************
- * Private Functions
- ****************************************************************************/
-
 /****************************************************************************
  * Public Functions
  ****************************************************************************/

arch/arm/src/armv7-a/arm_elf.c

@@ -48,18 +48,6 @@
 #include <nuttx/arch.h>
 #include <nuttx/binfmt/elf.h>
 
-/****************************************************************************
- * Pre-processor Definitions
- ****************************************************************************/
-
-/****************************************************************************
- * Private Data
- ****************************************************************************/
-
-/****************************************************************************
- * Private Functions
- ****************************************************************************/
-
 /****************************************************************************
  * Public Functions
  ****************************************************************************/

arch/arm/src/armv7-a/arm_fullcontextrestore.S

@@ -44,28 +44,12 @@
 
 	.file	"arm_fullcontextrestore.S"
 
-/****************************************************************************
- * Pre-processor Definitions
- ****************************************************************************/
-
 /****************************************************************************
  * Public Symbols
  ****************************************************************************/
 
 	.globl	up_fullcontextrestore
 
-/****************************************************************************
- * Private Data
- ****************************************************************************/
-
-/****************************************************************************
- * Public Data
- ****************************************************************************/
-
-/****************************************************************************
- * Private Functions
- ****************************************************************************/
-
 /****************************************************************************
  * Public Functions
  ****************************************************************************/

arch/arm/src/armv7-a/arm_gicv2.c

@@ -166,12 +166,12 @@ void arm_gic_initialize(void)
 
   /* Registers with 8-bits per interrupt */
 
-  putreg32(0x80808080, GIC_ICDIPR(0));         /* SGI[3:0] priority */
+  putreg32(0x80808080, GIC_ICDIPR(0));      /* SGI[3:0] priority */
-  putreg32(0x80808080, GIC_ICDIPR(4));         /* SGI[4:7] priority */
+  putreg32(0x80808080, GIC_ICDIPR(4));      /* SGI[4:7] priority */
-  putreg32(0x80808080, GIC_ICDIPR(8));         /* SGI[8:11] priority */
+  putreg32(0x80808080, GIC_ICDIPR(8));      /* SGI[8:11] priority */
-  putreg32(0x80808080, GIC_ICDIPR(12));        /* SGI[12:15] priority */
+  putreg32(0x80808080, GIC_ICDIPR(12));     /* SGI[12:15] priority */
-  putreg32(0x80000000, GIC_ICDIPR(24));        /* PPI[0] priority */
+  putreg32(0x80000000, GIC_ICDIPR(24));     /* PPI[0] priority */
-  putreg32(0x80808080, GIC_ICDIPR(28));        /* PPI[1:4] priority */
+  putreg32(0x80808080, GIC_ICDIPR(28));     /* PPI[1:4] priority */
 
   /* Set the binary point register.
    *
@@ -527,7 +527,7 @@ int up_prioritize_irq(int irq, int priority)
  * Name: arm_gic_irq_trigger
  *
  * Description:
- *   Set the trigger type for the specificd IRQ source and the current CPU.
+ *   Set the trigger type for the specified IRQ source and the current CPU.
  *
  *   Since this API is not supported on all architectures, it should be
  *   avoided in common implementations where possible.

arch/arm/src/armv7-a/arm_initialstate.c

@@ -48,18 +48,6 @@
 #include "up_internal.h"
 #include "up_arch.h"
 
-/****************************************************************************
- * Pre-processor Definitions
- ****************************************************************************/
-
-/****************************************************************************
- * Private Data
- ****************************************************************************/
-
-/****************************************************************************
- * Private Functions
- ****************************************************************************/
-
 /****************************************************************************
  * Public Functions
  ****************************************************************************/

arch/arm/src/armv7-a/arm_mmu.c

@@ -44,18 +44,6 @@
 #include "cache.h"
 #include "mmu.h"
 
-/****************************************************************************
- * Pre-processor Definitions
- ****************************************************************************/
-
-/****************************************************************************
- * Private Data
- ****************************************************************************/
-
-/****************************************************************************
- * Private Functions
- ****************************************************************************/
-
 /****************************************************************************
  * Public Functions
  ****************************************************************************/

arch/arm/src/armv7-a/arm_pgalloc.c

@@ -54,13 +54,6 @@
 
 #ifdef CONFIG_BUILD_KERNEL
 
-/****************************************************************************
- * Pre-processor Definitions
- ****************************************************************************/
-
-/****************************************************************************
- * Private Data
- ****************************************************************************/
 
 /****************************************************************************
  * Private Functions

arch/arm/src/armv7-a/arm_pginitialize.c

@@ -49,18 +49,6 @@
 
 #ifdef CONFIG_PAGING
 
-/****************************************************************************
- * Pre-processor Definitions
- ****************************************************************************/
-
-/****************************************************************************
- * Private Data
- ****************************************************************************/
-
-/****************************************************************************
- * Private Functions
- ****************************************************************************/
-
 /****************************************************************************
  * Public Functions
  ****************************************************************************/

arch/arm/src/armv7-a/arm_physpgaddr.c

@@ -55,17 +55,6 @@
 
 #ifdef CONFIG_MM_PGALLOC
 
-/****************************************************************************
- * Pre-processor Definitions
- ****************************************************************************/
-/****************************************************************************
- * Private Data
- ****************************************************************************/
-
-/****************************************************************************
- * Private Functions
- ****************************************************************************/
-
 /****************************************************************************
  * Public Functions
  ****************************************************************************/

arch/arm/src/armv7-a/arm_prefetchabort.c

@@ -61,18 +61,6 @@
 #include "sched/sched.h"
 #include "up_internal.h"
 
-/****************************************************************************
- * Pre-processor Definitions
- ****************************************************************************/
-
-/****************************************************************************
- * Private Data
- ****************************************************************************/
-
-/****************************************************************************
- * Private Functions
- ****************************************************************************/
-
 /****************************************************************************
  * Public Functions
  ****************************************************************************/

arch/arm/src/armv7-a/arm_schedulesigaction.c

@@ -53,18 +53,6 @@
 
 #ifndef CONFIG_DISABLE_SIGNALS
 
-/****************************************************************************
- * Pre-processor Definitions
- ****************************************************************************/
-
-/****************************************************************************
- * Private Data
- ****************************************************************************/
-
-/****************************************************************************
- * Private Functions
- ****************************************************************************/
-
 /****************************************************************************
  * Public Functions
  ****************************************************************************/

arch/arm/src/armv7-a/arm_signal_dispatch.c

@@ -47,18 +47,6 @@
 #if ((defined(CONFIG_BUILD_PROTECTED) && defined(__KERNEL__)) || \
       defined(CONFIG_BUILD_KERNEL)) && !defined(CONFIG_DISABLE_SIGNALS)
 
-/****************************************************************************
- * Pre-processor Definitions
- ****************************************************************************/
-
-/****************************************************************************
- * Private Data
- ****************************************************************************/
-
-/****************************************************************************
- * Private Functions
- ****************************************************************************/
-
 /****************************************************************************
  * Public Functions
  ****************************************************************************/

arch/arm/src/armv7-a/arm_syscall.c

@@ -77,7 +77,7 @@
 #endif
 
 /****************************************************************************
- * Private Data
+ * Private Functions
  ****************************************************************************/
 
 /****************************************************************************

arch/arm/src/armv7-a/arm_unblocktask.c

@@ -49,18 +49,6 @@
 #include "clock/clock.h"
 #include "up_internal.h"
 
-/****************************************************************************
- * Pre-processor Definitions
- ****************************************************************************/
-
-/****************************************************************************
- * Private Data
- ****************************************************************************/
-
-/****************************************************************************
- * Private Functions
- ****************************************************************************/
-
 /****************************************************************************
  * Public Functions
  ****************************************************************************/

arch/arm/src/armv7-a/arm_undefinedinsn.c

@@ -58,18 +58,6 @@
 
 #include "up_internal.h"
 
-/****************************************************************************
- * Pre-processor Definitions
- ****************************************************************************/
-
-/****************************************************************************
- * Private Data
- ****************************************************************************/
-
-/****************************************************************************
- * Private Functions
- ****************************************************************************/
-
 /****************************************************************************
  * Public Functions
  ****************************************************************************/

arch/arm/src/armv7-a/arm_va2pte.c

@@ -52,18 +52,6 @@
 
 #ifdef CONFIG_PAGING
 
-/****************************************************************************
- * Pre-processor Definitions
- ****************************************************************************/
-
-/****************************************************************************
- * Private Data
- ****************************************************************************/
-
-/****************************************************************************
- * Private Functions
- ****************************************************************************/
-
 /****************************************************************************
  * Public Functions
  ****************************************************************************/

arch/arm/src/armv7-a/arm_vectors.S

@@ -45,10 +45,6 @@
 
 	.file	"arm_vectors.S"
 
-/************************************************************************************
- * Pre-processor Definitions
- ************************************************************************************/
-
 /************************************************************************************
  * Private Data
  ************************************************************************************/
@@ -69,10 +65,6 @@ g_fiqtmp:
 	.word	0		/* Saved spsr */
 #endif
 
-/************************************************************************************
- * Assembly Macros
- ************************************************************************************/
-
 /************************************************************************************
  * Private Functions
  ************************************************************************************/

arch/arm/src/armv7-a/arm_virtpgaddr.c

@@ -43,18 +43,6 @@
 
 #if defined(CONFIG_MM_PGALLOC) && defined(CONFIG_ARCH_PGPOOL_MAPPING)
 
-/****************************************************************************
- * Pre-processor Definitions
- ****************************************************************************/
-
-/****************************************************************************
- * Private Data
- ****************************************************************************/
-
-/****************************************************************************
- * Private Functions
- ****************************************************************************/
-
 /****************************************************************************
  * Public Functions
  ****************************************************************************/

arch/arm/src/armv7-a/crt0.c

@@ -46,14 +46,6 @@
 
 #ifdef CONFIG_BUILD_KERNEL
 
-/****************************************************************************
- * Pre-processor Definitions
- ****************************************************************************/
-
-/****************************************************************************
- * Private Data
- ****************************************************************************/
-
 /****************************************************************************
  * Public Function Prototypes
  ****************************************************************************/

configs/sabre-6quad/README.txt

@@ -468,6 +468,13 @@ Open Issues:
    CPU (which may not be CPU0).  Perhaps that should be a spinlock to prohibit
    execution of interrupts on CPU0 when other CPUs are in a critical section?
 
+2. Cache Concurency.  This is a difficult problem.  There is logic in place now to
+   clean CPU0 D-cache before starting a new CPU and for invalidating the D-Cache
+   when the new CPU is started.
+
+   But there are many, many more cache coherency issues.  This could, in face, be
+   a showstopping issue.
+
 Configurations
 ==============
 

sched/irq/irq_dispatch.c

@@ -45,26 +45,6 @@
 
 #include "irq/irq.h"
 
-/****************************************************************************
- * Pre-processor Definitions
- ****************************************************************************/
-
-/****************************************************************************
- * Private Type Declarations
- ****************************************************************************/
-
-/****************************************************************************
- * Public Data
- ****************************************************************************/
-
-/****************************************************************************
- * Private Data
- ****************************************************************************/
-
-/****************************************************************************
- * Private Function Prototypes
- ****************************************************************************/
-
 /****************************************************************************
  * Public Functions
  ****************************************************************************/
@@ -102,4 +82,3 @@ void irq_dispatch(int irq, FAR void *context)
 
   vector(irq, context);
 }
-