<feed xmlns='http://www.w3.org/2005/Atom'>
<title>linux-stable.git/drivers/net, branch linux-2.6.19.y</title>
<subtitle>Linux kernel stable tree</subtitle>
<link rel='alternate' type='text/html' href='https://git.tavy.me/linux-stable.git/'/>
<entry>
<title>sky2: fix ram buffer allocation settings</title>
<updated>2007-03-03T00:32:46+00:00</updated>
<author>
<name>Stephen Hemminger</name>
<email>shemminger@linux-foundation.org</email>
</author>
<published>2007-02-27T01:18:39+00:00</published>
<link rel='alternate' type='text/html' href='https://git.tavy.me/linux-stable.git/commit/?id=074023e131b7ecba0f7d9ba43ba0df322fc8f129'/>
<id>074023e131b7ecba0f7d9ba43ba0df322fc8f129</id>
<content type='text'>
Different chipsets have different amount of ram buffer (some have none),
so need to make sure that driver does proper setup for all cases from 0 on
to 48K, in units of 1K.

This is a backport of the code from 2.6.19 or later

Signed-off-by: Stephen Hemminger &lt;shemminger@linux-foundation.org&gt;
Signed-off-by: Greg Kroah-Hartman &lt;gregkh@suse.de&gt;


</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
Different chipsets have different amount of ram buffer (some have none),
so need to make sure that driver does proper setup for all cases from 0 on
to 48K, in units of 1K.

This is a backport of the code from 2.6.19 or later

Signed-off-by: Stephen Hemminger &lt;shemminger@linux-foundation.org&gt;
Signed-off-by: Greg Kroah-Hartman &lt;gregkh@suse.de&gt;


</pre>
</div>
</content>
</entry>
<entry>
<title>sky2: dont flush good pause frames</title>
<updated>2007-03-03T00:32:46+00:00</updated>
<author>
<name>Stephen Hemminger</name>
<email>shemminger@linux-foundation.org</email>
</author>
<published>2007-02-27T01:18:38+00:00</published>
<link rel='alternate' type='text/html' href='https://git.tavy.me/linux-stable.git/commit/?id=1df6a4c30fbd43d9c3baf0dcdb2c60af688f0803'/>
<id>1df6a4c30fbd43d9c3baf0dcdb2c60af688f0803</id>
<content type='text'>
Don't mark pause frames as errors. This caused a bug that would
lockup transmitter, as well as all the other ports on a switch.

Signed-off-by: Stephen Hemminger &lt;shemminger@linux-foundation.org&gt;
Signed-off-by: Greg Kroah-Hartman &lt;gregkh@suse.de&gt;

</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
Don't mark pause frames as errors. This caused a bug that would
lockup transmitter, as well as all the other ports on a switch.

Signed-off-by: Stephen Hemminger &lt;shemminger@linux-foundation.org&gt;
Signed-off-by: Greg Kroah-Hartman &lt;gregkh@suse.de&gt;

</pre>
</div>
</content>
</entry>
<entry>
<title>sky2: transmit timeout deadlock</title>
<updated>2007-03-03T00:32:46+00:00</updated>
<author>
<name>Stephen Hemminger</name>
<email>shemminger@linux-foundation.org</email>
</author>
<published>2007-02-27T01:18:37+00:00</published>
<link rel='alternate' type='text/html' href='https://git.tavy.me/linux-stable.git/commit/?id=fead18b0e8a8ffc873482dee0e616d4d0d9592a6'/>
<id>fead18b0e8a8ffc873482dee0e616d4d0d9592a6</id>
<content type='text'>
The code in transmit timeout incorrectly assumed that netif_tx_lock
was not set.

Signed-off-by: Stephen Hemminger &lt;shemminger@linux-foundation.org&gt;
Signed-off-by: Greg Kroah-Hartman &lt;gregkh@suse.de&gt;


</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
The code in transmit timeout incorrectly assumed that netif_tx_lock
was not set.

Signed-off-by: Stephen Hemminger &lt;shemminger@linux-foundation.org&gt;
Signed-off-by: Greg Kroah-Hartman &lt;gregkh@suse.de&gt;


</pre>
</div>
</content>
</entry>
<entry>
<title>sky2: kfree_skb with IRQ with netconsole</title>
<updated>2007-03-03T00:32:46+00:00</updated>
<author>
<name>Stephen Hemminger</name>
<email>shemminger@linux-foundation.org</email>
</author>
<published>2007-02-27T01:18:36+00:00</published>
<link rel='alternate' type='text/html' href='https://git.tavy.me/linux-stable.git/commit/?id=eecf18ffe65123882562c54e55fd2d5e54ad4fef'/>
<id>eecf18ffe65123882562c54e55fd2d5e54ad4fef</id>
<content type='text'>
When using netconsole, it is possible for the driver to try and
free skb's with IRQ's disabled. This could cause problems if the skb
had a destructor function that didn't expect this. The documented semantics
is that skb-&gt;destructor callback happens in softirq.

