From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1751784AbdBAJAE (ORCPT ); Wed, 1 Feb 2017 04:00:04 -0500 Received: from mail.linuxfoundation.org ([140.211.169.12]:60280 "EHLO mail.linuxfoundation.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751560AbdBAJAC (ORCPT ); Wed, 1 Feb 2017 04:00:02 -0500 Date: Wed, 1 Feb 2017 10:00:06 +0100 From: "gregkh@linuxfoundation.org" To: Ajay Kaher Cc: "linux-usb@vger.kernel.org" , "linux-kernel@vger.kernel.org" , AMAN DEEP , HEMANSHU SRIVASTAVA Subject: Re: Re: Re: Subject: [PATCH v1] USB:Core: BugFix: Proper handling of Race Condition when two USB class drivers try to call init_usb_class simultaneously Message-ID: <20170201090006.GA25372@kroah.com> References: <20170130082525epcms5p5e2459e715ebb80cef1574946e3f6ffbf@epcms5p5> <20170131052146epcms5p79112e0ba47847b8b0d33aee7b66590c9@epcms5p7> <20170201072444epcms5p34768dd535eb6a99e3367befb375e663e@epcms5p3> MIME-Version: 1.0 Content-Type: text/plain; charset=iso-8859-1 Content-Disposition: inline Content-Transfer-Encoding: 8bit In-Reply-To: <20170201072444epcms5p34768dd535eb6a99e3367befb375e663e@epcms5p3> User-Agent: Mutt/1.7.2 (2016-11-26) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Wed, Feb 01, 2017 at 07:24:44AM +0000, Ajay Kaher wrote: >   > >> At boot time, probe function of multiple connected devices > >> (proprietary devices) execute simultaneously. > > > >What exactly do you mean here? How can probe happen "simultaneously"? > >The USB core prevents this, right? > > I have observed two scenarios to call probe function: > > Scenario #1: Driver inserted and attaching USB Device: > Yes, you are right, two probes at same time is not happening > in this scenario. > > Scenario #2: USB Device attached and inserting Driver: > In this case probe has been called in context of insmod, > refer following code flow: > init -> usb_register_driver -> driver_register -> bus_add_driver -> > driver_attach -> bus_for_each_dev -> __driver_attach -> > driver_probe_device -> usb_probe_interface -> probe -> usb_register_dev > > I have observed the crash in Scenario #2, as two probes executes at > same time in this scenario. And init_usb_class_mutex lock require to > prevent race condition. What about the fact that in __driver_attach() we call device_lock() so that probe never gets called at the same time for the same device? Or are you saying that you can load multiple USB modules at the same time? If so, how is insmod running on multiple cpus at the same time? I thought we had a global lock there to prevent that from happening (i.e. only one module can be loaded at a time.) Or is that what has recently changed? What is causing your modules to be loaded from userspace? What type of device is this happening for? And why haven't we seen this before? What kernel versions have you had a problem with this? And what for what drivers specifically? > >> And because of the following code path race condition happens: > >> probe->usb_register_dev->init_usb_class > > > >Why is this just showing up now, and hasn't been an issue for the decade > >or so this code has been around? What changed? > > > >> Tested with these changes, and problem has been solved. > > > >What changes? > > Tested with my patch (i.e. locking with init_usb_class_mutex). I don't see a patch here :( thanks, greg k-h