编译LDD第三版中scullc代码时遇到的问题及解决方法

      最近在学习Linux driver，按照LDD第三版书附赠的一些源代码在自己的机器上编译，由于自己的Linux系统是2.6.35的内核，而LDD书中附的代码是基于2.6.10版的，因此难免会出现一些问题，下面就是我在编译scullc代码中出现的问题和解决方法。

 

      编译中即有error也有warning，我这里先处理error

 

1.  *** CFLAGS was changed in "/.../scullc/scullctest/Makefile". Fix it to use EXTRA_CFLAGS.  Stop.

 

      这个错误是由于在新版本的内核中，CFLAGS已经改成EXTRA_CFLAGS了，因此修改办法很简单，将Makefile中的CFLAGS改成EXTRA_CFLAGS即可。

 

 

2. main.c:18: fatal error: linux/config.h: No such file or directory

 

     最新版代码中已经没有config.h文件，在main.c中删掉即可。

 

3. main.c:51: error: expected ‘=’, ‘,’, ‘;’, ‘asm’ or ‘__attribute__’ before ‘*’ token

 

      这个错误是说编译中找不到k mem_cache_t 这个类型，在linux/slab.h中找了一下，这个结构已经改为struct kmem_cache ，修改办法就是把k mem_cache_t改为 struct kmem_cache即可 。

 

4. main.c:439: error: macro "INIT_WORK" passed 3 arguments, but takes just 2

 

      在2.6.22版本以后的内核中，INIT_WORK已经做了大幅度的修改 ，INIT_WORK现在使用2个参数，去掉了最后一个data参数，传入func的声明也有所变化。如下所示：

 

INIT_WORK(struct work_struct *work, void (*function)(struct work_struct *));

 

      这时function以INIT_WROK中的第一参数work作为参数。因此在使用时需要将的struct work_struct *work加入到data所在的数据结构，在scullc这个例子中就是struct async_work *结构中，然后使用container_of这个函数来求出data的指针。

 

      基于此，scullc_do_deferred_op函数被修改为：

 

static void scullc_do_deferred_op(struct work_struct *p)
{
        struct async_work *stuff = container_of(p, struct async_work, work);
        aio_complete(stuff->iocb, stuff->result, 0);
        kfree(stuff);
}

 

5. main.c:560: error: too many arguments to function ‘kmem_cache_create’

 

       最新的内核中，kmem_cache_create函数的参数跟原来相比有变化，最后一个参数destructor已经被删掉，因此在main.c中将最后一个NULL删掉即可。

 

      处理掉上述的5条，实际上编译已经可以完成了，我大概试了一下，没发现问题。下面继续处理warning。

 

6. main.c:440: warning: passing argument 1 of ‘schedule_delayed_work’ from incompatible pointer type

 

       这个编译警告是因为最新内核中schedule_delayed_work函数参数有变化，现在第一个参数变成了struct delayed_work，不再是以前的struct work_struct。实际上这里的改动涉及到了struct work_struct这个结构自身的改动，原有的work_struct中的timer_list被移到了delayed_work中，如下：

struct delayed_work {
        struct work_struct work;
        struct timer_list timer;
};

 

       关于改动的详细内容可以参考http://blog.chinaunix.net/space.php?uid=14163325&do=blog&cuid=1388772

 

       我们可以看到，由于timer_list并未使用，如果不处理这个警告，似乎可能不会产生问题。但是保险起见还是最好修改一下，修改办法比较麻烦，

 

       1) 修改struct delayed_work，将struct work_struct work改为struct delayed_work work;

 

       2) 为了配合work的新类型，将INIT_WORK改为INIT_DELAYED_WORK

 

      3) 这里修改后，刚才4中提到的 container_of那里也要做相应修改，改为如下：

 

struct async_work *stuff = container_of(p, struct async_work, work.work );

 

 7. main.c:472: warning: initialization from incompatible pointer type

 

        这里是由于aio_read和aio_write函数有改变，最新的声明为：

ssize_t (*aio_read) (struct kiocb *iocb, const struct iovec *iov, unsigned long niov, loff_t pos);

ssize_t (*aio_write) (struct kiocb *iocb, const struct iovec *iov, unsigned long niov, loff_t pos);

      有关改动可以参考这里http://lwn.net/Articles/202449/和http://lwn.net/Articles/170954/，大只是说将原有单个buffer的形式改为了多个segment的vector形式。

      这里我根据对新接口的了解，除了修改相应的scullc_aio_read，scullc_aio_write和scullc_defer_op的参数外，还需要对scullc_defer_op函数中调用scullc_write和scullc_read的地方做一些修改，我修改的如下：

if (write)
          result = scullc_write(iocb->ki_filp, (char *)(iov->iov_base), iov->iov_len, &pos);
else
          result = scullc_read(iocb->ki_filp, (char *)(iov->iov_base), iov->iov_len, &pos);

      上述修改也只是根据个人理解所做的修改，不一定正确。

 

      做完上述修改后，代码改动的地方也不少，重新make一下，已经没有任何的error和warning了，执行一下代码，暂时没有发现问题，代码移植到此结束。

      再次感叹linux内核每次升级导致的头文件变化都要重新更改代码以适应新的内核，windows这点就好多了，从xp 32位到2008 64位，API改动很小，大多改动也都能兼容老版本，基本不需要改动代码。

 

     另外关于这篇文章，如果有读者发现我这里有什么错误的地方，欢迎指正。