8139cp: Fix one possible deadloop in cp_rx_poll
authorGao Feng <fgao@ikuai8.com>
Thu, 25 Aug 2016 01:45:39 +0000 (09:45 +0800)
committerDavid S. Miller <davem@davemloft.net>
Fri, 26 Aug 2016 00:02:48 +0000 (17:02 -0700)
commitb628d611a2a53858263fc419dba552f32431dba4
treea0f36ccc5eba2cc45525c0bd55c7d72b6a1db79b
parentf38ff2ee7727994685494bcc4d7c274b35b5418a
8139cp: Fix one possible deadloop in cp_rx_poll

When cp_rx_poll does not get enough packet, it will check the rx
interrupt status again. If so, it will jumpt to rx_status_loop again.
But the goto jump resets the rx variable as zero too.

As a result, it causes one possible deadloop. Assume this case,
rx_status_loop only gets the packet count which is less than budget,
and (cpr16(IntrStatus) & cp_rx_intr_mask) condition is always true.
It causes the deadloop happens and system is blocked.

Signed-off-by: Gao Feng <fgao@ikuai8.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
drivers/net/ethernet/realtek/8139cp.c