Page MenuHomeFreeBSD

cxgbe/t4_tom: the aio tx job queue must be empty by the time the toepcb is freed.
ClosedPublic

Authored by np on Jun 28 2019, 7:13 PM.
Tags
None
Referenced Files
Unknown Object (File)
Sat, Dec 13, 4:51 AM
Unknown Object (File)
Thu, Dec 11, 1:38 AM
Unknown Object (File)
Fri, Nov 28, 1:43 AM
Unknown Object (File)
Wed, Nov 26, 4:37 AM
Unknown Object (File)
Fri, Nov 21, 7:28 AM
Unknown Object (File)
Thu, Nov 20, 9:50 PM
Unknown Object (File)
Thu, Nov 20, 9:44 PM
Unknown Object (File)
Thu, Nov 20, 9:43 PM
Subscribers

Diff Detail

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

Event Timeline

sys/dev/cxgbe/tom/t4_tom.c
297 ↗(On Diff #59167)

Now that the jobq holds a reference on the socket (in effect), I think that hold on the socket prevents release_offload_resources() from being called? If soref() alone doesn't prevent that, then I'm less certain this will actually be true.

On further thought, I think this is ok. If the jobq isn't empty, then at least one job exists, and that job would hold a reference on the struct file and thus keep the socket "open".

This revision is now accepted and ready to land.Jun 28 2019, 7:19 PM