From mboxrd@z Thu Jan 1 00:00:00 1970 From: Stephen Hemminger Subject: Re: [PATCH v5 05/10] qede: Add core driver Date: Fri, 1 Apr 2016 08:41:42 -0700 Message-ID: <20160401084142.11bd9a06@xeon-e3> References: <1459476927-21387-1-git-send-email-rasesh.mody@qlogic.com> <1459476927-21387-6-git-send-email-rasesh.mody@qlogic.com> Mime-Version: 1.0 Content-Type: text/plain; charset=US-ASCII Content-Transfer-Encoding: 7bit Cc: , , , , , To: Rasesh Mody Return-path: Received: from mail-pa0-f46.google.com (mail-pa0-f46.google.com [209.85.220.46]) by dpdk.org (Postfix) with ESMTP id DD8CF2965 for ; Fri, 1 Apr 2016 17:41:28 +0200 (CEST) Received: by mail-pa0-f46.google.com with SMTP id fe3so93646852pab.1 for ; Fri, 01 Apr 2016 08:41:28 -0700 (PDT) In-Reply-To: <1459476927-21387-6-git-send-email-rasesh.mody@qlogic.com> List-Id: patches and discussions about DPDK List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: dev-bounces@dpdk.org Sender: "dev" On Thu, 31 Mar 2016 19:15:22 -0700 Rasesh Mody wrote: > +struct qed_filter_ucast_params { > + enum qed_filter_xcast_params_type type; > + uint8_t vlan_valid; > + uint16_t vlan; > + uint8_t mac_valid; > + unsigned char mac[ETHER_ADDR_LEN]; > +} __attribute__ ((__packed__)); Why packed? Why not just rearrange elements to eliminate the padding holes?