Remove "Old" variant of benchmark. Requires maintingin old code path.

route_test.go

@@ -876,29 +876,6 @@ func TestRouteFilterIterCanStop(t *testing.T) {
 	}
 }
 
-func BenchmarkRouteListFilteredOld(b *testing.B) {
-	tearDown := setUpNetlinkTest(b)
-	defer tearDown()
-
-	link, err := setUpRoutesBench(b)
-
-	b.ResetTimer()
-	b.ReportAllocs()
-	var routes []Route
-	for i := 0; i < b.N; i++ {
-		routes, err = pkgHandle.RouteListFilteredOld(FAMILY_V4, &Route{
-			LinkIndex: link.Attrs().Index,
-		}, RT_FILTER_OIF)
-		if err != nil {
-			b.Fatal(err)
-		}
-		if len(routes) != 65535 {
-			b.Fatal("Incorrect number of routes.", len(routes))
-		}
-	}
-	runtime.KeepAlive(routes)
-}
-
 func BenchmarkRouteListFilteredNew(b *testing.B) {
 	tearDown := setUpNetlinkTest(b)
 	defer tearDown()
@@ -922,74 +899,6 @@ func BenchmarkRouteListFilteredNew(b *testing.B) {
 	runtime.KeepAlive(routes)
 }
 
-// RouteListFiltered gets a list of routes in the system filtered with specified rules.
-// All rules must be defined in RouteFilter struct
-func (h *Handle) RouteListFilteredOld(family int, filter *Route, filterMask uint64) ([]Route, error) {
-	req := h.newNetlinkRequest(unix.RTM_GETROUTE, unix.NLM_F_DUMP)
-	rtmsg := &nl.RtMsg{}
-	rtmsg.Family = uint8(family)
-	msgs, err := h.routeHandle(filter, req, rtmsg)
-	if err != nil {
-		return nil, err
-	}
-
-	var res []Route
-	for _, m := range msgs {
-		msg := nl.DeserializeRtMsg(m)
-		if msg.Flags&unix.RTM_F_CLONED != 0 {
-			// Ignore cloned routes
-			continue
-		}
-		if msg.Table != unix.RT_TABLE_MAIN {
-			if filter == nil || filter != nil && filterMask&RT_FILTER_TABLE == 0 {
-				// Ignore non-main tables
-				continue
-			}
-		}
-		route, err := deserializeRoute(m)
-		if err != nil {
-			return nil, err
-		}
-		if filter != nil {
-			switch {
-			case filterMask&RT_FILTER_TABLE != 0 && filter.Table != unix.RT_TABLE_UNSPEC && route.Table != filter.Table:
-				continue
-			case filterMask&RT_FILTER_PROTOCOL != 0 && route.Protocol != filter.Protocol:
-				continue
-			case filterMask&RT_FILTER_SCOPE != 0 && route.Scope != filter.Scope:
-				continue
-			case filterMask&RT_FILTER_TYPE != 0 && route.Type != filter.Type:
-				continue
-			case filterMask&RT_FILTER_TOS != 0 && route.Tos != filter.Tos:
-				continue
-			case filterMask&RT_FILTER_REALM != 0 && route.Realm != filter.Realm:
-				continue
-			case filterMask&RT_FILTER_OIF != 0 && route.LinkIndex != filter.LinkIndex:
-				continue
-			case filterMask&RT_FILTER_IIF != 0 && route.ILinkIndex != filter.ILinkIndex:
-				continue
-			case filterMask&RT_FILTER_GW != 0 && !route.Gw.Equal(filter.Gw):
-				continue
-			case filterMask&RT_FILTER_SRC != 0 && !route.Src.Equal(filter.Src):
-				continue
-			case filterMask&RT_FILTER_DST != 0:
-				if filter.MPLSDst == nil || route.MPLSDst == nil || (*filter.MPLSDst) != (*route.MPLSDst) {
-					if filter.Dst == nil {
-						filter.Dst = genZeroIPNet(family)
-					}
-					if !ipNetEqual(route.Dst, filter.Dst) {
-						continue
-					}
-				}
-			case filterMask&RT_FILTER_HOPLIMIT != 0 && route.Hoplimit != filter.Hoplimit:
-				continue
-			}
-		}
-		res = append(res, route)
-	}
-	return res, nil
-}
-
 func BenchmarkRouteListIter(b *testing.B) {
 	tearDown := setUpNetlinkTest(b)
 	defer tearDown()