lwIP - A Lightweight TCP/IP stack - Patches: patch #5780, Implement SO_RCVTIMEO on UDP...
You are not allowed to post comments on this tracker with your current authentication level.
patch #5780: Implement SO_RCVTIMEO on UDP sockets/netconn
Submitter: | Frédéric Bernon <fbernon> | ||
Submitted: | Sun 04 Mar 2007 01:56:11 PM UTC | ||
Category: | None | Priority: | 5 - Normal |
Status: | Done | Privacy: | Public |
Assigned to: | fbernon | Open/Closed: | Closed |
Planned Release: | None |
Fri 20 Apr 2007 07:39:50 PM UTC, comment #4: |
Simon Goldschmidt <goldsimon> |
Tue 06 Mar 2007 04:19:38 PM UTC, comment #3: Looks OK to me. There are a number of other changes in the patch - whitespace mods, formatting, that sort of thing - which normally I'd prefer to see as a separate change, but I know how it is easier to clean these things up while you're working on something. |
Kieran Mansley <kieranm> |
Tue 06 Mar 2007 03:24:01 PM UTC, comment #2: I join the SO_RCVTIMO.patch file. About last remarks from Simon, I found that - in this case - adding a new sys_mbox_fetch_timeout will only increase source code size and footprint. This patch is very "small & simple". But, before commit that, I let you and others give me your comments.
|
Frédéric Bernon <fbernon> |
Mon 05 Mar 2007 07:41:42 AM UTC, comment #1: Implementing such general socket options (derived from linux etc.) is fine for me, as long as you can turn it off to save code space if you don't need it.
|
Simon Goldschmidt <goldsimon> |
Sun 04 Mar 2007 01:56:11 PM UTC, original submission:
I propose to add the possibility to use the socket's option name SO_RCVTIMEO on UDP sockets. It's very usefull in some UDP protocols, like SNTP or DNS, where you have to wait an answer which can never arrived (packet lost, server shutdown...). SO_RCVTIMEO is a very generic parameter. It allow to give a code very simple (you can do something like that with select, but it's not the simplest solution).
|
Frédéric Bernon <fbernon> |
Depends on the following items: None found
Items that depend on this one: None found
There are 0 votes so far. Votes easily highlight which items people would like to see resolved in priority, independently of the priority of the item set by tracker managers.
Follow 3 latest changes.
Date | Changed by | Updated Field | Previous Value | => | Replaced by |
---|---|---|---|---|---|
2007-03-06 | fbernon | Status | In Progress | Done | |
Open/Closed | Open | Closed | |||
2007-03-06 | fbernon | Attached File | - | Added SO_RCVTIMEO.patch, #12127 |
Powered by Savane 3.13-f14d.
Corresponding source code
I know I'm a bit late on this bug, since it is closed for over one month now, but shouldn't sys_mbox_fetch_timeout() return something like SYS_ARCH_TIMEOUT on timeout (no message) instead of setting *msg=NULL?