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

Reply via email to