[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH v2] net-hub: Drop can_receive
From: |
Fam Zheng |
Subject: |
Re: [Qemu-devel] [PATCH v2] net-hub: Drop can_receive |
Date: |
Tue, 7 Jul 2015 17:41:56 +0800 |
User-agent: |
Mutt/1.5.23 (2014-03-12) |
On Tue, 07/07 09:37, Stefan Hajnoczi wrote:
> On Tue, Jul 07, 2015 at 02:30:30PM +0800, Fam Zheng wrote:
> > This moves the semantics from net_hub_port_can_receive to receive
> > functions, by returning 0 if all receiving ports return 0. Also,
> > remember to flush the source port's queue in that case.
> >
> > Signed-off-by: Fam Zheng <address@hidden>
> > ---
> > net/hub.c | 54 +++++++++++++++++++++++++++++-------------------------
> > 1 file changed, 29 insertions(+), 25 deletions(-)
>
> This patch revision doesn't take into account the special case code in
> qemu_flush_or_purge_queued_packets(), which I mentioned in my reply to
> the previous revision of this patch.
>
> The queue is now flushed twice because you've introduced
> net_hub_port_send_cb() but qemu_flush_or_purge_queued_packets() already
> calls net_hub_flush().
>
> If you want to get rid of net_hub_flush(), that's great. But please
> remove the duplicate code.
Right, I missed that. I'll remove the duplicate and send again.
- [Qemu-devel] [PATCH v2] net-hub: Drop can_receive, Fam Zheng, 2015/07/07
- Re: [Qemu-devel] [PATCH v2] net-hub: Drop can_receive, Stefan Hajnoczi, 2015/07/07
- Re: [Qemu-devel] [PATCH v2] net-hub: Drop can_receive,
Fam Zheng <=
- Re: [Qemu-devel] [PATCH v2] net-hub: Drop can_receive, Stefan Hajnoczi, 2015/07/20
- Re: [Qemu-devel] [PATCH v2] net-hub: Drop can_receive, Fam Zheng, 2015/07/20
- Re: [Qemu-devel] [PATCH v2] net-hub: Drop can_receive, Stefan Hajnoczi, 2015/07/21
- Re: [Qemu-devel] [PATCH v2] net-hub: Drop can_receive, Fam Zheng, 2015/07/21
- Re: [Qemu-devel] [PATCH v2] net-hub: Drop can_receive, Stefan Hajnoczi, 2015/07/21