Skip site navigation (1)Skip section navigation (2)
Date:      Tue, 05 Sep 2006 21:56:12 +0200
From:      Andre Oppermann <andre@freebsd.org>
To:        Prafulla Deuskar <pdeuskar@freebsd.org>
Cc:        freebsd-net <freebsd-net@freebsd.org>, freebsd-current <freebsd-current@freebsd.org>, Jack Vogel <jfvogel@gmail.com>
Subject:   Re: RFC: TSO patch for current
Message-ID:  <44FDD65C.6070109@freebsd.org>
In-Reply-To: <20060905182313.GA85389@hub.freebsd.org>
References:  <2a41acea0609011551v40338539u4eef48d091dd12ab@mail.gmail.com> <44F9384C.9070902@freebsd.org> <2a41acea0609021741y481a04c0r42902166eaba78d7@mail.gmail.com> <20060905162542.GA63869@hub.freebsd.org> <44FDAF08.20407@freebsd.org> <20060905182313.GA85389@hub.freebsd.org>

next in thread | previous in thread | raw e-mail | index | archive | help
Prafulla Deuskar wrote:
> Andre Oppermann [andre@freebsd.org] wrote:
>> Prafulla Deuskar wrote:
>>> Jack Vogel [jfvogel@gmail.com] wrote:
>>>
>>>> On 9/2/06, Andre Oppermann <andre@freebsd.org> wrote:
>>>>
>>>>
>>>>> I can't comment on the em part but the tcp_output.c stuff looks
>>>>> very much like a straight port from NetBSD.  If we take code from
>>>>> the other BSDs we have to remark this in the emails we send with
>>>>> patches and the commit message (otherwise we get accused of 'stealing
>>>>> without attribution').
>>>> I dont know that I'd call it a straight port, rather I was working from 
>>>> some
>>>> prototype code that Prafulla had working back on 4.7, but I think at that
>>>> time that he may have patterned it after NetBSD. 
>>> I don't think NetBSD had TSO support in 2002 when I first did the 4.7 
>>> patch for internal testing.
>> OK, perhaps they lifted it from your patch then.  They just looks awfully
>> similiar that's why I thought it came from NetBSD.
>>
> Your patch looks good and is the way to go.
> 
> So after Jack confirms that your patch works with the em driver
> would you commit to to -current?

Absolutely.  :-)

> The driver related changes can follow..
> 
> Later we also need to fix ifconfig so that user can enable/disable TSO on the interface. 

I'll do that together with the TSO code.

-- 
Andre




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