Possible bug in AT91SAM7X EMAC driver
Posted by Christian Walter
on August 22, 2009
I found out that the AT91SAM7X emac driver from the 5.4.1 FreeRTOS distribution has a problem. If the pbuf passed to low_level_output contains more pbufs than the lEMACSend function has available transmit buffers all buffers will be filled but the transmission will not be started since the last buffer element can not be added to the transmit queue.
this renders the EMAC useless since it can no longer transmit any pakets. I would suggest one of the following options:
=> the pbuf should be reassembled in case its size exceeds the number of transmit buffers.
=> furthermore we should check upfront if it can fit into the buffers before allocating them.
The problem is not artifical and we have encountered it during unit testing of lwIP and FreeRTOS sicne we are using this combination commercially.
PS: Hi Richard - I have seen you have done great work on the coldfire - Thanks for making this one official.
RE: Possible bug in AT91SAM7X EMAC driver
Posted by Richard
on August 22, 2009
Thanks for pointing this out - I would be grateful to receive your modification so I can diff it with the original and understand the fix. I also have a different SAM7X MAC driver that somebody contributed that has some improvements in, maybe it also fixes this issue. Use r (dot) barry (at) freertos.org to send your code, or request the contributed driver.
Copyright (C) 2004-2010 Richard Barry. Copyright (C) 2010-2016 Real Time Engineers Ltd.
Any and all data, files, source code, html content and documentation included in the FreeRTOSTM distribution or available on this site are the exclusive property of Real Time Engineers Ltd..
See the files license.txt (included in the distribution) and this copyright notice for more information. FreeRTOSTM and FreeRTOS.orgTM are trade marks of Real Time Engineers Ltd.