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
