Hi Sabrina, I think I may have beaten you to the punch here by a few minutes. :)
The difference between our two versions is that you don't re-add the FRAGLIST attribute, whereas my patch does, and then it does the dynamic allocation. I suspect this might be a bit more robust. It also ensures that skb_cow_data is called on both paths. So perhaps let's roll with mine? Jason