On Wed, Apr 26, 2017 at 1:53 AM, Kalle Valo <kv...@codeaurora.org> wrote: > Arend Van Spriel <arend.vanspr...@broadcom.com> 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.vanspr...@broadcom.com> > > 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