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
F103040629: D46796.diff
Wed, Nov 20, 4:05 AM
Unknown Object (File)
Sun, Nov 17, 11:33 PM
Unknown Object (File)
Sat, Nov 9, 7:24 PM
Unknown Object (File)
Sat, Nov 9, 7:24 PM
Unknown Object (File)
Sat, Nov 9, 7:24 PM
Unknown Object (File)
Sat, Nov 9, 7:16 PM
Unknown Object (File)
Wed, Nov 6, 9:26 AM
Unknown Object (File)
Sat, Nov 2, 3:39 AM
View All 17 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 Not Applicable
Unit
Tests Not Applicable

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
usr.sbin/
acpi/
acpidump/
16 lines
1 line
14 lines
11 lines

Diff 144101

View Options

usr.sbin/acpi/acpidump/acpi.c

Loading...
View Options

usr.sbin/acpi/acpidump/acpidump.h

Loading...
View Options

usr.sbin/acpi/acpidump/acpidump.8

Loading...
View Options

usr.sbin/acpi/acpidump/acpidump.c

Loading...