Re: [PATCH v3 00/13] Request for Comments on SoftiWarp

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 




-----Jason Gunthorpe <jgg@xxxxxxxx> wrote: -----

>To: Bernard Metzler <BMT@xxxxxxxxxxxxxx>
>From: Jason Gunthorpe <jgg@xxxxxxxx>
>Date: 02/02/2018 07:57PM
>Cc: Steve Wise <swise@xxxxxxxxxxxxxxxxxxxxx>,
>linux-rdma@xxxxxxxxxxxxxxx
>Subject: Re: [PATCH v3 00/13] Request for Comments on SoftiWarp
>
>On Fri, Feb 02, 2018 at 02:37:52PM +0000, Bernard Metzler wrote:
>
>> I added libsiw to the current (as of today) rmda-core bundle
>> as found at
>https://urldefense.proofpoint.com/v2/url?u=https-3A__github.com_linux
>-2Drdma_rdma-2Dcore.git&d=DwIBAg&c=jf_iaSHvJObTbx-siA1ZOg&r=2TaYXQ0T-
>r8ZO1PP1alNwU_QJcRRLfmYTAgd3QCvqSc&m=UyHYNdR3HpzdZu5UEC-C3m-y2oY_mSux
>ocEezYKO-Sw&s=_1Odt9orPiTCNcrPz9U1gEwckeRc4RoeUwPsvPogtCI&e=, see
>
>Can you use the standard OFA license please? IBM is an OFA member and
>is behloden to the membership agreement requring the common dual
>license.
>

sorry to see that. What's exactly wrong with the current (dual) license 
statement from IBM? I checked several other user libs. All come with
some flavor of a dual BSD/GPLv2 license statement as this one,
as far as I can see.

>> It enables user level clients for the siw driver. Let me
>> know if we need a patch for user land as well for now.
>> I don't want to spam the list.
>
>We will eventually..
>
>+static const struct verbs_match_ent rnic_table[] = {
>+	VERBS_NAME_MATCH("siw", NULL),
>+	{},
>+};
>
>I really don't want to add more VERBS_NAME_MATCH, but I don't have an
>alternative ready right now :(
>

Hmm, I blindly followed what others did. This is different from
what libsiw had until now (doing its own sys entries reading/pattern
matching). I hope I did it right?
In any case, I hope this current approach allows for co-existence
of user libs from different vendors if heterogeneous hardware is
in use.

Thanks very much!
Bernard.

--
To unsubscribe from this list: send the line "unsubscribe linux-rdma" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html



[Index of Archives]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Photo]     [Yosemite News]     [Yosemite Photos]     [Linux Kernel]     [Linux SCSI]     [XFree86]

  Powered by Linux