On Wed, Oct 26, 2022 at 03:46:09PM +0300, Sagi Grimberg wrote: > This is clearer, but what I think would be even cleaner, is if we simply > move the whole first time to a different probe_work and treat it like it > is instead of relying on resources existence as a state indicators > (tagset/admin_q). The shared portion can move to helpers. I started playing with that a bit on my flight home this week. I think the right thing is to do away with the scheduled work for probe entirely and set the PROBE_PREFER_ASYNCHRONOUS flag in the driver, and that should also really help with probe error handling. But that's another fairly big set of changes over this already quite big series so I'd prefer to do it separately.
