CC3200: simplelink driver concurrency issue

Oleg Tcaregorodtcev

Intellectual 525 points

Part Number: CC3200

Hi,

I've noticed a concurrency problem with g_pCB->FlowContCB.TxPoolCnt (simplelink/source/driver.c):

in case there are 2 tasks that transmit/receive data over 2 simultaneously opened sockets,

g_pCB->FlowContCB.TxPoolCnt = ((_SlResponseHeader_t *)uBuf.TempBuf)->TxPoolCnt; (_SlDrvMsgRead)

may get called in the context of one task while another one is in between

if(g_pCB->FlowContCB.TxPoolCnt > FLOW_CONT_MIN + 1 )

and

VERIFY_PROTOCOL(g_pCB->FlowContCB.TxPoolCnt > FLOW_CONT_MIN + 1 );

g_pCB->FlowContCB.TxPoolCnt--;

(_SlDrvDataWriteOp)

Please find the fixed code attached (based on the 1.2.0 SDK):

driver.h

driver.c

The solution above fixed the issue for my use case (one task does only TX, another one does mostly RX). Please consider it to be included in the upstream SDK.

over 8 years ago

0 Hnz over 8 years ago

Guru 39650 points

Hi Oleg,

Please see my old thread ( e2e.ti.com/.../1786303 ).

Jan

0 Oleg Tcaregorodtcev over 8 years ago in reply to Hnz

Intellectual 525 points

Hi Hnz,

Thank you for the answer. The thread is rather long. Could you please give me a link to the exact message describing the solution you propose?

Oleg

0 Hnz over 8 years ago in reply to Oleg Tcaregorodtcev

Guru 39650 points

Hi Oleg,

Solution is not that sophisticated as your changes in driver, but works well also - e2e.ti.com/.../1817975

Jan

0 Oleg Tcaregorodtcev over 8 years ago in reply to Hnz

Intellectual 525 points

Hi Jan,

From my experience, it's better in a long term perspective to fix the issue, and not to hide it.

Oleg

Wi-Fi

Wi-Fi forum

CC3200: simplelink driver concurrency issue