* [PATCH] Fix logical and operator at interrupt handler thread
@ 2012-02-01 5:19 Geunsik Lim
2012-02-01 5:41 ` Eric Dumazet
0 siblings, 1 reply; 3+ messages in thread
From: Geunsik Lim @ 2012-02-01 5:19 UTC (permalink / raw)
To: Thomas Gleixner; +Cc: linux-kernel
From: Geunsik Lim <geunsik.lim@samsung.com>
'&' is a bitwise operation. (e.g: a & b )
Use '&&' and-operator instead of '&' bitwise operator.
Signed-off-by: Geunsik Lim <geunsik.lim@samsung.com>
---
kernel/irq/manage.c | 2 +-
1 files changed, 1 insertions(+), 1 deletions(-)
diff --git a/kernel/irq/manage.c b/kernel/irq/manage.c
index a9a9dbe..c0730ad 100644
--- a/kernel/irq/manage.c
+++ b/kernel/irq/manage.c
@@ -773,7 +773,7 @@ static int irq_thread(void *data)
struct irqaction *action);
int wake;
- if (force_irqthreads & test_bit(IRQTF_FORCED_THREAD,
+ if (force_irqthreads && test_bit(IRQTF_FORCED_THREAD,
&action->thread_flags))
handler_fn = irq_forced_thread_fn;
else
--
1.7.8.1
^ permalink raw reply related [flat|nested] 3+ messages in thread
* Re: [PATCH] Fix logical and operator at interrupt handler thread
2012-02-01 5:19 [PATCH] Fix logical and operator at interrupt handler thread Geunsik Lim
@ 2012-02-01 5:41 ` Eric Dumazet
2012-02-01 5:53 ` Eric Dumazet
0 siblings, 1 reply; 3+ messages in thread
From: Eric Dumazet @ 2012-02-01 5:41 UTC (permalink / raw)
To: Geunsik Lim; +Cc: Thomas Gleixner, linux-kernel
Le mercredi 01 février 2012 à 14:19 +0900, Geunsik Lim a écrit :
> From: Geunsik Lim <geunsik.lim@samsung.com>
>
> '&' is a bitwise operation. (e.g: a & b )
> Use '&&' and-operator instead of '&' bitwise operator.
>
> Signed-off-by: Geunsik Lim <geunsik.lim@samsung.com>
> ---
> kernel/irq/manage.c | 2 +-
> 1 files changed, 1 insertions(+), 1 deletions(-)
>
> diff --git a/kernel/irq/manage.c b/kernel/irq/manage.c
> index a9a9dbe..c0730ad 100644
> --- a/kernel/irq/manage.c
> +++ b/kernel/irq/manage.c
> @@ -773,7 +773,7 @@ static int irq_thread(void *data)
> struct irqaction *action);
> int wake;
>
> - if (force_irqthreads & test_bit(IRQTF_FORCED_THREAD,
> + if (force_irqthreads && test_bit(IRQTF_FORCED_THREAD,
> &action->thread_flags))
> handler_fn = irq_forced_thread_fn;
> else
This was intentional, to have a single conditional move :
ffffffff810b0868: 0f b6 15 19 f5 bd 00 movzbl 0xbdf519(%rip),%edx # ffffffff81c8fd88 <force_irqthreads>
ffffffff810b086f: 49 89 c5 mov %rax,%r13
ffffffff810b0872: 48 8b 43 40 mov 0x40(%rbx),%rax
ffffffff810b0876: be 01 00 00 00 mov $0x1,%esi
ffffffff810b087b: 48 c1 e8 04 shr $0x4,%rax
ffffffff810b087f: 85 c2 test %eax,%edx
ffffffff810b0881: 48 c7 c0 80 0a 0b 81 mov $0xffffffff810b0a80,%rax
ffffffff810b0888: 48 c7 c2 40 dc 80 81 mov $0xffffffff8180dc40,%rdx
ffffffff810b088f: 4c 0f 44 f8 cmove %rax,%r15
I guess you could instead use jump_label here to shorten the code.
^ permalink raw reply [flat|nested] 3+ messages in thread
* Re: [PATCH] Fix logical and operator at interrupt handler thread
2012-02-01 5:41 ` Eric Dumazet
@ 2012-02-01 5:53 ` Eric Dumazet
0 siblings, 0 replies; 3+ messages in thread
From: Eric Dumazet @ 2012-02-01 5:53 UTC (permalink / raw)
To: Geunsik Lim; +Cc: Thomas Gleixner, linux-kernel
Le mercredi 01 février 2012 à 06:41 +0100, Eric Dumazet a écrit :
> I guess you could instead use jump_label here to shorten the code.
(since its not hot path anyway, its probably not worth it)
^ permalink raw reply [flat|nested] 3+ messages in thread
end of thread, other threads:[~2012-02-01 5:53 UTC | newest]
Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2012-02-01 5:19 [PATCH] Fix logical and operator at interrupt handler thread Geunsik Lim
2012-02-01 5:41 ` Eric Dumazet
2012-02-01 5:53 ` Eric Dumazet
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).