routing: move rtentry and subscription code out of route_ctl.c
ClosedPublic
Actions

Authored by melifaro on Aug 8 2022, 1:57 PM.

Details

Reviewers

None

Group Reviewers

Commits

rGa4cfcd6b5d82: routing: move rtentry and subscription code out of route_ctl.c
rG5c4d2252d771: routing: move rtentry and subscription code out of route_ctl.c

Summary

route_ctl.c is getting bigger. Factor out non-relevant parts:

all rtentry logic, such as creation/destruction and accessors goes to net/route/route_rtentry.c
all rtable subscription logic goes to net/route/route_subscription.c

MFC after: 1 month

Diff Detail

Repository

rS FreeBSD src repository - subversion

Lint

Lint Passed

Unit

No Test Coverage

Build Status

Buildable 46784
Build 43673: arc lint + arc unit

Event Timeline

melifaro created this revision.Aug 8 2022, 1:57 PM

Herald added subscribers: glebius, ae, imp. · View Herald TranscriptAug 8 2022, 1:57 PM

melifaro requested review of this revision.Aug 8 2022, 1:57 PM

Harbormaster completed remote builds in B46784: Diff 108984.Aug 8 2022, 1:57 PM

melifaro added a reviewer: network.Aug 8 2022, 1:58 PM

melifaro added a parent revision: D36073: routing: add rib_<add|del>_route_px() functions operating with nexthops..

melifaro added a child revision: D36075: routing: populate fibs with interface routes after growing net.fibs..Aug 8 2022, 2:55 PM

This revision was not accepted when it landed; it landed in state Needs Review.Aug 10 2022, 7:05 PM

Closed by commit rG5c4d2252d771: routing: move rtentry and subscription code out of route_ctl.c (authored by melifaro). · Explain Why

This revision was automatically updated to reflect the committed changes.

melifaro added a commit: rG5c4d2252d771: routing: move rtentry and subscription code out of route_ctl.c.

melifaro added a commit: rGa4cfcd6b5d82: routing: move rtentry and subscription code out of route_ctl.c.Jan 13 2023, 9:26 PM

Revision Contents
Changeset List

Path

Size

sys/

conf/

files

2 lines

net/

route/

2 lines

438 lines

303 lines

213 lines

5 lines

Diff 108984

View Options

sys/conf/files

View Options

sys/net/route/route_ctl.h

View Options

sys/net/route/route_ctl.c

View Options

sys/net/route/route_rtentry.c

View Options

sys/net/route/route_subscription.c

View Options

routing: move rtentry and subscription code out of route_ctl.cClosedPublicActions

Details

Diff Detail

Event Timeline

Revision ContentsChangeset List

Diff 108984

sys/conf/files

sys/net/route/route_ctl.h

sys/net/route/route_ctl.c

sys/net/route/route_rtentry.c

sys/net/route/route_subscription.c

sys/net/route/route_var.h

routing: move rtentry and subscription code out of route_ctl.c
ClosedPublic
Actions

Revision Contents
Changeset List