RE: [EXTERNAL] Re: [net-next PATCH v5 1/6] octeontx2-pf: use xdp_return_frame() to free xdp buffers

[Suman Ghosh]

>> @@ -1465,10 +1476,14 @@ static bool otx2_xdp_rcv_pkt_handler(struct
>otx2_nic *pfvf,
>>  		trace_xdp_exception(pfvf->netdev, prog, act);
>>  		break;
>>  	case XDP_DROP:
>> +		cq->pool_ptrs++;
>> +		if (page->pp) {
>> +			page_pool_recycle_direct(pool->page_pool, page);
>> +			return true;
>> +		}
>>  		otx2_dma_unmap_page(pfvf, iova, pfvf->rbsize,
>>  				    DMA_FROM_DEVICE);
>>  		put_page(page);
>> -		cq->pool_ptrs++;
>>  		return true;
>
>The above seems to get shuffled around in the next patch anyway, so
>maybe it's best to do this here (completely untested):
>
>	case XDP_DROP:
>		cq->pool_ptrs++;
>		if (page->pp) {
>			page_pool_recycle_direct(pool->page_pool, page);
>		} else {
>			otx2_dma_unmap_page(pfvf, iova, pfvf->rbsize,
>					    DMA_FROM_DEVICE);
>			put_page(page);
>		}
>		return true;
>
>>  	}
>>  	return false;
[Suman] ack, will update in v6
>
>...