Skip site navigation (1)Skip section navigation (2)
Date:      Sat, 16 Apr 2005 10:01:33 -0700
From:      John-Mark Gurney <gurney_j@resnet.uoregon.edu>
To:        Pyun YongHyeon <yongari@rndsoft.co.kr>
Cc:        freebsd-sparc64@freebsd.org
Subject:   Re: em(4) patch
Message-ID:  <20050416170133.GA16099@funkthat.com>
In-Reply-To: <20050416004627.GA11603@michelle.rndsoft.co.kr>
References:  <20050414092608.GB2855@michelle.rndsoft.co.kr> <20050414192749.GO56487@funkthat.com> <20050415023902.GC7393@michelle.rndsoft.co.kr> <20050415030935.GQ56487@funkthat.com> <20050415042731.GD7393@michelle.rndsoft.co.kr> <20050415051112.GE7393@michelle.rndsoft.co.kr> <Pine.SOC.4.61.0504151208040.3463@tea.blinkenlights.nl> <20050415154309.GS56487@funkthat.com> <20050416004627.GA11603@michelle.rndsoft.co.kr>

next in thread | previous in thread | raw e-mail | index | archive | help
Pyun YongHyeon wrote this message on Sat, Apr 16, 2005 at 09:46 +0900:
> On Fri, Apr 15, 2005 at 08:43:09AM -0700, John-Mark Gurney wrote:
>  > Sten Spans wrote this message on Fri, Apr 15, 2005 at 12:09 +0200:
>  > > >With your changes JUMBO frames work here. This should also fix PR75794.
>  > > >It would be really great if your changes could be comitted.
>  > > 
>  > > Let's not forget ipv6 please, it needs this fix too,
>  > > if the code is imported.
>  > 
>  > And netatalk, netipsec and netipx too will probably need these changes..
>  > 
>  > Another reason I haven't brought in the code quite yet...
> 
> Hmm, this means we have to patch all filrewalls(pf/ipfw/ipf), carp(4)
> and possibily netgraph(4) too. I'm afraid this is not right direction
> to go. Alignment fixup would be best served in ether_input or in driver
> layer which seems to be more appropriate place. Maybe this is the
> reason why OpenBSD have such a complex alignment code in em driver.

Please make sure you know the entire problem before you say it's no
the correct way to go... Drivers like re and vge have to copy up an
entire cluster (2k), just to make alignment correct...  It's even more
ugly than patching everyone to self align...

Also, IMO, it's the correct layer to put the fix up in each firewalls,
etc..  Because the driver should not magicly know that the upper layers
need an alignment.. Plus, what happens if someone happens to use foobar
which only needs 2 byte alignment... Why should we continue to fixup
packets on alignment constrained machines when this layer doesn't need
it?

Hence why I am trying to get some performance measurements to see just
the performance difference the extra mbuf allocation makes over coping
data...

-- 
  John-Mark Gurney				Voice: +1 415 225 5579

     "All that I will do, has been done, All that I have, has not."


Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?20050416170133.GA16099>