This memory type allocator is used in both bgpd and library part.
to ease identification of the memory type, the allocator is put in
lib folder, instead of zebra folder.
Signed-off-by: Philippe Guibert
---
bgpd/bgp_nexthop.c | 2 +-
lib/memtypes.c | 2 +-
lib/nexthop.c | 4 ++--
3 files
From: David Lamparter
bgpd, ospf6d, isisd and some tests were reusing MTYPEs defined in the
library for its own use. This is bad practice and will break with the
later commit making the library's MTYPEs static.
Signed-off-by: David Lamparter
Signed-off-by: Philippe Guibert
---
isisd/isis_mem
because some compilers do not use this feature, it is provided an
API that forces the user to call its mem group and memtypes init.
Signed-off-by: Philippe Guibert
---
lib/memory.h | 24 ++--
1 file changed, 2 insertions(+), 22 deletions(-)
diff --git a/lib/memory.h b/lib/me
Hi quagga-dev,
This is a rework from [quagga-dev 14733], made by David Lamparter.
Among the changes done with previous version, the "contructor" usage
has been removed. This then should make possible to continue using
quagga on all platforms not supporting this compiler enhancement.
This patchset
From: David Lamparter
The following commit will recreate memory.[ch].
Signed-off-by: David Lamparter
Signed-off-by: Philippe Guibert
---
lib/Makefile.am | 6 +-
lib/memory.c | 484 ---
lib/memory.h | 98 +--
lib/memory_vt
On some specific cases, the user may wish not to export its memory
type and group. The user can then declare and initialises this
memory type, provided that he ensures also the initialisation. The
goal of that commit is initialising in quagga the static mtypes.
Signed-off-by: Philippe Guibert
---
From: David Lamparter
This is a rather large mechanical commit that splits up the memory types
defined in lib/memtypes.c and distributes them into *_memory.[ch] files
in the individual daemons.
The zebra change is slightly annoying because there is no nice place to
put the #include "zebra_memory.
This adapts the dump-at-exit handler and removes the old leftover code.
(Note the text in log_memtype_stderr was actually incorrect as the only
caller in bgpd cleans up configuration before calling it, i.e. any
remaining allocations are missing-cleanup bugs.)
Signed-off-by: David Lamparter
Signe
From: David Lamparter
Move over to the new allocation counting added in the previous commit.
(This is mostly mechanical.)
Signed-off-by: David Lamparter
Signed-off-by: Philippe Guibert
---
bgpd/bgp_main.c | 1 +
bgpd/bgp_vty.c | 1 +
isisd/isis_main.c |
From: David Lamparter
This rewrites Quagga's memory per-type allocation counting, without
using a fixed global list of types. Instead, source files can declare
memory types which get handled through constructor functions called by
the dynamic linker during startup.
Signed-off-by: David Lamparte
This commit centralises the memory group and memory types declarations
into one single place for each daemon/library. In this case, files
affected are /_memory.[ch].
In order to be used, the memory groups and memory types have to be
manually initialised. This is why the daemons are initialising the
This commit introduces a bash script called from Makefile from each
lib/daemon. Upon call, this script takes as input a txt file
describing the various memory group and type that need to be defined,
initialised and declared. The output of this script is 2 .c and .h
files used for compilation within
This work is enhancing RFC3107 encoding format for label in BGP NLRI messages.
This applies for instance to VPNv4 address family.
It is to be noted that this commit does not bring any API enhancement regarding
VTY configuration. However, the mecanic is ready to receive BGP updates with
multiple l
From: David Lamparter
API enhancement to be able to write and read 3 byte series in
stream structures.
Signed-off-by: David Lamparter
Signed-off-by: Philippe Guibert
---
lib/stream.c | 77
lib/stream.h | 4
2 files changed, 81
bgp NLRI has a specific format for label, terminating with bottom
of stack bit. This value is updated in all BGP attribute tests. A
specific test has been added to check for a list of labels contained
in a single NLRI.
Signed-off-by: Philippe Guibert
---
tests/bgp_mp_attr_test.c | 87 +
From: David Lamparter
Support for multiple label support handling in NLRI messages.
Ability to receive and sent BGP updates with NLRI containing multiple
labels. The commit follows label encoding rules, contained in RFC3107.
Signed-off-by: David Lamparter
Signed-off-by: Christian Franke
Signed
This commit introduces the BGP VRF configuration, and BGP VRF RIB
table.
It includes the ability for a BGP to configure its own route
distinguisher ( aka VRF). New vty commands introduced:
(config-router)# vrf rd
This structure permits configuring import and export route targets,
which is define
If the user preconfigures a vrf with import and export rules, before
creating a new network on vpnv4 address, then BGP update emitted will
contain the exported rts mentioned by the vrf export rule.
Signed-off-by: Philippe Guibert
---
bgpd/bgp_route.c | 11 +++
bgpd/bgp_route.h | 2 ++
2
This series of patches introduces the ability for a BGP router,
to configure a route distinguisher and associate import and
export route targets.
This permits for a BGP update to be sent with BGP extended communities
associated to the exported route targets of the RD.
This permits for a BGP update
Continous Integration Result: SUCCESSFUL
Congratulations, this patch passed basic tests
Tested-by: NetDEF CI System
This is an EXPERIMENTAL automated CI system.
For questions and feedback, feel free to email
Martin Winter .
Patches applied :
Patchwork 2089: http://patchwork.quagga.net/patch/
I don't understand the subject. This isn't restoring anything, but
adding a missing check from the patch to make NHT work without zebra? Or
did I somehow drop that chunk from the original patch?
Also, what about the tests on NEXTHOP_VALID in evaluate_paths? Do they
need to take bgp_zebra_num_c
Continous Integration Result: FAILED
See below for issues.
This is an EXPERIMENTAL automated CI system.
For questions and feedback, feel free to email
Martin Winter .
Patches applied :
Patchwork 2093: http://patchwork.quagga.net/patch/2093
[quagga-dev,16234,v3,01/11] *: memory type NEXTH
On 10/11/2016 7:04 AM, Paul Jakma wrote:
> I don't understand the subject. This isn't restoring anything, but
> adding a missing check from the patch to make NHT work without zebra? Or
> did I somehow drop that chunk from the original patch?
Yes, I patched a different spot - that is the one tha
Continous Integration Result: SUCCESSFUL
Congratulations, this patch passed basic tests
Tested-by: NetDEF CI System
This is an EXPERIMENTAL automated CI system.
For questions and feedback, feel free to email
Martin Winter .
Patches applied :
Patchwork 2103: http://patchwork.quagga.net/patch/
Continous Integration Result: SUCCESSFUL
Congratulations, this patch passed basic tests
Tested-by: NetDEF CI System
This is an EXPERIMENTAL automated CI system.
For questions and feedback, feel free to email
Martin Winter .
Patches applied :
Patchwork 2108: http://patchwork.quagga.net/patch/
---
bgpd/bgp_nht.c | 16 +++-
1 file changed, 11 insertions(+), 5 deletions(-)
diff --git a/bgpd/bgp_nht.c b/bgpd/bgp_nht.c
index b5d830e..171cb20 100644
--- a/bgpd/bgp_nht.c
+++ b/bgpd/bgp_nht.c
@@ -52,6 +52,13 @@ static int make_prefix(int afi, struct bgp_info *ri, struct
prefix *p
Continous Integration Result: SUCCESSFUL
Congratulations, this patch passed basic tests
Tested-by: NetDEF CI System
This is an EXPERIMENTAL automated CI system.
For questions and feedback, feel free to email
Martin Winter .
Patches applied :
Patchwork 2109: http://patchwork.quagga.net/patch/
> - Systematic way to collect patches could be:
>
> * Patchwork (it has issues over longer time spans, but prob ok on
> short ones)
>
> * Bugzilla
>
> * automated list crawler and applier (I'd prefer not to put closed
> tools in the middle of this though)
>
> Nothing is perfect
28 matches
Mail list logo