Signed-off-by: Stephen Hemminger &lt;shemminger@linux-foundation.org&gt;
Signed-off-by: Greg Kroah-Hartman &lt;gregkh@suse.de&gt;

</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
When using netconsole, it is possible for the driver to try and
free skb's with IRQ's disabled. This could cause problems if the skb
had a destructor function that didn't expect this. The documented semantics
is that skb-&gt;destructor callback happens in softirq.

Signed-off-by: Stephen Hemminger &lt;shemminger@linux-foundation.org&gt;
Signed-off-by: Greg Kroah-Hartman &lt;gregkh@suse.de&gt;

</pre>
</div>
</content>
</entry>
<entry>
<title>sky2: fix for NAPI with dual port boards</title>
<updated>2007-03-03T00:32:46+00:00</updated>
<author>
<name>Stephen Hemminger</name>
<email>shemminger@linux-foundation.org</email>
</author>
<published>2007-02-27T01:18:35+00:00</published>
<link rel='alternate' type='text/html' href='https://git.tavy.me/linux-stable.git/commit/?id=fe864ebed2a1e894552bb426bbb02a2e2c2f6878'/>
<id>fe864ebed2a1e894552bb426bbb02a2e2c2f6878</id>
<content type='text'>
This driver uses port 0 to handle receives on both ports. So
the netif_poll_disable call in dev_close would end up stopping the
second port on dual port cards.

Signed-off-by: Stephen Hemminger &lt;shemminger@linux-foundation.org&gt;
Signed-off-by: Greg Kroah-Hartman &lt;gregkh@suse.de&gt;


</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
This driver uses port 0 to handle receives on both ports. So
the netif_poll_disable call in dev_close would end up stopping the
second port on dual port cards.

Signed-off-by: Stephen Hemminger &lt;shemminger@linux-foundation.org&gt;
Signed-off-by: Greg Kroah-Hartman &lt;gregkh@suse.de&gt;


</pre>
</div>
</content>
</entry>
<entry>
<title>sky2: pci config blocks phy power control</title>
<updated>2007-03-03T00:32:46+00:00</updated>
<author>
<name>Stephen Hemminger</name>
<email>shemminger@linux-foundation.org</email>
</author>
<published>2007-02-27T01:18:34+00:00</published>
<link rel='alternate' type='text/html' href='https://git.tavy.me/linux-stable.git/commit/?id=6c51946a6ee266a8861cecc8277f5b7009851738'/>
<id>6c51946a6ee266a8861cecc8277f5b7009851738</id>
<content type='text'>
PHY power control won't work because writes to PCI config space
are not re-enabled.

Signed-off-by: Stephen Hemminger &lt;shemminger@linux-foundation.org&gt;
Signed-off-by: Greg Kroah-Hartman &lt;gregkh@suse.de&gt;


</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
PHY power control won't work because writes to PCI config space
are not re-enabled.

Signed-off-by: Stephen Hemminger &lt;shemminger@linux-foundation.org&gt;
Signed-off-by: Greg Kroah-Hartman &lt;gregkh@suse.de&gt;


</pre>
</div>
</content>
</entry>
<entry>
<title>netpoll: drivers must not enable IRQ unconditionally in their NAPI handler</title>
<updated>2007-03-03T00:32:45+00:00</updated>
<author>
<name>Francois Romieu</name>
<email>romieu@fr.zoreil.com</email>
</author>
<published>2007-02-21T22:43:47+00:00</published>
<link rel='alternate' type='text/html' href='https://git.tavy.me/linux-stable.git/commit/?id=4e0f3f7beb26b952cfa78998191f89dedaf61a0d'/>
<id>4e0f3f7beb26b952cfa78998191f89dedaf61a0d</id>
<content type='text'>
netpoll: drivers must not enable IRQ unconditionally in their NAPI handler

net/core/netpoll.c::netpoll_send_skb() calls the poll handler when
it is available. As netconsole can be used from almost any context,
IRQ must not be enabled blindly in the NAPI handler of a driver which
supports netpoll.

b57bd06655a028aba7b92e1c19c2093e7fcfb341 fixed the issue for the
8139too.c driver.

