* [PATCH] soc: ti: qmss: fix the case when !SMP
@ 2016-11-29 22:15 Grygorii Strashko
2016-11-30 11:37 ` Arnd Bergmann
0 siblings, 1 reply; 7+ messages in thread
From: Grygorii Strashko @ 2016-11-29 22:15 UTC (permalink / raw)
To: Murali Karicheri, Santosh Shilimkar
Cc: Sekhar Nori, linux-arm-kernel, linux-kernel, Grygorii Strashko
The irq_set_affinity_hint() will always fail when !SMP and
Networking will fail on Keystone 2 devices in this case.
Hence, fix by ignoring IRQ affinity settings when !SMP.
Cc: Murali Karicheri <m-karicheri2@ti.com>
Signed-off-by: Grygorii Strashko <grygorii.strashko@ti.com>
---
drivers/soc/ti/knav_qmss_queue.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/soc/ti/knav_qmss_queue.c b/drivers/soc/ti/knav_qmss_queue.c
index b73e353..3aa0470 100644
--- a/drivers/soc/ti/knav_qmss_queue.c
+++ b/drivers/soc/ti/knav_qmss_queue.c
@@ -1228,7 +1228,7 @@ static int knav_setup_queue_range(struct knav_device *kdev,
range->num_irqs++;
- if (oirq.args_count == 3)
+ if (IS_ENABLED(SMP) && oirq.args_count == 3)
range->irqs[i].cpu_map =
(oirq.args[2] & 0x0000ff00) >> 8;
}
--
2.10.1
^ permalink raw reply related [flat|nested] 7+ messages in thread
* Re: [PATCH] soc: ti: qmss: fix the case when !SMP
2016-11-29 22:15 [PATCH] soc: ti: qmss: fix the case when !SMP Grygorii Strashko
@ 2016-11-30 11:37 ` Arnd Bergmann
2016-11-30 16:10 ` Grygorii Strashko
2016-11-30 17:01 ` Santosh Shilimkar
0 siblings, 2 replies; 7+ messages in thread
From: Arnd Bergmann @ 2016-11-30 11:37 UTC (permalink / raw)
To: linux-arm-kernel
Cc: Grygorii Strashko, Murali Karicheri, Santosh Shilimkar,
Sekhar Nori, linux-kernel
On Tuesday, November 29, 2016 4:15:08 PM CET Grygorii Strashko wrote:
> range->num_irqs++;
>
> - if (oirq.args_count == 3)
> + if (IS_ENABLED(SMP) && oirq.args_count == 3)
> range->irqs[i].cpu_map =
> (oirq.args[2] & 0x0000ff00) >> 8;
>
I think you mean CONFIG_SMP, not SMP. With the change above, the
code will never be executed.
Arnd
^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: [PATCH] soc: ti: qmss: fix the case when !SMP
2016-11-30 11:37 ` Arnd Bergmann
@ 2016-11-30 16:10 ` Grygorii Strashko
2016-11-30 17:01 ` Santosh Shilimkar
1 sibling, 0 replies; 7+ messages in thread
From: Grygorii Strashko @ 2016-11-30 16:10 UTC (permalink / raw)
To: Arnd Bergmann, linux-arm-kernel
Cc: Murali Karicheri, Santosh Shilimkar, Sekhar Nori, linux-kernel
On 11/30/2016 05:37 AM, Arnd Bergmann wrote:
> On Tuesday, November 29, 2016 4:15:08 PM CET Grygorii Strashko wrote:
>> range->num_irqs++;
>>
>> - if (oirq.args_count == 3)
>> + if (IS_ENABLED(SMP) && oirq.args_count == 3)
>> range->irqs[i].cpu_map =
>> (oirq.args[2] & 0x0000ff00) >> 8;
>>
>
> I think you mean CONFIG_SMP, not SMP. With the change above, the
> code will never be executed.
>
Thanks. I'll resend.
--
regards,
-grygorii
^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: [PATCH] soc: ti: qmss: fix the case when !SMP
2016-11-30 11:37 ` Arnd Bergmann
2016-11-30 16:10 ` Grygorii Strashko
@ 2016-11-30 17:01 ` Santosh Shilimkar
2016-11-30 17:10 ` Grygorii Strashko
1 sibling, 1 reply; 7+ messages in thread
From: Santosh Shilimkar @ 2016-11-30 17:01 UTC (permalink / raw)
To: Arnd Bergmann, linux-arm-kernel
Cc: Grygorii Strashko, Murali Karicheri, Santosh Shilimkar,
Sekhar Nori, linux-kernel
Hi Grygorii,
On 11/30/2016 3:37 AM, Arnd Bergmann wrote:
> On Tuesday, November 29, 2016 4:15:08 PM CET Grygorii Strashko wrote:
>> range->num_irqs++;
>>
>> - if (oirq.args_count == 3)
>> + if (IS_ENABLED(SMP) && oirq.args_count == 3)
>> range->irqs[i].cpu_map =
>> (oirq.args[2] & 0x0000ff00) >> 8;
>>
>
> I think you mean CONFIG_SMP, not SMP. With the change above, the
> code will never be executed.
>
Is that the full patch ? Can you post updated patch wit above fixed
and copy me. I will pick it up.
Regards,
Santosh
^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: [PATCH] soc: ti: qmss: fix the case when !SMP
2016-11-30 17:01 ` Santosh Shilimkar
@ 2016-11-30 17:10 ` Grygorii Strashko
2016-11-30 17:14 ` Santosh Shilimkar
0 siblings, 1 reply; 7+ messages in thread
From: Grygorii Strashko @ 2016-11-30 17:10 UTC (permalink / raw)
To: Santosh Shilimkar, Arnd Bergmann, linux-arm-kernel
Cc: Murali Karicheri, Santosh Shilimkar, Sekhar Nori, linux-kernel
Hi Santosh,
On 11/30/2016 11:01 AM, Santosh Shilimkar wrote:
> Hi Grygorii,
>
> On 11/30/2016 3:37 AM, Arnd Bergmann wrote:
>> On Tuesday, November 29, 2016 4:15:08 PM CET Grygorii Strashko wrote:
>>> range->num_irqs++;
>>>
>>> - if (oirq.args_count == 3)
>>> + if (IS_ENABLED(SMP) && oirq.args_count == 3)
>>> range->irqs[i].cpu_map =
>>> (oirq.args[2] & 0x0000ff00) >> 8;
>>>
>>
>> I think you mean CONFIG_SMP, not SMP. With the change above, the
>> code will never be executed.
>>
> Is that the full patch ? Can you post updated patch wit above fixed
> and copy me. I will pick it up.
>
I've sent v2.
For some reason you e-email is not working - delivery failure
ssantosh@kernel.org
--
regards,
-grygorii
^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: [PATCH] soc: ti: qmss: fix the case when !SMP
2016-11-30 17:10 ` Grygorii Strashko
@ 2016-11-30 17:14 ` Santosh Shilimkar
2016-12-02 16:41 ` Santosh Shilimkar
0 siblings, 1 reply; 7+ messages in thread
From: Santosh Shilimkar @ 2016-11-30 17:14 UTC (permalink / raw)
To: Grygorii Strashko, Arnd Bergmann, linux-arm-kernel
Cc: Murali Karicheri, Santosh Shilimkar, Sekhar Nori, linux-kernel
On 11/30/2016 9:10 AM, Grygorii Strashko wrote:
>
> Hi Santosh,
>
> On 11/30/2016 11:01 AM, Santosh Shilimkar wrote:
>> Hi Grygorii,
>>
>> On 11/30/2016 3:37 AM, Arnd Bergmann wrote:
>>> On Tuesday, November 29, 2016 4:15:08 PM CET Grygorii Strashko wrote:
>>>> range->num_irqs++;
>>>>
>>>> - if (oirq.args_count == 3)
>>>> + if (IS_ENABLED(SMP) && oirq.args_count == 3)
>>>> range->irqs[i].cpu_map =
>>>> (oirq.args[2] & 0x0000ff00) >> 8;
>>>>
>>>
>>> I think you mean CONFIG_SMP, not SMP. With the change above, the
>>> code will never be executed.
>>>
>> Is that the full patch ? Can you post updated patch wit above fixed
>> and copy me. I will pick it up.
>>
>
> I've sent v2.
>
> For some reason you e-email is not working - delivery failure
> ssantosh@kernel.org
>
Weird. I was getting all the emails. Thanks for bouncing the thread.
I will check.
^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: [PATCH] soc: ti: qmss: fix the case when !SMP
2016-11-30 17:14 ` Santosh Shilimkar
@ 2016-12-02 16:41 ` Santosh Shilimkar
0 siblings, 0 replies; 7+ messages in thread
From: Santosh Shilimkar @ 2016-12-02 16:41 UTC (permalink / raw)
To: Grygorii Strashko, Arnd Bergmann, linux-arm-kernel
Cc: Murali Karicheri, Santosh Shilimkar, Sekhar Nori, linux-kernel
On 11/30/2016 9:14 AM, Santosh Shilimkar wrote:
> On 11/30/2016 9:10 AM, Grygorii Strashko wrote:
>>
[...]
>>
>> For some reason you e-email is not working - delivery failure
>> ssantosh@kernel.org
>>
> Weird. I was getting all the emails. Thanks for bouncing the thread.
> I will check.
>
Seems like there was issue with kernel.org mail host which I have
been told fixed now. You shouldn't see those bouncing anymore.
Regards,
Santosh
^ permalink raw reply [flat|nested] 7+ messages in thread
end of thread, other threads:[~2016-12-02 16:43 UTC | newest]
Thread overview: 7+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2016-11-29 22:15 [PATCH] soc: ti: qmss: fix the case when !SMP Grygorii Strashko
2016-11-30 11:37 ` Arnd Bergmann
2016-11-30 16:10 ` Grygorii Strashko
2016-11-30 17:01 ` Santosh Shilimkar
2016-11-30 17:10 ` Grygorii Strashko
2016-11-30 17:14 ` Santosh Shilimkar
2016-12-02 16:41 ` Santosh Shilimkar
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).