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
Unknown Object (File)
Thu, Sep 3, 4:21 AM
Unknown Object (File)
Fri, Aug 28, 10:00 AM
Unknown Object (File)
Fri, Aug 28, 9:48 AM
Unknown Object (File)
Fri, Aug 21, 11:18 PM
Unknown Object (File)
Fri, Aug 21, 6:09 AM
Unknown Object (File)
Thu, Aug 13, 2:30 PM
Unknown Object (File)
Thu, Aug 13, 12:22 AM
Unknown Object (File)
Wed, Aug 12, 11:11 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...