紧急求助

sky__sea

我现在写的一段内核代码，一运行就会当机，但是只是在某一个内核的suse上会这样，在大多数是没问题的，实在找不到解决办法，请各位高人帮忙看看，非常感谢！代码如下：
int test (struct file *pst_File)
{
        int res = -1;
        char *ac_Buf= NULL;
    char * pc_FdName = NULL;
    char file_path[PAGE_SIZE + 1] = {0};

    ac_Buf = kmalloc(PAGE_SIZE, GFP_ATOMIC);
    if (!ac_Buf)
         return -1;


    {
            pc_FdName = d_path(&(pst_File->f_path), ac_Buf, PAGE_SIZE);

        if (NULL != pc_FdName)
        {
                    memset(file_path, 0 , PAGE_SIZE);
                strncpy(file_path, pc_FdName, PAGE_SIZE);
                res = 0;
        }
        else
        {
                res = -1;
        }

    }
    kfree(ac_Buf);
    return res;

}

目的就是通过文件fd来获取到这个文件的全路径。

jeppeter

回复 1# sky__sea

你犯了一个严重问题，就是在内核栈上不能分配太大的内存，你一下子就分配了一个PAGE_SIZE的内容，而内核栈只有2 PAGE_SIZE，而且还有很多的task_struct的信息，这个实在太大了。所以，你把这个修改了，估计就好了，把这个在堆上分配就行了。
提醒一下，用论坛的代码写，要不然看起来真的很费劲。

nswcfd

可以检查一下这几个函数除了NULL之外是不是还会返回其它的error，例如-ENOMEM啥的

sky__sea

strncpy(file_path, pc_FdName, PAGE_SIZE);
我把这一行注释掉，就没有问题，加上就一定有问题，刚才我吧char file_path[PAGE_SIZE + 1] = {0};的大小定义改成10个字节，是没问题了，可是改成20个字节有不行，20个字节很小了吧，也不行，到底问题是在哪里，strncpy这一行怎么看都没有问题啊，另外，请教一下，根据fd获取文件的全路径，你们是怎么获取的呢？

sky__sea

我把file_path改成动态分配，真的不会当机了，可是为什么我静态分配20个字节就不行呢，堆栈应该还够用吧，我发现造成当机是因为非常频繁的调用这个函数，这个有什么关系？动态分配的话，是不是会影响效率，有大小限制吗？
再次谢谢楼上大神的帮助！

nswcfd

2楼说到点上了，是kernel stack大小的约束。

PS，strncpy的第三个参数一直是PAGE_SIZE么，还是有所调整（比如10或者20）?

nswcfd

可以统计一下当函数test被调用的时候，典型的sp的值分布（后13位即可），看看是不是离8k的边缘很近了。

action08

论坛有code标签的，我bash记录都直接贴的，看起来舒服感不一样

sky__sea

还是有问题，搞内核模块真痛苦，经常是这个环境不出问题，到另一个环境就崩溃，更郁闷的是没有任何系统日志，根本不知道哪里出的问题。我如果都改成kmalloc申请，也会崩溃，搞不清楚内核对内存有什么限制，申请的过于频繁是不是也会有问题？

sky__sea

我申请的也不大，也就1,200个字节。