<p><br>
On Sep 8, 2013 10:05 AM, "Joel Sherrill" <<a href="mailto:Joel.Sherrill@oarcorp.com">Joel.Sherrill@oarcorp.com</a>> wrote:<br>
><br>
> Given the talk of reorganizing the libcpu and libbsp structure, I would personally ignore the cache definitions BUT they feel VERY wrong to be defined in a bsp. They should be more related to a CPU core.<br>
><br>
I think it makes sense for (non-multilib) CPU-dependent code to be defined in libbsp/arch/shared.</p>
<p>> Bsp_reset should be defined by every bsp. That is a fairly recent addition and maybe those need the infinite loop stub added. It is also possible some had reset code at the bottom of start.S. double check .. May be a script issue missing some. Otherwise, we need to address.<br>
><br>
> I am suspicious of the bspprederiverhook missings. The bsp would not compile and link if this was missing.<br>
><br>
> I also don't like the explicit heap calls in gen5200. That is wrong.<br>
><br>
> Every bsp must have bspgetworkarea. So that must be a script issue. It wouldn't link.<br>
><br>
> _BSP_put_char is just a bad name choice. We should change those.<br>
><br>
> Calling _ fatal error routines seems wrong. Probably a mistake.<br>
><br>
> And why is a bsp directly calling _Thread _Start_multitasking. That seems wrong.<br>
><br>
> _BSP... Print exception frame has to be named incorrectly.<br>
><br>
> If we ignore the cache and isr CPU dependent helpers, I would worry about bsp reset and getworkarea. The latter almost certainly is a false positive. The first could be some false positives. <br>
><br>
> But as I mentioned, some are likely indicative of naming problems and can be fixed.<br>
><br>
><br>
><br>
> Gedare Bloom <<a href="mailto:gedare@rtems.org">gedare@rtems.org</a>> wrote:<br>
><br>
> You can add the script to rtems-testing.git (there is already check_bsp there i think). Check for using internal rtems functions versus defining them, e.g. by checking for a semicolon. Some of the uses you flag like the _CPU_Cache_xxx are defining those functions, which i think is ok.. Joel?<br>
><br>
> -Gedare<br>
><br>
> On Sep 8, 2013 5:59 AM, "Vipul Nayyar" <<a href="mailto:nayyar_vipul@yahoo.com">nayyar_vipul@yahoo.com</a>> wrote:<br>
>><br>
>> Hello Gedare,<br>
>><br>
>> I reviewed the current documentation for bsp development as you suggested. I didn't find any major discrepancies present already, but will look again closely. As far I think, the part about the PIC support which is available to every BSP is currently not present in the documentation, so I'll submit the addition to docs regarding this if you also agree to it. <br>
>><br>
>> My current script gives results on a per bsp basis. However, as you requested to classify the problems, I've done that by tweaking it a little. I've added the functionality to search for RTEMS internal functions being used in the bsp. From what I understood, these internal functions I've looked for in BSPs are taken from libcpu & cpukit. <br>
>><br>
>> My aim in developing this bash script was to identify flaws in BSPs , and also to provide a simple tool for future BSP developers so that they can check the file organization for their BSP. So, is it fine if I submit a patch to add this script to the RTEMS tree ??<br>
>><br>
>> Following, you'll find discrepancies found in current RTEMS tree regarding some serious flaws. The attached file all_warnings.txt classifies the flaws found in all bsps, either it being serious or not. The attached file 'per_bsp_observation.txt' contains result of the script showing serious flaws in all the bsps.<br>
>><br>
>><br>
>> **** RTEMS Internal functions being used in BSPs ****<br>
>> arm/csb336 : _BSP_poll_char()<br>
>><br>
>> arm/edb7312 : _CPU_ISR_install_vector()<br>
>><br>
>> arm/gp32 : _BSP_put_char()<br>
>><br>
>> arm/lpc24xx : _CPU_ISR_install_vector() _Heap_Extend()<br>
>><br>
>> arm/rtl22xx : _BSP_put_char()<br>
>><br>
>> m68k/av5282 : _CPU_cache_disable_instruction() _CPU_cache_enable_instruction() _CPU_cache_invalidate_entire_instruction() _CPU_cache_disable_data() _CPU_cache_enable_data() _CPU_cache_invalidate_entire_data() _CPU_cache_flush_entire_data() _CPU_cache_unfreeze_instruction() _CPU_cache_freeze_instruction() _CPU_cache_invalidate_1_instruction_line() _CPU_cache_unfreeze_data() _CPU_cache_freeze_data() _CPU_cache_invalidate_1_data_line() _CPU_cache_flush_1_data_line()<br>
>><br>
>> m68k/gen68340 : _CPU_ISR_install_raw_handler()<br>
>><br>
>> m68k/genmcf548x : _CPU_cache_disable_instruction() _CPU_cache_enable_instruction() _CPU_cache_invalidate_entire_instruction() _CPU_cache_disable_data() _CPU_cache_enable_data() _CPU_cache_invalidate_entire_data() _CPU_cache_flush_entire_data() _CPU_cache_unfreeze_instruction() _CPU_cache_freeze_instruction() _CPU_cache_invalidate_1_instruction_line() _CPU_cache_unfreeze_data() _CPU_cache_freeze_data() _CPU_cache_invalidate_1_data_line() _CPU_cache_flush_1_data_line() _Thread_Start_multitasking()<br>
>><br>
>> m68k/mcf5225x : _CPU_cache_disable_instruction() _CPU_cache_enable_instruction() _CPU_cache_invalidate_entire_instruction() _CPU_cache_disable_data() _CPU_cache_enable_data() _CPU_cache_invalidate_entire_data() _CPU_cache_flush_entire_data() _CPU_cache_unfreeze_instruction() _CPU_cache_freeze_instruction() _CPU_cache_invalidate_1_instruction_line() _CPU_cache_unfreeze_data() _CPU_cache_freeze_data() _CPU_cache_invalidate_1_data_line() _CPU_cache_flush_1_data_line()<br>
>><br>
>> m68k/mrm332 : _CPU_ISR_Set_level() _CPU_Fatal_halt()<br>
>><br>
>> m68k/uC5282 : _CPU_cache_disable_instruction() _CPU_cache_enable_instruction() _CPU_cache_invalidate_entire_instruction() _CPU_cache_disable_data() _CPU_cache_enable_data() _CPU_cache_invalidate_entire_data() _CPU_cache_flush_entire_data() _CPU_cache_unfreeze_instruction() _CPU_cache_freeze_instruction() _CPU_cache_invalidate_1_instruction_line() _CPU_cache_unfreeze_data() _CPU_cache_freeze_data() _CPU_cache_invalidate_1_data_line() _CPU_cache_flush_1_data_line()<br>
>><br>
>> powerpc/gen5200 : _Heap_Free() _Heap_Initialize() _Heap_Allocate()<br>
>><br>
>> powerpc/qoriq : _CPU_SMP_Send_interrupt()<br>
>><br>
>> sparc/erc32 : _BSP_Exception_frame_print() _CPU_ISR_install_raw_handler()<br>
>><br>
>> sparc/leon2 : _BSP_Exception_frame_print() _CPU_ISR_install_raw_handler()<br>
>><br>
>> sparc/leon3 : _BSP_Exception_frame_print() _CPU_ISR_install_raw_handler() _CPU_SMP_Send_interrupt()<br>
>><br>
>> **** bsggetworkarea not compiled ****<br>
>> powerpc/mpc55xxevb<br>
>><br>
>> **** bspreset.c not compiled ****<br>
>> arm/gumstix<br>
>> avr/avrtest<br>
>> bfin/bf537Stamp<br>
>> bfin/eZKit533<br>
>> bfin/TLL6527M<br>
>> h8300/h8sim<br>
>> lm32/milkymist<br>
>> m32c/m32cbsp<br>
>> m68k/av5282<br>
>> m68k/csb360<br>
>> m68k/gen68302<br>
>> m68k/gen68340<br>
>> m68k/gen68360<br>
>> m68k/genmcf548x<br>
>> m68k/idp<br>
>> m68k/mcf5206elite<br>
>> m68k/mcf52235<br>
>> m68k/mcf5225x<br>
>> m68k/mcf5235<br>
>> m68k/mcf5329<br>
>> m68k/mrm332<br>
>> m68k/mvme136<br>
>> m68k/mvme147<br>
>> m68k/mvme147s<br>
>> m68k/mvme162<br>
>> m68k/mvme167<br>
>> m68k/ods68302<br>
>> m68k/sim68000<br>
>> mips/genmongoosev<br>
>> mips/hurricane<br>
>> mips/jmr3904<br>
>> mips/rbtx4925<br>
>> mips/rbtx4938<br>
>> moxie/moxiesim<br>
>> nios2/nios2_iss<br>
>> powerpc/beatnik<br>
>> powerpc/ep1a<br>
>> powerpc/haleakala<br>
>> powerpc/mbx8xx<br>
>> powerpc/mpc55xxevb<br>
>> powerpc/mpc8260ads<br>
>> powerpc/mvme3100<br>
>> powerpc/psim<br>
>> powerpc/qemuppc<br>
>> powerpc/score603e<br>
>> powerpc/ss555<br>
>> powerpc/tqm8xx<br>
>> powerpc/virtex4<br>
>> powerpc/virtex5<br>
>> powerpc/virtex<br>
>> sh/gensh1<br>
>> sh/gensh2<br>
>> sh/gensh4<br>
>> sh/shsim<br>
>> sparc64/niagara<br>
>> sparc64/usiii<br>
>> sparc/erc32<br>
>> sparc/leon2<br>
>> sparc/leon3<br>
>><br>
>> **** start.S not compiled ****<br>
>> m68k/ods68302<br>
>> powerpc/haleakala<br>
>><br>
>> **** bsppredriverhook.c not compiled ****<br>
>> lm32/lm32_evr<br>
>> lm32/milkymist<br>
>> powerpc/mvme3100<br>
>> powerpc/score603e<br>
>> powerpc/virtex4<br>
>> powerpc/virtex5<br>
>> sparc/erc32<br>
>> sparc/leon2<br>
>> sparc/leon3<br>
>><br>
>> **** PIC Support not included *****<br>
>> arm/gdbarmsim<br>
>> avr/avrtest<br>
>> bfin/bf537Stamp<br>
>> bfin/eZKit533<br>
>> bfin/TLL6527M<br>
>> h8300/h8sim<br>
>> lm32/lm32_evr<br>
>> lm32/milkymist<br>
>> m32c/m32cbsp<br>
>> m32r/m32rsim<br>
>> m68k/av5282<br>
>> m68k/csb360<br>
>> m68k/gen68302<br>
>> m68k/gen68340<br>
>> m68k/gen68360<br>
>> m68k/genmcf548x<br>
>> m68k/idp<br>
>> m68k/mcf5206elite<br>
>> m68k/mcf52235<br>
>> m68k/mcf5225x<br>
>> m68k/mcf5235<br>
>> m68k/mcf5329<br>
>> m68k/mrm332<br>
>> m68k/mvme136<br>
>> m68k/mvme147<br>
>> m68k/mvme147s<br>
>> m68k/mvme162<br>
>> m68k/mvme167<br>
>> m68k/ods68302<br>
>> m68k/sim68000<br>
>> m68k/uC5282<br>
>> moxie/moxiesim<br>
>> nios2/nios2_iss<br>
>> powerpc/beatnik<br>
>> powerpc/ep1a<br>
>> powerpc/haleakala<br>
>> powerpc/motorola_powerpc<br>
>> powerpc/mvme3100<br>
>> powerpc/mvme5500<br>
>> powerpc/score603e<br>
>> powerpc/ss555<br>
>> powerpc/virtex4<br>
>> powerpc/virtex5<br>
>> sh/gensh1<br>
>> sh/gensh2<br>
>> sh/gensh4<br>
>> sh/shsim<br>
>> sparc64/niagara<br>
>> sparc64/usiii<br>
>> v850/gdbv850sim<br>
>> <br>
>> Hoping to hear from you soon !!<br>
>><br>
>><br>
>> Regards<br>
>> Vipul Nayyar <br>
>><br>
</p>