Signed-off-by: Francois Romieu &lt;romieu@fr.zoreil.com&gt;
Signed-off-by: Jeff Garzik &lt;jeff@garzik.org&gt;
Signed-off-by: Greg Kroah-Hartman &lt;gregkh@suse.de&gt;

</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
netpoll: drivers must not enable IRQ unconditionally in their NAPI handler

net/core/netpoll.c::netpoll_send_skb() calls the poll handler when
it is available. As netconsole can be used from almost any context,
IRQ must not be enabled blindly in the NAPI handler of a driver which
supports netpoll.

b57bd06655a028aba7b92e1c19c2093e7fcfb341 fixed the issue for the
8139too.c driver.

Signed-off-by: Francois Romieu &lt;romieu@fr.zoreil.com&gt;
Signed-off-by: Jeff Garzik &lt;jeff@garzik.org&gt;
Signed-off-by: Greg Kroah-Hartman &lt;gregkh@suse.de&gt;

</pre>
</div>
</content>
</entry>
<entry>
<title>net, 8139too.c: fix netpoll deadlock</title>
<updated>2007-02-24T00:24:27+00:00</updated>
<author>
<name>Ingo Molnar</name>
<email>mingo@elte.hu</email>
</author>
<published>2007-02-09T23:32:19+00:00</published>
<link rel='alternate' type='text/html' href='https://git.tavy.me/linux-stable.git/commit/?id=29b1320675ea8502790bb2dfd4098ab970ea6edb'/>
<id>29b1320675ea8502790bb2dfd4098ab970ea6edb</id>
<content type='text'>
fix deadlock in the 8139too driver: poll handlers should never forcibly
enable local interrupts, because they might be used by netpoll/printk
from IRQ context.

  =================================
  [ INFO: inconsistent lock state ]
  2.6.19 #11
  ---------------------------------
  inconsistent {softirq-on-W} -&gt; {in-softirq-W} usage.
  swapper/1 [HC0[0]:SC1[1]:HE1:SE0] takes:
   (&amp;npinfo-&gt;poll_lock){-+..}, at: [&lt;c0350a41&gt;] net_rx_action+0x64/0x1de
  {softirq-on-W} state was registered at:
    [&lt;c0134c86&gt;] mark_lock+0x5b/0x39c
    [&lt;c0135012&gt;] mark_held_locks+0x4b/0x68
    [&lt;c01351e9&gt;] trace_hardirqs_on+0x115/0x139
    [&lt;c02879e6&gt;] rtl8139_poll+0x3d7/0x3f4
    [&lt;c035c85d&gt;] netpoll_poll+0x82/0x32f
    [&lt;c035c775&gt;] netpoll_send_skb+0xc9/0x12f
    [&lt;c035cdcc&gt;] netpoll_send_udp+0x253/0x25b
    [&lt;c0288463&gt;] write_msg+0x40/0x65
    [&lt;c011cead&gt;] __call_console_drivers+0x45/0x51
    [&lt;c011cf16&gt;] _call_console_drivers+0x5d/0x61
    [&lt;c011d4fb&gt;] release_console_sem+0x11f/0x1d8
    [&lt;c011d7d7&gt;] register_console+0x1ac/0x1b3
    [&lt;c02883f8&gt;] init_netconsole+0x55/0x67
    [&lt;c010040c&gt;] init+0x9a/0x24e
    [&lt;c01049cf&gt;] kernel_thread_helper+0x7/0x10
    [&lt;ffffffff&gt;] 0xffffffff
  irq event stamp: 819992
  hardirqs last  enabled at (819992): [&lt;c0350a16&gt;] net_rx_action+0x39/0x1de
  hardirqs last disabled at (819991): [&lt;c0350b1e&gt;] net_rx_action+0x141/0x1de
  softirqs last  enabled at (817552): [&lt;c01214e4&gt;] __do_softirq+0xa3/0xa8
  softirqs last disabled at (819987): [&lt;c0106051&gt;] do_softirq+0x5b/0xc9

  other info that might help us debug this:
  no locks held by swapper/1.

  stack backtrace:
   [&lt;c0104d88&gt;] dump_trace+0x63/0x1e8
   [&lt;c0104f26&gt;] show_trace_log_lvl+0x19/0x2e
   [&lt;c010532d&gt;] show_trace+0x12/0x14
   [&lt;c0105343&gt;] dump_stack+0x14/0x16
   [&lt;c0134980&gt;] print_usage_bug+0x23c/0x246
   [&lt;c0134d33&gt;] mark_lock+0x108/0x39c
   [&lt;c01356a7&gt;] __lock_acquire+0x361/0x9ed
   [&lt;c0136018&gt;] lock_acquire+0x56/0x72
   [&lt;c03aff1f&gt;] _spin_lock+0x35/0x42
   [&lt;c0350a41&gt;] net_rx_action+0x64/0x1de
   [&lt;c0121493&gt;] __do_softirq+0x52/0xa8
   [&lt;c0106051&gt;] do_softirq+0x5b/0xc9
   [&lt;c0121338&gt;] irq_exit+0x3c/0x48
   [&lt;c0106163&gt;] do_IRQ+0xa4/0xbd
   [&lt;c01047c6&gt;] common_interrupt+0x2e/0x34
   [&lt;c011db92&gt;] vprintk+0x2c0/0x309
   [&lt;c011dbf6&gt;] printk+0x1b/0x1d
   [&lt;c01003f2&gt;] init+0x80/0x24e
   [&lt;c01049cf&gt;] kernel_thread_helper+0x7/0x10
   =======================

