lkml.org 
[lkml]   [2012]   [Aug]   [15]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
    Patch in this message
    /
    From
    Subject[v2.6.34-stable 023/165] rt2x00: do not drop usb dev reference counter on suspend
    Date
    From: Stanislaw Gruszka <sgruszka@redhat.com>

    -------------------
    This is a commit scheduled for the next v2.6.34 longterm release.
    http://git.kernel.org/?p=linux/kernel/git/paulg/longterm-queue-2.6.34.git
    If you see a problem with using this for longterm, please comment.
    -------------------

    commit 543cc38c8fe86deba4169977c61eb88491036837 upstream.

    When hibernating ->resume may not be called by usb core, but disconnect
    and probe instead, so we do not increase the counter after decreasing
    it in ->supend. As a result we free memory early, and get crash when
    unplugging usb dongle.

    BUG: unable to handle kernel paging request at 6b6b6b9f
    IP: [<c06909b0>] driver_sysfs_remove+0x10/0x30
    *pdpt = 0000000034f21001 *pde = 0000000000000000
    Pid: 20, comm: khubd Not tainted 3.1.0-rc1-wl+ #20 LENOVO 6369CTO/6369CTO
    EIP: 0060:[<c06909b0>] EFLAGS: 00010202 CPU: 1
    EIP is at driver_sysfs_remove+0x10/0x30
    EAX: 6b6b6b6b EBX: f52bba34 ECX: 00000000 EDX: 6b6b6b6b
    ESI: 6b6b6b6b EDI: c0a0ea20 EBP: f61c9e68 ESP: f61c9e64
    DS: 007b ES: 007b FS: 00d8 GS: 00e0 SS: 0068
    Process khubd (pid: 20, ti=f61c8000 task=f6138270 task.ti=f61c8000)
    Call Trace:
    [<c06909ef>] __device_release_driver+0x1f/0xa0
    [<c0690b20>] device_release_driver+0x20/0x40
    [<c068fd64>] bus_remove_device+0x84/0xe0
    [<c068e12a>] ? device_remove_attrs+0x2a/0x80
    [<c068e267>] device_del+0xe7/0x170
    [<c06d93d4>] usb_disconnect+0xd4/0x180
    [<c06d9d61>] hub_thread+0x691/0x1600
    [<c0473260>] ? wake_up_bit+0x30/0x30
    [<c0442a39>] ? complete+0x49/0x60
    [<c06d96d0>] ? hub_disconnect+0xd0/0xd0
    [<c06d96d0>] ? hub_disconnect+0xd0/0xd0
    [<c0472eb4>] kthread+0x74/0x80
    [<c0472e40>] ? kthread_worker_fn+0x150/0x150
    [<c0809b3e>] kernel_thread_helper+0x6/0x10

    Signed-off-by: Stanislaw Gruszka <sgruszka@redhat.com>
    Acked-by: Ivo van Doorn <IvDoorn@gmail.com>
    Signed-off-by: John W. Linville <linville@tuxdriver.com>
    Signed-off-by: Paul Gortmaker <paul.gortmaker@windriver.com>
    ---
    drivers/net/wireless/rt2x00/rt2x00usb.c | 14 +-------------
    1 file changed, 1 insertion(+), 13 deletions(-)

    diff --git a/drivers/net/wireless/rt2x00/rt2x00usb.c b/drivers/net/wireless/rt2x00/rt2x00usb.c
    index f9a7f8b..f423c65 100644
    --- a/drivers/net/wireless/rt2x00/rt2x00usb.c
    +++ b/drivers/net/wireless/rt2x00/rt2x00usb.c
    @@ -706,18 +706,8 @@ int rt2x00usb_suspend(struct usb_interface *usb_intf, pm_message_t state)
    {
    struct ieee80211_hw *hw = usb_get_intfdata(usb_intf);
    struct rt2x00_dev *rt2x00dev = hw->priv;
    - int retval;
    -
    - retval = rt2x00lib_suspend(rt2x00dev, state);
    - if (retval)
    - return retval;

    - /*
    - * Decrease usbdev refcount.
    - */
    - usb_put_dev(interface_to_usbdev(usb_intf));
    -
    - return 0;
    + return rt2x00lib_suspend(rt2x00dev, state);
    }
    EXPORT_SYMBOL_GPL(rt2x00usb_suspend);

    @@ -726,8 +716,6 @@ int rt2x00usb_resume(struct usb_interface *usb_intf)
    struct ieee80211_hw *hw = usb_get_intfdata(usb_intf);
    struct rt2x00_dev *rt2x00dev = hw->priv;

    - usb_get_dev(interface_to_usbdev(usb_intf));
    -
    return rt2x00lib_resume(rt2x00dev);
    }
    EXPORT_SYMBOL_GPL(rt2x00usb_resume);
    --
    1.7.12.rc2


    \
     
     \ /
      Last update: 2012-08-16 00:42    [W:4.461 / U:0.192 seconds]
    ©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site