Re: [PATCH net] net: thunderx: set tso_hdrs pointer to NULL in nicvf_free_snd_queue
> From: Lorenzo Bianconi > Date: Fri, 23 Nov 2018 18:28:01 +0100 > > > Reset snd_queue tso_hdrs pointer to NULL in nicvf_free_snd_queue routine > > since it is used to check if tso dma descriptor queue has been previously > > allocated. The issue can be triggered with the following reproducer: > > > > $ip link set dev enP2p1s0v0 xdpdrv obj xdp_dummy.o > > $ip link set dev enP2p1s0v0 xdpdrv off > ... > > where xdp_dummy.c is a simple bpf program that forwards the incoming > > frames to the network stack (available here: > > https://github.com/altoor/xdp_walkthrough_examples/blob/master/sample_1/xdp_dummy.c) > > > > Fixes: 05c773f52b96 ("net: thunderx: Add basic XDP support") > > Fixes: 4863dea3fab0 ("net: Adding support for Cavium ThunderX network > > controller") > > > > Signed-off-by: Lorenzo Bianconi > > Applied and queued up for -stable, but please in the future: > > 1) Do not break up long "Fixes: " tag lines, it must be keep as a single >uninterrupted line for grep'ability etc. > > 2) Do not put an empty line between "Fixes: " and other tags. All tags >are equal, and appear in a straight uninterrupted sequence of lines. > Hi David, ack, will do next time. Regards, Lorenzo > Thank you.
Re: [PATCH net] net: thunderx: set tso_hdrs pointer to NULL in nicvf_free_snd_queue
From: Lorenzo Bianconi Date: Fri, 23 Nov 2018 18:28:01 +0100 > Reset snd_queue tso_hdrs pointer to NULL in nicvf_free_snd_queue routine > since it is used to check if tso dma descriptor queue has been previously > allocated. The issue can be triggered with the following reproducer: > > $ip link set dev enP2p1s0v0 xdpdrv obj xdp_dummy.o > $ip link set dev enP2p1s0v0 xdpdrv off ... > where xdp_dummy.c is a simple bpf program that forwards the incoming > frames to the network stack (available here: > https://github.com/altoor/xdp_walkthrough_examples/blob/master/sample_1/xdp_dummy.c) > > Fixes: 05c773f52b96 ("net: thunderx: Add basic XDP support") > Fixes: 4863dea3fab0 ("net: Adding support for Cavium ThunderX network > controller") > > Signed-off-by: Lorenzo Bianconi Applied and queued up for -stable, but please in the future: 1) Do not break up long "Fixes: " tag lines, it must be keep as a single uninterrupted line for grep'ability etc. 2) Do not put an empty line between "Fixes: " and other tags. All tags are equal, and appear in a straight uninterrupted sequence of lines. Thank you.