Page MenuHomeFreeBSD
Differential D36161

mld6: use callout(9) directly instead of pr_slowtimo, pr_fasttimo
ClosedPublic
Actions

Authored by glebius on Aug 12 2022, 8:15 AM.
Tags
None
Referenced Files
Unknown Object (File)
Mon, Apr 13, 9:14 PM
Unknown Object (File)
Tue, Apr 7, 2:36 PM
Unknown Object (File)
Sun, Apr 5, 9:28 PM
Unknown Object (File)
Fri, Apr 3, 8:14 AM
Unknown Object (File)
Fri, Apr 3, 2:49 AM
Unknown Object (File)
Thu, Apr 2, 10:41 PM
Unknown Object (File)
Wed, Apr 1, 3:20 PM
Unknown Object (File)
Thu, Mar 26, 10:29 AM
View All Files
Subscribers
ae
bz
imp
melifaro

Details

Reviewers
melifaro
Group Reviewers
network
Commits
rGb730de8badbc: mld6: use callout(9) directly instead of pr_slowtimo, pr_fasttimo

Diff Detail

Repository
rG FreeBSD src repository
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

glebius created this revision.Aug 12 2022, 8:15 AM
Herald added subscribers: melifaro, bz, ae, imp. · View Herald TranscriptAug 12 2022, 8:15 AM
glebius requested review of this revision.Aug 12 2022, 8:15 AM
glebius added a parent revision: D36160: igmp: use callout(9) directly instead of pr_slowtimo, pr_fasttimo.
Harbormaster completed remote builds in B46885: Diff 109227.Aug 12 2022, 8:15 AM
glebius added a child revision: D36162: frag6: use callout(9) directly instead of pr_slowtimo.Aug 12 2022, 8:15 AM
glebius updated this revision to Diff 109231.Aug 12 2022, 8:20 AM

Rebase - part belonged to other diff.

Harbormaster completed remote builds in B46889: Diff 109231.Aug 12 2022, 8:20 AM
melifaro accepted this revision.Aug 12 2022, 11:08 AM
This revision is now accepted and ready to land.Aug 12 2022, 11:08 AM
melifaro added inline comments.Aug 12 2022, 11:21 AM
sys/netinet6/mld6.c
1726

Probably worth using callout_reset_sbt() to indicate absolute timeout.

melifaro added inline comments.Aug 12 2022, 11:27 AM
sys/netinet6/mld6.c
1298

We need to enter epoch here.

mld_fasttimo_vnet() calls mld_v2_dispatch_general_query() which requires epoch to be present.

glebius added inline comments.Aug 12 2022, 3:08 PM
sys/netinet6/mld6.c
1298

Thanks! Will do.

1726

I'd better leave functional improvements for future, not in this changeset.

glebius updated this revision to Diff 109249.Aug 12 2022, 3:28 PM

Fix epoch issues.

This revision now requires review to proceed.Aug 12 2022, 3:28 PM
Harbormaster completed remote builds in B46901: Diff 109249.Aug 12 2022, 3:28 PM
melifaro accepted this revision.Aug 17 2022, 4:00 PM
This revision is now accepted and ready to land.Aug 17 2022, 4:00 PM
glebius updated this revision to Diff 109471.Aug 17 2022, 5:00 PM

Rename PR_FOOHZ to MLD_FOOHZ

This revision now requires review to proceed.Aug 17 2022, 5:00 PM
Harbormaster completed remote builds in B47002: Diff 109471.Aug 17 2022, 5:00 PM
melifaro accepted this revision.Aug 17 2022, 6:09 PM
This revision is now accepted and ready to land.Aug 17 2022, 6:09 PM
Closed by commit rGb730de8badbc: mld6: use callout(9) directly instead of pr_slowtimo, pr_fasttimo (authored by glebius). · Explain WhyAug 17 2022, 6:54 PM
This revision was automatically updated to reflect the committed changes.
glebius added a commit: rGb730de8badbc: mld6: use callout(9) directly instead of pr_slowtimo, pr_fasttimo.

Revision Contents
Changeset List

PathSize
sys/
netinet/
2 lines
netinet6/
14 lines
3 lines
2 lines
42 lines
7 lines

Diff 109485

View Options

sys/netinet/icmp6.h

Loading...
View Options

sys/netinet6/icmp6.c

Loading...
View Options

sys/netinet6/in6.c

Loading...
View Options

sys/netinet6/in6_proto.c

Loading...
View Options

sys/netinet6/mld6.c

Loading...
View Options

sys/netinet6/mld6_var.h

Loading...