Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1754104Ab3CKPdO (ORCPT ); Mon, 11 Mar 2013 11:33:14 -0400 Received: from iolanthe.rowland.org ([192.131.102.54]:50231 "HELO iolanthe.rowland.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with SMTP id S1753864Ab3CKPct (ORCPT ); Mon, 11 Mar 2013 11:32:49 -0400 Date: Mon, 11 Mar 2013 11:32:48 -0400 (EDT) From: Alan Stern X-X-Sender: stern@iolanthe.rowland.org To: Ming Lei cc: Alexey Khoroshilov , Greg Kroah-Hartman , Hans de Goede , , , Subject: Re: [PATCH] usb/core/devio.c: Don't use GFP_KERNEL while we cannot reset a storage device In-Reply-To: Message-ID: MIME-Version: 1.0 Content-Type: TEXT/PLAIN; charset=US-ASCII Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 1356 Lines: 32 On Mon, 11 Mar 2013, Ming Lei wrote: > On Sat, Mar 9, 2013 at 12:50 AM, Alexey Khoroshilov > wrote: > > As it was described by Oliver Neukum in commit acbe2fe > > "USB: Don't use GFP_KERNEL while we cannot reset a storage device": > > > > Memory allocations with GFP_KERNEL can cause IO to a storage device > > which can fail resulting in a need to reset the device. Therefore > > GFP_KERNEL cannot be safely used between usb_lock_device() > > and usb_unlock_device(). Replace by GFP_NOIO. > > > > The patch fixes the same issue in usb/core/devio.c. > > All the allocations fixed are under usb_lock_device() from usbdev_do_ioctl(). > > I am wondering why the device lock is needed for usbdev_do_ioctl()? Looks > device lock isn't required for USB transfer of kernel driver. Of course you have to lock the device before changing its driver. What would happen if two different threads tried to change a device's driver at the same time? usbdev_do_ioctl() needs to acquire the device lock in order to prevent races with driver_disconnect() and usbdev_remove(). Alan Stern -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/