下面的代码是 linux kernel的代码,请指出其中的错误来: 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){ 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 08:03:10 浏览: 52
MapStruct实体转换及List转换的方法讲解
代码中有如下错误:
1. 第22行多了一个 i 字符,应该删掉。
2. 第38行中,kmalloc 的 GFP KERNEL 参数应该改为 GFP_KERNEL。
3. 第40行中,new_stu 指针后面的 * 应该改为 .,表示给指针所指的结构体成员赋值。
4. 第42行中,list_add_tail 函数的第一个参数应该是 &new_stu->list,而不是 new_stu。
5. 第43行中,打印字符串中间的“-”应该改为“=”。
修改后的代码如下:
```
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", stu->id, stu->name);
mutex_unlock(&student_mutex);
return 0;
}
```
阅读全文