From mboxrd@z Thu Jan 1 00:00:00 1970 From: Jens Axboe Subject: Re: qlogicpti driver hangs the machine Date: Mon, 29 Sep 2008 08:33:56 +0200 Message-ID: <20080929063355.GK2677@kernel.dk> References: <20080924.014553.221132445.davem@davemloft.net> <48DA01A6.1010807@panasas.com> <20080928.134336.26891578.davem@davemloft.net> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Return-path: Content-Disposition: inline In-Reply-To: <20080928.134336.26891578.davem@davemloft.net> Sender: sparclinux-owner@vger.kernel.org To: David Miller Cc: mroos@ut.ee, bharrosh@panasas.com, matthew@wil.cx, linux-scsi@vger.kernel.org, sparclinux@vger.kernel.org List-Id: linux-scsi@vger.kernel.org On Sun, Sep 28 2008, David Miller wrote: > From: Meelis Roos > Date: Wed, 24 Sep 2008 12:51:09 +0300 (EEST) > > > > Subject: [PATCH] fix fall out of sg-chaining patch > > > > > > I've reviewed all patches since Matthew's, and I find one small > > > problem. > > > > > > In the load_cmd() there is a compound loop where the first 4 sg's are > > > set then the rest are set into a memory structure in group of 7 sg's. > > > > > > Well the second 7-group and on is a bug because sg pointer does not advance. > > > This is a fall out from Jens's patch. > > > > > > I'm not sure if this is it but it should be fixed > > > > This definitely helps, > > Can we get this bug fix merged if it isn't already? I'll take care of it. -- Jens Axboe