| Commit message (Collapse) | Author | Age | Files | Lines |
| |
|
|
|
| |
Fix the pullup length of the shim header in mpls_do_error().
issue reported by Maxime Villard; OK deraadt@ claudio@
|
| |
|
|
| |
OK bluhm@
|
| |
|
|
|
|
|
| |
a new mbuf chain and this chain needs to be returned to the caller else
a use after free may happen.
Issue reported by Maxime Villard
OK bluhm@ deraadt@
|
| |
|
|
|
| |
Issue reported by Maxime Villard
OK bluhm@ deraadt@
|
| |
|
|
|
|
| |
proceeding, as described in ifconfig documentation. Discussion with claudio.
Related to a report from maxime.
ok claudio bluhm
|
| |
|
|
|
|
|
| |
This will help transitionning to an un-KERNEL_LOCK()ed IP
forwarding path.
Disucssed with bluhm@, ok claudio@
|
| |
|
|
| |
OK claudio@ mpi@
|
| |
|
|
|
|
|
| |
basic anyway. Simplifies the code a lot also by calling the mpls sysctl no
longer via the protosw but instead directly.
OK mpi@ on a previous diff. Also tested by renato@ who actually found a bug
which is now fixed.
|
| |
|
|
|
|
|
| |
This could result in a use after free if the route entry was holding
the last reference of the address descriptor.
ok jca@, bluhm@, claudio@
|
| |
|
|
|
|
| |
Being able to add route entries without configured addresses is a nice
feature but this is not my fight. So I'd rather no add another pointer
to ``struct rtentry'' if I'm not removing another one.
|
| |
|
|
|
|
|
|
|
| |
memory.
This will allow to unlink 'sruct rtentry' and 'struct ifaddr' to be able
to add route entries without needing an address.
ok sthen@, visa@, florian@
|
| |
|
|
| |
dlg@ and mpi@
|
| | |
|
| |
|
|
|
|
|
|
|
| |
them and they make everything so much harder with no gain. Remove the
ifp argument from mpls_input since it is not needed. On the input side
the lookup side is modified a bit when it comes to BOS handling.
Tested in a L3VPN setup with ldpd and bgpd. Commiting now so we can move
on with cleaning up rt_ifp usage. If this breaks L2VPN I will fix it once
reported. OK mpi@
|
| | |
|
| |
|
|
|
|
| |
specific places.
ok claudio@, benno@
|
| |
|
|
|
|
| |
needs to see lo0 in the output path.
ok claudio@
|
| |
|
|
|
|
|
|
|
| |
Use instead the RTF_LOCAL flag to loop local traffic back to the
corresponding protocol queue.
With this change rt_ifp is now always the same as rt_ifa->ifa_ifp.
ok claudio@
|
| |
|
|
|
|
| |
directly. Also protect non mp-safe functions while at it.
ok mpi@.
|
| |
|
|
| |
ok mpi@, claudio@.
|
| |
|
|
|
|
| |
with MPLS packets.
ok mpi@, claudio@
|
| |
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
receiving interface in the packet header of every mbuf.
The interface pointer should now be retrieved when necessary with
if_get(). If a NULL pointer is returned by if_get(), the interface
has probably been destroy/removed and the mbuf should be freed.
Such mechanism will simplify garbage collection of mbufs and limit
problems with dangling ifp pointers.
Tested by jmatthew@ and krw@, discussed with many.
ok mikeb@, bluhm@, dlg@
|
| |
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
with niqueues.
this change is so big because there's a lot of code that takes
pointers to different input queues (eg, ether_input picks between
ipv4, ipv6, pppoe, arp, and mpls input queues) and falls through
to code to enqueue packets against the pointer. if i changed only
one of the input queues id have to add sepearate code paths, one
for ifqueues and one for niqueues in each of these places
by flipping all these input queues at once i can keep the currently
common code common.
testing by mpi@ sthen@ and rafael zalamena
ok mpi@ sthen@ claudio@ henning@
|
| | |
|
| |
|
|
| |
ok mikeb@, krw@, bluhm@, tedu@
|
| |
|
|
|
|
| |
functionnality instead of a mix of enable/disable.
ok bluhm@, jca@
|
| |
|
|
|
|
| |
kill the macro.
ok mikeb@, henning@
|
| | |
|
| |
|
|
|
|
| |
global variables to in6.h.
ok deraadt@
|
| |
|
|
|
|
|
| |
created a bunch of useless dependencies. Remove this implicit
inclusion and do an explicit #include <netinet6/in6_var.h> when it
is needed.
OK mpi@ henning@
|
| |
|
|
| |
ok lteo@, claudio@, bluhm@, henning@
|
| | |
|
| |
|
|
| |
ok beck@, mikeb@
|
| | |
|
| |
|
|
|
| |
diff from form@ who's too busy drinking vodka with pelmeni to commit this;
ok claudio
|
| |
|
|
|
| |
explicit NULL labels. IMPLNULL labels should never ever be seen on the
wire but we're realists. michele@ agrees
|
| |
|
|
|
|
| |
TTL exceeded errors for IPv4 are handled. This makes traceroute through
MPLS tunnels work (including RFC 4950 MPLS extension header).
"best to get it in." deraadt@
|
| | |
|
| |
|
|
|
|
|
|
| |
by a pop operation we need to forward the packet to the specified nexthop
as is. This is done by calling the interface output routine directly.
This is different to the local operation since that one injects the packets
into ip_input() via netisr.
OK michele
|
| |
|
|
|
| |
on each interface there is no need for yet another knob.
OK michele@
|
| |
|
|
|
| |
which relaxes the position of these labels. After discussion with
Christophe Fillot (cf (at) utc fr)
|
| |
|
|
|
|
| |
does no overflow checking and does not set the congestion flag. Protocol
input queues (inet, inet6, ...) should always use IF_INPUT_ENQUEUE().
OK henning@
|
| |
|
|
|
|
|
|
|
|
|
| |
ether_output() and later on other L2 output functions use a trick and over-
load the ifp->if_output() function pointer on MPLS enabled interfaces to
go through mpls_output() which will then call the link level output function.
By setting IFXF_MPLS on an interface the output pointers are switched.
This now allows to cleanup the MPLS input and output pathes and fix mpe(4)
so that the MPLS code now actually works for both P and PE systems.
Tested by myself and michele
(A custom kernel with MPLS and mpe enabled is still needed).
|
| |
|
|
|
|
|
|
| |
accept flags for report and nocloning. Move the rtableid into struct route
(with a minor twist for now) and make a few more codepathes rdomain aware.
Appart from the pf.c and route.c bits the diff is mostly mechanical.
More to come...
OK michele, henning
|
| |
|
|
|
|
|
|
| |
The former is responsible to handle packets destined to localhost,
the latter handles packet that have to be forwarded or the ones
originated from localhost.
"i like it" claudio@
|
| |
|
|
|
|
| |
the MPLS cloud and join ipv4/ipv6 stack.
ok claudio@ laurent@
|
| |
|
|
|
|
|
|
|
| |
joining the ipv4/ipv6 stack.
Get also rid of the dependency on mpe(4) for packets not belonging
to VPN traffic.
ok laurent@, ok and input claudio@
|
| |
|
|
|
|
|
|
| |
We can now act as edge node and allow ipv4 packets to
enter a Label Switched Path and not just forwarding
MPLS packets.
OK claudio@
|
| |
|
|
|
|
|
|
|
| |
Now it contains just the label as it must be.
This introduces a ugly hack in rtentry that will be removed
as soon as possible.
OK claudio@
|
| |
|
|
|
|
|
|
| |
Remove smpls_in_iface as labels must be per platform unique.
Remove smpls_out_iface. It is redudant and can be inferred from nexthop.
Move smpls_operation to rt_flags and temporary remove smpls_out_exp.
ok claudio@
|