On Wed, Apr 26, 2017 at 1:53 AM, Kalle Valo <kvalo@xxxxxxxxxxxxxx> wrote: > Arend Van Spriel <arend.vanspriel@xxxxxxxxxxxx> wrote: >> An issue was found brcmfmac driver in which a skbuff in .start_xmit() >> callback was actually cloned. So instead of checking for sufficient >> headroom it should also be writable. Hence use skb_cow_head() to >> check and expand the headroom appropriately. >> >> Signed-off-by: Arend van Spriel <arend.vanspriel@xxxxxxxxxxxx> > > Steve, would you have time to run a quick test with this? > > Patch set to Deferred. > Happy to give it a quick spin on both of my platforms. @Arend: is there some demonstrable before/after that shows a problem I can detect at runtime? I understand your thought about putting a skb_clone() in there, but what are the expectations? And is any problem evident without explicitly modding the code with the clone? - Steve
