lkml.org 
[lkml]   [2020]   [Jul]   [7]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
Date
From
SubjectRe: [PATCH net-next 02/12] bridge: uapi: mrp: Extend MRP attributes for MRP interconnect
The 07/06/2020 12:27, David Miller wrote:
> EXTERNAL EMAIL: Do not click links or open attachments unless you know the content is safe
>
> From: Horatiu Vultur <horatiu.vultur@microchip.com>
> Date: Mon, 6 Jul 2020 11:18:32 +0200
>
> > +struct br_mrp_in_state {
> > + __u16 in_id;
> > + __u32 in_state;
> > +};
>
> Put the __u32 first then the __u16.
>
> > +struct br_mrp_in_role {
> > + __u16 in_id;
> > + __u32 ring_id;
> > + __u32 in_role;
> > + __u32 i_ifindex;
> > +};
>
> Likewise.
>
> > +struct br_mrp_start_in_test {
> > + __u16 in_id;
> > + __u32 interval;
> > + __u32 max_miss;
> > + __u32 period;
> > +};
>
> Likewise.
>
> > +struct br_mrp_in_test_hdr {
> > + __be16 id;
> > + __u8 sa[ETH_ALEN];
> > + __be16 port_role;
> > + __be16 state;
> > + __be16 transitions;
> > + __be32 timestamp;
> > +};
>
> Likewise. Put the larger members first. There is lots of unnecessary
> padding in this structure.

I will do the same here, except for the 'struct br_mrp_in_test_hdr'
because this represents the frame header for InTest frames. And this is
defined in the standard how it has to be. But I will do it for the other
structures.

>

--
/Horatiu

\
 
 \ /
  Last update: 2020-07-07 12:40    [W:0.089 / U:2.988 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site