もうひとつ原因があって irqreturn_tのtypedefがintからenum irqreturnへ変わり、IRQ_HANDLEDがマクロではなく列挙型になったためです。これにより2.4.xとのコンパチを保つための定番処理（※）の判定を誤って、irqreturn_tをvoidと定義してしまいます。

このため割り込みハンドラが値を返さない関数として定義されてしまい、ハンドラ内でreturn IRQ_HANDLED; などとしている部分がコンパイルエラーになります。

以上の変更点を踏まえたパッチは以下の通り。Realtekが提供しているr8168-8.012.00に当てます。


diff -r 2aec411ad986 src/r8168.h
--- a/src/r8168.h       Sat Aug 29 04:02:18 2009 +0900
+++ b/src/r8168.h       Fri Sep 04 23:14:44 2009 +0900
@@ -37,11 +37,13 @@
 #define CHECKSUM_PARTIAL CHECKSUM_HW
 #endif

+#if LINUX_VERSION_CODE < KERNEL_VERSION(2,6,30)
 #ifndef IRQ_HANDLED
 #define irqreturn_t void
 #define IRQ_HANDLED
 #define IRQ_NONE
 #endif
+#endif /* #if LINUX_VERSION_CODE < KERNEL_VERSION(2,6,30) */

 #ifndef HAVE_FREE_NETDEV
 #define free_netdev(x) kfree(x)
@@ -251,7 +253,11 @@
        #define RTL_GET_NETDEV(priv_ptr)                        struct net_device *dev = priv_ptr->dev;
        #define RTL_RX_QUOTA(ndev, budget)                      budget
        #define RTL_NAPI_QUOTA_UPDATE(ndev, work_done, budget)
- #if LINUX_VERSION_CODE < KERNEL_VERSION(2,6,29)
+ #if LINUX_VERSION_CODE < KERNEL_VERSION(2,6,31)
+       #define RTL_NETIF_RX_COMPLETE(dev, napi)                napi_complete(napi)
+       #define RTL_NETIF_RX_SCHEDULE_PREP(dev, napi)           napi_schedule_prep(napi)
+       #define __RTL_NETIF_RX_SCHEDULE(dev, napi)              __napi_schedule(napi)
+ #elif LINUX_VERSION_CODE < KERNEL_VERSION(2,6,29)
        #define RTL_NETIF_RX_COMPLETE(dev, napi)                netif_rx_complete(dev, napi)
        #define RTL_NETIF_RX_SCHEDULE_PREP(dev, napi)           netif_rx_schedule_prep(dev, napi)
        #define __RTL_NETIF_RX_SCHEDULE(dev, napi)              __netif_rx_schedule(dev, napi)

このパッチを使う方がもしいましたら、ご自由にどうぞ。一応、我が家のファイルサーバ上では元気に動いております。が、ご利用の際は自己責任でよろしくお願いいたします。

変化のとき

今回紹介した変更点が、いつカーネルに取り込まれたかについてはLinux kernelのgitリポジトリを調べてね…というのはちょっと無責任ですので、紹介しておきます。

細かいログは参考の章に譲り、変更点がマージされた時期と関連するバージョンタグだけ挙げます。時間は全てUTC換算です。

2009 04/07 21:25:01: Linux 2.6.30-rc1
2009 03/26 23:06:50: irqreturn_tのパッチがマージ
2009 03/23 23:12:14: Linux 2.6.29
2009 03/16 14:56:58: NAPIのパッチがマージ
2009 03/13 02:39:28: Linux 2.6.29-rc8

従って私のパッチでは、NAPIはLinux 2.6.29未満であれば古いAPIを使い、irqreturn_tはLinux 2.6.30未満であれば古い定義を使うようにしています。

（※）irqreturn_tを使うハンドラを、古い2.4.x系とコードコンパチにするコードとして、カーネルのコメント中で紹介されていた方法です。r8168が悪いわけではありません。

参考

以下はirqreturn_tが変化したときのログ。

irqreturn_tの定義が変わったときのマージログとオリジナルのパッチのログ


----- マージ
commit a8416961d32d8bb757bcbb86b72042b66d044510
Merge: 6671de3 fc2869f
Author: Linus Torvalds <torvalds@linux-foundation.org>
Date:   Thu Mar 26 16:06:50 2009 -0700

    Merge branch 'irq-for-linus' of git://git.kernel.org/pub/scm/linux/kernel/gi
t/tip/linux-2.6-tip

----- パッチ
commit bedd30d986a05e32dc3eab874e4b9ed8a38058bb
Author: Thomas Gleixner <tglx@linutronix.de>
Date:   Tue Sep 30 23:14:27 2008 +0200

    genirq: make irqreturn_t an enum

    Impact: cleanup

    Remove the 2.4 compabiliy cruft

もうひとつNAPIが変化したときのログ。

netif_rx系が削除されたときのマージログとオリジナルのパッチのログ


----- マージ
commit 8e91f178a2bb4a3e52e76f6263c251ffb816eb17
Merge: 8032b52 ea8dbdd
Author: Linus Torvalds <torvalds@linux-foundation.org>
Date:   Mon Mar 16 07:56:58 2009 -0700

    Merge git://git.kernel.org/pub/scm/linux/kernel/git/davem/net-2.6

----- パッチ
commit 9fae6c3f648e38f023b99b5f5a5280907b2e796e
Author: Ilya Yanok <yanok@emcraft.com>
Date:   Fri Mar 13 09:51:46 2009 -0700

    dnet: replace obsolete *netif_rx_* functions with *napi_*

    *netif_rx_* functions is obsolete and removed in newer kernels so
    we need to use corresponding *napi_* functions instead.

こんなとこかな。

編集者:すずき(2009/09/05 00:16)