Thread (15 messages) 15 messages, 4 authors, 2009-01-27

Re: [PANIC] lro + iscsi or lro + skb text search causes panic

From: David Miller <davem@davemloft.net>
Date: 2009-01-26 05:34:33

From: Mike Christie <redacted>
Date: Thu, 22 Jan 2009 22:22:42 -0600
Attached is a patch made against the scsi maintainer's tree (I think
it should also apply to linus's) that converts iscsi to use
skb_copy_bits. It is lightly tested. If there is no benefit in
having skb_find_text use skb_seq_read maybe we can just kill it, so
people do not have to maintain two helpers that provide similar
functionality.
Regardless of what we decide to do with iSCSI we have to make this
function work properly as long as there is a user in the tree.

This iterator can in fact be more efficient than using copy
bits since copy bits doesn't remember any state from previous
invocations whereas the iterator does.  So if you call it
multiple times on the same SKB that's a lot of wasted work.

In fact I'm pretty sure that's why we added it for textsearch,
because in that application we're constantly beating through
the same SKB via the testsearch state machine.

Therefore, keeping it's use for iSCSI would be beneficial.
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help