IMO, the way to start this would be to go back to SF and ask them if they'll contribute their newer codebase under the CDDL again (they have a shared codebase for their platforms, which is why some of the esoterica in that codebase exists); it shouldn't be "that hard" to convince someone since it's a shared codebase and they already got the relevant signoff for CDDL.
If so, then that answers some of the questions about cleaning up the bits of the codebase (versus attempting to not diverge from upstream); if not, that's another answer. It's also quite possible they don't have working 40GbE support on Solaris-type platforms yet, so what we get if we ask may not work... (The source tree for their sfutils stuff is also available, though I don't know that the Solaris-specific source, if any, is...could ask about that too.) Gabriele, would you like to ask, or should I? - Rich On Wed, Nov 19, 2014 at 10:08 PM, Gordon Ross <[email protected]> wrote: > As Rich mentioned, it looks like there was unfinished cleanup requested > during review: > > http://www.listbox.com/member/archive/182179/2014/01/search/c2Z4Z2U/sort/time_rev/page/1/?search_for=sfxge > > Would be great if someone could finish that up and propose this driver for > integration. > > > On Tue, Nov 18, 2014 at 10:46 AM, Gabriele Bulfon via illumos-discuss < > [email protected]> wrote: > >> Hi, >> >> thanks for the info. >> The driver is now installable directly from our latest IPS repo and >> merged into our illumos fork at GitHub. >> >> Gabriele. >> >> ------------------------------ >> >> >> *Da:* Rich via illumos-discuss <[email protected]> >> *A:* Illumos Discussion <[email protected]> Gabriele Bulfon < >> [email protected]> >> *Data:* 18 novembre 2014 13.35.18 CET >> *Oggetto:* Re: [discuss] sfxge webrev >> >> >> There were a number of issues raised when I initially put it up for >> comment, and I haven't had a chance to go back to it and clean them up. >> >> I've not had the driver cause any sort of system failure in years of use, >> for whatever that's worth. >> >> - Rich >> >> On Tue, Nov 18, 2014 at 4:08 AM, Gabriele Bulfon via illumos-discuss < >> [email protected]> wrote: >> >>> Hi, >>> is there any particular reason why this webrev is not yet included as >>> default inside illumos source tree? >>> May I consider it safe to add it in our fork? >>> >>> http://cr.illumos.org/~webrev/rincebrain/illumos-sfxge/ >>> >>> Thanks, >>> Gabriele. >>> >>> *illumos-discuss* | Archives >>> <https://www.listbox.com/member/archive/182180/=now> >>> <https://www.listbox.com/member/archive/rss/182180/21253829-71dc39ef> | >>> Modify <https://www.listbox.com/member/?&;> Your Subscription >>> <http://www.listbox.com> >>> >> >> *illumos-discuss* | Archives >> <https://www.listbox.com/member/archive/182180/=now> >> <https://www.listbox.com/member/archive/rss/182180/21175541-02f10c6f> | >> Modify <https://www.listbox.com/member/?&;> Your Subscription >> <http://www.listbox.com> >> >> *illumos-discuss* | Archives >> <https://www.listbox.com/member/archive/182180/=now> >> <https://www.listbox.com/member/archive/rss/182180/21175550-721fcbaf> | >> Modify >> <https://www.listbox.com/member/?&;> >> Your Subscription <http://www.listbox.com> >> > > ------------------------------------------- illumos-discuss Archives: https://www.listbox.com/member/archive/182180/=now RSS Feed: https://www.listbox.com/member/archive/rss/182180/21175430-2e6923be Modify Your Subscription: https://www.listbox.com/member/?member_id=21175430&id_secret=21175430-6a77cda4 Powered by Listbox: http://www.listbox.com
