Login | Register For Free | Help
Search for: (Advanced)

Mailing List Archive: Quagga: Dev

[PATCH] ospf6d: always remove the result of a previous SPF calculation

 

 

Quagga dev RSS feed   Index | Next | Previous | View Threaded


thomas.goff at boeing

Aug 19, 2011, 10:27 AM

Post #1 of 4 (471 views)
Permalink
[PATCH] ospf6d: always remove the result of a previous SPF calculation

This is needed to avoid stale routes in some cases; the regression was
introduced by commit 1d19234e.

* ospf6_spf.c: (ospf6_spf_calculation) Call ospf6_spf_table_finish()
before possibly returning if no router-LSA is found for the root of
the SPF tree.

Signed-off-by: Tom Goff <thomas.goff [at] boeing>
---
ospf6d/ospf6_spf.c | 4 ++--
1 files changed, 2 insertions(+), 2 deletions(-)

diff --git a/ospf6d/ospf6_spf.c b/ospf6d/ospf6_spf.c
index cb54961..f2bf0b6 100644
--- a/ospf6d/ospf6_spf.c
+++ b/ospf6d/ospf6_spf.c
@@ -392,6 +392,8 @@ ospf6_spf_calculation (u_int32_t router_id,
caddr_t lsdesc;
struct ospf6_lsa *lsa;

+ ospf6_spf_table_finish (result_table);
+
/* Install the calculating router itself as the root of the SPF tree */
/* construct root vertex */
lsa = ospf6_lsdb_lookup (htons (OSPF6_LSTYPE_ROUTER), htonl (0),
@@ -403,8 +405,6 @@ ospf6_spf_calculation (u_int32_t router_id,
candidate_list = pqueue_create ();
candidate_list->cmp = ospf6_vertex_cmp;

- ospf6_spf_table_finish (result_table);
-
root = ospf6_vertex_create (lsa);
root->area = oa;
root->cost = 0;
--
1.7.0.4

_______________________________________________
Quagga-dev mailing list
Quagga-dev [at] lists
http://lists.quagga.net/mailman/listinfo/quagga-dev


infrastation at yandex

Dec 13, 2011, 1:47 PM

Post #2 of 4 (362 views)
Permalink
Re: [PATCH] ospf6d: always remove the result of a previous SPF calculation [In reply to]

19.08.2011, 21:27, "Tom Goff" <thomas.goff [at] boeing>:
> This is needed to avoid stale routes in some cases; the regression was
> introduced by commit 1d19234e.
>
> * ospf6_spf.c: (ospf6_spf_calculation) Call ospf6_spf_table_finish()
> ššbefore possibly returning if no router-LSA is found for the root of
> ššthe SPF tree.

Hello.

1d19234e is in master; I presume, this change should also be there. Is this the final revision of the fix?

--
Denis Ovsienko
_______________________________________________
Quagga-dev mailing list
Quagga-dev [at] lists
http://lists.quagga.net/mailman/listinfo/quagga-dev


Thomas.Goff at boeing

Dec 13, 2011, 6:58 PM

Post #3 of 4 (360 views)
Permalink
Re: [PATCH] ospf6d: always remove the result of a previous SPF calculation [In reply to]

> -----Original Message-----
> From: Denis Ovsienko [mailto:infrastation [at] yandex]
> Sent: Tuesday, December 13, 2011 1:48 PM
> To: Goff, Thomas
> Cc: quagga-dev [at] lists
> Subject: Re: [quagga-dev 8768] [PATCH] ospf6d: always remove the result
> of a previous SPF calculation
>
> 19.08.2011, 21:27, "Tom Goff" <thomas.goff [at] boeing>:
> > This is needed to avoid stale routes in some cases; the regression
> was
> > introduced by commit 1d19234e.
> >
> > * ospf6_spf.c: (ospf6_spf_calculation) Call ospf6_spf_table_finish()
> >   before possibly returning if no router-LSA is found for the root of
> >   the SPF tree.
>
> Hello.
>
> 1d19234e is in master; I presume, this change should also be there. Is
> this the final revision of the fix?
>
> --
> Denis Ovsienko

Denis,

Yes, the patch I submitted should be included in the master branch and is the final version. It fixes a regression I inadvertently introduced with 1d19234e.

Thanks for looking at this.

Tom


_______________________________________________
Quagga-dev mailing list
Quagga-dev [at] lists
http://lists.quagga.net/mailman/listinfo/quagga-dev


infrastation at yandex

Dec 14, 2011, 2:16 AM

Post #4 of 4 (363 views)
Permalink
Re: [PATCH] ospf6d: always remove the result of a previous SPF calculation [In reply to]

> Yes, the patch I submitted should be included in the master branch and is the final version. šIt fixes a regression I inadvertently introduced with 1d19234e.
>
> Thanks for looking at this.
>
> ššTom

The fix is committed, thank you.

--
Denis Ovsienko
_______________________________________________
Quagga-dev mailing list
Quagga-dev [at] lists
http://lists.quagga.net/mailman/listinfo/quagga-dev

Quagga dev RSS feed   Index | Next | Previous | View Threaded
 
 


Interested in having your list archived? Contact Gossamer Threads
 
  Web Applications & Managed Hosting Powered by Gossamer Threads Inc.