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 PHC | |
Open Source and information security mailing list archives
| ||
|
Date: Wed, 30 Apr 2014 17:36:18 +0200 From: Stefan Wahren <stefan.wahren@...e.com> To: Arnd Bergmann <arnd@...db.de> CC: davem@...emloft.net, robh+dt@...nel.org, pawel.moll@....com, mark.rutland@....com, ijc+devicetree@...lion.org.uk, galak@...eaurora.org, f.fainelli@...il.com, netdev@...r.kernel.org, devicetree@...r.kernel.org Subject: Re: [PATCH RFC 2/2] net: qualcomm: new Ethernet over SPI driver for QCA7000 Hi Arnd, Am 29.04.2014 20:14, schrieb Arnd Bergmann: > On a related note, there is one part I don't understand: > > +netdev_tx_t > +qcaspi_netdev_xmit(struct sk_buff *skb, struct net_device *dev) > +{ > + u32 frame_len; > + u8 *ptmp; > + struct qcaspi *qca = netdev_priv(dev); > + u32 new_tail; > + struct sk_buff *tskb; > + u8 pad_len = 0; > + > + if (skb->len < QCAFRM_ETHMINLEN) > + pad_len = QCAFRM_ETHMINLEN - skb->len; > + > + if (qca->txq.skb[qca->txq.tail]) { > + netdev_warn(qca->net_dev, "queue was unexpectedly full!\n"); > + netif_stop_queue(qca->net_dev); > + qca->stats.queue_full++; > + return NETDEV_TX_BUSY; > + } > > You print a 'netdev_warn' message here when the queue is full, expecting > this to be rare. If the device is so slow, why doesn't this happen > all the time? > > Arnd i think i've found it. A little bit later in the function the "expected" queue full condition is handled. After the new skb is added to the buffer, the new tail is checked. In the case the tx queue is now full, netif_stop_queue is called. I've forgot to increment the queue_full counter for that case. After the skb has been transmitted to the QCA7000 over spi, the function netif_wake_queue is called. If i'm right, the message "queue was unexpectedly full" should never appear under normal condition. Stefan -- 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
Powered by blists - more mailing lists