lwIP - A Lightweight TCP/IP stack - Tasks: task #7269, Call netif_set_default during...
You are not allowed to post comments on this tracker with your current authentication level.
task #7269: Call netif_set_default during netif_add if not yet defined
Submitter: | Frédéric Bernon <fbernon> | ||
Submitted: | Mon 03 Sep 2007 10:06:21 AM UTC | ||
Category: | None | Should Start On: | Sun 02 Sep 2007 10:00:00 PM UTC |
Should be Finished on: | Sun 02 Sep 2007 10:00:00 PM UTC | Priority: | 5 - Normal |
Status: | Done | Privacy: | Public |
Assigned to: | fbernon | Percent Complete: | 100% |
Open/Closed: | Closed | Planned Release: | 1.3.0 |
Effort: | 0.00 |
Mon 03 Sep 2007 12:22:03 PM UTC, comment #4: |
Frédéric Bernon <fbernon> |
Mon 03 Sep 2007 10:52:21 AM UTC, comment #3: I agree with jonathan. Except for windows, in most OSes I know you have to set the default route explicitly.
|
Simon Goldschmidt <goldsimon> |
Mon 03 Sep 2007 10:38:30 AM UTC, comment #2:
|
Frédéric Bernon <fbernon> |
Mon 03 Sep 2007 10:17:38 AM UTC, comment #1: Unfortunately I think this netif_add change would be wrong. I think it is valid to only provide some defined routes, and no default route.
|
Jonathan Larmour <jifl> |
Mon 03 Sep 2007 10:06:21 AM UTC, original submission:
Is there any objections to patch netif_add like this ?
|
Frédéric Bernon <fbernon> |
No files currently attached
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 5 latest changes.
Date | Changed by | Updated Field | Previous Value | => | Replaced by |
---|---|---|---|---|---|
2007-09-03 | fbernon | Status | None | Done | |
Percent Complete | 0% | 100% | |||
Assigned to | None | fbernon | |||
Open/Closed | Open | Closed | |||
Planned Release | None | 1.3.0 |
Powered by Savane 3.13-caa5.
Corresponding source code
Ok, so, I will just fix the netif_remove, and let the netif_add like this.
If you have several netif, you should do an explicit netif_set_default (but, since we don't check in this part of the patch, it's just a remark).
I'm surprise to never have see any questions about a potential routing table. But it doesn't seems to be an usual need for embedded products...
Ok, I fix the netif_remove, and close this task...