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)
Tue, Mar 24, 6:59 AM
Unknown Object (File)
Tue, Mar 24, 6:40 AM
Unknown Object (File)
Mon, Mar 23, 11:51 AM
Unknown Object (File)
Mon, Mar 23, 11:46 AM
Unknown Object (File)
Sat, Mar 21, 11:30 PM
Unknown Object (File)
Sat, Mar 14, 5:31 PM
Unknown Object (File)
Sat, Mar 14, 3:28 AM
Unknown Object (File)
Wed, Mar 4, 6:16 PM
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...