On 26.6.2025. 15:07, Stefan Sperling wrote:
> On Thu, Jun 26, 2025 at 02:39:11PM +0200, Hrvoje Popovski wrote:
>> I've tested this diff with plain forwarding over bnxt and it seems to
>> work just fine.
> 
> While the diff changes bnx(4), it does not change bnxt(4).
> 
> bxnt already contains the required check:
> 
> int
> bnxt_intr(void *xq)
> {
> [...]
> 	txfree = 0;
> [...]
> 
> 		case CMPL_BASE_TYPE_TX_L2:
> 			if (ISSET(ifp->if_flags, IFF_RUNNING))
> 				bnxt_txeof(sc, tx, &txfree, cmpl);
> 			break;
> 
> [...]
> 	if (txfree != 0) {
> 		if (ifq_is_oactive(tx->tx_ifq))
> 			ifq_restart(tx->tx_ifq);
> 	}
> 


Ufff,  sorry for noise, I've saw bnxt, not bnx ...