ftgmac100: return error in ftgmac100_alloc_rx_buf
authorJoel Stanley <joel@jms.id.au>
Tue, 25 Jul 2017 00:49:01 +0000 (10:19 +0930)
committerDavid S. Miller <davem@davemloft.net>
Wed, 26 Jul 2017 04:21:44 +0000 (21:21 -0700)
commit6cee9d649cd1824a4d9ce6940dd716f2bf2ef24f
treee885c6f6a78ff16d35e8595c393a407376371961
parentafce615aaabfbaad02550e75c0bec106dafa1adf
ftgmac100: return error in ftgmac100_alloc_rx_buf

The error paths set err, but it's not returned.

I wondered if we should fix all of the callers to check the returned
value, but Ben explains why the code is this way:

> Most call sites ignore it on purpose. There's nothing we can do if
> we fail to get a buffer at interrupt time, so we point the buffer to
> the scratch page so the HW doesn't DMA into lalaland and lose the
> packet.
>
> The one call site that tests and can fail is the one used when brining
> the interface up. If we fail to allocate at that point, we fail the
> ifup. But as you noticed, I do have a bug not returning the error.

Acked-by: Benjamin Herrenschmidt <benh@kernel.crashing.org>
Signed-off-by: Joel Stanley <joel@jms.id.au>
Signed-off-by: David S. Miller <davem@davemloft.net>
drivers/net/ethernet/faraday/ftgmac100.c