2002-08-23 11:54:15

by sanket rathi

[permalink] [raw]
Subject: interrupt handler

hi,
Can i use spin lock in the interrupt handler for a singlre processor machine. because books says u can not use locks but spin lock is some thing diffrent

thanks in advance

--Sanket
---------
--
Get your free email from http://www.linuxmail.org


Powered by Outblaze


2002-08-23 12:03:07

by Gabor Kerenyi

[permalink] [raw]
Subject: Re: interrupt handler

8/23/2002 8:58:20 PM, "sanket rathi" <[email protected]> wrote:

>hi,
>Can i use spin lock in the interrupt handler for a singlre processor machine. because books says u can not use locks
>but spin lock is some thing diffrent

Yes you can. Spinlocks will be compiled in if you choose SMP even on single processor machine.
But look at the documentation in the kernel source.
I think "lock" in your message is a semaphore in real life.

Spinlocks never sleep - they don't perform task switch and you can use them anywhere, while semaphores
can sleep and you can use them only in user context (not in interrupt or bottom half)

Gabor


2002-08-23 12:11:32

by Richard B. Johnson

[permalink] [raw]
Subject: Re: interrupt handler

On Fri, 23 Aug 2002, sanket rathi wrote:

> hi,
> Can i use spin lock in the interrupt handler for a singlre processor
> machine. because books says u can not use locks but spin lock is some
> thing diffrent
>
> thanks in advance
>
> --Sanket
> ---------

Interrupts default to OFF within an interrupt handler. Given this,
why would you use a spin-lock within the ISR on a single-processor
machine?

To directly answer your question, YES, you can use a spin-lock
within an ISR even though it won't do anything except add code
on a single processor machine.

On multiple CPU machines, you can use the form of spin-lock that
does not save/restore interrupts within the ISR, and use the
save/restore versions, with the same lock variable, outside the
ISR.

Cheers,
Dick Johnson
Penguin : Linux version 2.4.18 on an i686 machine (797.90 BogoMips).
The US military has given us many words, FUBAR, SNAFU, now ENRON.
Yes, top management were graduates of West Point and Annapolis.

2002-08-23 12:50:52

by Gabor Kerenyi

[permalink] [raw]
Subject: Re: interrupt handler

8/23/2002 9:17:07 PM, "Richard B. Johnson" <[email protected]> wrote:

>On Fri, 23 Aug 2002, sanket rathi wrote:
>
>> hi,
>> Can i use spin lock in the interrupt handler for a singlre processor
>> machine. because books says u can not use locks but spin lock is some
>> thing diffrent
>>
>> thanks in advance
>>
>> --Sanket
>> ---------
>
>Interrupts default to OFF within an interrupt handler. Given this,
>why would you use a spin-lock within the ISR on a single-processor
>machine?

Because he would like to write a code that can be run on a computer
with more than one CPU.

Anyway, do anybody know what kind of advantages/disadvantages I can get
if I don't disable interrupts at all in my driver? Even if I have to use a circular
buffer or anything else? Is it worth trying to find such a solution or is it
a wasted time?

Gabor


2002-08-23 13:03:57

by Richard B. Johnson

[permalink] [raw]
Subject: Re: interrupt handler

On Fri, 23 Aug 2002, Kerenyi Gabor wrote:

> 8/23/2002 9:17:07 PM, "Richard B. Johnson" <[email protected]> wrote:
>
> >On Fri, 23 Aug 2002, sanket rathi wrote:
> >
> >> hi,
> >> Can i use spin lock in the interrupt handler for a singlre processor
> >> machine. because books says u can not use locks but spin lock is some
> >> thing diffrent
> >>
> >> thanks in advance
> >>
> >> --Sanket
> >> ---------
> >
> >Interrupts default to OFF within an interrupt handler. Given this,
> >why would you use a spin-lock within the ISR on a single-processor
> >machine?
>
> Because he would like to write a code that can be run on a computer
> with more than one CPU.
>
> Anyway, do anybody know what kind of advantages/disadvantages I can get
> if I don't disable interrupts at all in my driver? Even if I have to
> use a circular
> buffer or anything else? Is it worth trying to find such a solution or is it
> a wasted time?
>
> Gabor

If your ISR manipulates any data, which is quite likely, then
your driver code, that is outside the ISR, must be written
with the knowledge that an interrupt can happen at any time.

There are probably certain critical regions of code that must
be protected against modification from the ISR code. You need
to protect those critical regions with spin-locks.

Spin-locks have very little code. If there is no contention,
they do not affect performance in any measurable way. If there
is contention, they simply delay execution of the ISR to a time
where code is executing in a non-critical section. This delay
is necessary so, even though it does affect performance, the
system would not work without it.


Cheers,
Dick Johnson
Penguin : Linux version 2.4.18 on an i686 machine (797.90 BogoMips).
The US military has given us many words, FUBAR, SNAFU, now ENRON.
Yes, top management were graduates of West Point and Annapolis.

2002-08-23 13:28:07

by Gabor Kerenyi

[permalink] [raw]
Subject: Re: interrupt handler

8/23/2002 10:07:54 PM, "Richard B. Johnson" <[email protected]> wrote:

