On Wed, 2016-11-23 at 13:41 +0100, Phil Sutter wrote: > +struct vf_data_storage { > + unsigned char vf_mac[ETH_ALEN]; > + u16 pf_vlan; /* When set, guest VLAN config not allowed. */ > + u16 pf_qos; > + __be16 vlan_proto; > + u16 min_tx_rate; > + u16 max_tx_rate; > + u8 spoofchk_enabled; > + bool rss_query_enabled; > + u8 trusted; > + int link_state; > +};
Could you use proper indentation of field names ? struct vf_data_storage { u8 vf_mac[ETH_ALEN]; u16 pf_vlan; /* When set, guest VLAN config not allowed. */ u16 pf_qos; __be16 vlan_proto; u16 min_tx_rate; u16 max_tx_rate; u8 spoofchk_enabled; bool rss_query_enabled; u8 trusted; int link_state; }; struct dummy_priv { int num_vfs; struct vf_data_storage *vfinfo; }; Thanks.