Question

我遇到了将指针传递给kfifo的问题，我似乎无法解决这个问题。

一点背景;我已经接管了使用FPGA的Gumstix接口的内核驱动程序开发。这种情况下的FPGA基本上采用多个串行流，对数据进行一些处理，然后通过GPIO将其提供给Gumstix。由于这实际上是一个非常复杂的项目，我将不再详述 - 因为它将变成一个相当长的帖子，大部分是无关的信息。我为各个串行接口工作原型驱动程序 - 这个驱动程序就是所有这些组合。

在这个驱动程序的第一个/主文件（gpio.c）中，一些kfifo被初始化并存储在一个结构中的数组中：

#define NUM_DEVICES 31
#define DEVICE_FIFO_SIZE 65536

struct fpga_interface {
    ...
    struct kfifo fifos[NUM_DEVICES];
    spinlock_t fifo_locks[NUM_DEVICES];
    ...
};

static struct fpga_interface fpga;

int __init fpga_init_interface(void)
{
    int i;

    info("Initialising\n");

    // KFifo initialisation
    for(i=0; i<NUM_DEVICES; i++)
    {
        if(kfifo_alloc(&(fpga.fifos[i]), DEVICE_FIFO_SIZE, GFP_KERNEL)) {
            printk(KERN_ERR, "Error allocating the KFIFOs\n");
            return 1;
        }

        spin_lock_init(&fpga.fifo_locks[i]);

        info("Initialised KFIFO %d with addr=%x\n", i, &(fpga.fifos[i]));
    }
    ...
    info("Initialised\n");
}

void get_fifo(int dev, struct kfifo* fifo, spinlock_t* lock)
{
    fifo = &(fpga.fifos[dev]);
    lock = &(fpga.fifo_locks[dev]);

    info("Returning the KFIFO for devid=%d, fifo=%x, lock=%x", dev, fifo, lock);
}

然后在另一个文件（primary.c - 处理特定接口）中我们有：

struct fpga_primary {
    ....
    struct kfifo* fifo;
    spinlock_t* fifo_lock;
...
};

static struct fpga_primary primary;

int fpga_primary_init(void)
{
    info("Starting Primary\n");

    //Get our fifo and fifo lock
    get_fifo(1, primary.fifo, primary.lock_fifo);

    info("Got kfifo_addr=%x, lock_addr=%x\n", primary.fifo, primary.fifo_lock);
    ...
    info("Completed Primary\n");
}

现在，下面是来自gumstix的内核日志：

Nov 30 02:28:14 overo [17837.989776] GPIO: Initialising
Nov 30 02:28:14 overo [17837.997497] Initialised KFIFO 0 with addr=bf062bb4
Nov 30 02:28:14 overo [17838.005249] Initialised KFIFO 1 with addr=bf062bc8
Nov 30 02:28:14 overo [17838.012847] Initialised KFIFO 2 with addr=bf062bdc
...
Nov 30 02:28:14 overo [17838.233947] GPIO: Initialised
...
Nov 30 02:28:14 overo [17838.264984] PRIMARY: Starting Primary
Nov 30 02:28:14 overo [17838.272979] GPIO: Returning the KFIFO for devid=1, fifo=bf062bc8, lock=bf062e20
Nov 30 02:28:14 overo [17838.281280] PRIMARY: Got kfifo_addr=0, lock_addr=0
Nov 30 02:28:14 overo [17838.296203] PRIMARY: Completed Primary

你会注意到对get_fifo（...）的调用会返回NULL指针。这真的不好，我似乎无法找出原因。

我正在使用OpenEmbedded / BitBake标准环境，使用内核版本3.2.0和Angstrom 2011.03工具链。我正在使用Ubuntu 10.04。

有什么想法吗？我还在想，由于某种原因，我无法将指针传递到周围的kfifo ...但是我无法找到任何记录，为什么我不能/不应该这样做，考虑到我们将指向kfifo的指针传递给kifo_函数。

Answer 1

我认为错误的结果是由于函数void get_fifo（int dev，struct kfifo * fifo，spinlock_t * lock），在这个函数中，你将结果赋给局部变量：fifo和lock;当get_fifo（）返回时，局部变量将无效。

所以我的建议是：将get_fifo（）函数声明为：

    void get_fifo(int dev, struct kfifo** fifo, spinlock_t** lock)
    {
        *fifo = &(fpga.fifos[dev]);
        *lock = &(fpga.fifo_locks[dev]);

        info("Returning the KFIFO for devid=%d, fifo=%x, lock=%x", dev, *fifo, *lock);
    }

在函数int fpga_primary_init（void）中，将get_fifo（）调用为：

    get_fifo(1, &(primary.fifo), &(primary.lock_fifo));

我认为它会解决你的问题。

Linux内核驱动程序：返回指向kfifo的指针会产生空指针

1 个答案: