lwIP - A Lightweight TCP/IP stack - Patches: patch #9272, arp/ndp: allow overriding the...
You are not allowed to post comments on this tracker with your current authentication level.
patch #9272: arp/ndp: allow overriding the decision to copy pbufs
Submitter: | David van Moolenbroek <dcvmoole> | ||
Submitted: | Mon 27 Feb 2017 04:41:11 PM UTC | ||
Category: | ARP | Priority: | 3 - Low |
Status: | Done | Privacy: | Public |
Assigned to: | goldsimon | Open/Closed: | Closed |
Planned Release: | None |
Tue 28 Feb 2017 05:42:41 PM UTC, comment #5: |
David van Moolenbroek <dcvmoole> |
Tue 28 Feb 2017 08:39:49 AM UTC, comment #4: |
Simon Goldschmidt <goldsimon> |
Tue 28 Feb 2017 07:43:12 AM UTC, comment #3: Task #6735 is not much newer, being 9 years old now. And even back then, Kieran said in comment #33:
|
Simon Goldschmidt <goldsimon> |
Mon 27 Feb 2017 09:30:28 PM UTC, comment #2: That would be fine with me as well! The last comments from task #6735, which are more recent than bug #11400, suggested to me that at least at the time it was considered acceptable to reuse the pbuf, so I wasn't sure if I'd break anything changing that.
|
David van Moolenbroek <dcvmoole> |
Mon 27 Feb 2017 08:48:50 PM UTC, comment #1: Hm, to me the correct thing to do would be to only copy PBUF_REF.
|
Simon Goldschmidt <goldsimon> |
Mon 27 Feb 2017 04:41:11 PM UTC, original submission:
I realize that this is a contentious topic, especially because the long-term goal is a larger redesign, but after about a decade I'm guessing that that might take a while longer. ;) Therefore I'd like to propose a tiny change in this area that has substantial benefits for me right now at least [*]. I'd love to hear what you think.
|
David van Moolenbroek <dcvmoole> |
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 4 latest changes.
Date | Changed by | Updated Field | Previous Value | => | Replaced by |
---|---|---|---|---|---|
2017-02-28 | goldsimon | Status | None | Done | |
Assigned to | None | goldsimon | |||
Open/Closed | Open | Closed | |||
2017-02-27 | dcvmoole | Attached File | - | Added 0001-arp-ndp-allow-overriding-the-decision-to-copy-pbufs.patch, #39837 |
Powered by Savane 3.13-758e.
Corresponding source code
Thank you :)