On 2019/01/29 11:23, Jason Wang wrote:
> On 2019/1/29 上午8:45, Toshiaki Makita wrote:
...
>> @@ -2666,10 +2696,10 @@ static void free_unused_bufs(struct
>> virtnet_info *vi)
>>       for (i = 0; i < vi->max_queue_pairs; i++) {
>>           struct virtqueue *vq = vi->sq[i].vq;
>>           while ((buf = virtqueue_detach_unused_buf(vq)) != NULL) {
>> -            if (!is_xdp_raw_buffer_queue(vi, i))
>> +            if (!is_xdp_frame(buf))
> 
> 
> I believe this is the last user of is_xdp_raw_buffer_queue(), maybe you
> can sent a patch on top to remove it.

Actually patch2 added new users of it ;)

> 
> 
>>                   dev_kfree_skb(buf);
>>               else
>> -                xdp_return_frame(buf);
>> +                xdp_return_frame(ptr_to_xdp(buf));
>>           }
>>       }
>>   
> 
> 
> Acked-by: Jason Wang <jasow...@redhat.com>
> 

Thanks!

-- 
Toshiaki Makita

Reply via email to