>On Fri, 23 Aug 2002, Kerenyi Gabor wrote:
>> Anyway, do anybody know what kind of advantages/disadvantages I can get
>> if I don't disable interrupts at all in my driver? Even if I have to
>> use a circular
>> buffer or anything else? Is it worth trying to find such a solution or is it
>> a wasted time?
>>
>> Gabor
>
>If your ISR manipulates any data, which is quite likely, then
>your driver code, that is outside the ISR, must be written
>with the knowledge that an interrupt can happen at any time.

Well I wrote it in this way of course.

>There are probably certain critical regions of code that must
>be protected against modification from the ISR code. You need
>to protect those critical regions with spin-locks.

I know. But there are some technics that can be used to workaorund
the irq disabling thing.

>Spin-locks have very little code. If there is no contention,
>they do not affect performance in any measurable way. If there
>is contention, they simply delay execution of the ISR to a time
>where code is executing in a non-critical section. This delay
>is necessary so, even though it does affect performance, the
>system would not work without it.

I talked about irq disabling, not spinlocks. With or without spinlocks.
When you need synch between a bottom half(or user context) and irq
handler on single processor machine you can't use just spinlocks. you have to
disable the irq. so the question is about irq disabling to modify data
in a mutual way.
I solved it without disabling irq using a circular buffer (very simple one)
I think with this solution there is a better response time for the hardware
items. (they can get immediate response from the OS)

Gabor


2002-08-23 16:13:29

by Robert Love

[permalink] [raw]
Subject: Re: interrupt handler

On Fri, 2002-08-23 at 08:17, Richard B. Johnson wrote:

> Interrupts default to OFF within an interrupt handler. Given this,
> why would you use a spin-lock within the ISR on a single-processor
> machine?

Only the current interrupt handler is disabled... interrupts are
normally ON.

Robert Love

2002-08-23 16:40:35

by Richard B. Johnson

[permalink] [raw]
Subject: Re: interrupt handler

On 23 Aug 2002, Robert Love wrote:

> On Fri, 2002-08-23 at 08:17, Richard B. Johnson wrote:
>
> > Interrupts default to OFF within an interrupt handler. Given this,
> > why would you use a spin-lock within the ISR on a single-processor
> > machine?
>
> Only the current interrupt handler is disabled... interrupts are
> normally ON.
>
> Robert Love

No. Check out irq.c, line 446. The interrupts are turned back on
only if the flag did not have SA_INTERRUPT set. Certainly most
requests for interrupt services within drivers have SA_INTERRUPT
set.

This is linux-2.4.18 or 2.4.19. If the current code, 2.5+ enables
by default, it's broken and should be fixed.


Cheers,
Dick Johnson
Penguin : Linux version 2.4.18 on an i686 machine (797.90 BogoMips).
The US military has given us many words, FUBAR, SNAFU, now ENRON.
Yes, top management were graduates of West Point and Annapolis.

2002-08-23 16:48:12

by Robert Love

[permalink] [raw]
Subject: Re: interrupt handler

On Fri, 2002-08-23 at 12:45, Richard B. Johnson wrote:

> On 23 Aug 2002, Robert Love wrote:
> > Only the current interrupt handler is disabled... interrupts are
> > normally ON.
>
> No. Check out irq.c, line 446. The interrupts are turned back on
> only if the flag did not have SA_INTERRUPT set. Certainly most
> requests for interrupt services within drivers have SA_INTERRUPT
> set.

Sigh... SA_INTERRUPT is used only for fast interrupts. Certainly most
drivers do not have it (and most that do are probably from the way old
days when we went through great pains to distinguish between fast and
slow interrupt handlers).

Today, very few things should run with all interrupts disabled. That is
just dumb. In fact, on this system, it seems only the timer interrupt
sets SA_INTERRUPT...

Robert Love

2002-08-23 20:41:27

by George Anzinger

[permalink] [raw]
Subject: Re: interrupt handler

Robert Love wrote:
>
> On Fri, 2002-08-23 at 12:45, Richard B. Johnson wrote:
>
> > On 23 Aug 2002, Robert Love wrote:
> > > Only the current interrupt handler is disabled... interrupts are
> > > normally ON.
> >
> > No. Check out irq.c, line 446. The interrupts are turned back on
> > only if the flag did not have SA_INTERRUPT set. Certainly most
> > requests for interrupt services within drivers have SA_INTERRUPT
> > set.
>
> Sigh... SA_INTERRUPT is used only for fast interrupts. Certainly most
> drivers do not have it (and most that do are probably from the way old
> days when we went through great pains to distinguish between fast and
> slow interrupt handlers).
>
> Today, very few things should run with all interrupts disabled. That is
> just dumb. In fact, on this system, it seems only the timer interrupt
> sets SA_INTERRUPT...
>
And THAT makes sense as most of the timer interrupt is
processed holding the write_lock() on xtime which would need
to be an irq lock otherwise. If they were turned on the
system would have an additional interrupts on/off overhead.

> Robert Love
>
> -
> To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
> the body of a message to [email protected]
> More majordomo info at http://vger.kernel.org/majordomo-info.html
> Please read the FAQ at http://www.tux.org/lkml/

--
George Anzinger [email protected]
High-res-timers:
http://sourceforge.net/projects/high-res-timers/
Preemption patch:
http://www.kernel.org/pub/linux/kernel/people/rml