A что не нравится?

ИМХО все правильно - процесс у тебя спит и просыпается только после получения прерывания с клавиатуры, а wake_up происходит после того, как процесс уже проснулся. Если ты хочешь будить его из драйвера , то wake_up_interruptible должен вызываться из другой процедуры - скажем из обработчика poll, или из таймера, или из обработчика прерывания(где ты предварительно проверяешь поступили данные или нет)

хм. но процесс, заснувший на vsp_read я пытаюсь будить из vsp_write, тоесть из другой процедуры. Я наверно туплю, но не понял в чем фишка...

WBR, Dvorkin

Чудес на свете не бывает. wake_up -- если его вызвали -- должен
изменить состояние процесса.
Глядя на выдержку из кода, я заметил, что Вы не любите рисовать
"лишних" фигурных скобок. То, что бессмысленный printk меняет логику
исполнения наводит на мысли, верно? Убедитесь, пожалуйста, что
wake_up действительно вызывается.

Strange...

Да нет - это я туплю :)...
Действительно все правильно, но по всей видимости вся загвоздка в том коде, что у тебя поскипан. По-моему ситуация может быть такой - процесс просыпается, но по каким-то причинам буфер еще не обновился - он пытается прочитать его снова, видит длину <1 и снова засыпает, после чего буфер уже успешно обновляется. Понятно почему printk в этой ситуации помогает - буфер у тебя успевает обновиться за время выполнения printk

а че ты собсно не разруливаешь доступ к этому dev у тебя
ведь не атомарные операции над ним?

помоему было бы надежнее делать типа

read:
/*
 * TODO:
 *   o поднять семафор доступа к buff
 *   o проверить длину       | critical section
 *   o считать из бефера     | critical section
 *   o опустить семафор
 */

write:
/*
 * TODO:
 *   o поднять семафор доступа к buff
 *   o записать в буффер      | critical section
 *   o изменить длину буфера  | critical section
 *   o опустить семафор
 */

я не знаю специфику линукса, но обычно так делается ..

Вот чувак правильно сказал, а кто будет лочить dev перед тем как получить сайз из него? Пушкин?

лочить - не лочить баг не исчезает.

WBR, Dvorkin

Да нету никакого бага,,,

...разбирайся в том что у тебя поскипано. Вот модуль

#ifndef MODULE #define MODULE #endif #ifndef __KERNEL__ #define __KERNEL__ #endif

#ifndef EXPORT_SYMTAB # define EXPORT_SYMTAB #endif

#include <linux/module.h> #include <linux/kernel.h> #include <asm/system.h> #include <linux/fs.h> #include <linux/tqueue.h> #include <linux/config.h> #include <linux/types.h> #include <linux/poll.h> #include <linux/ioport.h> #include <asm/io.h> #include <asm/uaccess.h> #include <asm/semaphore.h>

int chj_major = 251;

MODULE_PARM(chj_major, "i");

char hold[1024]; int buff_len; wait_queue_head_t wqr, wqw; struct semaphore lock;

static ssize_t vsp_read( struct file *_file, char *_buff, size_t _len, loff_t *_off) { int i; if ( buff_len < 1) interruptible_sleep_on( &(wqr)); wake_up_interruptible( &wqw); down_interruptible(&lock); for (i=0;i<buff_len;i++){ hold[i]+=1; }; copy_to_user(_buff,hold,buff_len); buff_len=0; up(&lock); return i; }

static ssize_t vsp_write( struct file *_file, const char *_buff, size_t _len, loff_t *_off) { if ( buff_len >= 1024) interruptible_sleep_on( &(wqw)); down_interruptible(&lock); copy_from_user(hold,_buff,_len); buff_len+=_len; up(&lock); wake_up_interruptible( &wqr); return _len; } struct file_operations _fops={ read:vsp_read, write:vsp_write, } ;

int init_module() { int i,result = register_chrdev(chj_major,"Test",&_fops); void * z; if (result <0){ printk("<3>Can not gt major %d \n",chj_major); return result; } sema_init(&lock,1); init_waitqueue_head (&wqr); init_waitqueue_head (&wqw); return 0; }

void cleanup_module(void) { unregister_chrdev(chj_major,"Test"); }

mknod /dev/testscull c 251 1 cat /dev/testscull echo "1111" > /dev/testscull

у меня работает не лочится (ядро 2.4.18 стандартное из Слаквари)

Да нету никакого бага,,,

#ifndef MODULE
#define MODULE
#endif
#ifndef __KERNEL__
#define __KERNEL__
#endif

#ifndef EXPORT_SYMTAB
# define EXPORT_SYMTAB
#endif


#include <linux/module.h>
#include <linux/kernel.h>
#include <asm/system.h>
#include <linux/fs.h>
#include <linux/tqueue.h>
#include <linux/config.h>
#include <linux/types.h>
#include <linux/poll.h>
#include <linux/ioport.h>
#include <asm/io.h>
#include <asm/uaccess.h>
#include <asm/semaphore.h>

int chj_major = 251;

MODULE_PARM(chj_major, "i");


char hold[1024];
int buff_len;
wait_queue_head_t wqr, wqw;
struct semaphore lock;

static ssize_t vsp_read( struct file *_file, char *_buff, size_t _len, loff_t *_off) {
int i;
if ( buff_len < 1) interruptible_sleep_on( &(wqr));
wake_up_interruptible( &wqw);
down_interruptible(&lock);
for (i=0;i<buff_len;i++){
hold[i]+=1;
};
copy_to_user(_buff,hold,buff_len);
buff_len=0;
up(&lock);
return i;
}

static ssize_t vsp_write( struct file *_file, const char *_buff, size_t _len, loff_t *_off) {
if ( buff_len >= 1024)
interruptible_sleep_on( &(wqw));
down_interruptible(&lock);
copy_from_user(hold,_buff,_len);
buff_len+=_len;
up(&lock);
wake_up_interruptible( &wqr);
return _len;
}
struct file_operations _fops={
read:vsp_read,
write:vsp_write,
} ;


int init_module()
{
int i,result = register_chrdev(chj_major,"Test",&_fops);
void * z;
if (result <0){
printk("<3>Can not gt major %d \n",chj_major);
return result;
}
sema_init(&lock,1);
init_waitqueue_head (&wqr);
init_waitqueue_head (&wqw);
return 0;
}

void cleanup_module(void)
{
unregister_chrdev(chj_major,"Test");
}

By now looks sane ;-)