]> git.proxmox.com Git - mirror_frr.git/blobdiff - bgpd/bgp_evpn.c
Merge pull request #3282 from opensourcerouting/60-fix-bfd-ipv6-unnumbered
[mirror_frr.git] / bgpd / bgp_evpn.c
index 6ebfb7d7809a1bdeaab645c4c352f3a0d34d1f07..9e814516b73dfc31957b280020023ab0c717cac3 100644 (file)
@@ -173,7 +173,7 @@ static struct vrf_irt_node *vrf_import_rt_new(struct ecommunity_val *rt)
 
        bgp_def = bgp_get_default();
        if (!bgp_def) {
-               zlog_ferr(BGP_ERR_NO_DFLT,
+               flog_err(BGP_ERR_NO_DFLT,
                          "vrf import rt new - def instance not created yet");
                return NULL;
        }
@@ -204,7 +204,7 @@ static void vrf_import_rt_free(struct vrf_irt_node *irt)
 
        bgp_def = bgp_get_default();
        if (!bgp_def) {
-               zlog_ferr(BGP_ERR_NO_DFLT,
+               flog_err(BGP_ERR_NO_DFLT,
                          "vrf import rt free - def instance not created yet");
                return;
        }
@@ -226,7 +226,7 @@ static struct vrf_irt_node *lookup_vrf_import_rt(struct ecommunity_val *rt)
 
        bgp_def = bgp_get_default();
        if (!bgp_def) {
-               zlog_ferr(
+               flog_err(
                        BGP_ERR_NO_DFLT,
                        "vrf import rt lookup - def instance not created yet");
                return NULL;
@@ -624,7 +624,7 @@ static int bgp_zebra_send_remote_vtep(struct bgp *bgp, struct bgpevpn *vpn,
        if (is_evpn_prefix_ipaddr_v4(p))
                stream_put_in_addr(s, &p->prefix.imet_addr.ip.ipaddr_v4);
        else if (is_evpn_prefix_ipaddr_v6(p)) {
-               zlog_ferr(
+               flog_err(
                        BGP_ERR_VTEP_INVALID,
                        "Bad remote IP when trying to %s remote VTEP for VNI %u",
                        add ? "ADD" : "DEL", vpn->vni);
@@ -1295,7 +1295,7 @@ static int update_evpn_type4_route_entry(struct bgp *bgp,
         * We shouldn't see the same route from any other vtep.
         */
        if (remote_ri) {
-               zlog_ferr(
+               flog_err(
                        BGP_ERR_ES_INVALID,
                        "%u ERROR: local es route for ESI: %s Vtep %s also learnt from remote",
                        bgp->vrf_id,
@@ -1383,7 +1383,7 @@ static int update_evpn_type4_route(struct bgp *bgp,
                                            &attr, 1, &ri,
                                            &route_changed);
        if (ret != 0) {
-               zlog_ferr(
+               flog_err(
                        BGP_ERR_ES_INVALID,
                        "%u ERROR: Failed to updated ES route ESI: %s VTEP %s",
                        bgp->vrf_id,
@@ -1745,7 +1745,8 @@ static int update_evpn_route(struct bgp *bgp, struct bgpevpn *vpn,
        if (p->prefix.route_type == BGP_EVPN_MAC_IP_ROUTE &&
            (is_evpn_prefix_ipaddr_v4(p) ||
             !IN6_IS_ADDR_LINKLOCAL(&p->prefix.macip_addr.ip.ipaddr_v6)) &&
-           CHECK_FLAG(vpn->flags, VNI_FLAG_USE_TWO_LABELS))
+           CHECK_FLAG(vpn->flags, VNI_FLAG_USE_TWO_LABELS) &&
+           bgpevpn_get_l3vni(vpn))
                add_l3_ecomm = 1;
 
        /* Set up extended community. */
@@ -1997,7 +1998,8 @@ static int update_all_type2_routes(struct bgp *bgp, struct bgpevpn *vpn)
        /* Add L3 VNI RTs and RMAC for non IPv6 link-local attributes if
         * using L3 VNI for type-2 routes also.
         */
-       if (CHECK_FLAG(vpn->flags, VNI_FLAG_USE_TWO_LABELS))
+       if (CHECK_FLAG(vpn->flags, VNI_FLAG_USE_TWO_LABELS) &&
+           bgpevpn_get_l3vni(vpn))
                add_l3_ecomm = 1;
 
        build_evpn_route_extcomm(vpn, &attr, add_l3_ecomm);
@@ -2225,7 +2227,7 @@ static int delete_routes_for_es(struct bgp *bgp, struct evpnes *es)
        build_evpn_type4_prefix(&p, &es->esi, es->originator_ip.ipaddr_v4);
        ret = delete_evpn_type4_route(bgp, es, &p);
        if (ret) {
-               zlog_ferr(BGP_ERR_EVPN_ROUTE_DELETE,
+               flog_err(BGP_ERR_EVPN_ROUTE_DELETE,
                          "%u failed to delete type-4 route for ESI %s",
                          bgp->vrf_id, esi_to_str(&es->esi, buf, sizeof(buf)));
        }
@@ -2884,7 +2886,7 @@ static int install_uninstall_routes_for_es(struct bgp *bgp,
                                                bgp, es, evp, ri);
 
                                if (ret) {
-                                       zlog_ferr(
+                                       flog_err(
                                                BGP_ERR_EVPN_FAIL,
                                                "Failed to %s EVPN %s route in ESI %s",
                                                install ? "install"
@@ -2964,7 +2966,7 @@ static int install_uninstall_routes_for_vrf(struct bgp *bgp_vrf, int install)
                                                        bgp_vrf, evp, ri);
 
                                        if (ret) {
-                                               zlog_ferr(
+                                               flog_err(
                                                        BGP_ERR_EVPN_FAIL,
                                                        "Failed to %s EVPN %s route in VRF %s",
                                                        install ? "install"
@@ -3038,7 +3040,7 @@ static int install_uninstall_routes_for_vni(struct bgp *bgp,
                                                        bgp, vpn, evp, ri);
 
                                        if (ret) {
-                                               zlog_ferr(
+                                               flog_err(
                                                        BGP_ERR_EVPN_FAIL,
                                                        "%u: Failed to %s EVPN %s route in VNI %u",
                                                        bgp->vrf_id,
@@ -3141,7 +3143,7 @@ static int install_uninstall_route_in_es(struct bgp *bgp, struct evpnes *es,
                ret = uninstall_evpn_route_entry_in_es(bgp, es, evp, ri);
 
        if (ret) {
-               zlog_ferr(
+               flog_err(
                        BGP_ERR_EVPN_FAIL,
                        "%u: Failed to %s EVPN %s route in ESI %s", bgp->vrf_id,
                        install ? "install" : "uninstall", "ES",
@@ -3184,7 +3186,7 @@ static int install_uninstall_route_in_vrfs(struct bgp *bgp_def, afi_t afi,
                                                                ri);
 
                if (ret) {
-                       zlog_ferr(BGP_ERR_EVPN_FAIL,
+                       flog_err(BGP_ERR_EVPN_FAIL,
                                  "%u: Failed to %s prefix %s in VRF %s",
                                  bgp_def->vrf_id,
                                  install ? "install" : "uninstall",
@@ -3220,7 +3222,7 @@ static int install_uninstall_route_in_vnis(struct bgp *bgp, afi_t afi,
                        ret = uninstall_evpn_route_entry(bgp, vpn, evp, ri);
 
                if (ret) {
-                       zlog_ferr(
+                       flog_err(
                                BGP_ERR_EVPN_FAIL,
                                "%u: Failed to %s EVPN %s route in VNI %u",
                                bgp->vrf_id, install ? "install" : "uninstall",
@@ -3540,14 +3542,7 @@ static int delete_withdraw_vni_routes(struct bgp *bgp, struct bgpevpn *vpn)
  */
 static void update_router_id_vni(struct hash_backet *backet, struct bgp *bgp)
 {
-       struct bgpevpn *vpn;
-
-       vpn = (struct bgpevpn *)backet->data;
-
-       if (!vpn) {
-               zlog_warn("%s: VNI hash entry for VNI not found", __FUNCTION__);
-               return;
-       }
+       struct bgpevpn *vpn = (struct bgpevpn *)backet->data;
 
        /* Skip VNIs with configured RD. */
        if (is_rd_configured(vpn))
@@ -3565,14 +3560,7 @@ static void update_router_id_vni(struct hash_backet *backet, struct bgp *bgp)
  */
 static void withdraw_router_id_vni(struct hash_backet *backet, struct bgp *bgp)
 {
-       struct bgpevpn *vpn;
-
-       vpn = (struct bgpevpn *)backet->data;
-
-       if (!vpn) {
-               zlog_warn("%s: VNI hash entry for VNI not found", __FUNCTION__);
-               return;
-       }
+       struct bgpevpn *vpn = (struct bgpevpn *)backet->data;
 
        /* Skip VNIs with configured RD. */
        if (is_rd_configured(vpn))
@@ -3606,7 +3594,7 @@ static int process_type2_route(struct peer *peer, afi_t afi, safi_t safi,
         */
        if (psize != 33 && psize != 37 && psize != 49 && psize != 36
            && psize != 40 && psize != 52) {
-               zlog_ferr(BGP_ERR_EVPN_ROUTE_INVALID,
+               flog_err(BGP_ERR_EVPN_ROUTE_INVALID,
                          "%u:%s - Rx EVPN Type-2 NLRI with invalid length %d",
                          peer->bgp->vrf_id, peer->host, psize);
                return -1;
@@ -3643,7 +3631,7 @@ static int process_type2_route(struct peer *peer, afi_t afi, safi_t safi,
                memcpy(&p.prefix.macip_addr.mac.octet, pfx, ETH_ALEN);
                pfx += ETH_ALEN;
        } else {
-               zlog_ferr(
+               flog_err(
                        BGP_ERR_EVPN_ROUTE_INVALID,
                        "%u:%s - Rx EVPN Type-2 NLRI with unsupported MAC address length %d",
                        peer->bgp->vrf_id, peer->host, macaddr_len);
@@ -3655,7 +3643,7 @@ static int process_type2_route(struct peer *peer, afi_t afi, safi_t safi,
        ipaddr_len = *pfx++;
        if (ipaddr_len != 0 && ipaddr_len != IPV4_MAX_BITLEN
            && ipaddr_len != IPV6_MAX_BITLEN) {
-               zlog_ferr(
+               flog_err(
                        BGP_ERR_EVPN_ROUTE_INVALID,
                        "%u:%s - Rx EVPN Type-2 NLRI with unsupported IP address length %d",
                        peer->bgp->vrf_id, peer->host, ipaddr_len);
@@ -3717,7 +3705,7 @@ static int process_type3_route(struct peer *peer, afi_t afi, safi_t safi,
         * IP len (1) and IP (4 or 16).
         */
        if (psize != 17 && psize != 29) {
-               zlog_ferr(BGP_ERR_EVPN_ROUTE_INVALID,
+               flog_err(BGP_ERR_EVPN_ROUTE_INVALID,
                          "%u:%s - Rx EVPN Type-3 NLRI with invalid length %d",
                          peer->bgp->vrf_id, peer->host, psize);
                return -1;
@@ -3759,7 +3747,7 @@ static int process_type3_route(struct peer *peer, afi_t afi, safi_t safi,
                p.prefix.imet_addr.ip.ipa_type = IPADDR_V4;
                memcpy(&p.prefix.imet_addr.ip.ip.addr, pfx, IPV4_MAX_BYTELEN);
        } else {
-               zlog_ferr(
+               flog_err(
                        BGP_ERR_EVPN_ROUTE_INVALID,
                        "%u:%s - Rx EVPN Type-3 NLRI with unsupported IP address length %d",
                        peer->bgp->vrf_id, peer->host, ipaddr_len);
@@ -3796,7 +3784,7 @@ static int process_type4_route(struct peer *peer, afi_t afi, safi_t safi,
         *  RD (8), ESI (10), ip-len (1), ip (4 or 16)
         */
        if (psize != 23 && psize != 35) {
-               zlog_ferr(BGP_ERR_EVPN_ROUTE_INVALID,
+               flog_err(BGP_ERR_EVPN_ROUTE_INVALID,
                          "%u:%s - Rx EVPN Type-4 NLRI with invalid length %d",
                          peer->bgp->vrf_id, peer->host, psize);
                return -1;
@@ -3818,7 +3806,7 @@ static int process_type4_route(struct peer *peer, afi_t afi, safi_t safi,
        if (ipaddr_len == IPV4_MAX_BITLEN) {
                memcpy(&vtep_ip, pfx, IPV4_MAX_BYTELEN);
        } else {
-               zlog_ferr(
+               flog_err(
                        BGP_ERR_EVPN_ROUTE_INVALID,
                        "%u:%s - Rx EVPN Type-4 NLRI with unsupported IP address length %d",
                        peer->bgp->vrf_id, peer->host, ipaddr_len);
@@ -3861,7 +3849,7 @@ static int process_type5_route(struct peer *peer, afi_t afi, safi_t safi,
         * Note that the IP and GW should both be IPv4 or both IPv6.
         */
        if (psize != 34 && psize != 58) {
-               zlog_ferr(BGP_ERR_EVPN_ROUTE_INVALID,
+               flog_err(BGP_ERR_EVPN_ROUTE_INVALID,
                          "%u:%s - Rx EVPN Type-5 NLRI with invalid length %d",
                          peer->bgp->vrf_id, peer->host, psize);
                return -1;
@@ -3894,7 +3882,7 @@ static int process_type5_route(struct peer *peer, afi_t afi, safi_t safi,
        /* Fetch IP prefix length. */
        ippfx_len = *pfx++;
        if (ippfx_len > IPV6_MAX_BITLEN) {
-               zlog_ferr(
+               flog_err(
                        BGP_ERR_EVPN_ROUTE_INVALID,
                        "%u:%s - Rx EVPN Type-5 NLRI with invalid IP Prefix length %d",
                        peer->bgp->vrf_id, peer->host, ippfx_len);
@@ -4016,9 +4004,8 @@ static void cleanup_vni_on_disable(struct hash_backet *backet, struct bgp *bgp)
  */
 static void free_vni_entry(struct hash_backet *backet, struct bgp *bgp)
 {
-       struct bgpevpn *vpn;
+       struct bgpevpn *vpn = (struct bgpevpn *)backet->data;
 
-       vpn = (struct bgpevpn *)backet->data;
        delete_all_vni_routes(bgp, vpn);
        bgp_evpn_free(bgp, vpn);
 }
@@ -4090,11 +4077,6 @@ static void update_autort_vni(struct hash_backet *backet, struct bgp *bgp)
 {
        struct bgpevpn *vpn = backet->data;
 
-       if (!vpn) {
-               zlog_warn("%s: VNI hash entry for VNI not found", __PRETTY_FUNCTION__);
-               return;
-       }
-
        if (!is_import_rt_configured(vpn)) {
                if (is_vni_live(vpn))
                        bgp_evpn_uninstall_routes(bgp, vpn);
@@ -4127,7 +4109,7 @@ void bgp_evpn_withdraw_type5_route(struct bgp *bgp_vrf, struct prefix *p,
        build_type5_prefix_from_ip_prefix(&evp, p);
        ret = delete_evpn_type5_route(bgp_vrf, &evp);
        if (ret) {
-               zlog_ferr(
+               flog_err(
                        BGP_ERR_EVPN_ROUTE_DELETE,
                        "%u failed to delete type-5 route for prefix %s in vrf %s",
                        bgp_vrf->vrf_id, prefix2str(p, buf, sizeof(buf)),
@@ -4174,7 +4156,7 @@ void bgp_evpn_advertise_type5_route(struct bgp *bgp_vrf, struct prefix *p,
        build_type5_prefix_from_ip_prefix(&evp, p);
        ret = update_evpn_type5_route(bgp_vrf, &evp, src_attr);
        if (ret)
-               zlog_ferr(BGP_ERR_EVPN_ROUTE_CREATE,
+               flog_err(BGP_ERR_EVPN_ROUTE_CREATE,
                          "%u: Failed to create type-5 route for prefix %s",
                          bgp_vrf->vrf_id, prefix2str(p, buf, sizeof(buf)));
 }
@@ -4734,7 +4716,7 @@ int bgp_nlri_parse_evpn(struct peer *peer, struct attr *attr,
                        if (process_type2_route(peer, afi, safi,
                                                withdraw ? NULL : attr, pnt,
                                                psize, addpath_id)) {
-                               zlog_ferr(
+                               flog_err(
                                        BGP_ERR_EVPN_FAIL,
                                        "%u:%s - Error in processing EVPN type-2 NLRI size %d",
                                        peer->bgp->vrf_id, peer->host, psize);
@@ -4746,7 +4728,7 @@ int bgp_nlri_parse_evpn(struct peer *peer, struct attr *attr,
                        if (process_type3_route(peer, afi, safi,
                                                withdraw ? NULL : attr, pnt,
                                                psize, addpath_id)) {
-                               zlog_ferr(
+                               flog_err(
                                        BGP_ERR_PKT_PROCESS,
                                        "%u:%s - Error in processing EVPN type-3 NLRI size %d",
                                        peer->bgp->vrf_id, peer->host, psize);
@@ -4758,7 +4740,7 @@ int bgp_nlri_parse_evpn(struct peer *peer, struct attr *attr,
                        if (process_type4_route(peer, afi, safi,
                                                withdraw ? NULL : attr, pnt,
                                                psize, addpath_id)) {
-                               zlog_ferr(
+                               flog_err(
                                        BGP_ERR_PKT_PROCESS,
                                        "%u:%s - Error in processing EVPN type-4 NLRI size %d",
                                        peer->bgp->vrf_id, peer->host, psize);
@@ -4769,7 +4751,7 @@ int bgp_nlri_parse_evpn(struct peer *peer, struct attr *attr,
                case BGP_EVPN_IP_PREFIX_ROUTE:
                        if (process_type5_route(peer, afi, safi, attr, pnt,
                                                psize, addpath_id, withdraw)) {
-                               zlog_ferr(
+                               flog_err(
                                        BGP_ERR_PKT_PROCESS,
                                        "%u:%s - Error in processing EVPN type-5 NLRI size %d",
                                        peer->bgp->vrf_id, peer->host, psize);
@@ -4949,6 +4931,23 @@ void bgp_evpn_derive_auto_rd(struct bgp *bgp, struct bgpevpn *vpn)
        UNSET_FLAG(vpn->flags, VNI_FLAG_RD_CFGD);
 }
 
+/*
+ * Lookup L3-VNI
+ */
+bool bgp_evpn_lookup_l3vni_l2vni_table(vni_t vni)
+{
+       struct list *inst = bm->bgp;
+       struct listnode *node;
+       struct bgp *bgp_vrf;
+
+       for (ALL_LIST_ELEMENTS_RO(inst, node, bgp_vrf)) {
+               if (bgp_vrf->l3vni == vni)
+                       return true;
+       }
+
+       return false;
+}
+
 /*
  * Lookup VNI.
  */
@@ -5233,7 +5232,7 @@ int bgp_evpn_local_macip_add(struct bgp *bgp, vni_t vni, struct ethaddr *mac,
                char buf[ETHER_ADDR_STRLEN];
                char buf2[INET6_ADDRSTRLEN];
 
-               zlog_ferr(
+               flog_err(
                        BGP_ERR_EVPN_ROUTE_CREATE,
                        "%u:Failed to create Type-2 route, VNI %u %s MAC %s IP %s (flags: 0x%x)",
                        bgp->vrf_id, vpn->vni,
@@ -5251,13 +5250,12 @@ int bgp_evpn_local_macip_add(struct bgp *bgp, vni_t vni, struct ethaddr *mac,
 static void link_l2vni_hash_to_l3vni(struct hash_backet *backet,
                                     struct bgp *bgp_vrf)
 {
-       struct bgpevpn *vpn = NULL;
+       struct bgpevpn *vpn = (struct bgpevpn *)backet->data;
        struct bgp *bgp_def = NULL;
 
        bgp_def = bgp_get_default();
        assert(bgp_def);
 
-       vpn = (struct bgpevpn *)backet->data;
        if (vpn->tenant_vrf_id == bgp_vrf->vrf_id)
                bgpevpn_link_to_l3vni(vpn);
 }
@@ -5276,7 +5274,7 @@ int bgp_evpn_local_l3vni_add(vni_t l3vni, vrf_id_t vrf_id, struct ethaddr *rmac,
         */
        bgp_def = bgp_get_default();
        if (!bgp_def) {
-               zlog_ferr(
+               flog_err(
                        BGP_ERR_NO_DFLT,
                        "Cannot process L3VNI  %u ADD - default BGP instance not yet created",
                        l3vni);
@@ -5294,15 +5292,15 @@ int bgp_evpn_local_l3vni_add(vni_t l3vni, vrf_id_t vrf_id, struct ethaddr *rmac,
                              BGP_INSTANCE_TYPE_VRF);
                switch (ret) {
                case BGP_ERR_MULTIPLE_INSTANCE_NOT_SET:
-                       zlog_ferr(BGP_ERR_MULTI_INSTANCE,
+                       flog_err(BGP_ERR_MULTI_INSTANCE,
                                  "'bgp multiple-instance' not present\n");
                        return -1;
                case BGP_ERR_AS_MISMATCH:
-                       zlog_ferr(BGP_ERR_EVPN_AS_MISMATCH,
+                       flog_err(BGP_ERR_EVPN_AS_MISMATCH,
                                  "BGP is already running; AS is %u\n", as);
                        return -1;
                case BGP_ERR_INSTANCE_MISMATCH:
-                       zlog_ferr(BGP_ERR_EVPN_INSTANCE_MISMATCH,
+                       flog_err(BGP_ERR_EVPN_INSTANCE_MISMATCH,
                                  "BGP instance name and AS number mismatch\n");
                        return -1;
                }
@@ -5364,7 +5362,7 @@ int bgp_evpn_local_l3vni_del(vni_t l3vni, vrf_id_t vrf_id)
 
        bgp_vrf = bgp_lookup_by_vrf_id(vrf_id);
        if (!bgp_vrf) {
-               zlog_ferr(
+               flog_err(
                        BGP_ERR_NO_DFLT,
                        "Cannot process L3VNI %u Del - Could not find BGP instance",
                        l3vni);
@@ -5373,7 +5371,7 @@ int bgp_evpn_local_l3vni_del(vni_t l3vni, vrf_id_t vrf_id)
 
        bgp_def = bgp_get_default();
        if (!bgp_def) {
-               zlog_ferr(
+               flog_err(
                        BGP_ERR_NO_DFLT,
                        "Cannot process L3VNI %u Del - Could not find default BGP instance",
                        l3vni);
@@ -5504,7 +5502,7 @@ int bgp_evpn_local_vni_add(struct bgp *bgp, vni_t vni,
        if (!vpn) {
                vpn = bgp_evpn_new(bgp, vni, originator_ip, tenant_vrf_id);
                if (!vpn) {
-                       zlog_ferr(
+                       flog_err(
                                BGP_ERR_VNI,
                                "%u: Failed to allocate VNI entry for VNI %u - at Add",
                                bgp->vrf_id, vni);
@@ -5528,7 +5526,7 @@ int bgp_evpn_local_vni_add(struct bgp *bgp, vni_t vni,
        /* Create EVPN type-3 route and schedule for processing. */
        build_evpn_type3_prefix(&p, vpn->originator_ip);
        if (update_evpn_route(bgp, vpn, &p, 0)) {
-               zlog_ferr(BGP_ERR_EVPN_ROUTE_CREATE,
+               flog_err(BGP_ERR_EVPN_ROUTE_CREATE,
                          "%u: Type3 route creation failure for VNI %u",
                          bgp->vrf_id, vni);
                return -1;
@@ -5558,7 +5556,7 @@ int bgp_evpn_local_es_del(struct bgp *bgp,
        struct evpnes *es = NULL;
 
        if (!bgp->esihash) {
-               zlog_ferr(BGP_ERR_ES_CREATE, "%u: ESI hash not yet created",
+               flog_err(BGP_ERR_ES_CREATE, "%u: ESI hash not yet created",
                          bgp->vrf_id);
                return -1;
        }
@@ -5595,7 +5593,7 @@ int bgp_evpn_local_es_add(struct bgp *bgp,
        struct prefix_evpn p;
 
        if (!bgp->esihash) {
-               zlog_ferr(BGP_ERR_ES_CREATE, "%u: ESI hash not yet created",
+               flog_err(BGP_ERR_ES_CREATE, "%u: ESI hash not yet created",
                          bgp->vrf_id);
                return -1;
        }
@@ -5605,7 +5603,7 @@ int bgp_evpn_local_es_add(struct bgp *bgp,
        if (!es) {
                es = bgp_evpn_es_new(bgp, esi, originator_ip);
                if (!es) {
-                       zlog_ferr(
+                       flog_err(
                                BGP_ERR_ES_CREATE,
                                "%u: Failed to allocate ES entry for ESI %s - at Local ES Add",
                                bgp->vrf_id, esi_to_str(esi, buf, sizeof(buf)));
@@ -5617,7 +5615,7 @@ int bgp_evpn_local_es_add(struct bgp *bgp,
 
        build_evpn_type4_prefix(&p, esi, originator_ip->ipaddr_v4);
        if (update_evpn_type4_route(bgp, es, &p)) {
-               zlog_ferr(BGP_ERR_EVPN_ROUTE_CREATE,
+               flog_err(BGP_ERR_EVPN_ROUTE_CREATE,
                          "%u: Type4 route creation failure for ESI %s",
                          bgp->vrf_id, esi_to_str(esi, buf, sizeof(buf)));
                return -1;