[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <20200102170426.GA9282@ziepe.ca>
Date: Thu, 2 Jan 2020 13:04:26 -0400
From: Jason Gunthorpe <jgg@...pe.ca>
To: "Saleem, Shiraz" <shiraz.saleem@...el.com>
Cc: "Kirsher, Jeffrey T" <jeffrey.t.kirsher@...el.com>,
"davem@...emloft.net" <davem@...emloft.net>,
"gregkh@...uxfoundation.org" <gregkh@...uxfoundation.org>,
"netdev@...r.kernel.org" <netdev@...r.kernel.org>,
"linux-rdma@...r.kernel.org" <linux-rdma@...r.kernel.org>,
"nhorman@...hat.com" <nhorman@...hat.com>,
"sassmann@...hat.com" <sassmann@...hat.com>,
"parav@...lanox.com" <parav@...lanox.com>
Subject: Re: [PATCH v3 19/20] RDMA: Add irdma Kconfig/Makefile and remove
i40iw
On Thu, Jan 02, 2020 at 04:00:37PM +0000, Saleem, Shiraz wrote:
> > Subject: Re: [PATCH v3 19/20] RDMA: Add irdma Kconfig/Makefile and remove
> > i40iw
> >
> >
> > > > - The whole cqp_compl_thread thing looks really weird
> > > What is the concern?
> >
> > It looks like an open coded work queue
> >
>
> The cqp_compl_thread is not a work queue in the sense
> that no work is queued to it. It is a thread that is signaled to
> check for and handle CQP completion events if present.
How is that not a work queue? The work is the signal to handle CQP
completion events.
Jason
Powered by blists - more mailing lists