public inbox for linux-scsi@vger.kernel.org
 help / color / mirror / Atom feed
From: Jean Delvare <jdelvare@suse.de>
To: Joe Lawrence <Joe.Lawrence@stratus.com>
Cc: "Kai Mäkisara" <Kai.Makisara@kolumbus.fi>,
	"James E.J. Bottomley" <JBottomley@parallels.com>,
	linux-scsi@vger.kernel.org
Subject: Re: Kernel oops on st module cycling
Date: Thu, 07 Mar 2013 13:48:04 +0100	[thread overview]
Message-ID: <1362660484.4312.11.camel@amber.site> (raw)
In-Reply-To: <alpine.DEB.2.02.1302221017330.1736@jlaw-desktop.mno.stratus.com>

Hi Joe,

Thanks for your fast answer.

Le vendredi 22 février 2013 à 10:30 -0500, Joe Lawrence a écrit :
> I remember finding an st module load/unload kref accounting bug a while 
> ago: http://thread.gmane.org/gmane.linux.scsi/77539  I replied to the 
> report with a hack-patch that grabbed an extra reference to avoid the 
> crash.
> 
> There was an attempt at fixing this up in the block layer [1] but that 
> change was pulled when problems were found with that patch [2].
> 
> [1] https://lkml.org/lkml/2012/8/27/354
> [2] https://lkml.org/lkml/2012/9/22/113
> 
> Maybe this is the same bug?

Seems so. Meanwhile I saw you posted an update at:
http://marc.info/?l=linux-scsi&m=136249932603011&w=2

I have tested this patch successfully, and apparently others have as
well, so I would suggest to get this upstream ASAP. I think this fix is
a candidate for stable kernel series as well.

Note for backporters: the value returned by blk_get_queue() changed in
kernel 3.3, so care must be taken when backporting the fix to kernel 3.2
or older, otherwise success becomes failure and vice versa.

Thanks,
-- 
Jean Delvare
Suse L3

--
To unsubscribe from this list: send the line "unsubscribe linux-scsi" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

  reply	other threads:[~2013-03-07 12:48 UTC|newest]

Thread overview: 4+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2013-02-22 13:02 Kernel oops on st module cycling Jean Delvare
2013-02-22 15:30 ` Joe Lawrence
2013-03-07 12:48   ` Jean Delvare [this message]
2013-03-20 16:56     ` Jean Delvare

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=1362660484.4312.11.camel@amber.site \
    --to=jdelvare@suse.de \
    --cc=JBottomley@parallels.com \
    --cc=Joe.Lawrence@stratus.com \
    --cc=Kai.Makisara@kolumbus.fi \
    --cc=linux-scsi@vger.kernel.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox