On Thu, Jul 19, 2018 at 7:50 AM Tariq Toukan <tar...@mellanox.com> wrote: > --- a/net/core/dev_ioctl.c > +++ b/net/core/dev_ioctl.c > @@ -282,14 +282,7 @@ static int dev_ifsioc(struct net *net, struct ifreq > *ifr, unsigned int cmd) > return dev_mc_del_global(dev, ifr->ifr_hwaddr.sa_data); > > case SIOCSIFTXQLEN: > - if (ifr->ifr_qlen < 0) > - return -EINVAL;
Are you sure we can remove this if check too? The other one is safe to remove.