netdev - Re: [PATCH net-next v10 2/4] net: Introduce generic failover module

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Date:   Tue, 8 May 2018 17:02:27 +0800
From:   kbuild test robot <lkp@...el.com>
To:     Sridhar Samudrala <sridhar.samudrala@...el.com>
Cc:     kbuild-all@...org, mst@...hat.com, stephen@...workplumber.org,
        davem@...emloft.net, netdev@...r.kernel.org,
        virtualization@...ts.linux-foundation.org,
        virtio-dev@...ts.oasis-open.org, jesse.brandeburg@...el.com,
        alexander.h.duyck@...el.com, kubakici@...pl,
        sridhar.samudrala@...el.com, jasowang@...hat.com,
        loseweigh@...il.com, jiri@...nulli.us, aaron.f.brown@...el.com
Subject: Re: [PATCH net-next v10 2/4] net: Introduce generic failover module

Hi Sridhar,

I love your patch! Perhaps something to improve:

[auto build test WARNING on net-next/master]

url:    https://github.com/0day-ci/linux/commits/Sridhar-Samudrala/Enable-virtio_net-to-act-as-a-standby-for-a-passthru-device/20180508-123531
reproduce:
        # apt-get install sparse
        make ARCH=x86_64 allmodconfig
        make C=1 CF=-D__CHECK_ENDIAN__


sparse warnings: (new ones prefixed by >>)

>> net/core/net_failover.c:868:16: sparse: Using plain integer as NULL pointer
   net/core/net_failover.c:115:12: sparse: context imbalance in 'net_failover_select_queue' - wrong count at exit

vim +868 net/core/net_failover.c

   828	
   829	/**
   830	 * net_failover_register - Register a failover instance
   831	 *
   832	 * @dev: failover or standby netdev
   833	 * @ops: failover ops
   834	 *
   835	 * Paravirtual drivers supporting 3-netdev model call this routine indirectly
   836	 * via net_failover_create(). It passes failover netdev and ops will be NULL
   837	 * as the slave events are handled internally.
   838	 * Paravirtual drivers supporting 2-netdev model call this routine by passing
   839	 * standby netdev and ops that are called to handle slave register/unregister/
   840	 * link change events.
   841	 *
   842	 * Return: pointer to failover instance
   843	 */
   844	struct net_failover *net_failover_register(struct net_device *dev,
   845						   struct net_failover_ops *ops)
   846	{
   847		struct net_failover *failover;
   848	
   849		failover = kzalloc(sizeof(*failover), GFP_KERNEL);
   850		if (!failover)
   851			return ERR_PTR(-ENOMEM);
   852	
   853		rcu_assign_pointer(failover->ops, ops);
   854		dev_hold(dev);
   855		dev->priv_flags |= IFF_FAILOVER;
   856		rcu_assign_pointer(failover->failover_dev, dev);
   857	
   858		spin_lock(&net_failover_lock);
   859		list_add_tail(&failover->list, &net_failover_list);
   860		spin_unlock(&net_failover_lock);
   861	
   862		netdev_info(dev, "failover master:%s registered\n", dev->name);
   863	
   864		net_failover_existing_slave_register(dev);
   865	
   866		return failover;
   867	
 > 868		return 0;
   869	}
   870	EXPORT_SYMBOL_GPL(net_failover_register);
   871	

---
0-DAY kernel test infrastructure                Open Source Technology Center
https://lists.01.org/pipermail/kbuild-all                   Intel Corporation