Re: kernel oops when burning CDs

2001-06-06 Thread Douglas Gilbert

Tachino wrote:
> At Mon, 4 Jun 2001 22:43:30 +0100 (BST),
> Alan Cox wrote:
> > 
> > > I get an ooops and immediate kernel panic when I break (CTRL-C) cdrecord. I 
> > > can reproduce it anytime. I use 2.4.5-ac series. Obviously, Linus' 2.4.5 is 
> > > fine.
> > > I know, I know. I was supposed to make a serios oops report, BUT I wasn't 
> > 
> > Write down the EIP and the call trace then look them up in System.map. Also
> > include the hardware details. The -ac tree has a newer version of the scsi
> > generic code. It fixes some oopses but in your case it apparently added a new
> > failure case
>
>  Oops occures in SG driver. This patch fixes the problem.

[patch snipped. It looks fine]

Tachino,
Thanks for the patch. Alan Cox has incorporated it in lk 2.4.5-ac9 .
The problem only occurs when the sg driver is built in. [I do my
testing with sg built as a module.]

Currently sg version 3.1.18 is in the ac tree while the previous
version (3.1.17) is in the main branch. The code that oopsed is
there to temporarily bump the sg module count to inhibit rmmod
(and/or autoclean) until all SCSI commands on that device are
finished.

Doug Gilbert
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to [EMAIL PROTECTED]
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/



Re: kernel oops when burning CDs

2001-06-06 Thread Douglas Gilbert

Tachino wrote:
 At Mon, 4 Jun 2001 22:43:30 +0100 (BST),
 Alan Cox wrote:
  
   I get an ooops and immediate kernel panic when I break (CTRL-C) cdrecord. I 
   can reproduce it anytime. I use 2.4.5-ac series. Obviously, Linus' 2.4.5 is 
   fine.
   I know, I know. I was supposed to make a serios oops report, BUT I wasn't 
  
  Write down the EIP and the call trace then look them up in System.map. Also
  include the hardware details. The -ac tree has a newer version of the scsi
  generic code. It fixes some oopses but in your case it apparently added a new
  failure case

  Oops occures in SG driver. This patch fixes the problem.

[patch snipped. It looks fine]

Tachino,
Thanks for the patch. Alan Cox has incorporated it in lk 2.4.5-ac9 .
The problem only occurs when the sg driver is built in. [I do my
testing with sg built as a module.]

Currently sg version 3.1.18 is in the ac tree while the previous
version (3.1.17) is in the main branch. The code that oopsed is
there to temporarily bump the sg module count to inhibit rmmod
(and/or autoclean) until all SCSI commands on that device are
finished.

Doug Gilbert
-
To unsubscribe from this list: send the line unsubscribe linux-kernel in
the body of a message to [EMAIL PROTECTED]
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/



Re: kernel oops when burning CDs

2001-06-04 Thread Tachino Nobuhiro


Hello,

At Mon, 4 Jun 2001 22:43:30 +0100 (BST),
Alan Cox wrote:
> 
> > I get an ooops and immediate kernel panic when I break (CTRL-C) cdrecord. I 
> > can reproduce it anytime. I use 2.4.5-ac series. Obviously, Linus' 2.4.5 is 
> > fine.
> > I know, I know. I was supposed to make a serios oops report, BUT I wasn't 
> 
> Write down the EIP and the call trace then look them up in System.map. Also
> include the hardware details. The -ac tree has a newer version of the scsi
> generic code. It fixes some oopses but in your case it apparently added a new
> failure case

  Oops occures in SG driver. This patch fixes the problem.



diff -r -u linux.org/drivers/scsi/sg.c linux/drivers/scsi/sg.c
--- linux.org/drivers/scsi/sg.c Fri Jun  1 10:10:22 2001
+++ linux/drivers/scsi/sg.c Fri Jun  1 17:08:00 2001
@@ -1114,7 +1114,8 @@
 sg_remove_sfp(sdp, sfp);
sfp = NULL;
 }
-   __MOD_DEC_USE_COUNT(sg_template.module);
+   if (sg_template.module)
+   __MOD_DEC_USE_COUNT(sg_template.module);
if (sdp->device->host->hostt->module)
__MOD_DEC_USE_COUNT(sdp->device->host->hostt->module);
 }
