Wow! Such an awesome reproducer. Very detailed and easy to follow. Thanks for providing it.
I was able to verify the problem here, and looked at the upstream patch[1] that fixes it. The rationale makes sense to me, although that cleanup function is pretty involved and it's hard to say if there can be any fallout from moving the worker thread pool freeing action earlier.
I looked at the upstream repository and could not find any amends/fixes to the commit in question. It's present in Mantic and Noble, which is a good sign.
Do you have a PPA build that I can try with the reproducer to check the fix, please?
Otherwise, this LGTM and I'm +1 on proceeding with the SRU. Thanks!
Mauricio,
Wow! Such an awesome reproducer. Very detailed and easy to follow. Thanks for providing it.
I was able to verify the problem here, and looked at the upstream patch[1] that fixes it. The rationale makes sense to me, although that cleanup function is pretty involved and it's hard to say if there can be any fallout from moving the worker thread pool freeing action earlier.
I looked at the upstream repository and could not find any amends/fixes to the commit in question. It's present in Mantic and Noble, which is a good sign.
Do you have a PPA build that I can try with the reproducer to check the fix, please?
Otherwise, this LGTM and I'm +1 on proceeding with the SRU. Thanks!
[1]: For reference: https:/ /gitlab. com/libvirt/ libvirt/ -/commit/ 152770333449cd3 b78b4f5a9f1148f c1f482d842