<div dir="ltr"><dt><font face="arial, helvetica, sans-serif" style="color:rgb(0,0,0)">For tracing function entry and exit GCC has</font><font face="monospace" style="color:rgb(0,0,0);font-size:medium"> </font><font face="monospace, monospace" style="color:rgb(0,0,0)">-finstrument-functions.</font></dt></div><div class="gmail_extra"><br><div class="gmail_quote">On Sun, Dec 4, 2016 at 11:02 AM, Lou Berger <span dir="ltr"><<a href="mailto:lberger@labn.net" target="_blank">lberger@labn.net</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Donald,<br>
<br>
<br>
<br>
On 12/4/2016 9:16 AM, Donald Sharp wrote:<br>
> Look, I agree debugs are needed but most of what I removed were<br>
> unguarded debugs.<br>
  Please elaborate on what a "guarded debug" looks like to you -- do you<br>
mean controlled by a debug ... statement, is if something that is<br>
conditionally compiled?  Please point to an example of what you'd like<br>
to see.  The former is certainly a reasonable change.<br>
> These will simply just fill up the debug log with<br>
> useless cruft that makes it harder to actually debug issues because of<br>
> unnecessary data.<br>
Unnecessary to whom? If I'm debugging something related, you've made it<br>
impossible and also made it a lot of work to readd these statements.<br>
<br>
>  If you need these debugs then submit a patch so<br>
> that they can be turned on appropriately.<br>
We though the code was appropriate, please define what you think is<br>
appropriate.  Perhaps there's something to be added to HACKING.md to<br>
capture this view.<br>
<br>
>  I submit though that debugs<br>
> that say 'entering function', 'exiting function' don't do allot to<br>
> help you figure issues out.<br>
<br>
While that is certainly one perspective, it isn't one shared by the<br>
developer of that code.<br>
<br>
Lou<br>
<br>
><br>
> donald<br>
<div class="HOEnZb"><div class="h5">><br>
> On Sun, Dec 4, 2016 at 7:43 AM, Lou Berger <<a href="mailto:lberger@labn.net">lberger@labn.net</a>> wrote:<br>
>> Donald,<br>
>><br>
>> Huh?  Debug logs are there for when debugging is needed - and that's what<br>
>> these are all about. Simply removing them is wrong and make others lives<br>
>> much more difficult (ours) when debugging.<br>
>><br>
>> What logs are filling? (I.e Who normally runs with debug logging?)<br>
>><br>
>> What type of "guards" would you like to see?  We/you certain can add a<br>
>> debug_bgp_vnc flag if that's what you're thinking.<br>
>><br>
>> I really object to the summary removal of these debug log statements.<br>
>><br>
>> Lou<br>
>><br>
>><br>
>><br>
>> On December 3, 2016 10:40:12 PM Donald Sharp <<a href="mailto:sharpd@cumulusnetworks.com">sharpd@cumulusnetworks.com</a>><br>
>> wrote:<br>
>><br>
>>> Remove allot of unguarded debugs in bgp<br>
>>> that are causing the log file to be filed<br>
>>> up.<br>
>>><br>
>>> Signed-off-by: Donald Sharp <<a href="mailto:sharpd@cumulusnetworks.com">sharpd@cumulusnetworks.com</a>><br>
>>> ---<br>
>>>  bgpd/bgp_routemap.c         |  2 --<br>
>>>  bgpd/rfapi/bgp_rfapi_cfg.c  |  7 -------<br>
>>>  bgpd/rfapi/rfapi_import.c   | 24 -----------------------<br>
>>>  bgpd/rfapi/rfapi_monitor.c  |  4 ----<br>
>>>  bgpd/rfapi/rfapi_rib.c      | 10 ----------<br>
>>>  bgpd/rfapi/rfapi_vty.c      | 46<br>
>>> ------------------------------<wbr>---------------<br>
>>>  bgpd/rfapi/vnc_export_bgp.c | 14 --------------<br>
>>>  bgpd/rfapi/vnc_import_bgp.c |  2 --<br>
>>>  bgpd/rfapi/vnc_zebra.c      | 11 -----------<br>
>>>  9 files changed, 120 deletions(-)<br>
>>><br>
>>> diff --git a/bgpd/bgp_routemap.c b/bgpd/bgp_routemap.c<br>
>>> index ea42cb5..88a169a 100644<br>
>>> --- a/bgpd/bgp_routemap.c<br>
>>> +++ b/bgpd/bgp_routemap.c<br>
>>> @@ -2853,7 +2853,6 @@ bgp_route_map_process_update_<wbr>cb (char *rmap_name)<br>
>>>      bgp_route_map_process_update(<wbr>bgp, rmap_name, 1);<br>
>>><br>
>>>  #if ENABLE_BGP_VNC<br>
>>> -  zlog_debug("%s: calling vnc_routemap_update", __func__);<br>
>>>    vnc_routemap_update(bgp, __func__);<br>
>>>  #endif<br>
>>>    return 0;<br>
>>> @@ -2893,7 +2892,6 @@ bgp_route_map_mark_update (const char *rmap_name)<br>
>>>            for (ALL_LIST_ELEMENTS (bm->bgp, node, nnode, bgp))<br>
>>>              bgp_route_map_process_update(<wbr>bgp, rmap_name, 0);<br>
>>>   #if ENABLE_BGP_VNC<br>
>>> -          zlog_debug("%s: calling vnc_routemap_update", __func__);<br>
>>>            vnc_routemap_update(bgp, __func__);<br>
>>>  #endif<br>
>>>          }<br>
>>> diff --git a/bgpd/rfapi/bgp_rfapi_cfg.c b/bgpd/rfapi/bgp_rfapi_cfg.c<br>
>>> index d064c50..a0d80d6 100644<br>
>>> --- a/bgpd/rfapi/bgp_rfapi_cfg.c<br>
>>> +++ b/bgpd/rfapi/bgp_rfapi_cfg.c<br>
>>> @@ -774,7 +774,6 @@ vnc_redistribute_prechange (struct bgp *bgp)<br>
>>>    afi_t afi;<br>
>>>    int type;<br>
>>><br>
>>> -  zlog_debug ("%s: entry", __func__);<br>
>>>    memset (redist_was_enabled, 0, sizeof (redist_was_enabled));<br>
>>><br>
>>>    /*<br>
>>> @@ -794,7 +793,6 @@ vnc_redistribute_prechange (struct bgp *bgp)<br>
>>>              }<br>
>>>          }<br>
>>>      }<br>
>>> -  zlog_debug ("%s: return", __func__);<br>
>>>  }<br>
>>><br>
>>>  static void<br>
>>> @@ -803,7 +801,6 @@ vnc_redistribute_postchange (struct bgp *bgp)<br>
>>>    afi_t afi;<br>
>>>    int type;<br>
>>><br>
>>> -  zlog_debug ("%s: entry", __func__);<br>
>>>    /*<br>
>>>     * If we turned off redistribution above, turn it back on. Doing so<br>
>>>     * will tell zebra to resend the routes to us<br>
>>> @@ -818,7 +815,6 @@ vnc_redistribute_postchange (struct bgp *bgp)<br>
>>>              }<br>
>>>          }<br>
>>>      }<br>
>>> -  zlog_debug ("%s: return", __func__);<br>
>>>  }<br>
>>><br>
>>>  DEFUN (vnc_redistribute_rh_roo_<wbr>localadmin,<br>
>>> @@ -2498,8 +2494,6 @@ vnc_routemap_update (struct bgp *bgp, const char<br>
>>> *unused)<br>
>>>    struct rfapi_cfg *hc;<br>
>>>    int i;<br>
>>><br>
>>> -  zlog_debug ("%s(arg=%s)", __func__, unused);<br>
>>> -<br>
>>>    if (!bgp)<br>
>>>      {<br>
>>>        zlog_debug ("%s: No BGP process is configured", __func__);<br>
>>> @@ -2573,7 +2567,6 @@ vnc_routemap_update (struct bgp *bgp, const char<br>
>>> *unused)<br>
>>>    vnc_redistribute_prechange (bgp);<br>
>>>    vnc_redistribute_postchange (bgp);<br>
>>><br>
>>> -  zlog_debug ("%s done", __func__);<br>
>>>  }<br>
>>><br>
>>>  static void<br>
>>> diff --git a/bgpd/rfapi/rfapi_import.c b/bgpd/rfapi/rfapi_import.c<br>
>>> index 77da4f9..54643ee 100644<br>
>>> --- a/bgpd/rfapi/rfapi_import.c<br>
>>> +++ b/bgpd/rfapi/rfapi_import.c<br>
>>> @@ -1349,10 +1349,6 @@ rfapiRouteInfo2NextHopEntry (<br>
>>>    struct rfapi_next_hop_entry *new;<br>
>>>    int have_vnc_tunnel_un = 0;<br>
>>><br>
>>> -#if DEBUG_ENCAP_MONITOR<br>
>>> -  zlog_debug ("%s: entry, bi %p, rn %p", __func__, bi, rn);<br>
>>> -#endif<br>
>>> -<br>
>>>    new = XCALLOC (MTYPE_RFAPI_NEXTHOP, sizeof (struct<br>
>>> rfapi_next_hop_entry));<br>
>>>    assert (new);<br>
>>><br>
>>> @@ -1500,11 +1496,6 @@ rfapiRouteInfo2NextHopEntry (<br>
>>><br>
>>>            new->un_options = rfapi_encap_tlv_to_un_option (bi->attr);<br>
>>><br>
>>> -#if DEBUG_ENCAP_MONITOR<br>
>>> -          zlog_debug ("%s: line %d: have_vnc_tunnel_un=%d",<br>
>>> -                      __func__, __LINE__, have_vnc_tunnel_un);<br>
>>> -#endif<br>
>>> -<br>
>>>            if (!have_vnc_tunnel_un && bi && bi->extra)<br>
>>>              {<br>
>>>                /*<br>
>>> @@ -1960,7 +1951,6 @@ rfapiRouteTable2NextHopList (<br>
>>>          }<br>
>>>      }<br>
>>><br>
>>> -  zlog_debug ("%s: returning %d routes", __func__, count);<br>
>>>    return biglist;<br>
>>>  }<br>
>>><br>
>>> @@ -2055,7 +2045,6 @@ rfapiEthRouteTable2NextHopList (<br>
>>>          }<br>
>>>      }<br>
>>><br>
>>> -  zlog_debug ("%s: returning %d routes", __func__, count);<br>
>>>    return biglist;<br>
>>>  }<br>
>>><br>
>>> @@ -3117,10 +3106,6 @@ rfapiBgpInfoFilteredImportEnca<wbr>p (<br>
>>>        break;<br>
>>>      }<br>
>>><br>
>>> -  zlog_debug ("%s: entry: %s: prefix %s/%d", __func__,<br>
>>> -              action_str,<br>
>>> -              inet_ntop (p->family, &p->u.prefix, buf, BUFSIZ),<br>
>>> p->prefixlen);<br>
>>> -<br>
>>>    memset (&p_firstbi_old, 0, sizeof (p_firstbi_old));<br>
>>>    memset (&p_firstbi_new, 0, sizeof (p_firstbi_new));<br>
>>><br>
>>> @@ -3598,12 +3583,6 @@ rfapiBgpInfoFilteredImportVPN (<br>
>>>    if (import_table == bgp->rfapi->it_ce)<br>
>>>      is_it_ce = 1;<br>
>>><br>
>>> -  zlog_debug ("%s: entry: %s%s: prefix %s/%d: it %p, afi %s", __func__,<br>
>>> -              (is_it_ce ? "CE-IT " : ""),<br>
>>> -              action_str,<br>
>>> -              rfapi_ntop (p->family, &p->u.prefix, buf, BUFSIZ),<br>
>>> -              p->prefixlen, import_table, afi2str (afi));<br>
>>> -<br>
>>>    VNC_ITRCCK;<br>
>>><br>
>>>    /*<br>
>>> @@ -4695,9 +4674,6 @@ rfapiDeleteRemotePrefixesIt (<br>
>>>          buf_pfx[0] = '*';<br>
>>>          buf_pfx[1] = 0;<br>
>>>        }<br>
>>> -<br>
>>> -    zlog_debug ("%s: entry, p=%s, delete_active=%d, delete_holddown=%d",<br>
>>> -                __func__, buf_pfx, delete_active, delete_holddown);<br>
>>>    }<br>
>>>  #endif<br>
>>><br>
>>> diff --git a/bgpd/rfapi/rfapi_monitor.c b/bgpd/rfapi/rfapi_monitor.c<br>
>>> index 216b45e..8e33530 100644<br>
>>> --- a/bgpd/rfapi/rfapi_monitor.c<br>
>>> +++ b/bgpd/rfapi/rfapi_monitor.c<br>
>>> @@ -707,8 +707,6 @@ rfapiMonitorDelHd (struct rfapi_descriptor *rfd)<br>
>>>    struct bgp *bgp;<br>
>>>    int count = 0;<br>
>>><br>
>>> -  zlog_debug ("%s: entry rfd=%p", __func__, rfd);<br>
>>> -<br>
>>>    bgp = bgp_get_default ();<br>
>>><br>
>>>    if (rfd->mon)<br>
>>> @@ -1511,8 +1509,6 @@ rfapiMonitorEthDel (<br>
>>>    struct rfapi_monitor_eth mon_buf;<br>
>>>    int rc;<br>
>>><br>
>>> -  zlog_debug ("%s: entry rfd=%p", __func__, rfd);<br>
>>> -<br>
>>>    assert (rfd->mon_eth);<br>
>>><br>
>>>    memset ((void *) &mon_buf, 0, sizeof (mon_buf));<br>
>>> diff --git a/bgpd/rfapi/rfapi_rib.c b/bgpd/rfapi/rfapi_rib.c<br>
>>> index 896b5f5..40d00ca 100644<br>
>>> --- a/bgpd/rfapi/rfapi_rib.c<br>
>>> +++ b/bgpd/rfapi/rfapi_rib.c<br>
>>> @@ -1602,8 +1602,6 @@ rfapiRibUpdatePendingNode (<br>
>>>    int                          count = 0;<br>
>>>    char                         buf[BUFSIZ];<br>
>>><br>
>>> -  zlog_debug ("%s: entry", __func__);<br>
>>> -<br>
>>>    if (CHECK_FLAG (bgp->rfapi_cfg->flags,<br>
>>> BGP_VNC_CONFIG_CALLBACK_<wbr>DISABLE))<br>
>>>      return;<br>
>>><br>
>>> @@ -1619,9 +1617,6 @@ rfapiRibUpdatePendingNode (<br>
>>>    pn = route_node_get (rfd->rib_pending[afi], prefix);<br>
>>>    assert (pn);<br>
>>><br>
>>> -  zlog_debug ("%s: pn->info=%p, pn->aggregate=%p", __func__, pn->info,<br>
>>> -              pn->aggregate);<br>
>>> -<br>
>>>    if (pn->aggregate)<br>
>>>      {<br>
>>>        /*<br>
>>> @@ -2155,8 +2150,6 @@ rfapiRibPendingDeleteRoute (<br>
>>>    char buf[BUFSIZ];<br>
>>><br>
>>>    prefix2str (&it_node->p, buf, BUFSIZ);<br>
>>> -  zlog_debug ("%s: entry, it=%p, afi=%d, it_node=%p, pfx=%s",<br>
>>> -              __func__, it, afi, it_node, buf);<br>
>>><br>
>>>    if (AFI_ETHER == afi)<br>
>>>      {<br>
>>> @@ -2175,9 +2168,6 @@ rfapiRibPendingDeleteRoute (<br>
>>>         */<br>
>>>        if ((sl = RFAPI_MONITOR_ETH (it_node)))<br>
>>>          {<br>
>>> -<br>
>>> -          zlog_debug ("%s: route-specific skiplist: %p", __func__, sl);<br>
>>> -<br>
>>>            for (cursor = NULL, rc =<br>
>>>                 skiplist_next (sl, NULL, (void **) &m, (void **) &cursor);<br>
>>> !rc;<br>
>>>                 rc = skiplist_next (sl, NULL, (void **) &m, (void **)<br>
>>> &cursor))<br>
>>> diff --git a/bgpd/rfapi/rfapi_vty.c b/bgpd/rfapi/rfapi_vty.c<br>
>>> index c198564..1f25c72 100644<br>
>>> --- a/bgpd/rfapi/rfapi_vty.c<br>
>>> +++ b/bgpd/rfapi/rfapi_vty.c<br>
>>> @@ -3283,10 +3283,6 @@ rfapiDeleteLocalPrefixes (struct<br>
>>> rfapi_local_reg_delete_arg *cda)<br>
>>>    struct rfapi_next_hop_entry *tail = NULL;<br>
>>>    struct rfapi_cfg *rfapi_cfg;<br>
>>><br>
>>> -#if DEBUG_L2_EXTRA<br>
>>> -    zlog_debug ("%s: entry", __func__);<br>
>>> -#endif<br>
>>> -<br>
>>>    if (!bgp_default)<br>
>>>        return ENXIO;<br>
>>><br>
>>> @@ -3305,10 +3301,6 @@ rfapiDeleteLocalPrefixes (struct<br>
>>> rfapi_local_reg_delete_arg *cda)<br>
>>>        rfapiQprefix2Rprefix (pPrefix, &rprefix);<br>
>>>      }<br>
>>><br>
>>> -#if DEBUG_L2_EXTRA<br>
>>> -  zlog_debug ("%s: starting descriptor loop", __func__);<br>
>>> -#endif<br>
>>> -<br>
>>>    for (ALL_LIST_ELEMENTS_RO (&h->descriptors, node, rfd))<br>
>>>      {<br>
>>>        struct rfapi_adb *adb;<br>
>>> @@ -3317,10 +3309,6 @@ rfapiDeleteLocalPrefixes (struct<br>
>>> rfapi_local_reg_delete_arg *cda)<br>
>>>        struct nve_addr ha;<br>
>>>        struct nve_addr *hap;<br>
>>><br>
>>> -#if DEBUG_L2_EXTRA<br>
>>> -      zlog_debug ("%s: rfd=%p", __func__, rfd);<br>
>>> -#endif<br>
>>> -<br>
>>>        /*<br>
>>>         * match un, vn addresses of NVEs<br>
>>>         */<br>
>>> @@ -3329,10 +3317,6 @@ rfapiDeleteLocalPrefixes (struct<br>
>>> rfapi_local_reg_delete_arg *cda)<br>
>>>        if (pVn && (rfapi_ip_addr_cmp (pVn, &rfd->vn_addr)))<br>
>>>          continue;<br>
>>><br>
>>> -#if DEBUG_L2_EXTRA<br>
>>> -      zlog_debug ("%s: un, vn match", __func__);<br>
>>> -#endif<br>
>>> -<br>
>>>        /*<br>
>>>         * match prefix<br>
>>>         */<br>
>>> @@ -3372,10 +3356,6 @@ rfapiDeleteLocalPrefixes (struct<br>
>>> rfapi_local_reg_delete_arg *cda)<br>
>>>                {<br>
>>>                  if (!prefix_same (pPrefix, &adb->prefix_ip))<br>
>>>                    {<br>
>>> -#if DEBUG_L2_EXTRA<br>
>>> -                    zlog_debug ("%s: adb=%p, prefix doesn't match,<br>
>>> skipping",<br>
>>> -                                __func__, adb);<br>
>>> -#endif<br>
>>>                      continue;<br>
>>>                    }<br>
>>>                }<br>
>>> @@ -3385,10 +3365,6 @@ rfapiDeleteLocalPrefixes (struct<br>
>>> rfapi_local_reg_delete_arg *cda)<br>
>>>                      (cda->l2o.o.macaddr.octet,<br>
>>>                       adb->prefix_eth.u.prefix_eth.<wbr>octet, ETHER_ADDR_LEN))<br>
>>>                    {<br>
>>> -#if DEBUG_L2_EXTRA<br>
>>> -                    zlog_debug ("%s: adb=%p, macaddr doesn't match,<br>
>>> skipping",<br>
>>> -                                __func__, adb);<br>
>>> -#endif<br>
>>>                      continue;<br>
>>>                    }<br>
>>>                }<br>
>>> @@ -3397,19 +3373,10 @@ rfapiDeleteLocalPrefixes (struct<br>
>>> rfapi_local_reg_delete_arg *cda)<br>
>>>                {<br>
>>>                  if (cda->l2o.o.logical_net_id != adb->l2o.logical_net_id)<br>
>>>                    {<br>
>>> -#if DEBUG_L2_EXTRA<br>
>>> -                    zlog_debug ("%s: adb=%p, LNI doesn't match,<br>
>>> skipping",<br>
>>> -                                __func__, adb);<br>
>>> -#endif<br>
>>>                      continue;<br>
>>>                    }<br>
>>>                }<br>
>>><br>
>>> -#if DEBUG_L2_EXTRA<br>
>>> -            zlog_debug ("%s: ipN adding adb %p to delete list", __func__,<br>
>>> -                        adb);<br>
>>> -#endif<br>
>>> -<br>
>>>              listnode_add (adb_delete_list, adb);<br>
>>>            }<br>
>>><br>
>>> @@ -3454,10 +3421,6 @@ rfapiDeleteLocalPrefixes (struct<br>
>>> rfapi_local_reg_delete_arg *cda)<br>
>>>                  pVn = NULL;<br>
>>>                }<br>
>>><br>
>>> -#if DEBUG_L2_EXTRA<br>
>>> -            zlog_debug ("%s: ipN killing reg from adb %p ", __func__,<br>
>>> adb);<br>
>>> -#endif<br>
>>> -<br>
>>>              rc = rfapi_register (rfd, &rp, 0, NULL, pVn,<br>
>>> RFAPI_REGISTER_KILL);<br>
>>>              if (!rc)<br>
>>>                {<br>
>>> @@ -3508,10 +3471,6 @@ rfapiDeleteLocalPrefixes (struct<br>
>>> rfapi_local_reg_delete_arg *cda)<br>
>>>                          continue;<br>
>>>                        }<br>
>>>                    }<br>
>>> -#if DEBUG_L2_EXTRA<br>
>>> -                zlog_debug ("%s: ip0 adding adb %p to delete list",<br>
>>> -                            __func__, adb);<br>
>>> -#endif<br>
>>>                  listnode_add (adb_delete_list, adb);<br>
>>>                }<br>
>>><br>
>>> @@ -3527,11 +3486,6 @@ rfapiDeleteLocalPrefixes (struct<br>
>>> rfapi_local_reg_delete_arg *cda)<br>
>>>                  vn.type = RFAPI_VN_OPTION_TYPE_L2ADDR;<br>
>>>                  vn.v.l2addr = adb->l2o;<br>
>>><br>
>>> -#if DEBUG_L2_EXTRA<br>
>>> -                zlog_debug ("%s: ip0 killing reg from adb %p ",<br>
>>> -                            __func__, adb);<br>
>>> -#endif<br>
>>> -<br>
>>>                  rc = rfapi_register (rfd, &rp, 0, NULL, &vn,<br>
>>>                                       RFAPI_REGISTER_KILL);<br>
>>>                  if (!rc)<br>
>>> diff --git a/bgpd/rfapi/vnc_export_bgp.c b/bgpd/rfapi/vnc_export_bgp.c<br>
>>> index 6434c37..f1a9ecb 100644<br>
>>> --- a/bgpd/rfapi/vnc_export_bgp.c<br>
>>> +++ b/bgpd/rfapi/vnc_export_bgp.c<br>
>>> @@ -437,8 +437,6 @@ vnc_direct_bgp_vpn_enable_ce (struct bgp *bgp, afi_t<br>
>>> afi)<br>
>>>    struct route_node *rn;<br>
>>>    struct bgp_info *ri;<br>
>>><br>
>>> -  zlog_debug ("%s: entry, afi=%d", __func__, afi);<br>
>>> -<br>
>>>    if (!bgp)<br>
>>>      return;<br>
>>><br>
>>> @@ -499,8 +497,6 @@ vnc_direct_bgp_vpn_disable_ce (struct bgp *bgp, afi_t<br>
>>> afi)<br>
>>>  {<br>
>>>    struct bgp_node *rn;<br>
>>><br>
>>> -  zlog_debug ("%s: entry, afi=%d", __func__, afi);<br>
>>> -<br>
>>>    if (!bgp)<br>
>>>      return;<br>
>>><br>
>>> @@ -1276,8 +1272,6 @@ vnc_direct_bgp_add_group_afi (<br>
>>>    struct attr attr = { 0 };<br>
>>>    struct rfapi_import_table *import_table;<br>
>>><br>
>>> -  zlog_debug ("%s: entry", __func__);<br>
>>> -<br>
>>>    import_table = rfg->rfapi_import_table;<br>
>>>    if (!import_table)<br>
>>>      {<br>
>>> @@ -1417,8 +1411,6 @@ vnc_direct_bgp_del_group_afi (<br>
>>>    struct route_node *rn;<br>
>>>    struct rfapi_import_table *import_table;<br>
>>><br>
>>> -  zlog_debug ("%s: entry", __func__);<br>
>>> -<br>
>>>    import_table = rfg->rfapi_import_table;<br>
>>>    if (!import_table)<br>
>>>      {<br>
>>> @@ -1626,8 +1618,6 @@ vnc_direct_bgp_vpn_disable (struct bgp *bgp, afi_t<br>
>>> afi)<br>
>>>    struct rfapi_import_table *it;<br>
>>>    uint8_t family = afi2family (afi);<br>
>>><br>
>>> -  zlog_debug ("%s: entry, afi=%d", __func__, afi);<br>
>>> -<br>
>>>    if (!bgp)<br>
>>>      return;<br>
>>><br>
>>> @@ -1873,8 +1863,6 @@ vnc_direct_bgp_rh_vpn_enable (struct bgp *bgp, afi_t<br>
>>> afi)<br>
>>>    struct bgp_node *prn;<br>
>>>    struct rfapi_cfg *hc;<br>
>>><br>
>>> -  zlog_debug ("%s: entry, afi=%d", __func__, afi);<br>
>>> -<br>
>>>    if (!bgp)<br>
>>>      return;<br>
>>><br>
>>> @@ -2034,8 +2022,6 @@ vnc_direct_bgp_rh_vpn_disable (struct bgp *bgp,<br>
>>> afi_t afi)<br>
>>>  {<br>
>>>    struct bgp_node *rn;<br>
>>><br>
>>> -  zlog_debug ("%s: entry, afi=%d", __func__, afi);<br>
>>> -<br>
>>>    if (!bgp)<br>
>>>      return;<br>
>>><br>
>>> diff --git a/bgpd/rfapi/vnc_import_bgp.c b/bgpd/rfapi/vnc_import_bgp.c<br>
>>> index dc2640a..0ba927d 100644<br>
>>> --- a/bgpd/rfapi/vnc_import_bgp.c<br>
>>> +++ b/bgpd/rfapi/vnc_import_bgp.c<br>
>>> @@ -3097,7 +3097,6 @@ vnc_import_bgp_redist_disable (struct bgp *bgp,<br>
>>> afi_t afi)<br>
>>>      }<br>
>>><br>
>>>    bgp->rfapi_cfg->redist[afi][<wbr>ZEBRA_ROUTE_BGP_DIRECT] = 0;<br>
>>> -  zlog_debug ("%s: return", __func__);<br>
>>>  }<br>
>>><br>
>>><br>
>>> @@ -3145,5 +3144,4 @@ vnc_import_bgp_exterior_<wbr>redist_disable (struct bgp<br>
>>> *bgp, afi_t afi)<br>
>>>    }<br>
>>><br>
>>>    bgp->rfapi_cfg->redist[afi][<wbr>ZEBRA_ROUTE_BGP_DIRECT_EXT] = 0;<br>
>>> -  zlog_debug ("%s: return", __func__);<br>
>>>  }<br>
>>> diff --git a/bgpd/rfapi/vnc_zebra.c b/bgpd/rfapi/vnc_zebra.c<br>
>>> index e357ef6..bc77ca4 100644<br>
>>> --- a/bgpd/rfapi/vnc_zebra.c<br>
>>> +++ b/bgpd/rfapi/vnc_zebra.c<br>
>>> @@ -284,8 +284,6 @@ vnc_redistribute_withdraw (struct bgp *bgp, afi_t afi,<br>
>>> uint8_t type)<br>
>>>    struct bgp_node *prn;<br>
>>>    struct bgp_node *rn;<br>
>>><br>
>>> -  zlog_debug ("%s: entry", __func__);<br>
>>> -<br>
>>>    if (!bgp)<br>
>>>      return;<br>
>>>    if (!bgp->rfapi_cfg)<br>
>>> @@ -331,7 +329,6 @@ vnc_redistribute_withdraw (struct bgp *bgp, afi_t afi,<br>
>>> uint8_t type)<br>
>>>              }<br>
>>>          }<br>
>>>      }<br>
>>> -  zlog_debug ("%s: return", __func__);<br>
>>>  }<br>
>>><br>
>>>  /*<br>
>>> @@ -729,8 +726,6 @@ vnc_zebra_add_del_prefix (<br>
>>>    void *nh_ary = NULL;<br>
>>>    void *nhp_ary = NULL;<br>
>>><br>
>>> -  zlog_debug ("%s: entry, add=%d", __func__, add);<br>
>>> -<br>
>>>    if (zclient_vnc->sock < 0)<br>
>>>      return;<br>
>>><br>
>>> @@ -808,8 +803,6 @@ vnc_zebra_add_del_nve (<br>
>>>  //    struct prefix             *nhpp;<br>
>>>    void *pAddr;<br>
>>><br>
>>> -  zlog_debug ("%s: entry, add=%d", __func__, add);<br>
>>> -<br>
>>>    if (zclient_vnc->sock < 0)<br>
>>>      return;<br>
>>><br>
>>> @@ -909,7 +902,6 @@ vnc_zebra_add_del_group_afi (<br>
>>>    void *nh_ary = NULL;<br>
>>>    void *nhp_ary = NULL;<br>
>>><br>
>>> -  zlog_debug ("%s: entry", __func__);<br>
>>>    import_table = rfg->rfapi_import_table;<br>
>>>    if (!import_table)<br>
>>>      {<br>
>>> @@ -982,7 +974,6 @@ vnc_zebra_add_group (struct bgp *bgp, struct<br>
>>> rfapi_nve_group_cfg *rfg)<br>
>>>  void<br>
>>>  vnc_zebra_del_group (struct bgp *bgp, struct rfapi_nve_group_cfg *rfg)<br>
>>>  {<br>
>>> -  zlog_debug ("%s: entry", __func__);<br>
>>>    vnc_zebra_add_del_group_afi (bgp, rfg, AFI_IP, 0);<br>
>>>    vnc_zebra_add_del_group_afi (bgp, rfg, AFI_IP6, 0);<br>
>>>  }<br>
>>> @@ -1083,8 +1074,6 @@ vnc_redistribute_unset (struct bgp *bgp, afi_t afi,<br>
>>> int type)<br>
>>>    /* Withdraw redistributed routes from current BGP's routing table. */<br>
>>>    vnc_redistribute_withdraw (bgp, afi, type);<br>
>>><br>
>>> -  zlog_debug ("%s: return", __func__);<br>
>>> -<br>
>>>    return CMD_SUCCESS;<br>
>>>  }<br>
>>><br>
>>> --<br>
>>> 2.7.4<br>
>>><br>
>>><br>
>>> ______________________________<wbr>_________________<br>
>>> cmaster-next mailing list<br>
>>> <a href="mailto:cmaster-next@lists.nox.tf">cmaster-next@lists.nox.tf</a><br>
>>> <a href="https://lists.nox.tf/listinfo/cmaster-next" rel="noreferrer" target="_blank">https://lists.nox.tf/listinfo/<wbr>cmaster-next</a><br>
>><br>
>><br>
<br>
<br>
______________________________<wbr>_________________<br>
cmaster-next mailing list<br>
<a href="mailto:cmaster-next@lists.nox.tf">cmaster-next@lists.nox.tf</a><br>
<a href="https://lists.nox.tf/listinfo/cmaster-next" rel="noreferrer" target="_blank">https://lists.nox.tf/listinfo/<wbr>cmaster-next</a><br>
</div></div></blockquote></div><br></div>