@@ -1311,7 +1312,8 @@
sg_finish_rem_req(srp);
}
if (sfp->closed) {
-   __MOD_DEC_USE_COUNT(sg_template.module);
+   if (sg_template.module)
+   __MOD_DEC_USE_COUNT(sg_template.module);
if (sdp->device->host->hostt->module)
__MOD_DEC_USE_COUNT(sdp->device->host->hostt->module);
__sg_remove_sfp(sdp, sfp);
@@ -2207,7 +2209,8 @@
 else {
 sfp->closed = 1; /* flag dirty state on this fd */
/* MOD_INC's to inhibit unloading sg and associated adapter driver */
-   __MOD_INC_USE_COUNT(sg_template.module);
+   if (sg_template.module)
+   __MOD_INC_USE_COUNT(sg_template.module);
 if (sdp->device->host->hostt->module)
__MOD_INC_USE_COUNT(sdp->device->host->hostt->module);
 SCSI_LOG_TIMEOUT(1, printk(


-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to [EMAIL PROTECTED]
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/



kernel oops when burning CDs

2001-06-04 Thread Rudo Thomas

Hello everybody!

I get an ooops and immediate kernel panic when I break (CTRL-C) cdrecord. I 
can reproduce it anytime. I use 2.4.5-ac series. Obviously, Linus' 2.4.5 is 
fine.

I know, I know. I was supposed to make a serios oops report, BUT I wasn't 
able to get the serial console running. (I've read some people couldn't get 
it to run either.)

Can anyone help me with any of the problems?

Rudo Thomas.
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to [EMAIL PROTECTED]
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/



kernel oops when burning CDs

2001-06-04 Thread Rudo Thomas

Hello everybody!

I get an ooops and immediate kernel panic when I break (CTRL-C) cdrecord. I 
can reproduce it anytime. I use 2.4.5-ac series. Obviously, Linus' 2.4.5 is 
fine.

I know, I know. I was supposed to make a serios oops report, BUT I wasn't 
able to get the serial console running. (I've read some people couldn't get 
it to run either.)

Can anyone help me with any of the problems?

Rudo Thomas.
-
To unsubscribe from this list: send the line unsubscribe linux-kernel in
the body of a message to [EMAIL PROTECTED]
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/



Re: kernel oops when burning CDs

2001-06-04 Thread Tachino Nobuhiro


Hello,

At Mon, 4 Jun 2001 22:43:30 +0100 (BST),
Alan Cox wrote:
 
  I get an ooops and immediate kernel panic when I break (CTRL-C) cdrecord. I 
  can reproduce it anytime. I use 2.4.5-ac series. Obviously, Linus' 2.4.5 is 
  fine.
  I know, I know. I was supposed to make a serios oops report, BUT I wasn't 
 
 Write down the EIP and the call trace then look them up in System.map. Also
 include the hardware details. The -ac tree has a newer version of the scsi
 generic code. It fixes some oopses but in your case it apparently added a new
 failure case

  Oops occures in SG driver. This patch fixes the problem.



diff -r -u linux.org/drivers/scsi/sg.c linux/drivers/scsi/sg.c
--- linux.org/drivers/scsi/sg.c Fri Jun  1 10:10:22 2001
+++ linux/drivers/scsi/sg.c Fri Jun  1 17:08:00 2001
@@ -1114,7 +1114,8 @@
 sg_remove_sfp(sdp, sfp);
sfp = NULL;
 }
-   __MOD_DEC_USE_COUNT(sg_template.module);
+   if (sg_template.module)
+   __MOD_DEC_USE_COUNT(sg_template.module);
if (sdp-device-host-hostt-module)
__MOD_DEC_USE_COUNT(sdp-device-host-hostt-module);
 }
@@ -1311,7 +1312,8 @@
sg_finish_rem_req(srp);
}
if (sfp-closed) {
-   __MOD_DEC_USE_COUNT(sg_template.module);
+   if (sg_template.module)
+   __MOD_DEC_USE_COUNT(sg_template.module);
if (sdp-device-host-hostt-module)
__MOD_DEC_USE_COUNT(sdp-device-host-hostt-module);
__sg_remove_sfp(sdp, sfp);
@@ -2207,7 +2209,8 @@
 else {
 sfp-closed = 1; /* flag dirty state on this fd */
/* MOD_INC's to inhibit unloading sg and associated adapter driver */
-   __MOD_INC_USE_COUNT(sg_template.module);
+   if (sg_template.module)
+   __MOD_INC_USE_COUNT(sg_template.module);
 if (sdp-device-host-hostt-module)
__MOD_INC_USE_COUNT(sdp-device-host-hostt-module);
 SCSI_LOG_TIMEOUT(1, printk(


-
To unsubscribe from this list: send the line unsubscribe linux-kernel in
the body of a message to [EMAIL PROTECTED]
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/