Re: [PATCH] virtio_net: free transmit skbs in a timer

From: Avi Kivity
Date: Mon May 19 2008 - 06:26:49 EST


Rusty Russell wrote:
We should have thought of this before, though, especially as Xen does

this or something very similar:
/* Shared ring page */ \
struct __name##_sring { \
RING_IDX req_prod, req_event; \
RING_IDX rsp_prod, rsp_event; \
uint8_t pad[48]; \
union __name##_sring_entry ring[1]; /* variable-length */ \
}; \
req_event and rsp_event allow the other side to indicate when it wants a
notification.

Well, we do have such a thing, in the ring suppression flags.

Can you point me at this?

Note that DaveM is talking about moving network tx queue into the net drivers themselves, which will make them much more efficient (ie. drain entire queue before kick), which may again change the balance of what the Right Thing is.

That depends on whether Linux knows whether more packets are coming.

--
error compiling committee.c: too many arguments to function

--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/