Re: [PATCH] USB: core: Improve unlocking of a mutex in two functions

2017-11-05 Thread SF Markus Elfring
>> @@ -5529,8 +5528,7 @@ static int usb_reset_and_verify_device(struct >> usb_device *udev) >> dev_err(>dev, >> "can't restore configuration #%d (error=%d)\n", >> udev->actconfig->desc.bConfigurationValue, ret); >> -

Re: [PATCH] USB: core: Improve unlocking of a mutex in two functions

2017-11-05 Thread Greg Kroah-Hartman
On Sun, Nov 05, 2017 at 12:25:19AM +0100, Geert Uytterhoeven wrote: > On Sat, Nov 4, 2017 at 9:12 PM, SF Markus Elfring > wrote: > > From: Markus Elfring > > Date: Sat, 4 Nov 2017 21:00:46 +0100 > > > > * Add jump targets so that a

Re: [PATCH] USB: core: Improve unlocking of a mutex in two functions

2017-11-04 Thread Geert Uytterhoeven
On Sat, Nov 4, 2017 at 9:12 PM, SF Markus Elfring wrote: > From: Markus Elfring > Date: Sat, 4 Nov 2017 21:00:46 +0100 > > * Add jump targets so that a call of the function "mutex_unlock" is stored > only twice in these function

[PATCH] USB: core: Improve unlocking of a mutex in two functions

2017-11-04 Thread SF Markus Elfring
From: Markus Elfring Date: Sat, 4 Nov 2017 21:00:46 +0100 * Add jump targets so that a call of the function "mutex_unlock" is stored only twice in these function implementations. * Replace five calls by goto statements. This issue was detected by using the