[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index] Re: [Xen-devel] [PATCHv2 net] xen-netback: count number required slots for an skb more carefully
From: David Vrabel <david.vrabel@xxxxxxxxxx> Date: Wed, 11 Sep 2013 14:52:48 +0100 > From: David Vrabel <david.vrabel@xxxxxxxxxx> > > When a VM is providing an iSCSI target and the LUN is used by the > backend domain, the generated skbs for direct I/O writes to the disk > have large, multi-page skb->data but no frags. > > With some lengths and starting offsets, xen_netbk_count_skb_slots() > would be one short because the simple calculation of > DIV_ROUND_UP(skb_headlen(), PAGE_SIZE) was not accounting for the > decisions made by start_new_rx_buffer() which does not guarantee > responses are fully packed. ... > Fix this by counting the number of required slots more carefully. In > xen_netbk_count_skb_slots(), more closely follow the algorithm used by > xen_netbk_gop_skb() by introducing xen_netbk_count_frag_slots() which > is the dry-run equivalent of netbk_gop_frag_copy(). > > Signed-off-by: David Vrabel <david.vrabel@xxxxxxxxxx> > Acked-by: Ian Campbell <ian.campbell@xxxxxxxxxx> > -- > v2: xen_netbk_* -> xenvif_* to match new style Thanks for resolving this into a final patch after so much back and forth :-) I assume you want this queued up for -stable, and can you check if there is any non-trivial backporting for earlier kernels? _______________________________________________ Xen-devel mailing list Xen-devel@xxxxxxxxxxxxx http://lists.xen.org/xen-devel
|
Lists.xenproject.org is hosted with RackSpace, monitoring our |