Date: Mon, 13 Jun 2022 14:18:55 -0700 From: Eric Joyner <erj@freebsd.org> To: John Baldwin <jhb@freebsd.org> Cc: src-committers@freebsd.org, dev-commits-src-all@freebsd.org, dev-commits-src-main@freebsd.org Subject: Re: git: cdcd52d41e24 - main - irdma: Add RDMA driver for Intel(R) Ethernet Controller E810 Message-ID: <CAKdFRZibhP3Ap_S_Gj=CY3uhVdLxkS2C%2BaeqfZf18wyn==TUiA@mail.gmail.com> In-Reply-To: <c0b2824f-6386-b8ba-90f3-f347155ba87b@FreeBSD.org> References: <202205232354.24NNs69N032987@gitrepo.freebsd.org> <c0b2824f-6386-b8ba-90f3-f347155ba87b@FreeBSD.org>
next in thread | previous in thread | raw e-mail | index | archive | help
--00000000000085d1e305e15ad653 Content-Type: text/plain; charset="UTF-8" On Fri, Jun 10, 2022 at 3:29 PM John Baldwin <jhb@freebsd.org> wrote: > On 5/23/22 4:54 PM, Eric Joyner wrote: > > The branch main has been updated by erj: > > > > URL: > https://cgit.FreeBSD.org/src/commit/?id=cdcd52d41e246ba1c0fcfad0769bd691487355ef > > > > commit cdcd52d41e246ba1c0fcfad0769bd691487355ef > > Author: Bartosz Sobczak <bartosz.sobczak@intel.com> > > AuthorDate: 2022-05-23 23:39:27 +0000 > > Commit: Eric Joyner <erj@FreeBSD.org> > > CommitDate: 2022-05-23 23:52:49 +0000 > > > > irdma: Add RDMA driver for Intel(R) Ethernet Controller E810 > > > > This is an initial commit for RDMA FreeBSD driver for Intel(R) > Ethernet > > Controller E810, called irdma. Supporting both RoCEv2 and iWARP > > protocols in per-PF manner, RoCEv2 being the default. > > > > Testing has been done using krping tool, perftest, ucmatose, rping, > > ud_pingpong, rc_pingpong and others. > > > > Signed-off-by: Eric Joyner <erj@FreeBSD.org> > > > > Reviewed by: #manpages (pauamma_gundo.com) [documentation] > > MFC after: 1 week > > Relnotes: yes > > Sponsored by: Intel Corporation > > Differential Revision: https://reviews.freebsd.org/D34690 > > I'm trying to get the tree building again with GCC and ld.bfd does not like > libirdma.map at all complaining with an error: > > duplicate expression '*' in version information > > I looked at the version map, and it seems rather bogus to me and not like > version maps used anywhere else. It seems to be exporting all symbols in > two different versions? That would seem to defeat the point of having > symbol versions? > > /* Export symbols should be added below according to > Documentation/versioning.md document. */ > IRDMA_1.0 { > global: *; > local: *; > }; > > IRDMA_1.1 { > global: *; > } IRDMA_1.0; > > (I'm also not sure why lld accepts such a version map.) > > -- > John Baldwin > Hi, I don't know how to answer this; I'm going to forward this to the FreeBSD RDMA guy and another one (the first guy might still be out on vacation) and see if they can answer this. - Eric --00000000000085d1e305e15ad653 Content-Type: text/html; charset="UTF-8" Content-Transfer-Encoding: quoted-printable <div dir=3D"ltr"><div dir=3D"ltr">On Fri, Jun 10, 2022 at 3:29 PM John Bald= win <<a href=3D"mailto:jhb@freebsd.org">jhb@freebsd.org</a>> wrote:<b= r></div><div class=3D"gmail_quote"><blockquote class=3D"gmail_quote" style= =3D"margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding= -left:1ex">On 5/23/22 4:54 PM, Eric Joyner wrote:<br> > The branch main has been updated by erj:<br> > <br> > URL: <a href=3D"https://cgit.FreeBSD.org/src/commit/?id=3Dcdcd52d41e24= 6ba1c0fcfad0769bd691487355ef" rel=3D"noreferrer" target=3D"_blank">https://= cgit.FreeBSD.org/src/commit/?id=3Dcdcd52d41e246ba1c0fcfad0769bd691487355ef<= /a><br> > <br> > commit cdcd52d41e246ba1c0fcfad0769bd691487355ef<br> > Author:=C2=A0 =C2=A0 =C2=A0Bartosz Sobczak <<a href=3D"mailto:barto= sz.sobczak@intel.com" target=3D"_blank">bartosz.sobczak@intel.com</a>><b= r> > AuthorDate: 2022-05-23 23:39:27 +0000<br> > Commit:=C2=A0 =C2=A0 =C2=A0Eric Joyner <erj@FreeBSD.org><br> > CommitDate: 2022-05-23 23:52:49 +0000<br> > <br> >=C2=A0 =C2=A0 =C2=A0 irdma: Add RDMA driver for Intel(R) Ethernet Contr= oller E810<br> >=C2=A0 =C2=A0 =C2=A0 <br> >=C2=A0 =C2=A0 =C2=A0 This is an initial commit for RDMA FreeBSD driver = for Intel(R) Ethernet<br> >=C2=A0 =C2=A0 =C2=A0 Controller E810, called irdma.=C2=A0 Supporting bo= th RoCEv2 and iWARP<br> >=C2=A0 =C2=A0 =C2=A0 protocols in per-PF manner, RoCEv2 being the defau= lt.<br> >=C2=A0 =C2=A0 =C2=A0 <br> >=C2=A0 =C2=A0 =C2=A0 Testing has been done using krping tool, perftest,= ucmatose, rping,<br> >=C2=A0 =C2=A0 =C2=A0 ud_pingpong, rc_pingpong and others.<br> >=C2=A0 =C2=A0 =C2=A0 <br> >=C2=A0 =C2=A0 =C2=A0 Signed-off-by: Eric Joyner <erj@FreeBSD.org>= <br> >=C2=A0 =C2=A0 =C2=A0 <br> >=C2=A0 =C2=A0 =C2=A0 Reviewed by:=C2=A0 =C2=A0 #manpages (<a href=3D"ht= tp://pauamma_gundo.com" rel=3D"noreferrer" target=3D"_blank">pauamma_gundo.= com</a>) [documentation]<br> >=C2=A0 =C2=A0 =C2=A0 MFC after:=C2=A0 =C2=A0 =C2=A0 1 week<br> >=C2=A0 =C2=A0 =C2=A0 Relnotes:=C2=A0 =C2=A0 =C2=A0 =C2=A0yes<br> >=C2=A0 =C2=A0 =C2=A0 Sponsored by:=C2=A0 =C2=A0Intel Corporation<br> >=C2=A0 =C2=A0 =C2=A0 Differential Revision:=C2=A0 <a href=3D"https://re= views.freebsd.org/D34690" rel=3D"noreferrer" target=3D"_blank">https://revi= ews.freebsd.org/D34690</a><br> <br> I'm trying to get the tree building again with GCC and ld.bfd does not = like<br> libirdma.map at all complaining with an error:<br> <br> duplicate expression '*' in version information<br> <br> I looked at the version map, and it seems rather bogus to me and not like<b= r> version maps used anywhere else.=C2=A0 It seems to be exporting all symbols= in<br> two different versions?=C2=A0 That would seem to defeat the point of having= <br> symbol versions?<br> <br> /* Export symbols should be added below according to<br> =C2=A0 =C2=A0 Documentation/versioning.md document. */<br> IRDMA_1.0 {<br> =C2=A0 =C2=A0 =C2=A0 =C2=A0 global: *;<br> =C2=A0 =C2=A0 =C2=A0 =C2=A0 local: *;<br> };<br> <br> IRDMA_1.1 {<br> =C2=A0 =C2=A0 =C2=A0 =C2=A0 global: *;<br> } IRDMA_1.0;<br> <br> (I'm also not sure why lld accepts such a version map.)<br> <br> -- <br> John Baldwin<br></blockquote><div><br></div><div>Hi,</div><div><br></div><d= iv>I don't know how to answer this; I'm going to forward this to th= e FreeBSD RDMA guy and another one (the first guy might still be out on vac= ation) and see if they can answer this.</div><div><br></div><div>- Eric</di= v></div></div> --00000000000085d1e305e15ad653--
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?CAKdFRZibhP3Ap_S_Gj=CY3uhVdLxkS2C%2BaeqfZf18wyn==TUiA>