Page MenuHomeFreeBSD
Differential D46796

netlink: Handle RTA_PREFSRC on RTM_NEWROUTE
ClosedPublic
Actions

Authored by email_luiz.eng.br on Sep 26 2024, 3:36 PM.
Tags
None
Referenced Files
F163164391: D46796.id144101.diff
Mon, Jul 20, 4:24 PM
Unknown Object (File)
Sun, Jul 19, 2:32 PM
Unknown Object (File)
Thu, Jul 16, 8:38 AM
Unknown Object (File)
Sun, Jul 5, 2:05 AM
Unknown Object (File)
Sat, Jul 4, 3:57 AM
Unknown Object (File)
Thu, Jul 2, 1:43 AM
Unknown Object (File)
Thu, Jul 2, 12:32 AM
Unknown Object (File)
Wed, Jul 1, 11:25 PM
View All Files
Subscribers
glebius
imp
melifaro

Details

Reviewers
melifaro
Commits
rGf5a04b16b189: acpidump: add 's' option to parse dsdt and ssdt's separately. In some machine…
Summary

While investigating a problem with the source address selection for IPv4 over IPv6 nexthop routes, I decided to try setting the prefsrc on the routes exported from BIRD, only to realize that there was no support for it.

With this patch, I was able to set a proper source address for routes added via Netlink.

Sponsored by: InnoGames GmbH

Diff Detail

Repository
rG FreeBSD src repository
Lint
Lint Skipped
Unit
Tests Skipped

Event Timeline

email_luiz.eng.br created this revision.Sep 26 2024, 3:36 PM
Herald added a reviewer: melifaro. · View Herald TranscriptSep 26 2024, 3:36 PM
Herald added subscribers: glebius, melifaro, imp. · View Herald Transcript
email_luiz.eng.br requested review of this revision.Sep 26 2024, 3:36 PM
email_luiz.eng.br edited the summary of this revision. (Show Details)Sep 26 2024, 3:39 PM
melifaro added a comment.Sep 28 2024, 8:35 AM

Thank you for working on this!
Could you please add the patch with context & add a corresponding test?
Thank you!

email_luiz.eng.br added a comment.Oct 1 2024, 11:39 AM
In D46796#1067574, @melifaro wrote:

Could you please add the patch with context & add a corresponding test?

Will do, just give me a few days for the tests.

email_luiz.eng.br updated this revision to Diff 144089.Oct 1 2024, 11:58 AM
This revision was not accepted when it landed; it landed in state Needs Review.Oct 2 2024, 6:38 AM
Closed by commit rGf5a04b16b189: acpidump: add 's' option to parse dsdt and ssdt's separately. In some machine… (authored by takawata). · Explain Why
This revision was automatically updated to reflect the committed changes.
takawata added a commit: rGf5a04b16b189: acpidump: add 's' option to parse dsdt and ssdt's separately. In some machine….

Revision Contents
Changeset List

PathSize
sys/
netlink/
route/
25 lines

Diff 144089

View Options

sys/netlink/route/rt.c

Loading...