Skip site navigation (1)Skip section navigation (2)
Date:      Mon, 30 Apr 2007 09:50:48 +0800
From:      Ganbold <ganbold@micom.mng.net>
To:        Thomas Quinot <thomas@FreeBSD.ORG>
Cc:        freebsd-stable@FreeBSD.ORG, bug-followup@FreeBSD.ORG, linimon@FreeBSD.ORG, Nikolay Pavlov <quetzal@zone3000.net>, mjacob@FreeBSD.ORG
Subject:   Re: kern/112119: system hangs when starts k3b on RELENG_6
Message-ID:  <46354B78.5000909@micom.mng.net>
In-Reply-To: <20070427193119.GA95165@melamine.cuivre.fr.eu.org>
References:  <20070427150134.64D3713C448@mx1.freebsd.org> <20070427153218.GA9091@melamine.cuivre.fr.eu.org> <20070427174922.GA5655@zone3000.net> <46324CCF.7040109@samsco.org> <20070427193119.GA95165@melamine.cuivre.fr.eu.org>

next in thread | previous in thread | raw e-mail | index | archive | help
Thomas Quinot wrote:
> * Scott Long, 2007-04-27 :
>
>   
>> Oh hell, I know exactly what the problem is!  The opcode for a
>> TEST_UNIT_READY is 0x00.  This is probably the command that is
>> generating the CHECK_CONDITION.  The test for saved_cmd is entirely
>> bogus.
>>     
>
> Hmmmm. Looks like a very plausible culprit. Good catch Scott!
> (I felt there had to be something wrong when I wrote that test,
> incidentally, precisely because of TEST_UNIT_READY).
>
> Nikolay, Ganbold, (and others), here's another patch against 1.42.2.3,
> please let me know if it works for you.
>
> Thomas.
>
> Index: atapi-cam.c
> ===================================================================
> RCS file: /space/mirror/ncvs/src/sys/dev/ata/atapi-cam.c,v
> retrieving revision 1.50
> diff -u -r1.50 atapi-cam.c
> --- atapi-cam.c	14 Mar 2007 01:59:00 -0000	1.50
> +++ atapi-cam.c	27 Apr 2007 19:26:09 -0000
> @@ -729,7 +743,7 @@
>  		 * issued a REQUEST SENSE automatically and that operation
>  		 * returned without error.
>  		 */
> -		if (request->u.atapi.saved_cmd != 0 && request->error == 0) {
> +		if (request->u.atapi.sense.key != 0 && request->error == 0) {
>  		    bcopy (&request->u.atapi.sense, &csio->sense_data, sizeof(struct atapi_sense));
>  		    csio->ccb_h.status |= CAM_AUTOSNS_VALID;
>  		}
>
>
>
>   

Scott, Thomas, thank you very much for the effort fixing this problem.
k3b starts fine with this patch.

Ganbold





Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?46354B78.5000909>