From: Vadim Lomovtsev <vadim.lomovt...@caviumnetworks.com> Date: Fri, 6 Apr 2018 12:53:54 -0700
> @@ -1929,7 +1929,7 @@ static void nicvf_set_rx_mode_task(struct work_struct > *work_arg) > work.work); > struct nicvf *nic = container_of(vf_work, struct nicvf, rx_mode_work); > union nic_mbx mbx = {}; > - struct xcast_addr *xaddr, *next; > + int idx = 0; No need to initialize idx. > + for (idx = 0; idx < vf_work->mc->count; idx++) { As it is always explicitly initialized at, and only used inside of, this loop.