Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Seperate IPv6 and 6LoWPAN #404

Closed
wants to merge 1 commit into from
Closed

Seperate IPv6 and 6LoWPAN #404

wants to merge 1 commit into from

Conversation

mehlis
Copy link
Contributor

@mehlis mehlis commented Dec 6, 2013

alternative to #357

it builds for native and msba2

missing peace: start lowpan thread

@OlegHahm
Copy link
Member

OlegHahm commented Dec 7, 2013

There seems to be a lot of duplicate code in this PR. Maybe we could consolidate this.

@mehlis
Copy link
Contributor Author

mehlis commented Dec 12, 2013

consolidated -> rewrote my code -> force commit + push

problem: we now use lowpan as a layer, to access with the msg system: sending a radio_packet_t does not work because an ipv6 addr is longer that a layer 2 address....

todo: start lowpan layer thread

@OlegHahm
Copy link
Member

Maybe we could also merge recv_ieee802154_frame with the lowpan_layer_thread, in order to not have two threads running on this layer.

Regarding the problem with the address size: this could be solved by replacing the uint16_t for the src and dst member by using a typedef, right?

@mehlis
Copy link
Contributor Author

mehlis commented Dec 13, 2013

@OlegHahm I wanted to use these fields for the ip address:

uint16_t src; ///< Radio source address

but unfortunately they are too small....we can use a typedef to manage this (for the cc1100 hardware this filed has the size uint16_t but perhaps an other transceiver has an other mac layer address size)

@OlegHahm
Copy link
Member

Actuallay cc1100 has 8bit hardware addresses, but we should rethink this. The transceiver module is meant to provide a generic driver interface. Hence, it assumes a link layer address, not a network layer address.

@mehlis
Copy link
Contributor Author

mehlis commented Dec 15, 2013

totally right!

@miri64
Copy link
Member

miri64 commented Dec 27, 2013

After some consideration I am not sure if it is that good an idea to decide the usage of 6LoWPAN by a compiler set macro. Wouldn't it be better to make it dependable on the maximum packet size of the radio?

if (packet_size < 1280) (MTU of IPv6) use 6LoWPAN.

EDIT: This could also be determined at compile time.

@kaspar030
Copy link
Contributor

@authmillenon: well, if the minimum MTU is <1280, then IPv6 can't be used anyways, so the code can fallback to 6lowpan.

@OlegHahm
Copy link
Member

OlegHahm commented Jan 3, 2014

However, there might be other reasons to use 6lowpan.

@ghost ghost assigned mehlis Jan 20, 2014
@mehlis
Copy link
Contributor Author

mehlis commented Feb 10, 2014

as discussed, I close this PR, because it will get superseded by the general sixlowpan restructuring.

@mehlis mehlis closed this Feb 10, 2014
@mehlis mehlis deleted the sixlowpan-without-low branch March 28, 2014 21:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants