[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH V4 1/2] net/filter-mirror: implement filter-redi
From: |
Jason Wang |
Subject: |
Re: [Qemu-devel] [PATCH V4 1/2] net/filter-mirror: implement filter-redirector |
Date: |
Thu, 17 Mar 2016 14:28:30 +0800 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:38.0) Gecko/20100101 Thunderbird/38.5.1 |
On 03/17/2016 02:18 PM, Wen Congyang wrote:
>>>>>>> >>>>>> + s->state = 0;
>>>>>>> >>>>>> + /* FIXME: do something ? */
>>>>> >>>>
>>>>> >>>> This needs some thought, but at least reset the fd handler and state
>>>>> >>>> is
>>>>> >>>> needed.
>>> >> Maybe we can read the data from the chardev, and drop it?
>>> >>
>>> >> Thanks
>>> >> Wen Congyang
>>> >>
>> >
>> > No function difference, but reading and dropping will cost extra cpu
>> > which seems not good.
> If we reset and receive it again, the unread data will be auto dropped?
If socket receive queue is not full, the data will be buffered there
(but the queue will be purged after qemu process exits). Otherwise it
will be dropped.
> If so, I think reset the fd handler is better choice.
>
> Thanks
> Wen Congyang
>
- [Qemu-devel] [PATCH V4 0/2] Introduce filter-redirector, Zhang Chen, 2016/03/15
- [Qemu-devel] [PATCH V4 1/2] net/filter-mirror: implement filter-redirector, Zhang Chen, 2016/03/15
- Re: [Qemu-devel] [PATCH V4 1/2] net/filter-mirror: implement filter-redirector, Jason Wang, 2016/03/16
- Re: [Qemu-devel] [PATCH V4 1/2] net/filter-mirror: implement filter-redirector, Li Zhijian, 2016/03/16
- Re: [Qemu-devel] [PATCH V4 1/2] net/filter-mirror: implement filter-redirector, Wen Congyang, 2016/03/16
- Re: [Qemu-devel] [PATCH V4 1/2] net/filter-mirror: implement filter-redirector, Li Zhijian, 2016/03/16
- Re: [Qemu-devel] [PATCH V4 1/2] net/filter-mirror: implement filter-redirector, Jason Wang, 2016/03/17
- Re: [Qemu-devel] [PATCH V4 1/2] net/filter-mirror: implement filter-redirector, Wen Congyang, 2016/03/17
- Re: [Qemu-devel] [PATCH V4 1/2] net/filter-mirror: implement filter-redirector,
Jason Wang <=
[Qemu-devel] [PATCH V4 2/2] tests/test-filter-redirector: Add unit test for filter-redirector, Zhang Chen, 2016/03/15
Re: [Qemu-devel] [PATCH V4 0/2] Introduce filter-redirector, Jason Wang, 2016/03/16