alex/esp-idf
alex/esp-idf
1
0
Fork 0
mirror of https://github.com/espressif/esp-idf.git synced 2024-10-05 20:47:46 -04:00
Code Issues Actions 2 Packages Projects Releases Wiki Activity

bugfix(ringbuffer): Fix ringbuffer_send timeout bug

Browse Source
This commit is contained in:
kooho 2018-01-17 13:28:41 +08:00
parent f4009b94dc
commit a0a2422541
1 changed files with 13 additions and 7 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

10
components/freertos/ringbuf.c
View File

@ -602,7 +602,6 @@ BaseType_t xRingbufferSend(RingbufHandle_t ringbuf, void *data, size_t dataSize,
//we will need to wait some more. //we will need to wait some more.
if (ticks_to_wait != portMAX_DELAY) { if (ticks_to_wait != portMAX_DELAY) {
ticks_remaining = ticks_end - xTaskGetTickCount(); ticks_remaining = ticks_end - xTaskGetTickCount();
}
// ticks_remaining will always be less than or equal to the original ticks_to_wait, // ticks_remaining will always be less than or equal to the original ticks_to_wait,
// unless the timeout is reached - in which case it unsigned underflows to a much // unless the timeout is reached - in which case it unsigned underflows to a much
@ -610,8 +609,15 @@ BaseType_t xRingbufferSend(RingbufHandle_t ringbuf, void *data, size_t dataSize,
// //
// (Check is written this non-intuitive way to allow for the case where xTaskGetTickCount() // (Check is written this non-intuitive way to allow for the case where xTaskGetTickCount()
// has overflowed but the ticks_end value has not overflowed.) // has overflowed but the ticks_end value has not overflowed.)
if(ticks_remaining > ticks_to_wait) {
//Timeout, but there is not enough free space for the item that need to be sent.
xSemaphoreGive(rb->free_space_sem);
return pdFALSE;
} }
} while (ringbufferFreeMem(rb) < needed_size && ticks_remaining > 0 && ticks_remaining <= ticks_to_wait); }
}
} while (ringbufferFreeMem(rb) < needed_size);
//Lock the mux in order to make sure no one else is messing with the ringbuffer and do the copy. //Lock the mux in order to make sure no one else is messing with the ringbuffer and do the copy.
portENTER_CRITICAL(&rb->mux); portENTER_CRITICAL(&rb->mux);