[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-Id: <20180408.124200.2204489876311923873.davem@davemloft.net>
Date: Sun, 08 Apr 2018 12:42:00 -0400 (EDT)
From: David Miller <davem@...emloft.net>
To: Vadim.Lomovtsev@...iumnetworks.com
Cc: sgoutham@...ium.com, sunil.kovvuri@...il.com, rric@...nel.org,
linux-arm-kernel@...ts.infradead.org, netdev@...r.kernel.org,
linux-kernel@...r.kernel.org, dnelson@...hat.com,
robin.murphy@....com, hch@...radead.org, gustavo@...eddedor.com,
Vadim.Lomovtsev@...ium.com
Subject: Re: [PATCH v4] net: thunderx: rework mac addresses list to u64
array
From: Vadim Lomovtsev <Vadim.Lomovtsev@...iumnetworks.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.
Powered by blists - more mailing lists