netdev - Re: [PATCH NEXT 0/2] qlcnic: Add Qlogic 1/10Gb Ethernet driver for CNA devices

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-Id: <20100105.195156.39644124.davem@davemloft.net>
Date:	Tue, 05 Jan 2010 19:51:56 -0800 (PST)
From:	David Miller <davem@...emloft.net>
To:	dhananjay.phadke@...gic.com
Cc:	shemminger@...tta.com, amit.salecha@...gic.com,
	netdev@...r.kernel.org
Subject: Re: [PATCH NEXT 0/2] qlcnic: Add Qlogic 1/10Gb Ethernet driver for
 CNA devices

From: Dhananjay Phadke <dhananjay.phadke@...gic.com>
Date: Tue, 5 Jan 2010 17:56:00 -0800

>> But HW LRO is not bridge/forwarding/etc. agnostic because it loses
>> information.
>> 
>> As soon as I turn on forwarding to make a NAT box or whatever, all of
>> the hw LRO gets turned off and we're back to square one for local
>> connections on the machine.
> 
> I won't defend other vendors' implementation. Even if LRO was
> not turned off for bridge/forwarding, we still wouldn't accumulate
> those remote flows because the dest IP address is not configured in
> firmware in our case. This is the reason why we have a inet
> notification listener (to tell firmware about local addresses).

Ok, but you do realize that because GRO maintains enough state
we can reconstitute packets across forwarding and bridging
thus decreasing the per-packet costs in those situations as
well right?

That's why hardware vendors should strive to align their hw
LRO implementations with what GRO does, since it is truly
generic and allows to optimize all cases not just local
TCP streams.
--
To unsubscribe from this list: send the line "unsubscribe netdev" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html