Skip to content

ng_icmpv6: Initial import#2555

Merged
OlegHahm merged 4 commits intoRIOT-OS:masterfrom
miri64:ng_icmpv6/feat/initial
May 1, 2015
Merged

ng_icmpv6: Initial import#2555
OlegHahm merged 4 commits intoRIOT-OS:masterfrom
miri64:ng_icmpv6/feat/initial

Conversation

@miri64
Copy link
Copy Markdown
Member

@miri64 miri64 commented Mar 7, 2015

Depends on #2454 and all its dependencies.

@miri64 miri64 added Area: network Area: Networking State: WIP State: The PR is still work-in-progress and its code is not in its final presentable form yet Process: API change Integration Process: PR contains or issue proposes an API change. Should be handled with care. State: waiting for other PR State: The PR requires another PR to be merged first NSTF labels Mar 7, 2015
@miri64 miri64 added this to the Network Stack Task Force milestone Mar 7, 2015
@miri64 miri64 mentioned this pull request Mar 7, 2015
Closed
36 tasks
@miri64 miri64 force-pushed the ng_icmpv6/feat/initial branch 4 times, most recently from 7b5d706 to 89a23fd Compare March 12, 2015 01:28
@miri64
Copy link
Copy Markdown
Member Author

miri64 commented Mar 12, 2015

Rebased to current #2454

@miri64 miri64 force-pushed the ng_icmpv6/feat/initial branch 2 times, most recently from 7f1b556 to 33de13f Compare March 15, 2015 14:23
@miri64
Copy link
Copy Markdown
Member Author

miri64 commented Mar 15, 2015

Rebased to current #2454

@miri64 miri64 mentioned this pull request Mar 16, 2015
Closed
@miri64 miri64 force-pushed the ng_icmpv6/feat/initial branch 2 times, most recently from e36047f to 43e113f Compare March 19, 2015 14:58
@miri64
Copy link
Copy Markdown
Member Author

miri64 commented Mar 19, 2015

Rebased to current #2454

This was referenced Mar 19, 2015
Merged
Merged
@miri64 miri64 force-pushed the ng_icmpv6/feat/initial branch from ce1af46 to 8b66701 Compare March 25, 2015 01:56
@miri64
Copy link
Copy Markdown
Member Author

miri64 commented Mar 25, 2015

Rebased to current #2454

@miri64 miri64 force-pushed the ng_icmpv6/feat/initial branch from 6ae135e to 8f929a8 Compare March 30, 2015 00:33
@miri64
Copy link
Copy Markdown
Member Author

miri64 commented Mar 30, 2015

Rebased and adapted to changes in #2454. Since they were quite big, I squashed.

@miri64 miri64 removed the State: WIP State: The PR is still work-in-progress and its code is not in its final presentable form yet label Mar 30, 2015
@miri64
Copy link
Copy Markdown
Member Author

miri64 commented Mar 30, 2015

(No longer WIP)

cgundogan
cgundogan reviewed Apr 28, 2015
View reviewed changes
sys/net/network_layer/ng_icmpv6/ng_icmpv6.c Outdated
Copy link
Copy Markdown
Member

@cgundogan cgundogan Apr 28, 2015

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

too many sends!

@miri64
Copy link
Copy Markdown
Member Author

miri64 commented Apr 29, 2015

Addressed comments

OlegHahm
OlegHahm reviewed Apr 29, 2015
View reviewed changes
sys/net/network_layer/ng_icmpv6/echo/ng_icmpv6_echo.c Outdated
Copy link
Copy Markdown
Member

@OlegHahm OlegHahm Apr 29, 2015

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Any reason not to inline these functions now?

Copy link
Copy Markdown
Member Author

@miri64 miri64 Apr 29, 2015

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Because I need to expose the generic function for this. Anyways: just did that.

@OlegHahm OlegHahm added the CI: needs squashing Commits in this PR need to be squashed; If set, CI systems will mark this PR as unmergable label Apr 30, 2015
OlegHahm
OlegHahm reviewed Apr 30, 2015
View reviewed changes
sys/net/network_layer/ng_icmpv6/echo/ng_icmpv6_echo.c Outdated
Copy link
Copy Markdown
Member

@OlegHahm OlegHahm Apr 30, 2015

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Wouldn't this comment make more sense if written right before ng_pktbuf_hold()?

Copy link
Copy Markdown
Member Author

@miri64 miri64 Apr 30, 2015

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Yes.

@miri64
Copy link
Copy Markdown
Member Author

miri64 commented Apr 30, 2015

Addressed comments

@OlegHahm
Copy link
Copy Markdown
Member

OlegHahm commented Apr 30, 2015

ACK. Please squash and go if Travis agrees.

@miri64 miri64 force-pushed the ng_icmpv6/feat/initial branch from 5f77067 to 8be0772 Compare April 30, 2015 16:55
@miri64
Copy link
Copy Markdown
Member Author

miri64 commented Apr 30, 2015

Squashed

@miri64 miri64 mentioned this pull request Apr 30, 2015
Merged
@miri64 miri64 removed the CI: needs squashing Commits in this PR need to be squashed; If set, CI systems will mark this PR as unmergable label May 1, 2015
@miri64 miri64 force-pushed the ng_icmpv6/feat/initial branch from 8be0772 to 8e1f4a2 Compare May 1, 2015 06:32
@miri64 miri64 force-pushed the ng_icmpv6/feat/initial branch from 8e1f4a2 to 229f49a Compare May 1, 2015 06:34
@miri64
Copy link
Copy Markdown
Member Author

miri64 commented May 1, 2015

Had to fix something: since the types of the echo request/reply were now exposed, I needed to put them somewhere else than ng_icmpv6.h => introduced ng_icmpv6/types.h

@miri64
Copy link
Copy Markdown
Member Author

miri64 commented May 1, 2015

Please re-ACK.

@OlegHahm
Copy link
Copy Markdown
Member

OlegHahm commented May 1, 2015

I think that's fine. ACK and go.

OlegHahm added a commit that referenced this pull request May 1, 2015
@OlegHahm
Merge pull request #2555 from authmillenon/ng_icmpv6/feat/initial
d1bfa2f 
ng_icmpv6: Initial import
@OlegHahm OlegHahm merged commit d1bfa2f into RIOT-OS:master May 1, 2015
@miri64 miri64 deleted the ng_icmpv6/feat/initial branch May 1, 2015 11:45
@miri64 miri64 mentioned this pull request Feb 1, 2019
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

@OlegHahm OlegHahm

Labels

Area: network Area: Networking Process: API change Integration Process: PR contains or issue proposes an API change. Should be handled with care.

Projects

None yet

Milestone

Network Stack Task Force

Development

Successfully merging this pull request may close these issues.

5 participants

@miri64 @OlegHahm @cgundogan @kaspar030 @haukepetersen