Date: Fri, 23 Aug 2013 14:44:09 -0700 From: Alfred Perlstein <alfred@freebsd.org> To: Vijay Singh <vijju.singh@gmail.com> Cc: Adrian Chadd <adrian@freebsd.org>, "net@freebsd.org" <net@freebsd.org> Subject: Re: Making IB a first class citizen. Message-ID: <5217D7A9.109@freebsd.org> In-Reply-To: <536C3E8E-5998-4A3B-A019-127F6C00B277@gmail.com> References: <521792CF.2060905@freebsd.org> <CAJ-VmokUZkZoYXtvLfYorbBWPbQCj5M_xxUcu136VvhTV6Gzhw@mail.gmail.com> <536C3E8E-5998-4A3B-A019-127F6C00B277@gmail.com>
next in thread | previous in thread | raw e-mail | index | archive | help
On 8/23/13 2:29 PM, Vijay Singh wrote: > We've been running with this change at work for some time and it doesn't seem to be impacting performance at all. We have a statically routed environment though. Also if we really want to optimize for performance wrt routing then IMHO we need to bring back route caching to the tcpcb. Just a thought. Thanks Vijay, I'll give a little more time and then push this change in. -Alfred > 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. >> >> >> >> -adrian >> >> >> >> On 23 August 2013 09:50, Alfred Perlstein <alfred@freebsd.org> wrote: >> >>> Hello -net. >>> >>> This email is about making Infiniband a first class citizen of the FreeBSD >>> kernel. >>> >>> Right now we have one #ifdef OFED in the src tree that makes compiling >>> modules a real challenge: >>> >>> 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 for >>> 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. >>> >>> 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. >>> >>> If there's no objection I'd like to push this change into head in the next >>> day or two. The only difference is +16 bytes to the "struct llentry". >>> >>> Comments? >>> >>> ______________________________**_________________ >>> freebsd-net@freebsd.org mailing list >>> http://lists.freebsd.org/**mailman/listinfo/freebsd-net<http://lists.freebsd.org/mailman/listinfo/freebsd-net>; >>> To unsubscribe, send any mail to "freebsd-net-unsubscribe@**freebsd.org<freebsd-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" > _______________________________________________ > 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?5217D7A9.109>