On Friday 20 June 2008, Jarek Poplawski wrote:

quoted text
> This patch looks fine to me, but I wonder if, btw., this place can't
> be optimized a bit, so why can't we simply:
>
> 	if (spd_fill_page(spd, f->page, plen, poff, skb))
> 		goto err;
>
> in both cases, since nothing more can't be filled after this?

Yes, you are right. Here is the patch with your suggestions in place, tested. 

BTW, I have another trivial fix and a RFC related to tcp splice read that I've 
sent a while ago. Should I resend them as well? 

Thanks,
tavi