2004-11-05 Phil Spagnolo <phillip.a.spagnolo@boeing.com>

       * ospf6_flood.c:  When adding a thread to retransmit an lsa after
    rxmt_interval, the ospf6d used thread_add_event().  However,
    thread_add_event() executes immediately and ignores the delay parameter.
    This was causing unnecesary unicast LSU sends.  Adding the thread has
    been changed to use thread_add_timer() which will expire after delay.
    After the fix results appear consistent with expectation.

diff --git a/ospf6d/ChangeLog b/ospf6d/ChangeLog
index 5e16c702cb74c66a924e49b01a4750ad2b451228..9c884245920c67526f677a4d245e19f03b1dcfeb 100644 (file)
--- a/ospf6d/ChangeLog
+++ b/ospf6d/ChangeLog
@@ -1,3 +1,11 @@
+2004-11-05 Phil Spagnolo <phillip.a.spagnolo@boeing.com>
+       * ospf6_flood.c:  When adding a thread to retransmit an lsa after 
+    rxmt_interval, the ospf6d used thread_add_event().  However, 
+    thread_add_event() executes immediately and ignores the delay parameter.  
+    This was causing unnecesary unicast LSU sends.  Adding the thread has 
+    been changed to use thread_add_timer() which will expire after delay.  
+    After the fix results appear consistent with expectation.
+
 2004-10-12 Hasso Tepper <hasso at quagga.net>
 
        * ospf6_snmp.c: Remove defaults used to initialize smux connection to

diff --git a/ospf6d/ospf6_flood.c b/ospf6d/ospf6_flood.c
index acfe28ab926c5848912c2b3c0dc32438355cfad7..3c9987fe3e679fce36e364fcc499b8ca8e314725 100644 (file)
--- a/ospf6d/ospf6_flood.c
+++ b/ospf6d/ospf6_flood.c
@@ -335,7 +335,7 @@ ospf6_flood_interface (struct ospf6_neighbor *from,
       ospf6_lsdb_add (ospf6_lsa_copy (lsa), on->retrans_list);
       if (on->thread_send_lsupdate == NULL)
         on->thread_send_lsupdate =
-          thread_add_event (master, ospf6_lsupdate_send_neighbor,
+          thread_add_timer (master, ospf6_lsupdate_send_neighbor,
                             on, on->ospf6_if->rxmt_interval);
       retrans_added++;
     }