search

ciaa / firmware_v1

Firmware de la CIAA
http://www.proyecto-ciaa.com.ar
126 stars 120 forks source link

bugfix #432 RESOURCE = POSIXR for blinkink_lwip example #434

Closed mabeett closed 8 years ago

mabeett commented 8 years ago

Por favor estudiar (no conozco en detalle ciaaPOSIX). No agregué EVENT = POSIXE;, pero podria aplicar

mcerdeiro commented 8 years ago

Hola Matias,

Cual es el error? Se usan llamadas a posición desde esa tarea. En caso afirmativo hay que usar el recurso que usas y agregar el evento POSIXE va a menos que no se llame a ningún función bloqueante de posix.

Saludos Mariano Am 08.07.2016 7:46 PM schrieb "MAbeeTT (Matías Pecchia)" < notifications@github.com>:

Por favor estudiar (no conozco en detalle ciaaPOSIX). No agregué EVENT = POSIXE;, pero podria aplicar

— You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub https://github.com/ciaa/Firmware/pull/434#issuecomment-231425709, or mute the thread https://github.com/notifications/unsubscribe/AG7km-DaxpUh7_A3vzTyN82n6_YAjwaGks5qTozwgaJpZM4JIPvO .

mabeett commented 8 years ago

Se invoca a ciaaPOSIX_malloc(), que a su vez llama aciaaPOSIX_sem_wait()`. (ver ciaaPOSIX_stdlib.c). Como el recurso no existe se dispara el errorHook. en #432 dejé esta captura (no sé usar el debugger desde la línea de comandos). capture

jcecconi commented 8 years ago

Eso mismo estaba escribiendo...el recurso se necesita x el malloc de nuestro posiz, lo que no me explico es porque no esta si antes el ejemplo funcionaba...Se habrá borrado del oil sin querer en algún commit? El jul 8, 2016 3:01 PM, "MAbeeTT (Matías Pecchia)" notifications@github.com escribió:

Se invoca a ciaaPOSIX_malloc(), que a su vez llama aciaaPOSIX_sem_wait()`. (ver ciaaPOSIX_stdlib.c https://github.com/ciaa/Firmware/blob/1d1b0625a2387a74ee3f6ddb21d7247e6357b92a/modules/posix/src/ciaaPOSIX_stdlib.c#L128 ). Como el recurso no existe se dispara el errorHook. en #432 https://github.com/ciaa/Firmware/issues/432 dejé esta captura (no sé usar el debugger desde la línea de comandos). [image: capture] https://cloud.githubusercontent.com/assets/179218/16695841/584f117a-4518-11e6-85f3-efa382cf6e60.png

— You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub https://github.com/ciaa/Firmware/pull/434#issuecomment-231429769, or mute the thread https://github.com/notifications/unsubscribe/AGRsi0da-0swTWeL1ZUdSGA_BeclBnfTks5qTpCLgaJpZM4JIPvO .

mabeett commented 8 years ago

En 39d93f36b240508e03c07acb0f91b9d92e07a1f7 en blinking_lwip.oil está la tarea sin el recurso. Habría que poner a depurar en ese commit (suponiendo al ejemplo funcional) para ver si usa el malloc de ciaa_posix o el que trae LWIP.

mabeett commented 8 years ago

Mariano, no puedo (no sé) reponder la pregunta que hacés. =(

mcerdeiro commented 8 years ago

La pregunta era de Juan y creo que andaba por que antes malloc usaba sem_wait pero sem_wait era un stub, y ahora no hehe.

jcecconi commented 8 years ago

exactamente, ahi hice un seguimiento del tema. Yo el 13 de abril con este commit https://github.com/ciaa/Firmware/commit/1d1b0625a2387a74ee3f6ddb21d7247e6357b92a volvi a dejar andando el ejemplo de lwip...pero 3 dias despues con este otro commit https://github.com/ciaa/Firmware/commit/0bca134b81c60f3012a45d2024a6197619638caf Mariano le puso contenido a los semaforos y dejo de andar nuevamente el ejemplo....Como no tenemos los test funcionales en marcha nos tentereamos 3 meses despues... :(

2016-07-10 3:31 GMT-03:00 Mariano Cerdeiro notifications@github.com:

La pregunta era de Juan y creo que andaba por que antes malloc usaba sem_wait pero sem_wait era un stub, y ahora no hehe.

— You are receiving this because you commented. Reply to this email directly, view it on GitHub https://github.com/ciaa/Firmware/pull/434#issuecomment-231573413, or mute the thread https://github.com/notifications/unsubscribe/AGRsi4ffPArzYHa4IoyM9mYEOLX881Qkks5qUJHRgaJpZM4JIPvO .