Signed-off-by: Ingo Molnar &lt;mingo@elte.hu&gt;
Acked-by: Jeff Garzik &lt;jeff@garzik.org&gt;
Cc: Chuck Ebbert &lt;cebbert@redhat.com&gt;
Signed-off-by: Linus Torvalds &lt;torvalds@osdl.org&gt;
Signed-off-by: Greg Kroah-Hartman &lt;gregkh@suse.de&gt;

</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
fix deadlock in the 8139too driver: poll handlers should never forcibly
enable local interrupts, because they might be used by netpoll/printk
from IRQ context.

  =================================
  [ INFO: inconsistent lock state ]
  2.6.19 #11
  ---------------------------------
  inconsistent {softirq-on-W} -&gt; {in-softirq-W} usage.
  swapper/1 [HC0[0]:SC1[1]:HE1:SE0] takes:
   (&amp;npinfo-&gt;poll_lock){-+..}, at: [&lt;c0350a41&gt;] net_rx_action+0x64/0x1de
  {softirq-on-W} state was registered at:
    [&lt;c0134c86&gt;] mark_lock+0x5b/0x39c
    [&lt;c0135012&gt;] mark_held_locks+0x4b/0x68
    [&lt;c01351e9&gt;] trace_hardirqs_on+0x115/0x139
    [&lt;c02879e6&gt;] rtl8139_poll+0x3d7/0x3f4
    [&lt;c035c85d&gt;] netpoll_poll+0x82/0x32f
    [&lt;c035c775&gt;] netpoll_send_skb+0xc9/0x12f
    [&lt;c035cdcc&gt;] netpoll_send_udp+0x253/0x25b
    [&lt;c0288463&gt;] write_msg+0x40/0x65
    [&lt;c011cead&gt;] __call_console_drivers+0x45/0x51
    [&lt;c011cf16&gt;] _call_console_drivers+0x5d/0x61
    [&lt;c011d4fb&gt;] release_console_sem+0x11f/0x1d8
    [&lt;c011d7d7&gt;] register_console+0x1ac/0x1b3
    [&lt;c02883f8&gt;] init_netconsole+0x55/0x67
    [&lt;c010040c&gt;] init+0x9a/0x24e
    [&lt;c01049cf&gt;] kernel_thread_helper+0x7/0x10
    [&lt;ffffffff&gt;] 0xffffffff
  irq event stamp: 819992
  hardirqs last  enabled at (819992): [&lt;c0350a16&gt;] net_rx_action+0x39/0x1de
  hardirqs last disabled at (819991): [&lt;c0350b1e&gt;] net_rx_action+0x141/0x1de
  softirqs last  enabled at (817552): [&lt;c01214e4&gt;] __do_softirq+0xa3/0xa8
  softirqs last disabled at (819987): [&lt;c0106051&gt;] do_softirq+0x5b/0xc9

  other info that might help us debug this:
  no locks held by swapper/1.

  stack backtrace:
   [&lt;c0104d88&gt;] dump_trace+0x63/0x1e8
   [&lt;c0104f26&gt;] show_trace_log_lvl+0x19/0x2e
   [&lt;c010532d&gt;] show_trace+0x12/0x14
   [&lt;c0105343&gt;] dump_stack+0x14/0x16
   [&lt;c0134980&gt;] print_usage_bug+0x23c/0x246
   [&lt;c0134d33&gt;] mark_lock+0x108/0x39c
   [&lt;c01356a7&gt;] __lock_acquire+0x361/0x9ed
   [&lt;c0136018&gt;] lock_acquire+0x56/0x72
   [&lt;c03aff1f&gt;] _spin_lock+0x35/0x42
   [&lt;c0350a41&gt;] net_rx_action+0x64/0x1de
   [&lt;c0121493&gt;] __do_softirq+0x52/0xa8
   [&lt;c0106051&gt;] do_softirq+0x5b/0xc9
   [&lt;c0121338&gt;] irq_exit+0x3c/0x48
   [&lt;c0106163&gt;] do_IRQ+0xa4/0xbd
   [&lt;c01047c6&gt;] common_interrupt+0x2e/0x34
   [&lt;c011db92&gt;] vprintk+0x2c0/0x309
   [&lt;c011dbf6&gt;] printk+0x1b/0x1d
   [&lt;c01003f2&gt;] init+0x80/0x24e
   [&lt;c01049cf&gt;] kernel_thread_helper+0x7/0x10
   =======================

