[Bug 1680] New: _ISR_Handler_initialization improper alignment correction

[bugzilla-daemon at rtems.org]

https://www.rtems.org/bugzilla/show_bug.cgi?id=1680

           Summary: _ISR_Handler_initialization improper alignment
                    correction
           Product: RTEMS
           Version: HEAD
          Platform: All
        OS/Version: RTEMS
            Status: NEW
          Severity: normal
          Priority: P3
         Component: cpukit
        AssignedTo: joel.sherrill at oarcorp.com
        ReportedBy: giddyup44 at yahoo.com


Created an attachment (id=1021)
 --> (https://www.rtems.org/bugzilla/attachment.cgi?id=1021)
patch.

There appears to be a problem with the alignment of _CPU_Interrupt_stack_high,
at least I see a problem by inspection, although I haven't seen any symptoms.

Here is a diff showing what I think is a problem (and solution).
Index: isr.c
===================================================================
RCS file: /usr1/CVS/rtems/cpukit/score/src/isr.c,v
retrieving revision 1.19
diff -u -p -r1.19 isr.c
--- isr.c 29 Jun 2010 00:34:11 -0000  1.19
+++ isr.c 18 Aug 2010 18:12:24 -0000
@@ -63,7 +63,7 @@ void _ISR_Handler_initialization( void )
   );

   _CPU_Interrupt_stack_high = (void *)
-    ((uintptr_t) _CPU_Interrupt_stack_high & ~CPU_STACK_ALIGNMENT);
+    ((uintptr_t) _CPU_Interrupt_stack_high & ~(CPU_STACK_ALIGNMENT - 1));

   /* Interrupt stack might have to be aligned and/or setup
    * in a specific way.

-- 
Configure bugmail: https://www.rtems.org/bugzilla/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are watching all bug changes.