netdev - Re: [net-next:master 92/95] drivers/net/ethernet/broadcom/genet/bcmgenet.c:527:30-56: duplicated argument to && or ||

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>] [day] [month] [year] [list]

Message-ID: <alpine.DEB.2.10.1509180946350.2468@hadrien>
Date:	Fri, 18 Sep 2015 09:47:18 +0200 (CEST)
From:	Julia Lawall <julia.lawall@...6.fr>
To:	Florian Fainelli <f.fainelli@...il.com>
cc:	kbuild-all@...org, netdev@...r.kernel.org
Subject: Re: [net-next:master 92/95] drivers/net/ethernet/broadcom/genet/bcmgenet.c:527:30-56:
 duplicated argument to && or ||

Two uses of ec->tx_coalesce_usecs_high.  Please check.

julia

On Fri, 18 Sep 2015, kbuild test robot wrote:

> TO: Florian Fainelli <f.fainelli@...il.com>
> CC: kbuild-all@...org
> CC: netdev@...r.kernel.org
>
> tree:   https://git.kernel.org/pub/scm/linux/kernel/git/davem/net-next.git master
> head:   47bbbb30b4331ec58a74a66a044341f0114b02b3
> commit: 2f9130709d2c19e2eef336bda0fb0c807cbc1f64 [92/95] net: bcmgenet: Implement TX coalescing control knobs
> :::::: branch date: 2 hours ago
> :::::: commit date: 2 hours ago
>
> >> drivers/net/ethernet/broadcom/genet/bcmgenet.c:527:30-56: duplicated argument to && or ||
>
> git remote add net-next https://git.kernel.org/pub/scm/linux/kernel/git/davem/net-next.git
> git remote update net-next
> git checkout 2f9130709d2c19e2eef336bda0fb0c807cbc1f64
> vim +527 drivers/net/ethernet/broadcom/genet/bcmgenet.c
>
> 2f913070 Florian Fainelli 2015-09-16  511  }
> 2f913070 Florian Fainelli 2015-09-16  512
> 2f913070 Florian Fainelli 2015-09-16  513  static int bcmgenet_set_coalesce(struct net_device *dev,
> 2f913070 Florian Fainelli 2015-09-16  514  				 struct ethtool_coalesce *ec)
> 2f913070 Florian Fainelli 2015-09-16  515  {
> 2f913070 Florian Fainelli 2015-09-16  516  	struct bcmgenet_priv *priv = netdev_priv(dev);
> 2f913070 Florian Fainelli 2015-09-16  517  	unsigned int i;
> 2f913070 Florian Fainelli 2015-09-16  518
> 2f913070 Florian Fainelli 2015-09-16  519  	if (ec->tx_max_coalesced_frames > DMA_INTR_THRESHOLD_MASK ||
> 2f913070 Florian Fainelli 2015-09-16  520  	    ec->tx_max_coalesced_frames == 0)
> 2f913070 Florian Fainelli 2015-09-16  521  		return -EINVAL;
> 2f913070 Florian Fainelli 2015-09-16  522
> 2f913070 Florian Fainelli 2015-09-16  523  	/* GENET TDMA hardware does not support a configurable timeout, but will
> 2f913070 Florian Fainelli 2015-09-16  524  	 * always generate an interrupt either after MBDONE packets have been
> 2f913070 Florian Fainelli 2015-09-16  525  	 * transmitted, or when the ring is emtpy.
> 2f913070 Florian Fainelli 2015-09-16  526  	 */
> 2f913070 Florian Fainelli 2015-09-16 @527  	if (ec->tx_coalesce_usecs || ec->tx_coalesce_usecs_high ||
> 2f913070 Florian Fainelli 2015-09-16  528  	    ec->tx_coalesce_usecs_irq || ec->tx_coalesce_usecs_high ||
> 2f913070 Florian Fainelli 2015-09-16  529  	    ec->tx_coalesce_usecs_low)
> 2f913070 Florian Fainelli 2015-09-16  530  		return -EOPNOTSUPP;
> 2f913070 Florian Fainelli 2015-09-16  531
> 2f913070 Florian Fainelli 2015-09-16  532  	/* Program all TX queues with the same values, as there is no
> 2f913070 Florian Fainelli 2015-09-16  533  	 * ethtool knob to do coalescing on a per-queue basis
> 2f913070 Florian Fainelli 2015-09-16  534  	 */
> 2f913070 Florian Fainelli 2015-09-16  535  	for (i = 0; i < priv->hw_params->tx_queues; i++)
>
> ---
> 0-DAY kernel test infrastructure                Open Source Technology Center
> https://lists.01.org/pipermail/kbuild-all                   Intel Corporation
>
--
To unsubscribe from this list: send the line "unsubscribe netdev" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html