From patchwork Sat Mar 7 10:43:01 2009 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Jean Delvare X-Patchwork-Id: 10476 Received: from vger.kernel.org (vger.kernel.org [209.132.176.167]) by demeter.kernel.org (8.14.2/8.14.2) with ESMTP id n27AhVnx030895 for ; Sat, 7 Mar 2009 10:43:32 GMT Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1751513AbZCGKnL (ORCPT ); Sat, 7 Mar 2009 05:43:11 -0500 Received: (majordomo@vger.kernel.org) by vger.kernel.org id S1751436AbZCGKnL (ORCPT ); Sat, 7 Mar 2009 05:43:11 -0500 Received: from zone0.gcu-squad.org ([212.85.147.21]:41340 "EHLO services.gcu-squad.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751513AbZCGKnK (ORCPT ); Sat, 7 Mar 2009 05:43:10 -0500 Received: from jdelvare.pck.nerim.net ([62.212.121.182] helo=hyperion.delvare) by services.gcu-squad.org (GCU Mailer Daemon) with esmtpsa id 1Lfv3w-0006hN-Ca (TLSv1:AES256-SHA:256) (envelope-from ) ; Sat, 07 Mar 2009 12:51:28 +0100 Date: Sat, 7 Mar 2009 11:43:01 +0100 From: Jean Delvare To: LMML Cc: Devin Heitmueller Subject: [PATCH] em28xx: Prevent general protection fault on rmmod Message-ID: <20090307114301.1d4a12f6@hyperion.delvare> X-Mailer: Claws Mail 3.5.0 (GTK+ 2.14.4; x86_64-suse-linux-gnu) Mime-Version: 1.0 Sender: linux-media-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-media@vger.kernel.org The removal of the timer which polls the infrared input is racy. Replacing the timer with a delayed work solves the problem. Signed-off-by: Jean Delvare Cc: Devin Heitmueller --- linux/drivers/media/video/em28xx/em28xx-input.c | 24 ++++++----------------- 1 file changed, 7 insertions(+), 17 deletions(-) --- v4l-dvb.orig/linux/drivers/media/video/em28xx/em28xx-input.c 2009-03-06 13:59:37.000000000 +0100 +++ v4l-dvb/linux/drivers/media/video/em28xx/em28xx-input.c 2009-03-06 14:02:35.000000000 +0100 @@ -69,8 +69,7 @@ struct em28xx_IR { /* poll external decoder */ int polling; - struct work_struct work; - struct timer_list timer; + struct delayed_work work; unsigned int last_toggle:1; unsigned int last_readcount; unsigned int repeat_interval; @@ -298,13 +297,6 @@ static void em28xx_ir_handle_key(struct return; } -static void ir_timer(unsigned long data) -{ - struct em28xx_IR *ir = (struct em28xx_IR *)data; - - schedule_work(&ir->work); -} - #if LINUX_VERSION_CODE < KERNEL_VERSION(2, 6, 20) static void em28xx_ir_work(void *data) #else @@ -314,28 +306,26 @@ static void em28xx_ir_work(struct work_s #if LINUX_VERSION_CODE < KERNEL_VERSION(2, 6, 20) struct em28xx_IR *ir = data; #else - struct em28xx_IR *ir = container_of(work, struct em28xx_IR, work); + struct em28xx_IR *ir = container_of(work, struct em28xx_IR, work.work); #endif em28xx_ir_handle_key(ir); - mod_timer(&ir->timer, jiffies + msecs_to_jiffies(ir->polling)); + schedule_delayed_work(&ir->work, msecs_to_jiffies(ir->polling)); } static void em28xx_ir_start(struct em28xx_IR *ir) { - setup_timer(&ir->timer, ir_timer, (unsigned long)ir); #if LINUX_VERSION_CODE < KERNEL_VERSION(2, 6, 20) - INIT_WORK(&ir->work, em28xx_ir_work, ir); + INIT_DELAYED_WORK(&ir->work, em28xx_ir_work, ir); #else - INIT_WORK(&ir->work, em28xx_ir_work); + INIT_DELAYED_WORK(&ir->work, em28xx_ir_work); #endif - schedule_work(&ir->work); + schedule_delayed_work(&ir->work, 0); } static void em28xx_ir_stop(struct em28xx_IR *ir) { - del_timer_sync(&ir->timer); - flush_scheduled_work(); + cancel_delayed_work_sync(&ir->work); } int em28xx_ir_init(struct em28xx *dev)