Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1754495AbdCIMU4 (ORCPT ); Thu, 9 Mar 2017 07:20:56 -0500 Received: from mail.linuxfoundation.org ([140.211.169.12]:48474 "EHLO mail.linuxfoundation.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1752441AbdCIMUy (ORCPT ); Thu, 9 Mar 2017 07:20:54 -0500 Date: Thu, 9 Mar 2017 13:10:41 +0100 From: "gregkh@linuxfoundation.org" To: Ajay Kaher Cc: Alan Stern , "linux-usb@vger.kernel.org" , "linux-kernel@vger.kernel.org" , AMAN DEEP , HEMANSHU SRIVASTAVA Subject: Re: Subject: [PATCH v4] USB:Core: BugFix: Proper handling of Race Condition when two USB class drivers try to call init_usb_class simultaneously Message-ID: <20170309121041.GB13054@kroah.com> References: <20170309113425epcms5p783be90da82c183653eafed97413469c4@epcms5p7> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20170309113425epcms5p783be90da82c183653eafed97413469c4@epcms5p7> User-Agent: Mutt/1.8.0 (2017-02-23) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 2457 Lines: 73 On Thu, Mar 09, 2017 at 11:34:25AM +0000, Ajay Kaher wrote: > From febeb10887d5026a489658fd9e911656e76038ac Mon Sep 17 00:00:00 2001 > From: Ajay Kaher > Date: Thu, 9 Mar 2017 16:07:54 +0530 > Subject: [PATCH v4] USB:Core: BugFix: Proper handling of Race Condition when two > USB class drivers try to call init_usb_class simultaneously Why is your subject line have the word "subject" in it? Please fix your email client so you don't have the whole git commit header in the body of the email like you do here. Also, no need to say "Core:" or "BugFix:" > > There is race condition when two USB class drivers try to call > init_usb_class at the same time and leads to crash. > code path: probe->usb_register_dev->init_usb_class > > To solve this, mutex locking has been added in init_usb_class() and > destroy_usb_class(). > > As pointed by Alan, removed "if (usb_class)" test from destroy_usb_class() > because usb_class can never be NULL there. > > Signed-off-by: Ajay Kaher > Acked-by: Alan Stern > --- > drivers/usb/core/file.c | 9 +++++++-- > 1 file changed, 7 insertions(+), 2 deletions(-) > > diff --git a/drivers/usb/core/file.c b/drivers/usb/core/file.c > index 822ced9..422ce7b 100644 > --- a/drivers/usb/core/file.c > +++ b/drivers/usb/core/file.c > @@ -27,6 +27,7 @@ > #define MAX_USB_MINORS 256 > static const struct file_operations *usb_minors[MAX_USB_MINORS]; > static DECLARE_RWSEM(minor_rwsem); > +static DEFINE_MUTEX(init_usb_class_mutex); > > static int usb_open(struct inode *inode, struct file *file) > { > @@ -109,8 +110,9 @@ static void release_usb_class(struct kref *kref) > > static void destroy_usb_class(void) > { > - if (usb_class) > - kref_put(&usb_class->kref, release_usb_class); > + mutex_lock(&init_usb_class_mutex); > + kref_put(&usb_class->kref, release_usb_class); > + mutex_unlock(&init_usb_class_mutex); > } > > int usb_major_init(void) > @@ -171,7 +173,10 @@ int usb_register_dev(struct usb_interface *intf, > if (intf->minor >= 0) > return -EADDRINUSE; > > + mutex_lock(&init_usb_class_mutex); > retval = init_usb_class(); > + mutex_unlock(&init_usb_class_mutex); > + > if (retval) > return retval; > All tabs were turned into spaces and this patch can not be applied :( Please fix up and try again. Send a patch to yourself first to see if it works properly before sending it to us. greg k-h