Re: [PATCH v1 1/1 net-next] 8139cp: Fix one possible deadloop in cp_rx_poll

2016-08-25 Thread David Miller
From: f...@ikuai8.com Date: Thu, 25 Aug 2016 09:45:39 +0800 > From: Gao Feng > > 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 c

Re: [PATCH v1 1/1 net-next] 8139cp: Fix one possible deadloop in cp_rx_poll

2016-08-25 Thread Feng Gao
On Thu, Aug 25, 2016 at 9:45 AM, wrote: > From: Gao Feng > > 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 de

[PATCH v1 1/1 net-next] 8139cp: Fix one possible deadloop in cp_rx_poll

2016-08-24 Thread fgao
From: Gao Feng 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