Re: kern/170713: [cxgb] Driver must be loaded after boot due to timing issues checking for kern.ipc.nmb* values set via /boot/loader.conf

2012-08-22 Thread Garrett Cooper
The following reply was made to PR kern/170713; it has been noted by GNATS. From: Garrett Cooper To: Navdeep Parhar Cc: bug-follo...@freebsd.org Subject: Re: kern/170713: [cxgb] Driver must be loaded after boot due to timing issues checking for kern.ipc.nmb* values set via /boot/loader.conf Dat

Re: kern/170904: [ath] ath driver: configure related parameters when radar detection (DFS) is enabled

2012-08-22 Thread linimon
Synopsis: [ath] ath driver: configure related parameters when radar detection (DFS) is enabled Responsible-Changed-From-To: freebsd-net->freebsd-wireless Responsible-Changed-By: linimon Responsible-Changed-When: Thu Aug 23 03:47:27 UTC 2012 Responsible-Changed-Why: Set more canonical assignment.

Re: kern/170904: [ath] ath driver: configure related parameters when radar detection (DFS) is enabled

2012-08-22 Thread gjb
Synopsis: [ath] ath driver: configure related parameters when radar detection (DFS) is enabled Class-Changed-From-To: sw-bug->change-request Class-Changed-By: gjb Class-Changed-When: Thu Aug 23 03:40:21 UTC 2012 Class-Changed-Why: Undo class change with previous edit. http://www.freebsd.org/cg

Re: kern/170904: [ath] ath driver: configure related parameters when radar detection (DFS) is enabled

2012-08-22 Thread gjb
Old Synopsis: ath driver: configure related parameters when radar detection (DFS) is enabled New Synopsis: [ath] ath driver: configure related parameters when radar detection (DFS) is enabled Class-Changed-From-To: change-request->sw-bug Class-Changed-By: gjb Class-Changed-When: Thu Aug 23 03:38

Re: Incorrect ARP table entries

2012-08-22 Thread Peter Jeremy
On 2012-Aug-22 14:02:01 +1000, Peter Jeremy wrote: >I've run into a problem where the ARP table on several of my hosts is >apparently spontaneously replacing correct entries with incorrect MAC >addresses. I've done some digging with tcpdump and can't identify the >cause. I've tried to look in th

Re: kern/170713: [cxgb] Driver must be loaded after boot due to timing issues checking for kern.ipc.nmb* values set via /boot/loader.conf

2012-08-22 Thread Navdeep Parhar
The following reply was made to PR kern/170713; it has been noted by GNATS. From: Navdeep Parhar To: bug-follo...@freebsd.org, yaneg...@gmail.com Cc: Subject: Re: kern/170713: [cxgb] Driver must be loaded after boot due to timing issues checking for kern.ipc.nmb* values set via /boot/loader.co

Re: Replace bcopy() to update ether_addr

2012-08-22 Thread Luigi Rizzo
On Wed, Aug 22, 2012 at 03:21:06PM -0400, John Baldwin wrote: > On Wednesday, August 22, 2012 2:54:07 pm Adrian Chadd wrote: > > On 22 August 2012 05:02, John Baldwin wrote: > > > On Tuesday, August 21, 2012 12:34:42 pm Adrian Chadd wrote: > > >> Hi, > > >> > > >> What about just creating an ETHER

Re: Replace bcopy() to update ether_addr

2012-08-22 Thread Wojciech Puchar
but wonder whether the same does hold for MIPS/ARM. Getting it wrong there will lead to some very very poor performing code. 1) do - as already pointed out - standard copy of structure in C. 2) if compiler is found to generate bad code on some archs put assembly. 1 even if compiler is not smart

Re: Replace bcopy() to update ether_addr

2012-08-22 Thread John Baldwin
On Wednesday, August 22, 2012 2:54:07 pm Adrian Chadd wrote: > On 22 August 2012 05:02, John Baldwin wrote: > > On Tuesday, August 21, 2012 12:34:42 pm Adrian Chadd wrote: > >> Hi, > >> > >> What about just creating an ETHER_ADDR_COPY(dst, src) and putting that > >> in a relevant include file, the

