Donate to e Foundation | Murena handsets with /e/OS | Own a part of Murena! Learn more

Commit 01c5952d authored by David Mosberger's avatar David Mosberger Committed by TARKZiM
Browse files

drivers: usb: core: Minimize irq disabling in usb_sg_cancel() 



commit 5f2e5fb873e269fcb806165715d237f0de4ecf1d upstream.

Restructure usb_sg_cancel() so we don't have to disable interrupts
while cancelling the URBs.

Suggested-by: default avatarAlan Stern <stern@rowland.harvard.edu>
Signed-off-by: default avatarDavid Mosberger <davidm@egauge.net>
Signed-off-by: default avatarGreg Kroah-Hartman <gregkh@linuxfoundation.org>
Signed-off-by: default avatarBen Hutchings <ben@decadent.org.uk>
Signed-off-by: default avatarKevin F. Haggerty <haggertk@lineageos.org>
Change-Id: Ic298903a2b2b5c8cc0378bad2c6dabfc08099fa8
parent 67e03559

drivers/usb/core/message.c

+17 −20
Original line number Diff line number Diff line
@@ -578,18 +578,18 @@ EXPORT_SYMBOL_GPL(usb_sg_wait); 
void usb_sg_cancel(struct usb_sg_request *io)

{

	unsigned long flags;

	int i, retval;



	spin_lock_irqsave(&io->lock, flags);



	/* shut everything down, if it didn't already */

	if (!io->status) {

		int i;



	if (io->status) {

		spin_unlock_irqrestore(&io->lock, flags);

		return;

	}

	/* shut everything down */

	io->status = -ECONNRESET;

		spin_unlock(&io->lock);

		for (i = 0; i < io->entries; i++) {

			int retval;

	spin_unlock_irqrestore(&io->lock, flags);



	for (i = io->entries - 1; i >= 0; --i) {

		usb_block_urb(io->urbs[i]);



		retval = usb_unlink_urb(io->urbs[i]);
@@ -600,9 +600,6 @@ void usb_sg_cancel(struct usb_sg_request *io) 
			dev_warn(&io->dev->dev, "%s, unlink --> %d\n",

				 __func__, retval);

	}

		spin_lock(&io->lock);

	}

	spin_unlock_irqrestore(&io->lock, flags);

}

EXPORT_SYMBOL_GPL(usb_sg_cancel);