Jia-Ju Bai <baijiaju1...@gmail.com> wrote:

> +     mutex_lock(&user->file_lock);
>       sbefifo_release_command(user);
>       free_page((unsigned long)user->cmd_page);
> +     mutex_unlock(&user->file_lock);

It shouldn't be necessary to do the free_page() call inside the locked
section.

David

Reply via email to