On Thu, 2019-09-19 at 19:37 +0530, Allen wrote: > > > > + trans_pcie->rba.alloc_wq = alloc_workqueue("rb_allocator", > > + WQ_HIGHPRI | WQ_UNBOUND, 1); > > + if (!trans_pcie->rba.alloc_wq) { > > I would like to stick to if(unlikely(!trans_pcie->rba.alloc_wq) just > for consistency. That's just clutter, this path gets called exactly once in the lifetime of most systems ... > Let me know if I could add your SOB and send out V2. No no, I've already sent the patch on the way internally :) johannes