Page MenuHomeFreeBSD
Differential D13989

Reduce code duplication for inpcb route caching
ClosedPublic
Actions

Authored by rstone on Jan 19 2018, 9:40 PM.
Tags
None
Referenced Files
Unknown Object (File)
Thu, Dec 26, 12:49 PM
Unknown Object (File)
Nov 30 2024, 3:09 AM
Unknown Object (File)
Nov 19 2024, 4:06 AM
Unknown Object (File)
Nov 14 2024, 4:23 AM
Unknown Object (File)
Nov 13 2024, 5:12 AM
Unknown Object (File)
Nov 1 2024, 4:43 AM
Unknown Object (File)
Oct 30 2024, 4:57 PM
Unknown Object (File)
Sep 30 2024, 10:50 PM
View All 62 Files
Subscribers
ae
imp
jim_netgate.com
vangyzen
wollman

Details

Reviewers
• karels
Commits
rS328271: Reduce code duplication for inpcb route caching
Summary

Add a new macro to clear both the L3 and L2 route caches, to
hopefully prevent future instances where only the L3 cache was
cleared when both should have been.

Diff Detail

Repository
rS FreeBSD src repository - subversion
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

rstone created this revision.Jan 19 2018, 9:40 PM
Herald added a subscriber: ae. · View Herald TranscriptJan 19 2018, 9:40 PM
Harbormaster completed remote builds in B14490: Diff 38222.Jan 19 2018, 9:40 PM
vangyzen added a subscriber: vangyzen.Jan 19 2018, 9:58 PM
vangyzen added inline comments.
sys/net/route.h
419 ↗(On Diff #38222)

I generally prefer inline functions over macros, but this is fine.

sys/netinet/ip_output.c
305 ↗(On Diff #38222)

This drops the assignment of NULL to the local rte.

jim_netgate.com added a subscriber: jim_netgate.com.Jan 19 2018, 10:09 PM
rstone updated this revision to Diff 38234.Jan 19 2018, 11:36 PM

NULL-out rte after route cache is invalidated

Harbormaster completed remote builds in B14496: Diff 38234.Jan 19 2018, 11:36 PM
rstone marked 2 inline comments as done.Jan 19 2018, 11:38 PM
• karels added a comment.Jan 20 2018, 2:37 AM

Was there a place where only L3 was invalidated?

In any case, this seems like a good change.

wollman added a subscriber: wollman.Jan 20 2018, 5:57 PM
• karels accepted this revision.Jan 22 2018, 5:38 AM
This revision is now accepted and ready to land.Jan 22 2018, 5:38 AM
Closed by commit rS328271: Reduce code duplication for inpcb route caching (authored by rstone). · Explain WhyJan 23 2018, 3:16 AM
This revision was automatically updated to reflect the committed changes.
Herald added a subscriber: imp. · View Herald TranscriptJan 23 2018, 3:16 AM

Revision Contents
Changeset List

PathSize
head/
sys/
net/
17 lines
netinet/
8 lines
7 lines

Diff 38338

View Options

head/sys/net/route.h

Loading...
View Options

head/sys/netinet/in_pcb.c

Loading...
View Options

head/sys/netinet/ip_output.c

Loading...