From mboxrd@z Thu Jan 1 00:00:00 1970 From: David Miller Subject: Re: [PATCH] openvswitch: fix sw_flow_key alignment Date: Sat, 31 Aug 2013 00:16:50 -0400 (EDT) Message-ID: <20130831.001650.130549489746201651.davem@davemloft.net> References: <20130830.144211.1550393719904034493.davem@davemloft.net> Mime-Version: 1.0 Content-Type: Text/Plain; charset=us-ascii Content-Transfer-Encoding: 7bit Cc: azhou@nicira.com, dev@openvswitch.org, netdev@vger.kernel.org To: jesse@nicira.com Return-path: Received: from shards.monkeyblade.net ([149.20.54.216]:33142 "EHLO shards.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1754013Ab3HaEQx (ORCPT ); Sat, 31 Aug 2013 00:16:53 -0400 In-Reply-To: Sender: netdev-owner@vger.kernel.org List-ID: From: Jesse Gross Date: Fri, 30 Aug 2013 16:20:25 -0700 > I looked through the struct definition and I think that the idea of > manually padding as Geert did in his patch will be difficult to > maintain over time (and actually there are a few that he missed) since > there are a number of different structs/unions contained in there. You have to be mindful of the gaps and wasted space for performance reasons anyways.