[lkml]   [2003]   [Aug]   [1]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
Messages in this thread
    SubjectRe: volatile variable
    On Fri, 1 Aug 2003, Dinesh  Gandhewar wrote:

    > Hello,
    > If a system call is having following code.
    > add current process into wait quque ;
    > while (1)
    > { set task state INTERRUPTIBLE ;
    > if (a > 0)
    > break ;
    > schedule() ;
    > }
    > set task state RUNNING ;
    > remove current from wait queue ;
    > If an interrupt service is having following code
    > set a = 512 ;
    > 'a' is a global variable shared in ISR and system call
    > Do I need to define a as 'volatile int a ;' ? Why?
    > Thanks & Regards,
    > Dinesh

    First, there are already procedures available to do just
    what you seem to want to do, interruptible_sleep_on() and
    interruptible_sleep_on_timeout(). These take care of the
    ugly details that can trip up compilers.

    In any event in your loop, variable 'a', has already been
    read by the code the compiler generates. There is nothing
    else in the loop that touches that variable. Therefore
    the compiler is free to (correctly) assume that whatever
    it was when it was first read is what it will continue to
    be. The compiler will therefore optimise it to be a single
    read and compare. So, the loop will continue forever if
    'a' started as zero because the compiler correctly knows
    that it cannot possibly change in the only execution
    path that it knows about.

    To tell the compiler that there are other possible execution
    paths in which the variable could be modified, you need to
    declare the variable as "volatile". The variable's storage
    hasn't changed. The size of the variable hasn't changed.
    All you have done is told the compiler to read that variable
    every time because it could have changed.

    Dick Johnson
    Penguin : Linux version 2.4.20 on an i686 machine (797.90 BogoMips).
    Note 96.31% of all statistics are fiction.

    To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
    the body of a message to
    More majordomo info at
    Please read the FAQ at

     \ /
      Last update: 2005-03-22 13:47    [W:0.060 / U:143.516 seconds]
    ©2003-2016 Jasper Spaans. hosted at Digital OceanAdvertise on this site