Ядро Panic после изменений в sys_close - PullRequest
Новая
       77

Ядро Panic после изменений в sys_close

0 голосов
/ 12 ноября 2009

Я прохожу курс по операционным системам, и мы работаем над Linux Red Hat 8.0. В рамках задания мне пришлось изменить sys close и sys open.Изменения в sys close прошли без инцидентов, но когда я ввожу изменения в sys close, внезапно ОС обнаруживает ошибку во время загрузки, заявляя, что не может смонтировать root fs, и вызывает панику.EIP, как сообщается, находится вблизи sys, когда это произойдет.

Вот изменения, которые я сделал (ищите комментарий «Дополнения HW1»): В fs / open.c: 

asmlinkage long sys_open(const char * filename, int flags, int mode)
{
    char * tmp;
    int fd, error;
    event_t* new_event;

#if BITS_PER_LONG != 32
    flags |= O_LARGEFILE;
#endif
    tmp = getname(filename);
    fd = PTR_ERR(tmp);
    if (!IS_ERR(tmp)) {
        fd = get_unused_fd();
        if (fd >= 0) {
            struct file *f = filp_open(tmp, flags, mode);
            error = PTR_ERR(f);
            if (IS_ERR(f))
                goto out_error;
            fd_install(fd, f);
        }
        /* HW1 additions */
        if (current->record_flag==1){
            new_event=(event_t*)kmalloc(sizeof(event_t), GFP_KERNEL);
            if (!new_event){
                new_event->type=Open;
                strcpy(new_event->filename, tmp);
                file_queue_add(*new_event, current->queue);
            }
        }
        /* End HW1 additions */
out:
        putname(tmp);
    }
    return fd;

out_error:
    put_unused_fd(fd);
    fd = error;
    goto out;
}

asmlinkage long sys_close(unsigned int fd)
{
    struct file * filp;
    struct files_struct *files = current->files;
    event_t* new_event;
    char* tmp = files->fd[fd]->f_dentry->d_name.name;   

    write_lock(&files->file_lock);
    if (fd >= files->max_fds)
        goto out_unlock;
    filp = files->fd[fd];
    if (!filp)
        goto out_unlock;
    files->fd[fd] = NULL;
    FD_CLR(fd, files->close_on_exec);
    __put_unused_fd(files, fd);
    write_unlock(&files->file_lock);
    /* HW1 additions */    
    if(current->record_flag == 1){
        new_event=(event_t*)kmalloc(sizeof(event_t), GFP_KERNEL);
        if (!new_event){
            new_event->type=Close;
            strcpy(new_event->filename, tmp);
            file_queue_add(*new_event, current->queue);
        }
    }
    /* End HW1 additions */
    return filp_close(filp, files);

out_unlock:
    write_unlock(&files->file_lock);
    return -EBADF;
}

Структура task_struct, определенная в schedule.h, была изменена вend to include: 

unsigned int record_flag; /* when zero: do not record. when one: record. */
file_queue* queue;

И очередь файлов, а также событие t определены в отдельном файле следующим образом: 

typedef enum {Open, Close} EventType;

typedef struct event_t{
    EventType type;
    char filename[256];
}event_t;

typedef struct file_quque_t{
    event_t queue[101];
    int head, tail; 
}file_queue;

Добавление очереди файлов работает следующим образом: 

void file_queue_add(event_t event, file_queue* queue){ 
    queue->queue[queue->head]=event;
    queue->head = (queue->head+1) % 101;
    if (queue->head==queue->tail){
        queue->tail=(queue->tail+1) % 101;
    }
}

Ответы [ 2 ]

2 голосов
/ 13 ноября 2009 
if (!new_event) {
    new_event->type = …

Это эквивалентно if (new_event == NULL). Я думаю, что вы имеете в виду if (new_event != NULL), который обычно пишут в ядре как if (new_event).

0 голосов
/ 22 ноября 2009

Можете ли вы опубликовать стека дамп ошибки. Я не вижу места, где структуре queue_info выделяется память. Еще одна вещь: вы не можете быть уверены, что процесс record_flag будет всегда нулевым, если не назначен в ядре, потому что ядро ​​- это долго выполняющаяся программа, а память содержит мусор.

Также можно проверить точное местоположение в функции, посмотрев на трассировку стека.

...