[Date Prev][Date Next] [Thread Prev][Thread Next] [Date Index] [Thread Index]

Bug#684441: Kernel panic nearly all of the first boot on everyday's morning



On Fri, 2012-08-10 at 10:47 +0800, YunQiang Su wrote:
> Package: linux
> Version: 3.2.23-1
> 
> Nearly every morning when I boot my laptop, it will kernel panic, then
> press button to halt,
> and power on again.
> 
> It seems that the kernel never panic when the second, third, forth ...
> boot: only the first time.
> 
> The attachment is the screenshot and the output of lshw .
> 
> Both the kernel 3.2 in testing and 3.5 in experimental have the problem.

Please test the attached patch, following instructions at
<http://kernel-handbook.alioth.debian.org/ch-common-tasks.html#s-common-official>

Ben.

-- 
Ben Hutchings
Sturgeon's Law: Ninety percent of everything is crap.
From ac6ba83515001f68722f5846f2a95ee42377bd71 Mon Sep 17 00:00:00 2001
From: Ben Hutchings <ben@decadent.org.uk>
Date: Sun, 12 Aug 2012 22:47:41 +0100
Subject: [PATCH] [media] rc: ite-cir: Initialise ite_dev::rdev earlier

ite_dev::rdev is currently initialised in ite_probe() after
rc_register_device() returns.  If a newly registered device is opened
quickly enough, we may enable interrupts and try to use ite_dev::rdev
before it has been initialised.  Move it up to the earliest point we
can, right after calling rc_allocate_device().

Signed-off-by: Ben Hutchings <ben@decadent.org.uk>
---
 drivers/media/rc/ite-cir.c |    2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/media/rc/ite-cir.c b/drivers/media/rc/ite-cir.c
index 0e49c99..c06992e 100644
--- a/drivers/media/rc/ite-cir.c
+++ b/drivers/media/rc/ite-cir.c
@@ -1473,6 +1473,7 @@ static int ite_probe(struct pnp_dev *pdev, const struct pnp_device_id
 	rdev = rc_allocate_device();
 	if (!rdev)
 		goto failure;
+	itdev->rdev = rdev;
 
 	ret = -ENODEV;
 
@@ -1604,7 +1605,6 @@ static int ite_probe(struct pnp_dev *pdev, const struct pnp_device_id
 	if (ret)
 		goto failure;
 
-	itdev->rdev = rdev;
 	ite_pr(KERN_NOTICE, "driver has been successfully loaded\n");
 
 	return 0;

Attachment: signature.asc
Description: This is a digitally signed message part


Reply to: