下面的代码是 linux kernel的代码,请指出其中的错误来: struct student{ int id; char name[20]; struct list_head 1ist; }; static LIST_HEAD(student_list); static DEFINE MUTEX(student_mutex); static int add_student(struct student *stu) { struct student *new_stu = NULL; if(!stu) { printk(KERN_ ERR " Invalid argument\n"); return -EINVAL; } if (stu->id <= 0){ i printk(KERN_ ERR "Invalid id\n"); return -EINVAL; } if(!mutex_trylock(&student_mutex)){ printk(KERN_ INFO "Failed to lock mutex\n"); return-EBUSY; } new_stu = kmalloc(sizeof(struct student), GFP KERNEL); if(!new_ stu) { printk(KERN ERR "Failed to allocate memory\n"); return -ENOMEM; *new_stu= *stu; list_add_ tail(&new_stu->list, &student_list); printk(KERN_ INFO "Added student: id=%d, name-%s\n", stu->id, stu->name); mutex_unlock(&student_mutex); return 0; }
时间: 2024-01-16 13:03:09 浏览: 65
MapStruct实体转换及List转换的方法讲解
这段代码中有一些错误,具体如下:
1. 结构体 `student` 中的变量 `list` 应该改为 `list_head`,即 `struct list_head list;`。
2. 函数 `add_student` 中定义的指针变量 `new_stu` 没有进行初始化,应该将其初始化为 `NULL`。
3. 函数 `add_student` 中的 `if (!mutex_trylock(&student_mutex))` 应该改为 `if (mutex_trylock(&student_mutex))`,因为函数 `mutex_trylock` 返回值为 0 表示锁已经被占用,非 0 表示成功获取到锁。
4. 函数 `add_student` 中的 `new_stu = kmalloc(sizeof(struct student), GFP KERNEL);` 应该改为 `new_stu = kmalloc(sizeof(struct student), GFP_KERNEL);`,因为 `GFP_KERNEL` 是一个宏定义,需要在代码中使用。
5. 函数 `add_student` 中的 `*new_stu = *stu;` 应该放在 `if(!new_stu)` 的外面,否则会出现没有释放 `new_stu` 的情况。
6. 函数 `add_student` 中的 `list_add_ tail(&new_stu->list, &student_list);` 应该改为 `list_add_tail(&new_stu->list, &student_list);`,因为函数名写错了。
7. 函数 `add_student` 中的 `printk(KERN_ INFO "Added student: id=%d, name-%s\n", stu->id, stu->name);` 应该改为 `printk(KERN_INFO "Added student: id=%d, name-%s\n", new_stu->id, new_stu->name);`,因为要打印的是新添加的学生信息。
修改后的代码如下所示:
```
struct student {
int id;
char name[20];
struct list_head list;
};
static LIST_HEAD(student_list);
static DEFINE_MUTEX(student_mutex);
static int add_student(struct student *stu) {
struct student *new_stu = NULL;
if (!stu) {
printk(KERN_ERR "Invalid argument\n");
return -EINVAL;
}
if (stu->id <= 0) {
printk(KERN_ERR "Invalid id\n");
return -EINVAL;
}
if (!mutex_trylock(&student_mutex)) {
printk(KERN_INFO "Failed to lock mutex\n");
return -EBUSY;
}
new_stu = kmalloc(sizeof(struct student), GFP_KERNEL);
if (!new_stu) {
printk(KERN_ERR "Failed to allocate memory\n");
mutex_unlock(&student_mutex);
return -ENOMEM;
}
*new_stu = *stu;
list_add_tail(&new_stu->list, &student_list);
printk(KERN_INFO "Added student: id=%d, name-%s\n", new_stu->id, new_stu->name);
mutex_unlock(&student_mutex);
return 0;
}
```
阅读全文