Re: Marvell if_mvgbe

[Masanobu SAITOH <msaitoh%execsw.org@localhost>]

Hello, Robert.

(2012/10/17 20:09), Robert Swindells wrote:
> Masanobu SAITOH wrote:
>> Modified Files:
>>      src/sys/dev/marvell: if_mvgbe.c mvgbereg.h
>>
>> Log Message:
>> Add code to modify interrupt coalescing with sysctl.
>>   - both TX side an RX side.
>>   - different setting for each port
>>   - TX side is hw.mvgbe.mvgbe*.ipginttx
>>   - RX side is hw.mvgbe.mvgbe*.ipgintrx
>>   - The default value is 768.
>>   - The lowest value is 0
>>   - For highest value, 0x3777 is used for V1, and 0xffff is used for V2.
> 
> Are you sure it makes sense to add this for tx ?
> 
> Robert Swindells

 The TX's value was taken from FreeBSD. For TX side, it's not related
to the *delay* of each packet. If the value is big, the TX's descriptor
ring may be full. I'll do some tests.

-- 
-----------------------------------------------
                SAITOH Masanobu (msaitoh%execsw.org@localhost
                                 msaitoh%netbsd.org@localhost)