Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

networking.7: erase confusing caveat #1316

Merged
merged 2 commits into from
Jul 13, 2024

Conversation

concussious
Copy link
Contributor

service routing restart isn't necessary and actually breaks routing.

Fixes: 75eda00
MFC after: 3 days
Reported by: Graham Perrin

@concussious
Copy link
Contributor Author

Tagging @grahamperrin

@concussious concussious marked this pull request as ready for review July 5, 2024 15:31
@grahamperrin
Copy link
Contributor

I guess it's OK, thanks.

@gmshake
Copy link
Contributor

gmshake commented Jul 8, 2024

I guess the workaround was copied from static network configuration. For DHCP it makes troubles. Maybe it is more clear to change the commit message to networking.7: Stop advising restarting routing for DHCP network .

@concussious concussious force-pushed the networking.7 branch 2 times, most recently from 3736f30 to cf560df Compare July 8, 2024 04:38
@concussious concussious changed the title networking.7: stop advising restarting routing networking.7: erase confusing caveat Jul 8, 2024
concussious added a commit to concussious/freebsd-src that referenced this pull request Jul 8, 2024
+ service routing restart breaks on dhcp
+ change my copyright from inbox to home

Fixes:		75eda00
MFC after:	3 days
Pointy hat:	me
Reported by:	Graham Perrin
Reviewed by:	zlei
Pull Request:	github.com/freebsd/pull/1316
concussious added a commit to concussious/freebsd-src that referenced this pull request Jul 8, 2024
+ service routing restart breaks on dhcp
+ change my copyright from inbox to home

Fixes:		75eda00
MFC after:	3 days
Pointy hat:	me
Reported by:	Graham Perrin, zlei
Pull Request:	freebsd#1316
@concussious
Copy link
Contributor Author

concussious commented Jul 8, 2024

I don't want to break 50 chr limit.
How's this instead?

Thanks.

@bsdimp bsdimp self-assigned this Jul 12, 2024
@bsdimp bsdimp added the ready label Jul 12, 2024
@bsdimp
Copy link
Member

bsdimp commented Jul 12, 2024

I'll land this later after I test out my multiple PR CI landing script.
I may change 'erase' to 'remove' since that's better english and if it's over 50 chars, I'm good with that extra char. :)

concussious added a commit to concussious/freebsd-src that referenced this pull request Jul 12, 2024
+ service routing restart breaks on dhcp
+ change my copyright from inbox to home

Fixes:		75eda00
MFC after:	3 days
Pointy hat:	me
Reported by:	Graham Perrin, zlei
Reviewed by:	imp
Pull Request:	freebsd#1316
@concussious
Copy link
Contributor Author

Changed it myself to save you some typing, thank you.

bsdimp pushed a commit to concussious/freebsd-src that referenced this pull request Jul 13, 2024
+ service routing restart breaks on dhcp
+ change my copyright from inbox to home

Fixes:		75eda00
MFC after:	3 days
Reported by:	Graham Perrin, zlei
Reviewed by:	imp
Pull Request:	freebsd#1316
@bsdimp bsdimp force-pushed the networking.7 branch 2 times, most recently from 3cfc78f to a6ca6df Compare July 13, 2024 03:39
concussious and others added 2 commits July 12, 2024 21:40
+ service routing restart breaks on dhcp
+ change my copyright from inbox to home

Fixes:		75eda00
MFC after:	3 days
Reported by:	Graham Perrin, zlei
Reviewed by:	imp
Pull Request:	freebsd#1316
This code is using the vnode after it has been released and causing a
panic when a p9fs shared volume is unmounted. In fact, it seems like it's
just duplicated code left behind from a bad merge.

PR:		279887
Reported by:	Michael Dexter

Reviewed by: imp
Pull Request: freebsd#1323
@freebsd-git freebsd-git merged commit a6ca6df into freebsd:main Jul 13, 2024
14 of 16 checks passed
freebsd-git pushed a commit that referenced this pull request Jul 20, 2024
+ service routing restart breaks on dhcp
+ change my copyright from inbox to home

Fixes:		75eda00
MFC after:	3 days
Reported by:	Graham Perrin, zlei
Reviewed by:	imp
Pull Request:	#1316

(cherry picked from commit c55f56d)
bsdjhb pushed a commit to bsdjhb/cheribsd that referenced this pull request Oct 30, 2024
+ service routing restart breaks on dhcp
+ change my copyright from inbox to home

Fixes:		75eda00
MFC after:	3 days
Reported by:	Graham Perrin, zlei
Reviewed by:	imp
Pull Request:	freebsd/freebsd-src#1316
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants