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 PHC | |
Open Source and information security mailing list archives
| ||
|
Date: Wed, 31 Dec 2014 09:30:31 -0500 From: Jesse Gross <jesse@...ira.com> To: Or Gerlitz <gerlitz.or@...il.com> Cc: David Miller <davem@...emloft.net>, Linux Netdev List <netdev@...r.kernel.org> Subject: Re: [PATCH net-next 1/2] net: Add Transparent Ethernet Bridging GRO support. On Wed, Dec 31, 2014 at 4:19 AM, Or Gerlitz <gerlitz.or@...il.com> wrote: > On Wed, Dec 31, 2014 at 5:10 AM, Jesse Gross <jesse@...ira.com> wrote: >> Currently the only tunnel protocol that supports GRO with encapsulated >> Ethernet is VXLAN. This pulls out the Ethernet code into a proper layer >> so that it can be used by other tunnel protocols such as GRE and Geneve. > > Hi Jesse, > > Thanks for taking care of that, I also had it coded under the > intention of adding GRO support for OVS's TEB based GRE, but didn't > make it to submit before your post... anyway, I would recommend that > you break this patch into two: > > 1. basic TEB GRO support > 2. refactoring of the VXLAN GRO logic to use it This patch is really just moving code so breaking it into two steps would essentially mean introducing duplicate code and then deleting the first version in the next patch. It's already a pretty short patch and I think splitting it would actually make it harder to verify that it is correct. -- 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
Powered by blists - more mailing lists