Date: Mon, 2 Jun 2014 22:01:00 -0700 From: Adrian Chadd <adrian@freebsd.org> To: Yong-Hyeon Pyun <pyunyh@gmail.com> Cc: "svn-src-head@freebsd.org" <svn-src-head@freebsd.org>, "svn-src-all@freebsd.org" <svn-src-all@freebsd.org>, Marcel Moolenaar <marcel@freebsd.org>, "src-committers@freebsd.org" <src-committers@freebsd.org>, Marcel Moolenaar <marcel@xcllnt.net> Subject: Re: svn commit: r266974 - in head/sys: dev/dc dev/fxp dev/mii dev/netmap kern net Message-ID: <CAJ-Vmomt%2BHN7veD79a7h%2Bv9P4R6LdhAJpRB79yjwGd3H%2BdSpow@mail.gmail.com> In-Reply-To: <20140603012939.GA2998@michelle.cdnetworks.com> References: <201406021754.s52Hsd1B039620@svn.freebsd.org> <CAJ-VmonRqXrzrqQys_9s7-o8DFmOE_c_BZWupezm0i%2BW7E09vA@mail.gmail.com> <CAJ-VmonDJCLYWBPiz1ntFRm8aj4WQd5iHMKK0Wm3hcdjZmxghw@mail.gmail.com> <DA4CB2BA-A551-47A4-9E44-766FA29E2EF2@xcllnt.net> <20140603012939.GA2998@michelle.cdnetworks.com>
next in thread | previous in thread | raw e-mail | index | archive | help
On 2 June 2014 18:29, Yonghyeon PYUN <pyunyh@gmail.com> wrote: > On Mon, Jun 02, 2014 at 11:42:10AM -0700, Marcel Moolenaar wrote: >> >> On Jun 2, 2014, at 11:27 AM, Adrian Chadd <adrian@freebsd.org> wrote: >> >> > .. and actually, bikeshedding for a moment, would we be able to move a >> > lot of these accessor methods over to inlines? Would that break the >> > Juniper way of doing things? >> >> That would definitely break Juniper as it doesn't give a stable >> ABI. >> >> I've suggested an approach that allows for both, but it was deemed >> unnecessary. The argument being that the function call overhead is >> negligible. >> >> We can always revisit that decision if needed... >> > > The function call overheads shall show measurable differences on > slow boxes. This change adds several function calls in driver's > fast path(interrupt handler, packet statistics, checksum > offloading checking and etc) and these functions would be called > on every TX/RX packet. > It would be great if there is a way to minimize function call > overheads in fast path. I wonder if in the short term we should just use inlines for now, at least so the methodization can get done without hurting people on ARM/MIPS. ARM/MIPS people (me included) - let's see if we can generate some number? -a
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?CAJ-Vmomt%2BHN7veD79a7h%2Bv9P4R6LdhAJpRB79yjwGd3H%2BdSpow>