> Under heavy TX load, bnx2x_poll() can loop forever and trigger soft lockup 
> bugs.
> 
> A napi poll handler must yield after one TX completion round, risk of 
> livelock is
> too high otherwise.
> 
> Bug is very easy to trigger using a debug build, and udp flood, because of 
> added
> cpu cycles in TX completion, and we do not receive enough packets to break the
> loop.

Eric - I understand what you're doing and it looks fine [to me, at least].
Out of curiosity, do you know whether removing the loop damages any
other flow, i.e., by slowing transmitter as transmission rings gets filled
completely between consecutive NAPI runs? 
N�����r��y����b�X��ǧv�^�)޺{.n�+���z�^�)����w*jg��������ݢj/���z�ޖ��2�ޙ����&�)ߡ�a�����G���h��j:+v���w��٥

Reply via email to