[原创] 总结一下， 我们写module的时候， 常用的..

－－－－－－－－－－－－－－－－－－－－－－－－－－－－－－－－－－－－－－－－－－－－－－－－－
用信号量：

static DECLARE_MUTEX(char_sem);
static unsigned long char_inuse;

static int device_open(struct inode *inode, struct file *file)
{
    static int counter = 0;
    down_interruptible(&char_sem);

    if(char_inuse == 1) {
        up(&char_sem);
        return -EBUSY;
    }
    char_inuse = 1;

    up(&char_sem);
    
    ...........
}

static int device_release(struct inode *inode, struct file *file)
{
    char_inuse = 0;
    MOD_DEC_USE_COUNT;
    
    return 0;
}

－－－－－－－－－－－－－－－－－－－－－－－－－－－－－－－－－－－

用原子变量：
static atomic_t inuse_atomic = ATOMIC_INIT(-1);

/* 这个动作有可能并发，这里我又改成原子变量了
 */
static int device_open(struct inode *inode, struct file *file)
{
    
    if( ! atomic_inc_and_test(&inuse_atomic)) //原子加1 ， 并判断是否＝＝0 , -1 , 表示无人使用此设备

    {
        dbg("%d process is opening this device \n",inuse_atomic);//如果值是1 ， 说明刚好有一个进程在访问该设备

        dbg("open device failure \n");
        return -EBUSY;
    }
    //try_module_get(THIS_MODULE);

    MOD_INC_USE_COUNT;

    return SUCCESS;
}

static int device_release(struct inode *inode, struct file *file)
{

    atomic_set(&inuse_atomic,-1);//重新再把它置成-1

    MOD_DEC_USE_COUNT;
    
    return 0;
}

－－－－－－－－－－－－－－－－－－－－－－－－－－－－－－
用原子的bit操作

static int chardev_users = 0; //防止互斥， 这里用更严格的test_and_set_bit() 函数


static int device_open(struct inode *inode, struct file *file)
{
    static int counter = 0;
    //    if (Device_Open)

    //        return -EBUSY;

    //    Device_Open++;

    //    上面的办法太不可靠了。用原子操作或者bit操作才是最可靠的。

    if(test_and_set_bit(0,&chardev_users))
        return -EBUSY;

    //sprintf(msg, "I already told you %d times Hello world!\n", counter++);

    //memset(msg,'\0',sizeof(msg));

    msg_Ptr = msg;
    //try_module_get(THIS_MODULE);

    MOD_INC_USE_COUNT; //increase the flag to present the module is used

/* strategy :
 * you should here allocate resources , not in init_module() ,if(refer == 0)
 */

    return SUCCESS;
}

static int device_release(struct inode *inode, struct file *file)
{
    //    Device_Open--;        /* We're now ready for our next caller */


    //chardev_users =0; //直到现在才可以被别人open()

    //set_bit() is better by me

    if( !test_and_clear_bit(0,&chardev_users)) //bit0 should be 1

        dbg("It looks wrong ,lock may bad \n");

    MOD_DEC_USE_COUNT;