Open ghananigans opened 3 years ago
A friendly reminder that this issue had no activity for 120 days.
@manninglucas is this still relevant?
I think it's more appropriate to have the enhancement label here. It's not really an issue right now, but could be a good idea to implement in the future. This idea also probably needs more detailed design work.
The Packet buffer currently holds only fields that are only set by some layer and then read by layers above it. Lower layers "promise" to never touch upper layers' fields.
E.g. Network layer adds network layer information and transport layer uses that when processing a packet. The link layer never touches those fields.
We should consider improving the type safety of the packet buffer by wrapping lower layer types with a new type that has its layer's information.
E.g. Instead of
We can use
An alternative is to wrap lower layer types with another layer's type (LinkPacket in a NetworkPacket) or some other idea.
cc @tamird @hbhasker