* [PATCH] usb: host: iounmap before return
@ 2015-12-06 13:47 Geyslan G. Bem
2015-12-07 9:30 ` Lu Baolu
0 siblings, 1 reply; 5+ messages in thread
From: Geyslan G. Bem @ 2015-12-06 13:47 UTC (permalink / raw)
To: peter.senna
Cc: Geyslan G. Bem, Mathias Nyman, Greg Kroah-Hartman, linux-usb,
linux-kernel
This patch fixes a 'quirk_usb_handoff_xhci()' branch return that was not unmapping correctly.
Coccinelle: scripts/coccinelle/free/iounmap.cocci
Signed-off-by: Geyslan G. Bem <geyslan@gmail.com>
---
drivers/usb/host/pci-quirks.c | 3 ++-
1 file changed, 2 insertions(+), 1 deletion(-)
diff --git a/drivers/usb/host/pci-quirks.c b/drivers/usb/host/pci-quirks.c
index f940056..64150dd 100644
--- a/drivers/usb/host/pci-quirks.c
+++ b/drivers/usb/host/pci-quirks.c
@@ -990,7 +990,7 @@ static void quirk_usb_handoff_xhci(struct pci_dev *pdev)
/* We're reading garbage from the controller */
dev_warn(&pdev->dev,
"xHCI controller failing to respond");
- return;
+ goto out;
}
if (!ext_cap_offset)
@@ -1062,6 +1062,7 @@ hc_init:
XHCI_MAX_HALT_USEC, val);
}
+out:
iounmap(base);
}
--
2.6.3
^ permalink raw reply related [flat|nested] 5+ messages in thread
* Re: [PATCH] usb: host: iounmap before return
2015-12-06 13:47 [PATCH] usb: host: iounmap before return Geyslan G. Bem
@ 2015-12-07 9:30 ` Lu Baolu
2015-12-07 10:26 ` Geyslan G. Bem
2015-12-07 11:37 ` Sergei Shtylyov
0 siblings, 2 replies; 5+ messages in thread
From: Lu Baolu @ 2015-12-07 9:30 UTC (permalink / raw)
To: Geyslan G. Bem, peter.senna
Cc: Mathias Nyman, Greg Kroah-Hartman, linux-usb, linux-kernel
On 12/06/2015 09:47 PM, Geyslan G. Bem wrote:
> This patch fixes a 'quirk_usb_handoff_xhci()' branch return that was not unmapping correctly.
>
> Coccinelle: scripts/coccinelle/free/iounmap.cocci
>
> Signed-off-by: Geyslan G. Bem <geyslan@gmail.com>
> ---
> drivers/usb/host/pci-quirks.c | 3 ++-
> 1 file changed, 2 insertions(+), 1 deletion(-)
>
> diff --git a/drivers/usb/host/pci-quirks.c b/drivers/usb/host/pci-quirks.c
> index f940056..64150dd 100644
> --- a/drivers/usb/host/pci-quirks.c
> +++ b/drivers/usb/host/pci-quirks.c
> @@ -990,7 +990,7 @@ static void quirk_usb_handoff_xhci(struct pci_dev *pdev)
> /* We're reading garbage from the controller */
> dev_warn(&pdev->dev,
> "xHCI controller failing to respond");
> - return;
> + goto out;
If "out" is only used here, why not iounmap and return directly here?
> }
>
> if (!ext_cap_offset)
> @@ -1062,6 +1062,7 @@ hc_init:
> XHCI_MAX_HALT_USEC, val);
> }
>
> +out:
> iounmap(base);
> }
>
^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [PATCH] usb: host: iounmap before return
2015-12-07 9:30 ` Lu Baolu
@ 2015-12-07 10:26 ` Geyslan G. Bem
2015-12-07 11:37 ` Sergei Shtylyov
1 sibling, 0 replies; 5+ messages in thread
From: Geyslan G. Bem @ 2015-12-07 10:26 UTC (permalink / raw)
To: Lu Baolu
Cc: Peter Senna Tschudin, Mathias Nyman, Greg Kroah-Hartman, linux-usb, LKML
2015-12-07 6:30 GMT-03:00 Lu Baolu <baolu.lu@linux.intel.com>:
>
>
> On 12/06/2015 09:47 PM, Geyslan G. Bem wrote:
>> This patch fixes a 'quirk_usb_handoff_xhci()' branch return that was not unmapping correctly.
>>
>> Coccinelle: scripts/coccinelle/free/iounmap.cocci
>>
>> Signed-off-by: Geyslan G. Bem <geyslan@gmail.com>
>> ---
>> drivers/usb/host/pci-quirks.c | 3 ++-
>> 1 file changed, 2 insertions(+), 1 deletion(-)
>>
>> diff --git a/drivers/usb/host/pci-quirks.c b/drivers/usb/host/pci-quirks.c
>> index f940056..64150dd 100644
>> --- a/drivers/usb/host/pci-quirks.c
>> +++ b/drivers/usb/host/pci-quirks.c
>> @@ -990,7 +990,7 @@ static void quirk_usb_handoff_xhci(struct pci_dev *pdev)
>> /* We're reading garbage from the controller */
>> dev_warn(&pdev->dev,
>> "xHCI controller failing to respond");
>> - return;
>> + goto out;
>
> If "out" is only used here, why not iounmap and return directly here?
Could be directly. I just think that code recurrence can make things
complicated in future. What do you think?
Ah, seems that this patch is not aligned with linux-next. I'll make a
new one soon.
>
>> }
>>
>> if (!ext_cap_offset)
>> @@ -1062,6 +1062,7 @@ hc_init:
>> XHCI_MAX_HALT_USEC, val);
>> }
>>
>> +out:
>> iounmap(base);
>> }
>>
>
--
Regards,
Geyslan G. Bem
hackingbits.com
^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [PATCH] usb: host: iounmap before return
2015-12-07 9:30 ` Lu Baolu
2015-12-07 10:26 ` Geyslan G. Bem
@ 2015-12-07 11:37 ` Sergei Shtylyov
2015-12-07 23:39 ` Geyslan G. Bem
1 sibling, 1 reply; 5+ messages in thread
From: Sergei Shtylyov @ 2015-12-07 11:37 UTC (permalink / raw)
To: Lu Baolu, Geyslan G. Bem, peter.senna
Cc: Mathias Nyman, Greg Kroah-Hartman, linux-usb, linux-kernel
Hello.
On 12/07/2015 12:30 PM, Lu Baolu wrote:
>> This patch fixes a 'quirk_usb_handoff_xhci()' branch return that was not unmapping correctly.
>>
>> Coccinelle: scripts/coccinelle/free/iounmap.cocci
>>
>> Signed-off-by: Geyslan G. Bem <geyslan@gmail.com>
>> ---
>> drivers/usb/host/pci-quirks.c | 3 ++-
>> 1 file changed, 2 insertions(+), 1 deletion(-)
>>
>> diff --git a/drivers/usb/host/pci-quirks.c b/drivers/usb/host/pci-quirks.c
>> index f940056..64150dd 100644
>> --- a/drivers/usb/host/pci-quirks.c
>> +++ b/drivers/usb/host/pci-quirks.c
>> @@ -990,7 +990,7 @@ static void quirk_usb_handoff_xhci(struct pci_dev *pdev)
>> /* We're reading garbage from the controller */
>> dev_warn(&pdev->dev,
>> "xHCI controller failing to respond");
>> - return;
>> + goto out;
>
> If "out" is only used here, why not iounmap and return directly here?
Why repeat the code? Nah, bad idea.
>> }
>>
>> if (!ext_cap_offset)
>> @@ -1062,6 +1062,7 @@ hc_init:
>> XHCI_MAX_HALT_USEC, val);
>> }
>>
>> +out:
>> iounmap(base);
>> }
>>
MBR, Sergei
^ permalink raw reply [flat|nested] 5+ messages in thread* Re: [PATCH] usb: host: iounmap before return
2015-12-07 11:37 ` Sergei Shtylyov
@ 2015-12-07 23:39 ` Geyslan G. Bem
0 siblings, 0 replies; 5+ messages in thread
From: Geyslan G. Bem @ 2015-12-07 23:39 UTC (permalink / raw)
To: Sergei Shtylyov
Cc: Lu Baolu, Peter Senna Tschudin, Mathias Nyman,
Greg Kroah-Hartman, linux-usb, LKML
2015-12-07 8:37 GMT-03:00 Sergei Shtylyov <sergei.shtylyov@cogentembedded.com>:
> Hello.
>
> On 12/07/2015 12:30 PM, Lu Baolu wrote:
>
>>> This patch fixes a 'quirk_usb_handoff_xhci()' branch return that was not
>>> unmapping correctly.
>>>
>>> Coccinelle: scripts/coccinelle/free/iounmap.cocci
>>>
>>> Signed-off-by: Geyslan G. Bem <geyslan@gmail.com>
>>> ---
>>> drivers/usb/host/pci-quirks.c | 3 ++-
>>> 1 file changed, 2 insertions(+), 1 deletion(-)
>>>
>>> diff --git a/drivers/usb/host/pci-quirks.c
>>> b/drivers/usb/host/pci-quirks.c
>>> index f940056..64150dd 100644
>>> --- a/drivers/usb/host/pci-quirks.c
>>> +++ b/drivers/usb/host/pci-quirks.c
>>> @@ -990,7 +990,7 @@ static void quirk_usb_handoff_xhci(struct pci_dev
>>> *pdev)
>>> /* We're reading garbage from the controller */
>>> dev_warn(&pdev->dev,
>>> "xHCI controller failing to respond");
>>> - return;
>>> + goto out;
>>
>>
>> If "out" is only used here, why not iounmap and return directly here?
>
>
> Why repeat the code? Nah, bad idea.
>
>>> }
>>>
>>> if (!ext_cap_offset)
>>> @@ -1062,6 +1062,7 @@ hc_init:
>>> XHCI_MAX_HALT_USEC, val);
>>> }
>>>
>>> +out:
>>> iounmap(base);
>>> }
>>>
>
> MBR, Sergei
>
Look for:
[PATCH v2] usb: host: iounmap before return
--
Regards,
Geyslan G. Bem
hackingbits.com
^ permalink raw reply [flat|nested] 5+ messages in thread
end of thread, other threads:[~2015-12-07 23:40 UTC | newest]
Thread overview: 5+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2015-12-06 13:47 [PATCH] usb: host: iounmap before return Geyslan G. Bem
2015-12-07 9:30 ` Lu Baolu
2015-12-07 10:26 ` Geyslan G. Bem
2015-12-07 11:37 ` Sergei Shtylyov
2015-12-07 23:39 ` Geyslan G. Bem
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.