Thanks very much, Chuck -- glad to see things getting simplified! Rob
> On Feb 18, 2020, at 12:39 PM, Chuck Silvers <c...@chuq.com> wrote: > > eventually I realized that the aiodone_queue workqueue thread was > made redundant long ago... we no longer need this mechanism to hand off > any specific iodone processing to a worker thread because these days > all iodone processing is done in a (softint) worker thread. > I just commited a patch to remove this workqueue thread and associated code. > > -Chuck > > > On Wed, Feb 12, 2020 at 05:03:39PM -0800, Rob Newberry wrote: >> Thanks very much, Chuck -- this patch fixed my problem. >> >> I noticed you removed a couple of KASSERTs -- shouldn't those be cases be >> EVEN MORE true now than they were before? Given what I debugged, I'm >> wondering if the asserts would help make sure future code doesn't end up >> trying to do something similar in the future... >> >> Rob