Page MenuHomeFreeBSD

Fix sleepq_add panic happening with too wide net epoch.
ClosedPublic

Authored by melifaro on Oct 17 2020, 11:28 AM.
Tags
None
Referenced Files
Unknown Object (File)
Dec 20 2023, 4:21 AM
Unknown Object (File)
Sep 7 2023, 9:11 AM
Unknown Object (File)
Sep 7 2023, 9:11 AM
Unknown Object (File)
Sep 7 2023, 9:10 AM
Unknown Object (File)
Sep 7 2023, 9:10 AM
Unknown Object (File)
Sep 1 2023, 8:35 PM
Unknown Object (File)
Sep 1 2023, 1:29 AM
Unknown Object (File)
Jun 25 2023, 5:15 PM

Details

Summary

Fix panic described in kern/250413 introduced by D25445.

Diff Detail

Repository
rS FreeBSD src repository - subversion
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

melifaro added a reviewer: network.
melifaro added a subscriber: kib.

It seems the only solution here is taking ifnet reference. I'm not sure about PCB, probably it can not disappear here.

sys/netinet/in_mcast.c
2087 ↗(On Diff #78344)

IN_MULTI_LOCK() is sx lock, so you still can sleep while in epoch.

This revision is now accepted and ready to land.Oct 17 2020, 5:01 PM
head/sys/netinet/in_mcast.c
2085

I believe it would be not too hard to move this EXIT to before corresponding breaks inside switch. Current structure where ENTER is done inside switch, and you expect all cases escape with epoch entered, is too weird/fragile.