Received-SPF: pass (google.com: best guess record for domain of linux-nfs-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) client-ip=209.132.180.67;
Content-Type: text/plain;
        charset=us-ascii
Mime-Version: 1.0 (Mac OS X Mail 12.4 \(3445.104.11\))
Subject: Re: [PATCH v3] rdma: Enable ib_alloc_cq to spread work over a
 device's comp_vectors
From:   Chuck Lever <chuck.lever@oracle.com>
In-Reply-To: <f181b5b6-df7c-d657-4ec6-4a4e56a9b5ff@acm.org>
Date:   Mon, 5 Aug 2019 13:15:22 -0400
Cc:     jgg@ziepe.ca, linux-rdma@vger.kernel.org,
        linux-cifs@vger.kernel.org,
        Linux NFS Mailing List <linux-nfs@vger.kernel.org>,
        v9fs-developer@lists.sourceforge.net
Content-Transfer-Encoding: quoted-printable
Message-Id: <87D288BC-65F8-4678-A8C5-DEF7C3355BC7@oracle.com>
References: <20190729171923.13428.52555.stgit@manet.1015granger.net>
 <f181b5b6-df7c-d657-4ec6-4a4e56a9b5ff@acm.org>
To:     Bart Van Assche <bvanassche@acm.org>
Sender: linux-nfs-owner@vger.kernel.org
Precedence: bulk

Hi Bart-

> On Aug 5, 2019, at 12:09 PM, Bart Van Assche <bvanassche@acm.org> =
wrote:
>=20
> On 7/29/19 10:22 AM, Chuck Lever wrote:
>> diff --git a/drivers/infiniband/ulp/srpt/ib_srpt.c =
b/drivers/infiniband/ulp/srpt/ib_srpt.c
>> index 1a039f1..e25c70a 100644
>> --- a/drivers/infiniband/ulp/srpt/ib_srpt.c
>> +++ b/drivers/infiniband/ulp/srpt/ib_srpt.c
>> @@ -1767,8 +1767,8 @@ static int srpt_create_ch_ib(struct =
srpt_rdma_ch *ch)
>>  		goto out;
>>    retry:
>> -	ch->cq =3D ib_alloc_cq(sdev->device, ch, ch->rq_size + sq_size,
>> -			0 /* XXX: spread CQs */, IB_POLL_WORKQUEUE);
>> +	ch->cq =3D ib_alloc_cq_any(sdev->device, ch, ch->rq_size + =
sq_size,
>> +				 IB_POLL_WORKQUEUE);
>>  	if (IS_ERR(ch->cq)) {
>>  		ret =3D PTR_ERR(ch->cq);
>>  		pr_err("failed to create CQ cqe=3D %d ret=3D %d\n",
> Hi Chuck,
>=20
> Please Cc me for future srp and srpt patches. I think my name appears =
next to both drivers in the MAINTAINERS file.

I see your name listed, but I thought the rule was to Cc: the mailing =
list
which is listed for that component. My bad.

--
Chuck Lever