On Thu, Jun 30, 2016 at 13:13:15 +0000, Yuval Mintz wrote:
> > + /* verify MAC addr is valid */
> > + if (!is_zero_ether_addr(mac) && !is_valid_ether_addr(mac) &&
> > + is_multicast_ether_addr(mac)) {
>
> This is really odd as verification goes; Currently this is a very elaborate
> way of checking for multicast, but I guess it's probably a mistake.
>
My bad, will send a V2
- [PATCH net-next 0/3] Add option to specify PF parameter ... Hariprasad Shenai
- [PATCH net-next 2/3] cxgb4/cxgb4vf: Add set VF mac ... Hariprasad Shenai
- RE: [PATCH net-next 2/3] cxgb4/cxgb4vf: Add set... Yuval Mintz
- Re: [PATCH net-next 2/3] cxgb4/cxgb4vf: Add... Hariprasad Shenai
- [PATCH net-next iproute2 3/3] ip: Add option to spe... Hariprasad Shenai
- [PATCH net-next 1/3] net: Add provision to specify ... Hariprasad Shenai
