función cat llamando a read() infinitas veces

linux-kernel linux-device-driver

Question

Estoy trabajando en un controlador de dispositivo de caracteres simple. He implementado funciones de lectura y escritura en el módulo, el problema es que cuando trato de leer el archivo del dispositivo usando cat /dev/devicefile , entra en un bucle infinito, es decir, lee los mismos datos repetidamente. ¿Alguien puede sugerirme alguna solución a este problema? A continuación se muestra mi código de conductor.

#include<linux/module.h>
#include<linux/fs.h>
#include<linux/string.h>
#include<asm/uaccess.h>
#include<linux/init.h>
MODULE_LICENSE("GPL");
MODULE_DESCRIPTION("character device driver");
MODULE_AUTHOR("Srinivas");

static char msg[100]={0};

static int t;

static int dev_open(struct inode *, struct file *);
static int dev_rls(struct inode *, struct file *);
static ssize_t dev_read(struct file *, char *,size_t, loff_t *);
static ssize_t dev_write(struct file *, const char *, size_t,loff_t *);

static struct file_operations fops =
{
    .read = dev_read,
    .open = dev_open,
    .write = dev_write,
    .release = dev_rls,
};
static int himodule( void )
{
    t = 0;
    t = register_chrdev(0, "chardevdriver", &fops);
    if (t < 0)
        printk(KERN_ALERT"device registration failed/n");
    else
        printk(KERN_ALERT"device registered successfully/n");

    printk(KERN_ALERT"major number is %d", t);
    return 0;
}
static void byemodule(void)
{
    unregister_chrdev(t, "chardevdriver");
    printk(KERN_ALERT"successfully unregistered/n");
}

static int dev_open(struct inode *inod, struct file *fil)
{
    printk(KERN_ALERT"inside the dev open");
    return 0;
}
static ssize_t dev_read(struct file *filp, char *buff, size_t len, loff_t *off)
{
    short count = 0;
    while (msg[count] != 0) {
        put_user(msg[count], buff++);
        count++;
    }
    return count;
}

static ssize_t dev_write(struct file *filp, const char *buff, size_t len, loff_t *off)
{
    short count = 0;
    printk(KERN_ALERT"inside write/n");

    memset(msg,0,100);

    printk(KERN_ALERT" size of len is %zd",len);

    while (len > 0) {
        msg[count] = buff[count];
        len--;
        count++;
    }
    return count;
}

static int dev_rls(struct inode *inod,struct file *fil)
{
    printk(KERN_ALERT"device closed/n");
    return 0;
}
module_init(himodule);
module_exit(byemodule);

Answer 1

Este problema se puede resolver configurando correctamente *off (cuarto parámetro de my_read() ).

Debe devolver el recuento por primera vez y cero a partir de la segunda vez.

if(*off == 0) {
    while (msg[count] != 0) {
        put_user(msg[count], buff++);
        count++;
        (*off)++;
    }
    return count;
}
else
return 0;

Answer 2

No respeta el tamaño del búfer pasado a la función dev_read , por lo que puede invocar un comportamiento indefinido en cat . Prueba esto:

static ssize_t dev_read( struct file *filp, char *buff, size_t len, loff_t  *off )
{
    size_t count = 0;
    printk( KERN_ALERT"inside read %d/n", *off );
    while( msg[count] != 0 && count < len )
    {
        put_user( msg[count], buff++ );
        count++;
    }
    return count;
}

Answer 3

.read función .read también debe procesar correctamente sus argumentos len y off . La forma más sencilla de implementar la lectura del archivo con memoria intermedia es usar el ayudante simple_read_from_buffer :

static ssize_t dev_read(struct file *filp, char *buff, size_t len, loff_t *off)
{
    return simple_read_from_buffer(buff, len, off, msg, 100);
}

Puede inspeccionar el código de ese ayudante (definido en fs/libfs.c ) con fines educativos.

Por cierto, para su método .write podría usar simple_write_to_buffer helper.