On Thu, Jan 17, 2013 at 07:07:11AM +0100, Luigi Rizzo wrote: > The comment at the beginning of net/queue.c says that packets that > cannot be sent by qemu_net_queue_send() should not be enqueued > unless a callback is set. > > This patch implements this behaviour, that prevents a queue to grow > unbounded (e.g. when a network backend is not connected). > > Also for good measure the patch implements bounded size queues > (though it should not be necessary now because each source can only have > one packet queued). When a packet is dropped because excessive > queue size the callback is not supposed to be called.
Although I appreciate the semantics that the comment tries to establish, the code doesn't behave like this today and we cannot drop packets in cases where we relied on queuing them. More changes will be required to make the hub, USB, pcap scenario I described previously work. Stefan