Signed-off-by: Ingo Molnar &lt;mingo@elte.hu&gt;
Acked-by: Jeff Garzik &lt;jeff@garzik.org&gt;
Cc: Chuck Ebbert &lt;cebbert@redhat.com&gt;
Signed-off-by: Linus Torvalds &lt;torvalds@osdl.org&gt;
Signed-off-by: Greg Kroah-Hartman &lt;gregkh@suse.de&gt;

</pre>
</div>
</content>
</entry>
<entry>
<title>prism54: correct assignment of DOT1XENABLE in WE-19 codepaths</title>
<updated>2007-02-24T00:24:27+00:00</updated>
<author>
<name>Dan Williams</name>
<email>dcbw@redhat.com</email>
</author>
<published>2007-02-13T21:07:27+00:00</published>
<link rel='alternate' type='text/html' href='https://git.tavy.me/linux-stable.git/commit/?id=8c1501237aa91f5d167b6647970e9947cd6c0f62'/>
<id>8c1501237aa91f5d167b6647970e9947cd6c0f62</id>
<content type='text'>
Correct assignment of DOT1XENABLE in WE-19 codepaths.
RX_UNENCRYPTED_EAPOL = 1 really means setting DOT1XENABLE _off_, and
vice versa.  The original WE-19 patch erroneously reversed that.  This
patch fixes association with unencrypted and WEP networks when using
wpa_supplicant.

It also adds two missing break statements that, left out, could result
in incorrect card configuration.

Applies to (I think) 2.6.19 and later.

Signed-off-by: Dan Williams &lt;dcbw@redhat.com&gt;
Signed-off-by: Greg Kroah-Hartman &lt;gregkh@suse.de&gt;

</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
Correct assignment of DOT1XENABLE in WE-19 codepaths.
RX_UNENCRYPTED_EAPOL = 1 really means setting DOT1XENABLE _off_, and
vice versa.  The original WE-19 patch erroneously reversed that.  This
patch fixes association with unencrypted and WEP networks when using
wpa_supplicant.

It also adds two missing break statements that, left out, could result
in incorrect card configuration.

Applies to (I think) 2.6.19 and later.

Signed-off-by: Dan Williams &lt;dcbw@redhat.com&gt;
Signed-off-by: Greg Kroah-Hartman &lt;gregkh@suse.de&gt;

</pre>
</div>
</content>
</entry>
<entry>
<title>bcm43xx: Fix for oops on ampdu status</title>
<updated>2007-02-24T00:24:26+00:00</updated>
<author>
<name>Michael Buesch</name>
<email>mb@bu3sch.de</email>
</author>
<published>2007-02-06T17:47:08+00:00</published>
<link rel='alternate' type='text/html' href='https://git.tavy.me/linux-stable.git/commit/?id=d0a0ad6ff5d23e30548c62ea9192dc4d4ab92773'/>
<id>d0a0ad6ff5d23e30548c62ea9192dc4d4ab92773</id>
<content type='text'>
If bcm43xx were to process an afterburner (ampdu) status response, Linux would oops. The
ampdu and intermediate status bits are properly named.

Signed-off-by: Michael Buesch &lt;mb@bu3sch.de&gt;
Signed-off-by: Larry Finger &lt;Larry.Finger@lwfinger.net&gt;
Signed-off-by: Greg Kroah-Hartman &lt;gregkh@suse.de&gt;

</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
If bcm43xx were to process an afterburner (ampdu) status response, Linux would oops. The
ampdu and intermediate status bits are properly named.

Signed-off-by: Michael Buesch &lt;mb@bu3sch.de&gt;
Signed-off-by: Larry Finger &lt;Larry.Finger@lwfinger.net&gt;
Signed-off-by: Greg Kroah-Hartman &lt;gregkh@suse.de&gt;

</pre>
</div>
</content>
</entry>
</feed>
