[Top][All Lists]

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

[lwip-devel] [task #7490] Add return value to sys_mbox_post

From: Simon Goldschmidt
Subject: [lwip-devel] [task #7490] Add return value to sys_mbox_post
Date: Sat, 05 Jan 2008 11:17:34 +0000
User-agent: Mozilla/5.0 (Windows; U; Windows NT 5.1; de; rv: Gecko/20071127 Firefox/

Follow-up Comment #23, task #7490 (project lwip):

to sys_arch.txt: To prevent people porting lwIP to get confused about the
'size' argument, I'd add something like 'Elements stored in mboxes are
pointers.' to the description of sys_mbox_new() so that people can calculate
the actual memory size needed. Also, 0 = max or 0 = default should be
mentioned somewhere, I think (since TCPIP_MBOX_SIZE is 0 by default).

to 1/:
I would rather include a new function (tcpip_trycallback or something), that
way, it is not linked when not used.

to 4/:
Resetting the pcb callbacks, calling tcp_close (or tcp_abort), freeing
conn->mbox, conn->recvmbox and conn should be done. This can be done in an
extra function to be combined with do_close_internal() and netconn_delete()
(draining the mboxes doesn't hurt).

About using sys_mbox_trypost in api_msg.c:recv_tcp(): Does this really work?
We should take a look at tcp_in.c to see if retransmitted data is given to the
application again on receiving a second time... after all I think tcp might
already have ACKed that data... ??

Other than that, I think the patch is fine.


Reply to this item at:


  Nachricht geschickt von/durch Savannah

reply via email to

[Prev in Thread] Current Thread [Next in Thread]