Date: Fri, 23 Aug 2013 14:29:49 -0700 From: Vijay Singh <vijju.singh@gmail.com> To: Adrian Chadd <adrian@freebsd.org> Cc: Alfred Perlstein <alfred@freebsd.org>, "net@freebsd.org" <net@freebsd.org> Subject: Re: Making IB a first class citizen. Message-ID: <536C3E8E-5998-4A3B-A019-127F6C00B277@gmail.com> In-Reply-To: <CAJ-VmokUZkZoYXtvLfYorbBWPbQCj5M_xxUcu136VvhTV6Gzhw@mail.gmail.com> References: <521792CF.2060905@freebsd.org> <CAJ-VmokUZkZoYXtvLfYorbBWPbQCj5M_xxUcu136VvhTV6Gzhw@mail.gmail.com>
next in thread | previous in thread | raw e-mail | index | archive | help
We've been running with this change at work for some time and it doesn't see= m to be impacting performance at all. We have a statically routed environmen= t though. Also if we really want to optimize for performance wrt routing the= n IMHO we need to bring back route caching to the tcpcb. Just a thought. Sent from my iPhone On Aug 23, 2013, at 1:52 PM, Adrian Chadd <adrian@freebsd.org> wrote: > .. should just check to see what impact it has on performance in the > general case. that may change the cache behaviour of the ARP / routing > table code. >=20 >=20 >=20 > -adrian >=20 >=20 >=20 > On 23 August 2013 09:50, Alfred Perlstein <alfred@freebsd.org> wrote: >=20 >> Hello -net. >>=20 >> This email is about making Infiniband a first class citizen of the FreeBS= D >> kernel. >>=20 >> Right now we have one #ifdef OFED in the src tree that makes compiling >> modules a real challenge: >>=20 >> In sys/net/if_llatbl.h the "struct llentry" size changes based on if OFED= >> is compiled in or not, only by 16 bytes because Infiniband uses 20bytes f= or >> MAC. I am wondering if it would be OK to just unifdef this part to make >> inifiband a first class citizen of the kernel. Otherwise maybe we can >> reverse the ifdef so that it's WITHOUT_OFED and by default have it on. >>=20 >> I understand that we can not do this for FreeBSD 9.x due to breaking >> network ABI, however I think we still have time to do so in FreeBSD 10.x.= >>=20 >> If there's no objection I'd like to push this change into head in the nex= t >> day or two. The only difference is +16 bytes to the "struct llentry". >>=20 >> Comments? >>=20 >> ______________________________**_________________ >> freebsd-net@freebsd.org mailing list >> http://lists.freebsd.org/**mailman/listinfo/freebsd-net<http://lists.free= bsd.org/mailman/listinfo/freebsd-net> >> To unsubscribe, send any mail to "freebsd-net-unsubscribe@**freebsd.org<f= reebsd-net-unsubscribe@freebsd.org> >> " > _______________________________________________ > freebsd-net@freebsd.org mailing list > http://lists.freebsd.org/mailman/listinfo/freebsd-net > To unsubscribe, send any mail to "freebsd-net-unsubscribe@freebsd.org"
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?536C3E8E-5998-4A3B-A019-127F6C00B277>