Parche para el módulo cm2020

A partir del kernel 2.6.17, la macro MODULE_PARM ha sido eliminada. Esta macro estaba deprecada desde el 2004 (según LWN - API changes in the 2.6 kernel series). Como el módulo de la tarjeta Omnikey Cardman 2020 utiliza esta macro, he tenido que parchearlo, otra vez, para que funcione.

También se utiliza otra función deprecada, aunque todavía no ha sido eliminada. Pero mejor prevenir que curar. En este caso se trata de usb_unlink_urb(), por lo que en su lugar utilizaremos usb_kill_urb(). Para esta ocasión sí que he puesto la versión del kernel (2.6.8) y no la actual (2.6.17) porque he leído en algún changelog que afectaba al rendimiento, al escribir más comentarios en el syslog de los necesarios.

• Archivo cardman.c.patch --- cm2020_installer_v2_4_1_src/src/cm2020/cardman.c 2006-01-31 16:36:42.000000000 +0100 +++ cm2020_installer_v2_4_1_src-parcheado/src/cm2020/cardman.c 2006-10-02 12:13:43.000000000 +0200 @@ -63,7 +63,12 @@ #ifdef CM2020_DEBUG static int cm_debug = CM2020_DEBUG; +#if LINUX_VERSION_CODE <= KERNEL_VERSION(2,6,16) MODULE_PARM(cm_debug, "i"); +#else +module_param( cm_debug, int, 0); +#endif + #define DEBUG(n, args...) if (cm_debug >= (n)) printk(KERN_INFO args) #else #define DEBUG(n, args...) @@ -77,6 +82,7 @@ #if LINUX_VERSION_CODE >= KERNEL_VERSION(2,6,0) #define FILL_CONTROL_URB usb_fill_control_urb #define FILL_INT_URB usb_fill_int_urb +#define verify_area(type, addr, size) access_ok(type, addr, size) /* prevent races between open() and disconnect() */ static DECLARE_MUTEX (disconnect_sem); #endif @@ -165,14 +171,16 @@ static struct usb_class_driver cmu_class = { .name= "usb/cm%d", .fops= &cmu_fops, +#if LINUX_VERSION_CODE < KERNEL_VERSION(2,6,15) .mode= S_IFCHR | S_IRUSR | S_IWUSR | S_IRGRP | S_IWGRP | S_IROTH, +#endif .minor_base= CM2020_MINOR, }; #endif STATIC struct usb_driver cmu_driver = { -#if LINUX_VERSION_CODE >= KERNEL_VERSION(2,4,20) - .owner= THIS_MODULE, +#if LINUX_VERSION_CODE >= KERNEL_VERSION(2,4,20) && LINUX_VERSION_CODE < KERNEL_VERSION(2,6,16) + .owner= THIS_MODULE, #endif .name= "cardman", .probe= cmu_probe, @@ -243,9 +251,16 @@ STATIC inline int xusb_unlink_urb(struct urb *urb) { int i; - DEBUG(3, "-> usb_unlink_urb(urb=%p)\n", urb); - i=usb_unlink_urb(urb); - DEBUG(3, "<- usb_unlink_urb(urb=%p rc=%d)\n",urb,i); + #if LINUX_VERSION_CODE >= KERNEL_VERSION(2,6,8) + DEBUG(3, "-> usb_kill_urb(urb=%p)\n", urb); + usb_kill_urb(urb); + i=-1; + DEBUG(3, "<- usb_kill_urb(urb=%p rc=%d)\n",urb,i); + #else + DEBUG(3, "-> usb_unlink_urb(urb=%p)\n", urb); + i=usb_unlink_urb(urb); + DEBUG(3, "<- usb_unlink_urb(urb=%p rc=%d)\n",urb,i); + #endif return i; } @@ -1775,12 +1790,12 @@ if (_IOC_DIR(cmd)&_IOC_READ) { DEBUG(5,"ioctl: _ioc_dir=read\n"); if (arg==0) rc=-EFAULT; - else rc=verify_area(VERIFY_WRITE,(void *)arg,size); + else rc=(access_ok(VERIFY_WRITE,(void *)arg,size)) ? 0 : -EFAULT; } if (_IOC_DIR(cmd)&_IOC_WRITE) { DEBUG(5,"ioctl: _ioc_dir=write\n"); if (arg==0) rc=-EFAULT; - else rc=verify_area(VERIFY_READ,(void *)arg,size); + else rc=(access_ok(VERIFY_READ,(void *)arg,size)) ? 0 : -EFAULT; } if (rc) { DEBUG(4, "<- ioctl (unknown comamnd)\n");