> 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��٥