Re: [PATCH net-next] ethernet/atheros/alx: sanitize buffer sizing and padding

From: David Miller
Date: Wed Jan 06 2016 - 01:23:34 EST


From: Jarod Wilson <jarod@xxxxxxxxxx>
Date: Tue, 5 Jan 2016 18:12:34 -0500

> - if ((raw_mtu + 8) < ALX_TXQ1_JUMBO_TSO_TH)
> - val = (raw_mtu + 8 + 7) >> 3;
> + if ((raw_mtu) < ALX_TXQ1_JUMBO_TSO_TH)
> + val = (raw_mtu + 7) >> 3;

The parenthesis around 'raw_mtu' is superfluous, please
remove it.

Otherwise this looks great, please resubmit with that fixed
up.

Thanks.
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/