logo       

Re: [PATCH] IPv6: Improvement of Source Address Selection: msg#00306

Subject: Re: [PATCH] IPv6: Improvement of Source Address Selection
   From: YOSHIFUJI Hideaki / 吉藤英明 <yoshfuji@xxxxxxxxxxxxxx>
   Date: Sat, 28 Sep 2002 00:17:42 +0900 (JST)

Please redesign this structure.

   +struct addrselect_attrs {
   +    struct inet6_ifaddr *ifp;
   +    int     match;
   +    int     deprecated;
   +    int     home;
   +    int     temporary;
   +    int     device;
   +    int     scope;
   +    int     label;
   +    int     matchlen;
   +};

This is much larger than it needs to be.  Please replace these "int"
binary states with single "u32 flags;" and appropriate bit
definitions.

This structure sits on the stack, so it is important to be
as small as we can easily make it.

Otherwise I have no problems with the patch, Alexey?





<Prev in Thread] Current Thread [Next in Thread>