Page MenuHomeFreeBSD

ktls: fix vnet-related panic in ktls_reset_receive_tag()
ClosedPublic

Authored by mm on Jan 11 2024, 12:29 PM.
Tags
None
Referenced Files
Unknown Object (File)
Fri, Apr 26, 7:42 PM
Unknown Object (File)
Thu, Apr 25, 11:56 PM
Unknown Object (File)
Thu, Apr 25, 6:55 PM
Unknown Object (File)
Sat, Apr 6, 1:40 PM
Unknown Object (File)
Sat, Apr 6, 11:19 AM
Unknown Object (File)
Mar 18 2024, 8:46 AM
Unknown Object (File)
Mar 18 2024, 8:45 AM
Unknown Object (File)
Mar 18 2024, 8:45 AM
Subscribers

Details

Summary

I have reproducibly triggered a panic with sorele() in ktls_reset_receive_tag() being out of vnet context when using a mlx5 with KTLS with lagg and vlan.

Diff Detail

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

Event Timeline

mm requested review of this revision.Jan 11 2024, 12:29 PM
This revision is now accepted and ready to land.Jan 11 2024, 3:00 PM

For future reference, uploading diffs with context (e.g. with git-arc) makes reviewing easier. I don't think we need the vnet around if_rele() (in case it calls if_free), so I think this is correct.

In D43400#989566, @jhb wrote:

For future reference, uploading diffs with context (e.g. with git-arc) makes reviewing easier. I don't think we need the vnet around if_rele() (in case it calls if_free), so I think this is correct.

You can also provide context on diffs using 'git diff -U999999'.