bugfix> c > 投稿

主にプロセスのリストと各プロセスで開かれているファイルのリストを維持することになっているいくつかのシステムコールを実装しています。ただし、コードのデバッグに苦労しており、何度も何度も再コンパイルする必要があります。

おそらく私の主な問題は次のとおりです。

  1. プロセスを保持するリストをいつどのように初期化できますか?現在、初期化する前にリストが空かどうかを確認していますが、機能しません。 head == head->prev->next も試しましたしかし、失敗しました。

  2. 関連する「比較」関数を考慮してlist_sortを呼び出す正しい方法は何ですか?エラーが発生しています

[ 1827.710802] BUG: unable to handle kernel paging request at fffff550

[ 1827.710808] IP: [] compare+0x8/0x20

[ 1827.710814] *pde = 00a32067 *pte = 00000000

これがコードです。特に init をレビューしてくれて本当にありがとう他のユーザーが依存する正しい動作を行うシステムコール。

#include <linux/kernel.h>
#include <linux/module.h>
#include <linux/sched.h>
#include <linux/pid.h>
#include <linux/list.h>
#include <linux/fs.h>
#include <linux/proc_fs.h>
#include <linux/fdtable.h>
#include <linux/list_sort.h>
#include <linux/slab.h>

struct fileDescriptor {
    int fd;
    struct list_head list;
};
struct processInfo {
    struct task_struct task;
    int len_files;
    struct fileDescriptor listHead;
    struct list_head list;
};
struct processInfo listHead;
int compare(void* priv, struct list_head *a, struct list_head *b)
{
    struct processInfo *p1 = container_of(a, struct processInfo, list);
    struct processInfo *p2 = container_of(b, struct processInfo, list);
    if (p1->task.pid > p2->task.pid) return -1;
    else return 1;
}

asmlinkage long sys_init_process_list(pid_t p)
{   
    struct pid* pid;
    struct task_struct *task;
    struct files_struct *processFiles;
    struct fdtable *filesTable;
    struct processInfo *newProcess;
    if(list_empty(&listHead.list))
    {
        printk("____list_empty_____\n");
        INIT_LIST_HEAD(&listHead.list);
    }
    else
        printk("____list_not_empty_____\n");

    pid = find_get_pid(p);
    if (pid == NULL)
    {
        sys_init_process_list(1);
        return 1;
    }
    else 
    {
        struct list_head *list;
        struct task_struct *childTask;
        struct fileDescriptor *newfd;
        int i = 0;
        task = pid_task(pid, PIDTYPE_PID);
        newProcess = (struct processInfo*) kmalloc(sizeof(*newProcess), GFP_KERNEL);
        //copy task structure
        newProcess->task = *task;
        //adding file descriptors
        INIT_LIST_HEAD(&(newProcess->listHead).list);
        processFiles = task->files;
        filesTable = files_fdtable(processFiles);
        while (filesTable->fd[i] != NULL)
        {
            newfd = (struct fileDescriptor*) kmalloc(sizeof(*newfd), GFP_KERNEL);
            newfd->fd = i;
            INIT_LIST_HEAD(&newfd->list);
            list_add_tail(&(newfd->list), &(newProcess->listHead).list);
            i++;
        }
        newProcess->len_files = i;
        INIT_LIST_HEAD(&newProcess->list);
        // add the new node to mylist
        list_add_tail(&(newProcess->list), &(listHead.list));        
        //adding childern
        list_for_each(list, &task->children)
        {
            childTask = list_entry(list, struct task_struct, sibling);
            sys_init_process_list(childTask->pid);
        }
        return 0;
    }
}
//////////////////////////////////////////
asmlinkage long sys_sort_process_list(void)
{
    if(list_empty(&listHead.list))
    {
        printk("empty list\n");
        return 1;
    }
    list_sort(NULL, &listHead.list, compare);
    return 0;
}
////////////////////////////////////////
asmlinkage long sys_print_process_list(void)
{
    struct processInfo *aProcess;
    if (sys_sort_process_list())
    {
        printk("empty list\n");
        return 1;
    }
    list_for_each_entry(aProcess, &listHead.list, list)
    {
        printk("%d, %d\n", (aProcess->task).pid, aProcess->len_files);
    }
    return 0;
}
//////////////////////////////////////////////////
asmlinkage long sys_clear_process_list(void)
{   
    struct processInfo *aProcess, *tmp;
    if(list_empty(&listHead.list))
    {
        printk("empty list\n");
        return 1;
    }
    printk("deleting the list");
    list_for_each_entry_safe(aProcess, tmp, &listHead.list, list)
    {
        printk("freeing %d", (aProcess->task).pid);
        list_del(&aProcess->list);
        kfree(aProcess);
    }
    return 0;
}

編集:コンパイル時に行われる「静的定義」の問題をなんとか解決することができました。つまり、

struct processInfo listHead;

struct processInfo listHead =
{
    .list = LIST_HEAD_INIT(listHead.list)
}

そして今、すべてがうまく機能しているようです。しかし、これが最善の方法であるかどうかはわかりません。そのため、他の可能な解決策が追加される可能性があるため、質問を開いたままにします。