Re: Replace bcopy() to update ether_addr

2012-08-22 Thread Warner Losh
On Aug 22, 2012, at 12:54 PM, Adrian Chadd wrote: > On 22 August 2012 05:02, John Baldwin wrote: >> On Tuesday, August 21, 2012 12:34:42 pm Adrian Chadd wrote: >>> Hi, >>> >>> What about just creating an ETHER_ADDR_COPY(dst, src) and putting that >>> in a relevant include file, then hide the ug

Re: Replace bcopy() to update ether_addr

2012-08-22 Thread Adrian Chadd
On 22 August 2012 05:02, John Baldwin wrote: > On Tuesday, August 21, 2012 12:34:42 pm Adrian Chadd wrote: >> Hi, >> >> What about just creating an ETHER_ADDR_COPY(dst, src) and putting that >> in a relevant include file, then hide the ugliness there? >> >> The same benefits will likely appear whe

Re: speed tests (Re: Replace bcopy() to update ether_addr)

2012-08-22 Thread Luigi Rizzo
On Wed, Aug 22, 2012 at 05:26:47PM +0300, Mitya wrote: > 22.08.2012 17:36, Luigi Rizzo ??: > >On Wed, Aug 22, 2012 at 02:32:21AM +, Bruce Evans wrote: > >>luigi wrote: > >> > >>>even more orthogonal: > >>> > >>>I found that copying 8n + (5, 6 or 7) bytes was much much slower than >

Re: speed tests (Re: Replace bcopy() to update ether_addr)

2012-08-22 Thread Mitya
22.08.2012 17:36, Luigi Rizzo написал: On Wed, Aug 22, 2012 at 02:32:21AM +, Bruce Evans wrote: luigi wrote: even more orthogonal: I found that copying 8n + (5, 6 or 7) bytes was much much slower than copying a multiple of 8 bytes. For n=0, 1,2,4,8 bytes are efficient, other cases are slo

speed tests (Re: Replace bcopy() to update ether_addr)

2012-08-22 Thread Luigi Rizzo
On Wed, Aug 22, 2012 at 02:32:21AM +, Bruce Evans wrote: > luigi wrote: > > > even more orthogonal: > > > > I found that copying 8n + (5, 6 or 7) bytes was much much slower than > > copying a multiple of 8 bytes. For n=0, 1,2,4,8 bytes are efficient, > > other cases are slow (turned into 2 or

Re: Replace bcopy() to update ether_addr

2012-08-22 Thread Warner Losh
On Aug 22, 2012, at 6:02 AM, John Baldwin wrote: > On Tuesday, August 21, 2012 12:34:42 pm Adrian Chadd wrote: >> Hi, >> >> What about just creating an ETHER_ADDR_COPY(dst, src) and putting that >> in a relevant include file, then hide the ugliness there? >> >> The same benefits will likely app

Re: Replace bcopy() to update ether_addr

2012-08-22 Thread John Baldwin
On Tuesday, August 21, 2012 12:34:42 pm Adrian Chadd wrote: > Hi, > > What about just creating an ETHER_ADDR_COPY(dst, src) and putting that > in a relevant include file, then hide the ugliness there? > > The same benefits will likely appear when copying wifi MAC addresses > to/from headers. > >

Re: Replace bcopy() to update ether_addr

2012-08-22 Thread Bruce Evans
mitya wrote: > 22.08.2012 05:07, Bruce Evans íàïèñàë: > >> On Mon, Aug 20, 2012 at 05:46:12PM +0300, Mitya wrote: > >>> Hi. > >>> I found some overhead code in /src/sys/net/if_ethersubr.c and > >>> /src/sys/netgraph/ng_ether.c > >>> > >>> It contains strings, like bcopy(src, dst, ETHER_ADDR_LEN);