enable-smp on xilinx_zynq_zedboard

[Sebastian Huber]

Hello Ric,

yes, this is one of the rare cases in which a 
rtems_interrupt_local_disable|enable() makes sense. I fixed it on the 
master and 4.11.

On 20/07/15 21:25, Ric Claus wrote:
> Hello All,
>
>    Today I git cloned the RTEMS master and configured with:
>
> ../../src/rtems/configure --target=arm-rtems4.11 --enable-posix \
> --enable-networking --enable-cxx --enable-maintainer-mode --enable-tests \
> --enable-rtemsbsp=xilinx_zynq_zedboard --enable-smp \
> --prefix=/home/claus/development/rtems/4.11 BSP_PRINT_EXCEPTION_CONTEXT=1
>
> and built.  The build failed with:
>
> arm-rtems4.11-gcc -B../../../../../xilinx_zynq_zedboard/lib/ -specs bsp_specs -qrtems -march=armv7-a -mthumb -mfpu=neon -mfloat-abi=hard -mtune=cortex-a9 -O2 -g -Wall -Wmissing-prototypes -Wimplicit-function-declaration -Wstrict-prototypes -Wnested-externs    -march=armv7-a -mthumb -mfpu=neon -mfloat-abi=hard -mtune=cortex-a9   -o hello.exe init.o
> ../../../../../xilinx_zynq_zedboard/lib/librtemsbsp.a(libbsp_a-cache_manager.o): In function `arm_cache_l1_get_data_cache_size':
> /home/claus/development/rtems/bld/xilinx_zynq/zedboard/smp/arm-rtems4.11/c/xilinx_zynq_zedboard/lib/libbsp/arm/xilinx-zynq/../../../../../../../../../../../src/rtems/c/src/lib/libbsp/arm/xilinx-zynq/../shared/arm-l2c-310/../include/arm-cache-l1.h:380: undefined reference to `rtems_interrupt_disable'
> /home/claus/development/rtems/bld/xilinx_zynq/zedboard/smp/arm-rtems4.11/c/xilinx_zynq_zedboard/lib/libbsp/arm/xilinx-zynq/../../../../../../../../../../../src/rtems/c/src/lib/libbsp/arm/xilinx-zynq/../shared/arm-l2c-310/../include/arm-cache-l1.h:386: undefined reference to `rtems_interrupt_enable'
> ../../../../../xilinx_zynq_zedboard/lib/librtemsbsp.a(libbsp_a-cache_manager.o): In function `arm_cache_l1_get_instruction_cache_size':
> /home/claus/development/rtems/bld/xilinx_zynq/zedboard/smp/arm-rtems4.11/c/xilinx_zynq_zedboard/lib/libbsp/arm/xilinx-zynq/../../../../../../../../../../../src/rtems/c/src/lib/libbsp/arm/xilinx-zynq/../shared/arm-l2c-310/../include/arm-cache-l1.h:401: undefined reference to `rtems_interrupt_disable'
> /home/claus/development/rtems/bld/xilinx_zynq/zedboard/smp/arm-rtems4.11/c/xilinx_zynq_zedboard/lib/libbsp/arm/xilinx-zynq/../../../../../../../../../../../src/rtems/c/src/lib/libbsp/arm/xilinx-zynq/../shared/arm-l2c-310/../include/arm-cache-l1.h:407: undefined reference to `rtems_interrupt_enable’
>
> I remember Sebastian’s patch email titled "[PATCH] rtems: Add rtems_interrupt_local_disable|enable()” of a month ago.  Since the L1 cache functions of the Zynq are visible only to the local core, is it appropriate to replace the rtems_interrupt_disable|enable functions with rtems_interrupt_local_disable|enable in this file?  Or do we need a more involved solution, in which case, could I please get some guidance?
>
>    Thanks very much!
>
>    Cheers,
> 	Ric
>
> _______________________________________________
> users mailing list
> users at rtems.org
> http://lists.rtems.org/mailman/listinfo/users

-- 
Sebastian Huber, embedded brains GmbH

Address : Dornierstr. 4, D-82178 Puchheim, Germany
Phone   : +49 89 189 47 41-16
Fax     : +49 89 189 47 41-09
E-Mail  : sebastian.huber at embedded-brains.de
PGP     : Public key available on request.

Diese Nachricht ist keine geschäftliche Mitteilung im Sinne des EHUG.